diff options
author | Jeremie Guichard <djebrest@gmail.com> | 2017-03-03 13:22:22 +0700 |
---|---|---|
committer | Dirk Hohndel <dirk@hohndel.org> | 2017-03-04 12:03:33 -0800 |
commit | 7329629b6ec6e373020907cc0e43dc311d360777 (patch) | |
tree | 311e1ad3e82865b9c6b75bb9e1cecf6aa1e6031e /tests/testmerge.cpp | |
parent | c8065a0275204876f9958dea6dc71f736d06e5e0 (diff) | |
download | subsurface-7329629b6ec6e373020907cc0e43dc311d360777.tar.gz |
Use proper order in QCOMPARE arguments
Expected value is the second argument of QCOMPARE,
having the arguments in the right order avoid confusion
when looking at error message in case of test failure.
Signed-off-by: Jeremie Guichard <djebrest@gmail.com>
Diffstat (limited to 'tests/testmerge.cpp')
-rw-r--r-- | tests/testmerge.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/tests/testmerge.cpp b/tests/testmerge.cpp index 4ff317ad3..450aba2e9 100644 --- a/tests/testmerge.cpp +++ b/tests/testmerge.cpp @@ -29,7 +29,7 @@ void TestMerge::testMergeEmpty() QStringList readin = orgS.readAll().split("\n"); QStringList written = outS.readAll().split("\n"); while(readin.size() && written.size()){ - QCOMPARE(readin.takeFirst().trimmed(), written.takeFirst().trimmed()); + QCOMPARE(written.takeFirst().trimmed(), readin.takeFirst().trimmed()); } clear_dive_file_data(); } @@ -53,7 +53,7 @@ void TestMerge::testMergeBackwards() QStringList readin = orgS.readAll().split("\n"); QStringList written = outS.readAll().split("\n"); while(readin.size() && written.size()){ - QCOMPARE(readin.takeFirst().trimmed(), written.takeFirst().trimmed()); + QCOMPARE(written.takeFirst().trimmed(), readin.takeFirst().trimmed()); } clear_dive_file_data(); } |