diff options
author | Maksim Denisov <denisov.maksim@huawei.com> | 2024-06-19 12:17:24 +0200 |
---|---|---|
committer | Maksim Denisov <denisov.maksim@huawei.com> | 2024-06-25 13:59:48 +0200 |
commit | 6f48dfe97ccad8ba251bd09464dbd8d4427014c2 (patch) | |
tree | dd1fa2556a9586926108fca9e6c47f56e86c56da | |
parent | a98bc0cffd5a86515ef9c1f3f0426162f9100f86 (diff) | |
download | justbuild-6f48dfe97ccad8ba251bd09464dbd8d4427014c2.tar.gz |
Remove GraphTraverser's constructor used for tests only.
-rw-r--r-- | src/buildtool/graph_traverser/graph_traverser.hpp | 20 | ||||
-rw-r--r-- | test/buildtool/graph_traverser/graph_traverser.test.hpp | 45 |
2 files changed, 30 insertions, 35 deletions
diff --git a/src/buildtool/graph_traverser/graph_traverser.hpp b/src/buildtool/graph_traverser/graph_traverser.hpp index 194ae6d4..2c6794a6 100644 --- a/src/buildtool/graph_traverser/graph_traverser.hpp +++ b/src/buildtool/graph_traverser/graph_traverser.hpp @@ -82,26 +82,6 @@ class GraphTraverser { std::vector<std::pair<std::map<std::string, std::string>, ServerAddress>> dispatch_list, gsl::not_null<Statistics*> const& stats, - gsl::not_null<Progress*> const& progress) - : clargs_{std::move(clargs)}, - repo_config_{repo_config}, - platform_properties_{std::move(platform_properties)}, - dispatch_list_{std::move(dispatch_list)}, - stats_{stats}, - progress_{progress}, - local_api_{CreateExecutionApi(std::nullopt, - std::make_optional(repo_config))}, - remote_api_{CreateExecutionApi(RemoteExecutionConfig::RemoteAddress(), - std::make_optional(repo_config))}, - reporter_{[](auto done, auto cv) {}} {} - - explicit GraphTraverser( - CommandLineArguments clargs, - gsl::not_null<const RepositoryConfig*> const& repo_config, - std::map<std::string, std::string> platform_properties, - std::vector<std::pair<std::map<std::string, std::string>, - ServerAddress>> dispatch_list, - gsl::not_null<Statistics*> const& stats, gsl::not_null<Progress*> const& progress, progress_reporter_t reporter, Logger const* logger = nullptr) diff --git a/test/buildtool/graph_traverser/graph_traverser.test.hpp b/test/buildtool/graph_traverser/graph_traverser.test.hpp index 2ec011d4..d9ecbe54 100644 --- a/test/buildtool/graph_traverser/graph_traverser.test.hpp +++ b/test/buildtool/graph_traverser/graph_traverser.test.hpp @@ -161,7 +161,8 @@ inline void SetLauncher() { RemoteExecutionConfig::PlatformProperties(), RemoteExecutionConfig::DispatchList(), &stats, - &progress}; + &progress, + [](auto done, auto cv) {}}; auto const result = gt.BuildAndStage(clargs.graph_description, clargs.artifacts); @@ -186,7 +187,8 @@ inline void SetLauncher() { RemoteExecutionConfig::PlatformProperties(), RemoteExecutionConfig::DispatchList(), &stats, - &progress}; + &progress, + [](auto done, auto cv) {}}; auto const exec_result = gt_get_exec.BuildAndStage( clargs_exec.graph_description, clargs_exec.artifacts); @@ -217,7 +219,8 @@ inline void SetLauncher() { RemoteExecutionConfig::PlatformProperties(), RemoteExecutionConfig::DispatchList(), &stats, - &progress}; + &progress, + [](auto done, auto cv) {}}; auto const result = gt.BuildAndStage(clargs.graph_description, clargs.artifacts); @@ -244,7 +247,8 @@ inline void SetLauncher() { RemoteExecutionConfig::PlatformProperties(), RemoteExecutionConfig::DispatchList(), &stats, - &progress}; + &progress, + [](auto done, auto cv) {}}; auto const result = gt.BuildAndStage(clargs.graph_description, clargs.artifacts); @@ -259,7 +263,8 @@ inline void SetLauncher() { RemoteExecutionConfig::PlatformProperties(), RemoteExecutionConfig::DispatchList(), &stats, - &progress}; + &progress, + [](auto done, auto cv) {}}; auto const full_build_result = gt_full_build.BuildAndStage( clargs_full_build.graph_description, clargs_full_build.artifacts); @@ -290,7 +295,8 @@ inline void SetLauncher() { RemoteExecutionConfig::PlatformProperties(), RemoteExecutionConfig::DispatchList(), &stats, - &progress}; + &progress, + [](auto done, auto cv) {}}; auto const cpp_result = gt_upload.BuildAndStage( clargs_update_cpp.graph_description, clargs_update_cpp.artifacts); @@ -311,7 +317,8 @@ inline void SetLauncher() { RemoteExecutionConfig::PlatformProperties(), RemoteExecutionConfig::DispatchList(), &stats, - &progress}; + &progress, + [](auto done, auto cv) {}}; auto const result = gt.BuildAndStage(clargs.graph_description, clargs.artifacts); @@ -340,7 +347,8 @@ static void TestBlobsUploadedAndUsed(bool is_hermetic = true) { RemoteExecutionConfig::PlatformProperties(), RemoteExecutionConfig::DispatchList(), &stats, - &progress}; + &progress, + [](auto done, auto cv) {}}; auto const result = gt.BuildAndStage(clargs.graph_description, clargs.artifacts); @@ -374,7 +382,8 @@ static void TestEnvironmentVariablesSetAndUsed(bool is_hermetic = true) { RemoteExecutionConfig::PlatformProperties(), RemoteExecutionConfig::DispatchList(), &stats, - &progress}; + &progress, + [](auto done, auto cv) {}}; auto const result = gt.BuildAndStage(clargs.graph_description, clargs.artifacts); @@ -408,7 +417,8 @@ static void TestTreesUsed(bool is_hermetic = true) { RemoteExecutionConfig::PlatformProperties(), RemoteExecutionConfig::DispatchList(), &stats, - &progress}; + &progress, + [](auto done, auto cv) {}}; auto const result = gt.BuildAndStage(clargs.graph_description, clargs.artifacts); @@ -442,7 +452,8 @@ static void TestNestedTreesUsed(bool is_hermetic = true) { RemoteExecutionConfig::PlatformProperties(), RemoteExecutionConfig::DispatchList(), &stats, - &progress}; + &progress, + [](auto done, auto cv) {}}; auto const result = gt.BuildAndStage(clargs.graph_description, clargs.artifacts); @@ -478,7 +489,8 @@ static void TestFlakyHelloWorldDetected(bool /*is_hermetic*/ = true) { RemoteExecutionConfig::PlatformProperties(), RemoteExecutionConfig::DispatchList(), &stats, - &progress}; + &progress, + [](auto done, auto cv) {}}; auto const result = gt.BuildAndStage(clargs.graph_description, clargs.artifacts); @@ -497,7 +509,8 @@ static void TestFlakyHelloWorldDetected(bool /*is_hermetic*/ = true) { RemoteExecutionConfig::PlatformProperties(), RemoteExecutionConfig::DispatchList(), &stats, - &progress}; + &progress, + [](auto done, auto cv) {}}; REQUIRE(gt_output.BuildAndStage(clargs_output.graph_description, clargs_output.artifacts)); CHECK(stats.ActionsFlakyCounter() == 1); @@ -514,7 +527,8 @@ static void TestFlakyHelloWorldDetected(bool /*is_hermetic*/ = true) { RemoteExecutionConfig::PlatformProperties(), RemoteExecutionConfig::DispatchList(), &stats, - &progress}; + &progress, + [](auto done, auto cv) {}}; REQUIRE(gt_stripped.BuildAndStage(clargs_stripped.graph_description, clargs_stripped.artifacts)); CHECK(stats.ActionsFlakyCounter() == 1); @@ -530,7 +544,8 @@ static void TestFlakyHelloWorldDetected(bool /*is_hermetic*/ = true) { RemoteExecutionConfig::PlatformProperties(), RemoteExecutionConfig::DispatchList(), &stats, - &progress}; + &progress, + [](auto done, auto cv) {}}; REQUIRE(gt_ctimes.BuildAndStage(clargs_ctimes.graph_description, clargs_ctimes.artifacts)); CHECK(stats.ActionsFlakyCounter() == 2); |