From acd81410b5205cb18a4bba3fea75ff2389f00635 Mon Sep 17 00:00:00 2001 From: Maksim Denisov Date: Thu, 11 Jul 2024 15:14:05 +0200 Subject: ArtifactFactory: remove FromDescription method ...since it is used in tests only. --- test/buildtool/execution_api/local/local_execution.test.cpp | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) (limited to 'test/buildtool/execution_api/local/local_execution.test.cpp') diff --git a/test/buildtool/execution_api/local/local_execution.test.cpp b/test/buildtool/execution_api/local/local_execution.test.cpp index bbd48ddc..4c2e309c 100644 --- a/test/buildtool/execution_api/local/local_execution.test.cpp +++ b/test/buildtool/execution_api/local/local_execution.test.cpp @@ -19,7 +19,7 @@ #include #include "catch2/catch_test_macros.hpp" -#include "src/buildtool/common/artifact_factory.hpp" +#include "src/buildtool/common/artifact_description.hpp" #include "src/buildtool/common/repository_config.hpp" #include "src/buildtool/execution_api/local/config.hpp" #include "src/buildtool/execution_api/local/local_api.hpp" @@ -233,11 +233,10 @@ TEST_CASE("LocalExecution: One input copied to output", "[execution_api]") { std::vector const cmdline = {"cp", input_path, output_path}; auto local_artifact_opt = - ArtifactFactory::FromDescription(ArtifactFactory::DescribeKnownArtifact( - test_digest.hash(), test_digest.size(), ObjectType::File)); - REQUIRE(local_artifact_opt); + ArtifactDescription::CreateKnown(test_digest, ObjectType::File) + .ToArtifact(); auto local_artifact = - DependencyGraph::ArtifactNode{std::move(*local_artifact_opt)}; + DependencyGraph::ArtifactNode{std::move(local_artifact_opt)}; auto action = api.CreateAction(*api.UploadTree({{input_path, &local_artifact}}), -- cgit v1.2.3