Skip to content

Commit

Permalink
Opt-in to some tests for TOML
Browse files Browse the repository at this point in the history
  • Loading branch information
franzpoeschel committed May 9, 2023
1 parent a5fc44b commit ad4183c
Showing 1 changed file with 19 additions and 1 deletion.
20 changes: 19 additions & 1 deletion test/SerialIOTest.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -81,7 +81,10 @@ std::vector<std::string> testedFileExtensions()
allExtensions.begin(), allExtensions.end(), [](std::string const &ext) {
// sst and ssc need a receiver for testing
// bp4 is already tested via bp
return ext == "sst" || ext == "ssc" || ext == "bp4";
// toml parsing is very slow and its implementation is equivalent to
// the json backend, so it is only activated for selected tests
return ext == "sst" || ext == "ssc" || ext == "bp4" ||
ext == "toml";
});
return {allExtensions.begin(), newEnd};
}
Expand Down Expand Up @@ -1590,6 +1593,11 @@ TEST_CASE("dtype_test", "[serial]")
{
dtype_test(t);
}
/*
* TOML backend is not generally tested for performance reasons, opt in to
* testing it here.
*/
dtype_test("toml");
}

inline void write_test(const std::string &backend)
Expand Down Expand Up @@ -2188,6 +2196,11 @@ TEST_CASE("fileBased_write_test", "[serial]")
{
fileBased_write_test(t);
}
/*
* TOML backend is not generally tested for performance reasons, opt in to
* testing it here.
*/
fileBased_write_test("toml");
}

inline void sample_write_thetaMode(std::string file_ending)
Expand Down Expand Up @@ -7238,4 +7251,9 @@ TEST_CASE("groupbased_read_write", "[serial]")
}
}
}
/*
* TOML backend is not generally tested for performance reasons, opt in to
* testing it here.
*/
groupbased_read_write("toml");
}

0 comments on commit ad4183c

Please sign in to comment.