Removed executor fixture and using aliases for functional tests (#56)

* removed executor fixture and using aliases * updated changelog
AgnostiqHQ · Dec 6, 2022 · eb663c9 · eb663c9
1 parent 946df71
commit eb663c9
Show file tree

Hide file tree

Showing 5 changed files with 6 additions and 33 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -7,6 +7,10 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [UNRELEASED]
 
+### Changed
+
+- Using executor aliases instead of classes for functional tests
+
 ## [0.24.0] - 2022-12-06
 
 ### Changed

diff --git a/tests/functional_tests/basic_workflow_test.py b/tests/functional_tests/basic_workflow_test.py
@@ -21,14 +21,12 @@
 import covalent as ct
 import pytest
 
-from tests.functional_tests.fixtures.executor import executor
-
 # Basic Workflow
 
 
 @pytest.mark.functional_tests
 def test_basic_workflow():
-    @ct.electron(executor=executor)
+    @ct.electron(executor="ecs")
     def join_words(a, b):
         return ", ".join([a, b])
 

diff --git a/tests/functional_tests/fixtures/__init__.py b/tests/functional_tests/fixtures/__init__.py
diff --git a/tests/functional_tests/fixtures/executor.py b/tests/functional_tests/fixtures/executor.py
diff --git a/tests/functional_tests/svm_workflow_test.py b/tests/functional_tests/svm_workflow_test.py
@@ -22,8 +22,6 @@
 import covalent as ct
 import pytest
 
-from tests.functional_tests.fixtures.executor import executor
-
 deps_pip = ct.DepsPip(packages=["numpy==1.22.4", "scikit-learn==1.1.2"])
 
 
@@ -40,7 +38,7 @@ def load_data():
         iris.target = iris.target[perm]
         return iris.data, iris.target
 
-    @ct.electron(executor=executor, deps_pip=deps_pip)
+    @ct.electron(executor="ecs", deps_pip=deps_pip)
     def train_svm(data, C, gamma):
         X, y = data
         clf = svm.SVC(C=C, gamma=gamma)