-
Notifications
You must be signed in to change notification settings - Fork 86
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
[nasa/nos3#176] Basic unit tests #347
Conversation
…ect if building unit tests;
… comment out invalid tests as functionality disabled in NOS3;
…s in current tests, added build-test command to iterate faster on specific tests;
To test this I did the following:
Confirmed tests passed manually at the end. Unfortunately, some tests may fail depending on the resources in your environment. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
When I ran with a NOS3 virtual machine, things appear to all work; it has errors and failed tests when running on just a regular machine, though.
Was able to replicate. Simply running |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
My comment is similar. First time two tests failed, reran and one test failed. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All tests passed for me
Added ability to run unit tests in the sample app. Requires the following pull requests to be completed first: