From c14a86d1cf2759466ce30dab9eddd2ac69e86a68 Mon Sep 17 00:00:00 2001 From: "Lukacs T. Berki" Date: Thu, 22 Jun 2023 20:30:52 +0000 Subject: [PATCH] Update tests for a changed status message. Test: Presubmit. Change-Id: I55d82442ba80d4954b0d2e091ad98d27622a9eb4 --- ui/status/kati.go | 2 +- ui/status/kati_test.go | 12 ++++++------ 2 files changed, 7 insertions(+), 7 deletions(-) diff --git a/ui/status/kati.go b/ui/status/kati.go index 1485c8dd8..dbb0ce3df 100644 --- a/ui/status/kati.go +++ b/ui/status/kati.go @@ -24,7 +24,7 @@ import ( ) var katiError = regexp.MustCompile(`^(\033\[1m)?[^ ]+:[0-9]+: (\033\[31m)?error:`) -var katiIncludeRe = regexp.MustCompile(`^(\[(\d+)/(\d+)] )?((including [^ ]+|initializing (build|packaging) system|finishing (build|packaging) rules|writing (build|packaging) rules) ...)$`) +var katiIncludeRe = regexp.MustCompile(`^(\[(\d+)/(\d+)] )?((including [^ ]+|initializing (legacy Make module parser|packaging system)|finishing (legacy Make module parsing|packaging rules)|writing (legacy Make module|packaging) rules) ...)$`) var katiLogRe = regexp.MustCompile(`^\*kati\*: `) var katiNinjaMissing = regexp.MustCompile("^[^ ]+ is missing, regenerating...$") diff --git a/ui/status/kati_test.go b/ui/status/kati_test.go index f2cb81326..fd1b4b575 100644 --- a/ui/status/kati_test.go +++ b/ui/status/kati_test.go @@ -65,7 +65,7 @@ func TestKatiNormalCase(t *testing.T) { parser.parseLine("out/build-aosp_arm.ninja is missing, regenerating...") output.Expect(t, Counts{}) - parser.parseLine("[1/1] initializing build system ...") + parser.parseLine("[1/1] initializing legacy Make module parser ...") output.Expect(t, Counts{ TotalActions: 1, RunningActions: 1, @@ -86,14 +86,14 @@ func TestKatiNormalCase(t *testing.T) { parser.parseLine(msg) // Start the next line to flush the previous result - parser.parseLine("[4/5] finishing build rules ...") + parser.parseLine("[4/5] finishing legacy Make module parsing ...") msg += "\n" if output.result.Output != msg { t.Errorf("output for action did not match:\nwant: %q\n got: %q\n", msg, output.result.Output) } - parser.parseLine("[5/5] writing build rules ...") + parser.parseLine("[5/5] writing legacy Make module rules ...") parser.parseLine("*kati*: verbose msg") parser.flushAction() @@ -118,7 +118,7 @@ func TestKatiExtraIncludes(t *testing.T) { st: status.StartTool(), } - parser.parseLine("[1/1] initializing build system ...") + parser.parseLine("[1/1] initializing legacy Make module parser ...") parser.parseLine("[2/5] including out/soong/Android-aosp_arm.mk ...") output.Expect(t, Counts{ TotalActions: 5, @@ -145,7 +145,7 @@ func TestKatiExtraIncludes(t *testing.T) { FinishedActions: 3, }) - parser.parseLine("[3/5] finishing build rules ...") + parser.parseLine("[3/5] finishing legacy Make module parsing ...") output.Expect(t, Counts{ TotalActions: 7, @@ -164,7 +164,7 @@ func TestKatiFailOnError(t *testing.T) { st: status.StartTool(), } - parser.parseLine("[1/1] initializing build system ...") + parser.parseLine("[1/1] initializing legacy Make module parser ...") parser.parseLine("[2/5] inclduing out/soong/Android-aosp_arm.mk ...") parser.parseLine("build/make/tools/Android.mk:19: error: testing") parser.flushAction()