diff options
| author | Tobias Bosch <tbosch@google.com> | 2019-07-11 01:19:42 -0700 |
|---|---|---|
| committer | Tobias Bosch <tbosch@google.com> | 2019-07-12 08:25:40 +0000 |
| commit | 9780ea97662c429f6dcb53fb2ef90e98fe1a5f1b (patch) | |
| tree | 671f3f511abcbc383b45f1fc3954bb447494e36d /compiler_wrapper/compiler_wrapper_test.go | |
| parent | 9d60930e882d0e39b48e6dfab0bffa12f6f544ee (diff) | |
| download | toolchain-utils-9780ea97662c429f6dcb53fb2ef90e98fe1a5f1b.tar.gz | |
Add support for calling bisect driver
Also removes forwarding to old wrapper as we now have all
functionality in the new wrapper.
BUG=chromium:773875
TEST=unit test, ran command for bisect driver manually in terminal.
Change-Id: Icea8a20030bc72e46d6802934acd1d657f62d766
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/1697929
Reviewed-by: George Burgess <gbiv@chromium.org>
Tested-by: Tobias Bosch <tbosch@google.com>
Diffstat (limited to 'compiler_wrapper/compiler_wrapper_test.go')
| -rw-r--r-- | compiler_wrapper/compiler_wrapper_test.go | 28 |
1 files changed, 27 insertions, 1 deletions
diff --git a/compiler_wrapper/compiler_wrapper_test.go b/compiler_wrapper/compiler_wrapper_test.go index 0a041fe8..5e347ec1 100644 --- a/compiler_wrapper/compiler_wrapper_test.go +++ b/compiler_wrapper/compiler_wrapper_test.go @@ -128,7 +128,7 @@ func TestErrorOnLogRusageAndForceDisableWError(t *testing.T) { withTestContext(t, func(ctx *testContext) { ctx.env = []string{ "FORCE_DISABLE_WERROR=1", - "GETRUSAGE=" + filepath.Join(ctx.tempDir, "rusage.log"), + "GETRUSAGE=rusage.log", } stderr := ctx.mustFail(callCompiler(ctx, ctx.cfg, ctx.newCommand(gccX86_64, mainCc))) if err := verifyNonInternalError(stderr, "GETRUSAGE is meaningless with FORCE_DISABLE_WERROR"); err != nil { @@ -137,6 +137,32 @@ func TestErrorOnLogRusageAndForceDisableWError(t *testing.T) { }) } +func TestErrorOnLogRusageAndBisect(t *testing.T) { + withTestContext(t, func(ctx *testContext) { + ctx.env = []string{ + "BISECT_STAGE=xyz", + "GETRUSAGE=rusage.log", + } + stderr := ctx.mustFail(callCompiler(ctx, ctx.cfg, ctx.newCommand(gccX86_64, mainCc))) + if err := verifyNonInternalError(stderr, "BISECT_STAGE is meaningless with GETRUSAGE"); err != nil { + t.Error(err) + } + }) +} + +func TestErrorOnBisectAndForceDisableWError(t *testing.T) { + withTestContext(t, func(ctx *testContext) { + ctx.env = []string{ + "BISECT_STAGE=xyz", + "FORCE_DISABLE_WERROR=1", + } + stderr := ctx.mustFail(callCompiler(ctx, ctx.cfg, ctx.newCommand(gccX86_64, mainCc))) + if err := verifyNonInternalError(stderr, "BISECT_STAGE is meaningless with FORCE_DISABLE_WERROR"); err != nil { + t.Error(err) + } + }) +} + func TestPrintUserCompilerError(t *testing.T) { buffer := bytes.Buffer{} printCompilerError(&buffer, newUserErrorf("abcd")) |