Skip to content

Systemtest python-bareos: split tests in separate files#944

Merged
joergsteffens merged 4 commits intomasterfrom
dev/joergs/master/systemtests-python
Sep 30, 2021
Merged

Systemtest python-bareos: split tests in separate files#944
joergsteffens merged 4 commits intomasterfrom
dev/joergs/master/systemtests-python

Conversation

@joergsteffens
Copy link
Member

@joergsteffens joergsteffens commented Sep 29, 2021

Split monolitic systemtests/tests/python-bareos/python-bareos-unittest.py
into one file per test class.
Move commonly used functionality to modules in
systemtests/python-modules/bareos_unittest/.
Run each Python file (test_*.py) as a separate ctest.
Reuse the bareos_unittest also from python-pam.

Please check

  • Short description and the purpose of this PR is present above this paragraph
  • Your name is present in the AUTHORS file (optional)

If you have any questions or problems, please give a comment in the PR.

Helpful documentation and best practices

Checklist for the reviewer of the PR (will be processed by the Bareos team)

General
  • PR name is meaningful
  • Purpose of the PR is understood
  • Separate commit for this PR in the CHANGELOG.md, PR number referenced is same
  • Commit descriptions are understandable and well formatted
Source code quality
  • Source code changes are understandable
  • Variable and function names are meaningful
  • Code comments are correct (logically and spelling)
  • Required documentation changes are present and part of the PR
  • bareos-check-sources --since-merge does not report any problems
  • git status should not report modifications in the source tree after building and testing

@joergsteffens joergsteffens force-pushed the dev/joergs/master/systemtests-python branch from 69f1bba to 695b0bb Compare September 29, 2021 15:06
@arogge arogge self-assigned this Sep 29, 2021
@joergsteffens joergsteffens force-pushed the dev/joergs/master/systemtests-python branch from bd690bf to f59731d Compare September 30, 2021 12:51
@joergsteffens joergsteffens force-pushed the dev/joergs/master/systemtests-python branch from f59731d to c598340 Compare September 30, 2021 13:59
Split monolitic systemtests/tests/python-bareos/python-bareos-unittest.py
into one file per test class.
Move commonly used functionality to modules in
systemtests/python-modules/bareos_unittest/.
Use ``unittest discover`` to run all Python tests in the directory named test_*.py.
Run each Python file (test_*.py) as a separate test.
Refactor python-pam to use the systemtest Python module.
@joergsteffens joergsteffens force-pushed the dev/joergs/master/systemtests-python branch from c598340 to fa4f5c7 Compare September 30, 2021 14:11
@joergsteffens joergsteffens merged commit 4d163c0 into master Sep 30, 2021
@joergsteffens joergsteffens deleted the dev/joergs/master/systemtests-python branch September 30, 2021 14:12
benibr pushed a commit to benibr/bareos that referenced this pull request Oct 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants