diff options
Diffstat (limited to 'test/buildtool/common/artifact_factory.test.cpp')
-rw-r--r-- | test/buildtool/common/artifact_factory.test.cpp | 20 |
1 files changed, 9 insertions, 11 deletions
diff --git a/test/buildtool/common/artifact_factory.test.cpp b/test/buildtool/common/artifact_factory.test.cpp index 64a6c782..222ffd2b 100644 --- a/test/buildtool/common/artifact_factory.test.cpp +++ b/test/buildtool/common/artifact_factory.test.cpp @@ -16,23 +16,22 @@ #include "catch2/catch_test_macros.hpp" #include "nlohmann/json.hpp" -#include "src/buildtool/common/artifact_factory.hpp" +#include "src/buildtool/common/artifact_description.hpp" #include "src/buildtool/file_system/object_type.hpp" TEST_CASE("Description missing mandatory key/value pair", "[artifact_factory]") { - nlohmann::json const missing_type = {{"data", {{"path", "some/path"}}}}; - CHECK(not ArtifactFactory::FromDescription(missing_type)); + CHECK(not ArtifactDescription::FromJson(missing_type)); nlohmann::json const missing_data = {{"type", "LOCAL"}}; - CHECK(not ArtifactFactory::FromDescription(missing_data)); + CHECK(not ArtifactDescription::FromJson(missing_data)); } TEST_CASE("Local artifact description contains incorrect value for \"data\"", "[artifact_factory]") { nlohmann::json const local_art_missing_path = { {"type", "LOCAL"}, {"data", nlohmann::json::object()}}; - CHECK(not ArtifactFactory::FromDescription(local_art_missing_path)); + CHECK(not ArtifactDescription::FromJson(local_art_missing_path)); } TEST_CASE("Known artifact description contains incorrect value for \"data\"", @@ -43,20 +42,19 @@ TEST_CASE("Known artifact description contains incorrect value for \"data\"", nlohmann::json const known_art_missing_id = { {"type", "KNOWN"}, {"data", {{"size", 15}, {"file_type", file_type}}}}; - CHECK(not ArtifactFactory::FromDescription(known_art_missing_id)); + CHECK(not ArtifactDescription::FromJson(known_art_missing_id)); } SECTION("missing \"size\"") { nlohmann::json const known_art_missing_size = { {"type", "KNOWN"}, {"data", {{"id", "known_input"}, {"file_type", file_type}}}}; - CHECK(not ArtifactFactory::FromDescription(known_art_missing_size)); + CHECK(not ArtifactDescription::FromJson(known_art_missing_size)); } SECTION("missing \"file_type\"") { nlohmann::json const known_art_missing_file_type = { {"type", "KNOWN"}, {"data", {{"id", "known_input"}, {"size", 15}}}}; - CHECK( - not ArtifactFactory::FromDescription(known_art_missing_file_type)); + CHECK(not ArtifactDescription::FromJson(known_art_missing_file_type)); } } @@ -64,9 +62,9 @@ TEST_CASE("Action artifact description contains incorrect value for \"data\"", "[artifact_factory]") { nlohmann::json const action_art_missing_id = { {"type", "ACTION"}, {"data", {{"path", "output/path"}}}}; - CHECK(not ArtifactFactory::FromDescription(action_art_missing_id)); + CHECK(not ArtifactDescription::FromJson(action_art_missing_id)); nlohmann::json const action_art_missing_path = { {"type", "ACTION"}, {"data", {{"id", "action_id"}}}}; - CHECK(not ArtifactFactory::FromDescription(action_art_missing_path)); + CHECK(not ArtifactDescription::FromJson(action_art_missing_path)); } |