Pitest #714
Closed
Pitest #714
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…y on in.specmatic.test.*
* main: (52 commits) Updated WSDL tests for attributes Updated some tests Updated some tests Created a method to log all the facts associated with a request, so that we can identify the different scenarios that are being tested ( for wsdl attribute tests) Validated actual counts of expected tests instead of just asserting if the count is greater than 1. Modified the stub tests for attributes to make them more lightweight and precise (without having to actually run the stub) minor changes, clean up Added tests to validate behaviour of mandatory attributes in wsdl in stub mode. Removed explicit filtering of XMLPattern attributes to only retain the 'SPECMATIC_XML_ATTRIBUTE_PREFIX' attribute when de-referencing. As due to this, any other custom attribute defined, gets ignored. Minor refactorings Update the version to 0.76.0 Addressed review comments: 1. Added default value(null) for the configFileName param in loadSpecmaticJsonConfig(), so that we don't explicitly need to pass a null. 2. Improved the info message if no report configuration section is found in specmatic.json. 3. Improved the error message displayed if the API Coverage report generated is blank ( provided guidance on scenarios where a blank report is generated) 4.Extracted code in OpenApiCoverageReportInput.generate() to separate methods with descriptive names to eliminate the need for comments. clean up Auto-sized the width of the remarks column based on the longest remarks string. Left aligned all the text columns in the API Coverage report. Added a 'Remarks' column in the API Coverage report to indicate if the endpoint is covered, missing in spec, or not implemented in code. Renamed the 'count' column to '#excercised' in the API Coverage Summary report. minor changes Fixed issues in marking tests as not implemented. Marked all tests which have failed whose path/method are not found in the actuator as not implemented. If an operation is defined in the spec, but not implemented in code, it won't get counted for coverage. ...
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What:
Why:
How:
Checklist:
Issue ID:
Closes: #123