-
Notifications
You must be signed in to change notification settings - Fork 28
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
Test.updates #73
Test.updates #73
Conversation
Added setup and teardown functions for the bats tests
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.
Sounds good, I'll start looking into the other test files
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.
Nice! The bats supported mechanisms for the test directory and teardown is much cleaner, consistent, and safer, agreed.
I suggested adding (or uncommenting, really) a nccmp to the scatter test but it was a preexisting condition so won't hold this up for it.
@@ -79,103 +68,4 @@ | |||
--dst_cold_restart dst_cold_restart/land.res \ | |||
--remap_file remap_file_C48_to_C192 --print_memory | |||
[ "$status" -eq 0 ] | |||
|
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.
@underwoo @ceblanton
Are we getting rid of these other remap_land tests?
Nevermind.
@rem1776 I updated a few of the tests using additional
bats
items. This should do a better job at cleaning up after the tests (even failed tests). I needed to do this while tracking down a failing tests -- the old method did not clean up after a test failed, causing future tests to fail.I suggest updating the other tests with similar modifications.