Update CODING_GUIDELINES.md
This commit is contained in:
parent
c7ae1ee978
commit
7638d8ff46
1 changed files with 4 additions and 4 deletions
|
@ -28,11 +28,11 @@ PEP8 and basic style checks
|
||||||
Testing
|
Testing
|
||||||
=======
|
=======
|
||||||
|
|
||||||
* Much of ansible's testing needs are in integration, not unit tests. We're working on releasing wide array of integration tests that use modules in a live environment.
|
* Much of ansible's testing needs are in integration, not unit tests. Add module tests there.
|
||||||
* That being said, there are unit tests
|
* That being said, there are unit tests too!
|
||||||
* Code written must absolutely pass unit tests (i.e. "make tests")
|
* Code written must absolutely pass tests (i.e. "make tests")
|
||||||
* You should anticipate any error paths in your code and test down those error paths.
|
* You should anticipate any error paths in your code and test down those error paths.
|
||||||
* Additions to unit tests for core code is welcome, but modules tend to be more integration-testey, so it's not always possible to add them (examples: ec2, etc).
|
* Additions to tests for core code is welcome, but not always possible. Be sure things are at least well tested manually in that case.
|
||||||
|
|
||||||
Whitespace
|
Whitespace
|
||||||
==========
|
==========
|
||||||
|
|
Loading…
Reference in a new issue