From f7e4ebf0e2913ff1b987e40baab5c0c809e6c0de Mon Sep 17 00:00:00 2001 From: Sascha Roloff Date: Thu, 13 Oct 2022 11:39:13 +0200 Subject: Remove default value of ArtifactDigest::Create template parameter This enforces the explicit specification, which object type, either file or tree, should be used to create an artifact digest. This also avoids subtile errors at locations as in the previous commit, where files as well as trees are supposed to be handled, but digest creation mistakenly defaults to file object type. --- .../buildtool/execution_api/bazel/bazel_execution_client.test.cpp | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) (limited to 'test/buildtool/execution_api/bazel/bazel_execution_client.test.cpp') diff --git a/test/buildtool/execution_api/bazel/bazel_execution_client.test.cpp b/test/buildtool/execution_api/bazel/bazel_execution_client.test.cpp index d5e00ad8..d136e818 100755 --- a/test/buildtool/execution_api/bazel/bazel_execution_client.test.cpp +++ b/test/buildtool/execution_api/bazel/bazel_execution_client.test.cpp @@ -25,8 +25,8 @@ TEST_CASE("Bazel internals: Execution Client", "[execution_api]") { std::string instance_name{"remote-execution"}; std::string content("test"); - auto test_digest = - static_cast(ArtifactDigest::Create(content)); + auto test_digest = static_cast( + ArtifactDigest::Create(content)); BazelExecutionClient execution_client(info->host, info->port); @@ -93,8 +93,8 @@ TEST_CASE("Bazel internals: Execution Client using env variables", std::string instance_name{"remote-execution"}; std::string content("contents of env variable"); - auto test_digest = - static_cast(ArtifactDigest::Create(content)); + auto test_digest = static_cast( + ArtifactDigest::Create(content)); BazelExecutionClient execution_client(info->host, info->port); -- cgit v1.2.3