summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--src/buildtool/graph_traverser/graph_traverser.hpp20
-rw-r--r--test/buildtool/graph_traverser/graph_traverser.test.hpp45
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);