fix(travis): fix pytest codestyle and docstyle for makefile and travis

.travis.yml

@@ -9,7 +9,7 @@ install:
 script:
   - flake8
   - mypy bff tests
-  - pytest --codestyle --docstyle
+  - pytest --pycodestyle --pydocstyle
   - pytest --pylint --pylint-rcfile=.pylintrc --pylint-error-types=CWEF
   - pytest --cov=bff tests
 after_success:

Makefile

@@ -44,7 +44,7 @@ lint:
 style:
 	flake8
 	mypy bff tests
-	pytest --codestyle --docstyle
+	pytest --pycodestyle --pydocstyle
 
 .PHONY: coverage
 coverage:

setup.cfg

@@ -12,25 +12,32 @@ description-file = README.md
 [flake8]
 max-line-length = 100
 ignore = F841,   # Local variable name is assigned to but never used.
-         W503    # Line break occurred before a binary operator
+         W503    # Line break occurred before a binary operator.
 exclude =
     .git,
-    venv*,
+    .eggs,
+    __pycache__,
+    build/,
+    dist/,
+    doc/,
+    venv*/,
     versioneer.py,
     bff/_version.py,
-    setup.py
-    doc
-
-[tool:pytest]
-testpaths = bff
-docstyle_convention = numpy
-docstyle_add_ignore = D401
-codestyle_max_line_length = 100
-markers = mpl_image_compare: mark a test for image comparison.
 
 [mypy]
 ignore_missing_imports=True
 no_implicit_optional=True
 
 [mypy-bff._version]
 ignore_errors=True
+
+[pycodestyle]
+max_line_length = 100
+
+[pydocstyle]
+convention = numpy
+add_ignore = D401   # First line should be in imperative mood.
+
+[tool:pytest]
+testpaths = tests
+markers = mpl_image_compare: mark a test for image comparison.

tests/test_config.py

@@ -21,7 +21,7 @@ def test_create_config(self):
         Test of loading the config when it does not exists.
         """
         # Default configuration
-        config_default = FancyConfig()
+        __ = FancyConfig()
 
         default_path_ok = Path(__file__).resolve().parent.parent.joinpath('bff/config.yml')
         default_path_ko = Path.home().resolve().joinpath('config.yml')
@@ -93,7 +93,7 @@ def test_access_config(self):
 
         # Check the error message using a mock.
         with unittest.mock.patch('logging.Logger.error') as mock_logging:
-            config['error']
+            __ = config['error']
             mock_logging.assert_called_with('Configuration for error does not exist.')
 
         # Remove the file.