diff options
| author |  Berthold Stoeger <bstoeger@mail.tuwien.ac.at> | 2020-04-23 23:32:42 +0200 |
|---|---|---|
| committer |  Dirk Hohndel <dirk@hohndel.org> | 2020-04-24 10:40:12 -0700 |
| commit | 17526ded58703064dedebdf53669e12f0ab4f43e (patch) | |
| tree | ee7f16edfa84884db388f55546ae8b75f2f9ad97 /tests | |
| parent | 52f93ea32661e25ee55206a1e452ef73a0aff3db (diff) | |
| download | subsurface-17526ded58703064dedebdf53669e12f0ab4f43e.tar.gz | |
cleanup: move clearing of device nodes to clear_dive_file_data()
It makes no sense to keep the device nodes if all the other data
is cleared. Let's do this automatically and not explicitly.
This ensures that the function is also called on mobile.
Currently it was only called on desktop.
Weirdly, the parser-tests were expecting that the device nodes
were not reset by clear_dive_file_data() and therefore divecomputers
were accumulating in the test results. Thus, the additional
computers had to be removed from the expected test results.
Signed-off-by: Berthold Stoeger <bstoeger@mail.tuwien.ac.at>
Diffstat (limited to 'tests')
| -rw-r--r-- | tests/testparse.cpp | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/tests/testparse.cpp b/tests/testparse.cpp index 06c110acb..39e8a605d 100644 --- a/tests/testparse.cpp +++ b/tests/testparse.cpp @@ -286,11 +286,6 @@ void TestParse::testParseDLD() fprintf(stderr, "number of dives from DLD: %d \n", dive_table.nr); // Compare output - /* - * DC is not cleared from previous tests with the - * clear_dive_file_data(), so we do have an additional DC nick - * name field on the log. - */ QCOMPARE(save_dives("./testdldout.ssrf"), 0); FILE_COMPARE("./testdldout.ssrf", SUBSURFACE_TEST_DATA "/dives/TestDiveDivelogsDE.xml") |