feat: re-organize and create complete implementation unit test coverage for 492 2D features#346
Merged
Merged
Conversation
…ge for 492 2D features
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
This PR brings together full unit test coverage for all 492 2D features. Of the 492 features 375 of them have comparable independent third party libraries. 214 of the 375 agree with the 3rd party tools. There are no third-party equivalent for 117 of the features and 32 of them are implementation or convention dependent. Analysis and audit of the remaining unvetted features would follow. The tests for which we don't have an external V&V script are clearly marked as such in their name.
Preliminary analysis for the features (161) that don't agree with third party tools is completed. More analysis, improvements, bug fixes and updates will follow this PR.
The independent third-party V&V scripts are here : V&V scripts for 2D Nyxus features