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

Added support for non-strict unordered assertion mode. #48

Conversation

mehmet-aslan
Copy link
Contributor

Added a new mode "NON_STRICT_UNORDERED" to DatabaseAssertionMode enum to be able to compare actual and expected data ignoring order of rows. Corresponding class to this mode sorts two data sets using SortedTable.

When using only NON_STRICT mode, our test cases pass sometimes but fails sometimes because the order of actual data can not match expected data every time so we need to this behaviour.

@buildhive
Copy link

springtestdbunit » spring-test-dbunit #28 FAILURE
Looks like there's a problem with this pull request
(what's this?)

@mzagar
Copy link
Contributor

mzagar commented Jun 2, 2014

Maybe we could make this to be the default behavior of NON_STRICT mode?
I don't see any reason not to (we wouldn't be breaking backward compatibility)...

@jmerljak
Copy link

jmerljak commented Jun 2, 2014

either way, unordered non-strict comparison is a must and should be implemented/merged as soon as possible.

@philwebb philwebb closed this in ae83828 Sep 17, 2014
philwebb added a commit that referenced this pull request Sep 17, 2014
* non-strict-unordered-assertion:
  Add DatabaseAssertionMode.NON_STRICT_UNORDERED
@philwebb
Copy link
Contributor

Merged with a few tweaks. Thanks!

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.

None yet

5 participants