-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add missing docstring in tests #39
Labels
Milestone
Comments
czerus
added
documentation
Improvements or additions to documentation
enhancement
New feature or request
labels
Feb 5, 2021
czerus
pushed a commit
that referenced
this issue
May 18, 2021
Following changes has been made: * Split tests into 2 files. One for testing 'webdriver' (system identification) and second for 'pydriver.py' (actual API tests) * Move fixtures according to pytest guide to 'conftest.py' file * Move remaining helper methods to 'helpers.py' module * Use pytest's 'assert' instead of custom asserts as they are more versatile and together with 'pytest-clarity' plugin provides better visibility of actual and expected * Add missing docstrings to helpers, fixtures and tests * Replace bunch of arguments in parametrized tests with class instances holding structured input * Add pytest-clarity plugin to show whole diff when assert comparing long string and list of strings fails. In order to show this enriched diff '-vv' option is needed when running tests. Updated sv and psv jobs and README.md. * Bump version of black for flake8-black. Old version was failing to parse properly 'test_pydriver.py' file. This require also python version bump in 'pyproject.toml' to version required by newest version of 'black'. Fixes: #38, #39
czerus
pushed a commit
that referenced
this issue
May 18, 2021
Following changes has been made: * Split tests into 2 files. One for testing 'webdriver' (system identification) and second for 'pydriver.py' (actual API tests) * Move fixtures according to pytest guide to 'conftest.py' file * Move remaining helper methods to 'helpers.py' module * Use pytest's 'assert' instead of custom asserts as they are more versatile and together with 'pytest-clarity' plugin provides better visibility of actual and expected * Add missing docstrings to helpers, fixtures and tests * Replace bunch of arguments in parametrized tests with class instances holding structured input * Add pytest-clarity plugin to show whole diff when assert comparing long string and list of strings fails. In order to show this enriched diff '-vv' option is needed when running tests. Updated sv and psv jobs and README.md. * Bump version of black for flake8-black. Old version was failing to parse properly 'test_pydriver.py' file. This require also python version bump in 'pyproject.toml' to version required by newest version of 'black'. Fixes: #38, #39
czerus
pushed a commit
that referenced
this issue
May 21, 2021
Following changes has been made: * Split tests into 2 files. One for testing 'webdriver' (system identification) and second for 'pydriver.py' (actual API tests) * Move fixtures according to pytest guide to 'conftest.py' file * Move remaining helper methods to 'helpers.py' module * Use pytest's 'assert' instead of custom asserts as they are more versatile and together with 'pytest-clarity' plugin provides better visibility of actual and expected * Add missing docstrings to helpers, fixtures and tests * Replace bunch of arguments in parametrized tests with class instances holding structured input * Add pytest-clarity plugin to show whole diff when assert comparing long string and list of strings fails. In order to show this enriched diff '-vv' option is needed when running tests. Updated sv and psv jobs and README.md. * Bump version of black for flake8-black. Old version was failing to parse properly 'test_pydriver.py' file. This require also python version bump in 'pyproject.toml' to version required by newest version of 'black'. Fixes: #38, #39
czerus
pushed a commit
that referenced
this issue
May 25, 2021
Following changes has been made: * Split tests into 2 files. One for testing 'webdriver' (system identification) and second for 'pydriver.py' (actual API tests) * Move fixtures according to pytest guide to 'conftest.py' file * Move remaining helper methods to 'helpers.py' module * Use pytest's 'assert' instead of custom asserts as they are more versatile and together with 'pytest-clarity' plugin provides better visibility of actual and expected * Add missing docstrings to helpers, fixtures and tests * Replace bunch of arguments in parametrized tests with class instances holding structured input * Add pytest-clarity plugin to show whole diff when assert comparing long string and list of strings fails. In order to show this enriched diff '-vv' option is needed when running tests. Updated sv and psv jobs and README.md. * Bump version of black for flake8-black. Old version was failing to parse properly 'test_pydriver.py' file. This require also python version bump in 'pyproject.toml' to version required by newest version of 'black'. Fixes: #38, #39
czerus
added a commit
that referenced
this issue
Jun 10, 2021
Following changes has been made: * Split tests into 2 files. One for testing 'webdriver' (system identification) and second for 'pydriver.py' (actual API tests) * Move fixtures according to pytest guide to 'conftest.py' file * Move remaining helper methods to 'helpers.py' module * Use pytest's 'assert' instead of custom asserts as they are more versatile and together with 'pytest-clarity' plugin provides better visibility of actual and expected * Add missing docstrings to helpers, fixtures and tests * Replace bunch of arguments in parametrized tests with class instances holding structured input * Add pytest-clarity plugin to show whole diff when assert comparing long string and list of strings fails. In order to show this enriched diff '-vv' option is needed when running tests. Updated sv and psv jobs and README.md. * Bump version of black for flake8-black. Old version was failing to parse properly 'test_pydriver.py' file. This require also python version bump in 'pyproject.toml' to version required by newest version of 'black'. Fixes: #38, #39
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
No description provided.
The text was updated successfully, but these errors were encountered: