From 4a824ad8e526d8b46afc33396d4bb78d109637dd Mon Sep 17 00:00:00 2001 From: kmlefran Date: Fri, 27 Sep 2024 09:20:00 -0400 Subject: [PATCH] Fix Tests/linting --- src/aiida_aimall/controllers.py | 2 +- tests/conftest.py | 1 + tests/controllers/test_gaussiansubmissioncontroller.py | 2 -- tests/parsers/test_aimqbbaseparser.py | 1 + tests/parsers/test_aimqbgroupparser.py | 1 + tests/workchains/test_aimreorworkchain.py | 1 + tests/workchains/test_qmtoaimworkchain.py | 1 + tests/workchains/test_smilestogaussian.py | 1 + tests/workchains/test_subparamchain.py | 1 + 9 files changed, 8 insertions(+), 3 deletions(-) diff --git a/src/aiida_aimall/controllers.py b/src/aiida_aimall/controllers.py index babbc05..62d665a 100644 --- a/src/aiida_aimall/controllers.py +++ b/src/aiida_aimall/controllers.py @@ -7,7 +7,7 @@ and `SmilesToGaussianWorkChain`. """ - +# pylint:disable=too-many-positional-arguments from aiida import orm from aiida.orm import Dict, Int, Str from aiida.plugins import CalculationFactory, WorkflowFactory diff --git a/tests/conftest.py b/tests/conftest.py index 9565929..a7ac22a 100644 --- a/tests/conftest.py +++ b/tests/conftest.py @@ -1,5 +1,6 @@ # pylint: disable=redefined-outer-name,too-many-statements """Initialise a text database and profile for pytest.""" +# pylint:disable=too-many-positional-arguments import io import os diff --git a/tests/controllers/test_gaussiansubmissioncontroller.py b/tests/controllers/test_gaussiansubmissioncontroller.py index 810c576..e17e9a0 100644 --- a/tests/controllers/test_gaussiansubmissioncontroller.py +++ b/tests/controllers/test_gaussiansubmissioncontroller.py @@ -20,7 +20,6 @@ def test_unstored_parentgrouplabel_returns_error(): group_label="opt_workchain", max_concurrent=1, code_label="test.aimall.aimqb", - wfxgroup="test", gauss_sp_params={}, wfxname="output.wfx", ) @@ -41,7 +40,6 @@ def test_gaussiansubmission_controller(fixture_code): max_concurrent=1, code_label=code.label + "@" + code.computer.label, gauss_sp_params={}, - wfxgroup="test", wfxname="output.wfx", ) assert con.get_extra_unique_keys() == ("smiles",) diff --git a/tests/parsers/test_aimqbbaseparser.py b/tests/parsers/test_aimqbbaseparser.py index f58901a..7bd4f28 100644 --- a/tests/parsers/test_aimqbbaseparser.py +++ b/tests/parsers/test_aimqbbaseparser.py @@ -1,5 +1,6 @@ # pylint: disable=redefined-outer-name """Tests for the `AimqbParser""" +# pylint:disable=too-many-positional-arguments import pytest diff --git a/tests/parsers/test_aimqbgroupparser.py b/tests/parsers/test_aimqbgroupparser.py index b28c90d..778d834 100644 --- a/tests/parsers/test_aimqbgroupparser.py +++ b/tests/parsers/test_aimqbgroupparser.py @@ -1,4 +1,5 @@ """Test aiida-aimall group parser""" +# pylint:disable=too-many-positional-arguments import os import pytest diff --git a/tests/workchains/test_aimreorworkchain.py b/tests/workchains/test_aimreorworkchain.py index 8e2b4df..7e6fb30 100644 --- a/tests/workchains/test_aimreorworkchain.py +++ b/tests/workchains/test_aimreorworkchain.py @@ -1,4 +1,5 @@ """Tests for AimReor Workchain""" +# pylint:disable=too-many-positional-arguments # import os from aiida.common import LinkType diff --git a/tests/workchains/test_qmtoaimworkchain.py b/tests/workchains/test_qmtoaimworkchain.py index 39c2661..a6eb8a2 100644 --- a/tests/workchains/test_qmtoaimworkchain.py +++ b/tests/workchains/test_qmtoaimworkchain.py @@ -1,4 +1,5 @@ """Tests for aiida_aimall.workchains.QMToAIMWorkchain""" +# pylint:disable=too-many-positional-arguments from aiida.common import LinkType from aiida.orm import Dict, SinglefileData from plumpy.utils import AttributesFrozendict diff --git a/tests/workchains/test_smilestogaussian.py b/tests/workchains/test_smilestogaussian.py index 58bdc20..686006c 100644 --- a/tests/workchains/test_smilestogaussian.py +++ b/tests/workchains/test_smilestogaussian.py @@ -1,4 +1,5 @@ """Tests for SmilesToGaussianWorkchain""" +# pylint:disable=too-many-positional-arguments import os import cclib diff --git a/tests/workchains/test_subparamchain.py b/tests/workchains/test_subparamchain.py index 2c6e1d5..da0d276 100644 --- a/tests/workchains/test_subparamchain.py +++ b/tests/workchains/test_subparamchain.py @@ -1,4 +1,5 @@ """Tests for aiida_aimall.workchains.SubstituentParameterWorkchain""" +# pylint:disable=too-many-positional-arguments import io import ase.io