Skip to content

Commit

Permalink
fix comment and ruff ignore D (missing docs) in tests
Browse files Browse the repository at this point in the history
  • Loading branch information
janosh committed Feb 2, 2024
1 parent 930c83b commit 9cc0b23
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 10 deletions.
6 changes: 3 additions & 3 deletions pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -7,13 +7,13 @@ name = "custodian"
version = "2024.1.9"
description = "A simple JIT job management framework in Python."
authors = [
{ name = "Shyue Ping Ong", email = "ongsp@ucsd.edu" },
{ name = "Janosh Riebesell", email = "janosh.riebesell@gmail.com" },
{ name = "Matthew Horton" },
{ name = "Samuel M. Blau" },
{ name = "Shyue Ping Ong", email = "ongsp@ucsd.edu" },
{ name = "Stephen Dacek" },
{ name = "William Davidson Richards" },
{ name = "Xiaohui Qu" },
{ name = "Janosh Riebesell", email = "janosh.riebesell@gmail.com" },
]
maintainers = [{ name = "Janosh Riebesell" }, { name = "Shyue Ping Ong" }]
readme = "README.md"
Expand Down Expand Up @@ -116,7 +116,7 @@ lint.isort.split-on-trailing-comma = false

[tool.ruff.lint.per-file-ignores]
"__init__.py" = ["F401"]
"*/tests/*" = ["D"]
"tests/*" = ["D"]
"tasks.py" = ["D", "E"]

[tool.pytest.ini_options]
Expand Down
10 changes: 4 additions & 6 deletions tests/ansible/test_interpreter.py
Original file line number Diff line number Diff line change
@@ -1,5 +1,9 @@
"""Created on Jun 1, 2012."""

import pytest

from custodian.ansible.actions import FileActions
from custodian.ansible.interpreter import Modder

__author__ = "Shyue Ping Ong"
__copyright__ = "Copyright 2012, The Materials Project"
Expand All @@ -9,12 +13,6 @@
__date__ = "Jun 1, 2012"


import pytest

from custodian.ansible.actions import FileActions
from custodian.ansible.interpreter import Modder


class TestModder:
def test_dict_modify(self):
modder = Modder()
Expand Down
2 changes: 1 addition & 1 deletion tests/vasp/test_handlers.py
Original file line number Diff line number Diff line change
Expand Up @@ -1118,7 +1118,7 @@ def test_correct(self) -> None:
incar = Incar.from_file("INCAR")
assert incar["ALGO"].lower() == algo

# now test meta-GGA and hybrid, should only directly to ALGO = all from fast
# now test meta-GGA and hybrid, should go directly from ALGO = fast to all
for updates in [{"METAGGA": "SCAN"}, {"LHFCALC": True, "GGA": "PE"}]:
incar = Incar(original_incar) # incar.copy() returns dict
incar.update(updates)
Expand Down

0 comments on commit 9cc0b23

Please sign in to comment.