f5-common-python icon indicating copy to clipboard operation
f5-common-python copied to clipboard

Standarize testing patterns

Open wojtek0806 opened this issue 8 years ago • 3 comments

As per comment in PR #1040, I would like us to discuss the established patterns for testing in this library, these will include:

  • What we test in unit and functional tests, mainly the scope of those tests.
  • The format of test fixtures, e.g out with teardown() in with yield
  • Factory classes to reduce the amount of repetitive code.
  • When not to test certain elements, e.g due to specifics of BIGIP operation, or limitations, whether its feasible to work around those.
  • Adherence to code coverage, e.g. no 100% code coverage (whenever possible) means no PR approval

@caphrim007 @jasonrahm @zancas @pjbreaux

wojtek0806 avatar Mar 20 '17 11:03 wojtek0806

closing as there seems to be no interest in this

wojtek0806 avatar Sep 01 '17 08:09 wojtek0806

re-opening. this still needs to be fixed. there is no lack of interest

caphrim007 avatar Sep 01 '17 14:09 caphrim007

since we are on this topic, adding code coverage tests at some point should happen. We can push whatever we need now but to test coverage with PR and remain consistent, we would have to go through each lib and address this in separate PRs @caphrim007

wojtek0806 avatar Sep 01 '17 14:09 wojtek0806