Add pre-commit yaml

Author: iivanou

.gitignore

@@ -1,19 +1,37 @@
-# Byte-compiled
+# Byte-compiled / optimized / DLL files
 __pycache__/
 *.py[cod]
 *$py.class
 
+# C extensions
+*.so
+
 # Distribution / packaging
-.idea
 .Python
 build/
 develop-eggs/
 dist/
+downloads/
+eggs/
+.eggs/
+lib/
+lib64/
+parts/
+sdist/
+var/
+wheels/
+pip-wheel-metadata/
+share/python-wheels/
 *.egg-info/
 .installed.cfg
 *.egg
 MANIFEST
-.eggs
+
+# PyInstaller
+#  Usually these files are written by a python script from a template
+#  before PyInstaller builds the exe, so as to inject date/other infos into it.
+*.manifest
+*.spec
 
 # Installer logs
 pip-log.txt
@@ -26,14 +44,87 @@ htmlcov/
 .coverage
 .coverage.*
 .cache
+nosetests.xml
 coverage.xml
 *.cover
+*.py,cover
+.hypothesis/
 .pytest_cache/
+.idea/
+
+# Translations
+*.mo
+*.pot
+
+# Django stuff:
+*.log
+local_settings.py
+db.sqlite3
+db.sqlite3-journal
+
+# Flask stuff:
+instance/
+.webassets-cache
+
+# Scrapy stuff:
+.scrapy
+
+# Sphinx documentation
+docs/_build/
+
+# PyBuilder
+target/
 
 # Jupyter Notebook
 .ipynb_checkpoints
 
+# IPython
+profile_default/
+ipython_config.py
+
+# pyenv
+.python-version
+
+# pipenv
+#   According to pypa/pipenv#598, it is recommended to include Pipfile.lock in version control.
+#   However, in case of collaboration, if having platform-specific dependencies or dependencies
+#   having no cross-platform support, pipenv may install dependencies that don't work, or not
+#   install all needed dependencies.
+#Pipfile.lock
+
+# PEP 582; used by e.g. github.com/David-OConnor/pyflow
+__pypackages__/
+
+# Celery stuff
+celerybeat-schedule
+celerybeat.pid
+
+# SageMath parsed files
+*.sage.py
+
 # Environments
 .env
 .venv
-/venv
+env/
+venv/
+ENV/
+env.bak/
+venv.bak/
+
+# Spyder project settings
+.spyderproject
+.spyproject
+
+# Rope project settings
+.ropeproject
+
+# mkdocs documentation
+/site
+
+# mypy
+.mypy_cache/
+.dmypy.json
+dmypy.json
+
+# Pyre type checker
+.pyre/

.pre-commit-config.yaml

@@ -0,0 +1,22 @@
+# See https://pre-commit.com for more information
+# See https://pre-commit.com/hooks.html for more hooks
+repos:
+-   repo: https://github.com/pre-commit/pre-commit-hooks
+    rev: v3.4.0
+    hooks:
+    -   id: trailing-whitespace
+    -   id: end-of-file-fixer
+    -   id: check-yaml
+    -   id: check-added-large-files
+-   repo: https://github.com/PyCQA/pydocstyle
+    rev: 6.0.0
+    hooks:
+    -   id: pydocstyle
+-   repo: https://github.com/Lucas-C/pre-commit-hooks-markup
+    rev: v1.0.1
+    hooks:
+    -   id: rst-linter
+-   repo: https://gitlab.com/pycqa/flake8.git
+    rev: 3.9.0
+    hooks:
+    -   id: flake8

CONTRIBUTING.rst

@@ -88,4 +88,3 @@ Ready to contribute? Here's how to set up `agent-python-pytest` for local develo
     $ git push origin name-of-your-bugfix-or-feature
 
 7. Submit a pull request through the GitHub website.
-

README.rst

@@ -205,7 +205,7 @@ To run test with Report Portal you must provide '--reportportal' flag:
 
 
 Test issue info
-~~~~~~~~~
+~~~~~~~~~~~~~~~
 
 Some pytest marks could be used to specify information about skipped or failed test result.
 List of this marks should be specified in pytest ini file (see :code:`rp_issue_marks`).
@@ -226,14 +226,14 @@ Example:
 
 
 Send attachement (screenshots)
-----------------
+------------------------------
 
 https://github.com/reportportal/client-Python#send-attachement-screenshots
 
 
 
 Troubleshooting
-~~~~~~~~~
+~~~~~~~~~~~~~~~
 
 In case you have connectivity issues (or similar problems) with Report Portal,
 it's possible to ignore exceptions raised by :code:`pytest_reportportal` plugin.
@@ -275,4 +275,3 @@ Copyright Notice
 Licensed under the `Apache 2.0`_ license (see the LICENSE file).
 
 .. _Apache 2.0:  https://www.apache.org/licenses/LICENSE-2.0
-

tests/test_service.py

@@ -65,7 +65,7 @@ def test_get_item_parameters(mocked_item, rp_service):
 @mock.patch('reportportal_client.service.ReportPortalService.start_test_item')
 def test_code_ref_bypass(mocked_item_start, mocked_item, mocked_session,
                          rp_service):
-    """ Test that a test code reference constructed and bypassed to a client.
+    """Test that a test code reference constructed and bypassed to a client.
 
     :param mocked_item_start: mocked start_test_item method reference
     :param mocked_item:       a mocked test item