Prepare plugin detection for removal of the cmx_3600 adapter (#1488)

* add "extract_adapters" to CI triggers * use "otiod" as override example Signed-off-by: apetrynet <[email protected]>
AcademySoftwareFoundation · Apr 4, 2024 · 4da693c · 4da693c
1 parent 1ad891d
commit 4da693c
Show file tree

Hide file tree

Showing 4 changed files with 9 additions and 9 deletions.
diff --git a/.github/workflows/docs.yml b/.github/workflows/docs.yml
@@ -2,9 +2,9 @@ name: docs
 
 on:
   push:
-    branches: [ main ]
+    branches: [ main, extract_adapters ]
   pull_request:
-    branches: [ main ]
+    branches: [ main, extract_adapters ]
 
 jobs:
   check-links:

diff --git a/.github/workflows/python-package.yml b/.github/workflows/python-package.yml
@@ -11,9 +11,9 @@ env:
 
 on:
   push:
-    branches: [ main ]
+    branches: [ main, extract_adapters ]
   pull_request:
-    branches: [ main ]
+    branches: [ main, extract_adapters ]
 
 concurrency:
   group: ${{ github.workflow }}-${{ github.ref }}

diff --git a/tests/baselines/plugin_module/otio_override_adapter/plugin_manifest.json b/tests/baselines/plugin_module/otio_override_adapter/plugin_manifest.json
@@ -3,9 +3,9 @@
     "adapters": [
         {
             "OTIO_SCHEMA" : "Adapter.1",
-            "name" : "cmx_3600",
+            "name" : "otiod",
             "filepath" : "adapter.py",
-            "suffixes" : ["edl"]
+            "suffixes" : ["otiod"]
         }
     ]
 }
diff --git a/tests/test_plugin_detection.py b/tests/test_plugin_detection.py
@@ -83,11 +83,11 @@ def test_override_adapter(self):
         # Test that entrypoint plugins load before builtin and contrib
         man = otio.plugins.manifest.load_manifest()
 
-        # The override_adapter creates another cmx_3600 adapter
+        # The override_adapter creates another otiod adapter
         adapters = [adapter for adapter in man.adapters
-                    if adapter.name == "cmx_3600"]
+                    if adapter.name == "otiod"]
 
-        # More then one cmx_3600 adapter should exist.
+        # More then one otiod adapter should exist.
         self.assertTrue(len(adapters) > 1)
 
         # Override adapter should be the first adapter found