From 55776eca5f7bc13e22bc1e32cb5dfaa3bd822592 Mon Sep 17 00:00:00 2001 From: Etienne Perot Date: Thu, 23 Jun 2022 12:31:07 -0700 Subject: [PATCH] integration_test: Make `TestTmpMountWithSize` error message less confusing. If I understand correctly, this test expects the second write to fail, but if it succeeds, the error message it shows suggests that it should have failed. This CL rewords this message to reflect that. #codehealth PiperOrigin-RevId: 456835892 --- test/e2e/integration_test.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/test/e2e/integration_test.go b/test/e2e/integration_test.go index 285620715..8caf32eb8 100644 --- a/test/e2e/integration_test.go +++ b/test/e2e/integration_test.go @@ -944,7 +944,7 @@ func TestTmpMountWithSize(t *testing.T) { } echoOutput, err := d.Exec(ctx, dockerutil.ExecOpts{}, "/bin/sh", "-c", "echo world > /tmp/foo/test2.txt") if err == nil { - t.Fatalf("docker exec size check failed: %v", err) + t.Fatalf("docker exec size check unexpectedly succeeded (output: %v)", echoOutput) } wantErr := "No space left on device" if !strings.Contains(echoOutput, wantErr) {