Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Integration test improvements #18

Draft
wants to merge 13 commits into
base: mf-develop
Choose a base branch
from

Conversation

Isaac-DeFrain
Copy link

@Isaac-DeFrain Isaac-DeFrain commented Jul 4, 2023

Explain your changes:

This PR introduces major refactoring and simplification to the Test_common module and all integration tests.

  • adds a test_name val to the output module of the Integration_test_lib.Intf.Test functor
  • adds an optional prefix to the Logger for including test names in the logs
  • eliminates the pesky dsl, network, and node types from tests
  • refactors the Test_common module to make it more user-friendly
  • adds comments for all Test_common functions
  • eliminates a few of the persistent compilation errors in various apps

Explain how you tested your changes:

That's really the problem isn't it... This is the first step in making the testing process more accessible. But, it compiles!


@Isaac-DeFrain Isaac-DeFrain force-pushed the integration-test-improvements branch from 40fb3b7 to 170411c Compare July 5, 2023 01:56
@Isaac-DeFrain
Copy link
Author

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant