Closed jmisur closed 2 years ago
@fbenz I removed the default false
param from tests and used the creator method for true
tests.
I just had a look at this. It seems to be fine. Just one thought: Should we create a test where not all documented fields are present and assert that an exception is thrown (because there are some fields which are not documented)?
I just had a look at this. It seems to be fine. Just one thought: Should we create a test where not all documented fields are present and assert that an exception is thrown (because there are some fields which are not documented)?
There are 2 tests already failOnUndocumentedParameters and failOnUndocumentedFields.
🏓 @fbenz
Fixes #448 #449