diff options
Diffstat (limited to 'test/buildtool/execution_api')
3 files changed, 17 insertions, 3 deletions
diff --git a/test/buildtool/execution_api/bazel/bazel_cas_client.test.cpp b/test/buildtool/execution_api/bazel/bazel_cas_client.test.cpp index 488483ae..031da8f0 100644 --- a/test/buildtool/execution_api/bazel/bazel_cas_client.test.cpp +++ b/test/buildtool/execution_api/bazel/bazel_cas_client.test.cpp @@ -79,6 +79,8 @@ TEST_CASE("Bazel internals: CAS Client", "[execution_api]") { auto blobs = cas_client.BatchReadBlobs(instance_name, {digest}); REQUIRE(blobs.size() == 1); CHECK(blobs.begin()->GetDigest() == digest); - CHECK(*blobs.begin()->ReadContent() == content); + auto const read_content = blobs.begin()->ReadContent(); + CHECK(read_content != nullptr); + CHECK(*read_content == content); } } diff --git a/test/buildtool/execution_api/bazel/bazel_network.test.cpp b/test/buildtool/execution_api/bazel/bazel_network.test.cpp index 1bc6c0ac..3cb0ef35 100644 --- a/test/buildtool/execution_api/bazel/bazel_network.test.cpp +++ b/test/buildtool/execution_api/bazel/bazel_network.test.cpp @@ -99,6 +99,9 @@ TEST_CASE("Bazel network: write/read blobs", "[execution_api]") { // Check order maintained REQUIRE(blobs.size() == 5); + for (auto const& blob : blobs) { + REQUIRE(blob.ReadContent() != nullptr); + } CHECK(*blobs[0].ReadContent() == content_foo); CHECK(*blobs[1].ReadContent() == content_bar); CHECK(*blobs[2].ReadContent() == content_baz); @@ -160,6 +163,9 @@ TEST_CASE("Bazel network: read blobs with unknown size", "[execution_api]") { // Check order maintained REQUIRE(blobs.size() == 2); + for (auto const& blob : blobs) { + REQUIRE(blob.ReadContent() != nullptr); + } CHECK(*blobs[0].ReadContent() == content_foo); CHECK(*blobs[1].ReadContent() == content_bar); } diff --git a/test/buildtool/execution_api/bazel/bytestream_client.test.cpp b/test/buildtool/execution_api/bazel/bytestream_client.test.cpp index 4676a5c6..991dcf72 100644 --- a/test/buildtool/execution_api/bazel/bytestream_client.test.cpp +++ b/test/buildtool/execution_api/bazel/bytestream_client.test.cpp @@ -61,7 +61,10 @@ TEST_CASE("ByteStream Client: Transfer single blob", "[execution_api]") { auto const downloaded_blob = stream.Read(instance_name, digest); REQUIRE(downloaded_blob.has_value()); - CHECK(*downloaded_blob->ReadContent() == content); + + auto const downloaded_content = downloaded_blob->ReadContent(); + REQUIRE(downloaded_content != nullptr); + CHECK(*downloaded_content == content); } SECTION("Small blob with wrong digest") { @@ -97,7 +100,10 @@ TEST_CASE("ByteStream Client: Transfer single blob", "[execution_api]") { SECTION("Download large blob") { auto const downloaded_blob = stream.Read(instance_name, digest); REQUIRE(downloaded_blob.has_value()); - CHECK(*downloaded_blob->ReadContent() == content); + + auto const downloaded_content = downloaded_blob->ReadContent(); + REQUIRE(downloaded_content != nullptr); + CHECK(*downloaded_content == content); } SECTION("Incrementally download large blob") { |