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
Fixing broken tests #40
Conversation
I'd recommend to keep this on hold until #38 is finished |
Alright. Is there a way for me to label this pull request on hold? |
/cc @aalhour |
Hello @hbsock, The migration of the tests package to use a unit testing framework is done, can you please resolve the merge conflicts that you have in this PR? Cheers, |
Sure. |
9a0c1c3
to
2aa58ed
Compare
It seems the unit tests for SortedDictionaryTests already had the changes I originally had, so I only updated the DLinkedListTest. |
Hello @hbsock, Can you please try to rebase your branch once more against master? I have fixed the issue causing the Travis builds to fail. Cheers, |
Sure. |
2aa58ed
to
a495ee2
Compare
@aalhour could this PR be merged now? Thank you. |
Absolutely. Thanks for contributing. |
* Fixed doubly linked list test with null enumerator. * Fix DLinkedListTest.
Fixing broken tests on SortedDictionaryTests and DLinkedListTest