[test] Fix expected line number in TestRunner.py

[StephanTLavavej]

My #73541 added lines to llvm/utils/lit/tests/Inputs/testrunner-custom-parsers/test.txt so what was previously on line 7 is now on line 12.

Before:

llvm-project/llvm/utils/lit/tests/Inputs/testrunner-custom-parsers/test.txt

Lines 7 to 8 in 28412d1

	// MY_RUN: foo \
	// MY_RUN: bar

After:

llvm-project/llvm/utils/lit/tests/Inputs/testrunner-custom-parsers/test.txt

Lines 12 to 13 in 6fb7c2d

	// MY_RUN: foo \
	// MY_RUN: bar

This didn't show up in the PR checks, but caused a buildbot failure after merging, https://lab.llvm.org/buildbot/#/builders/139/builds/54597 :

# | ====================================================================== # | FAIL: test_commands (__main__.TestIntegratedTestKeywordParser) # | ---------------------------------------------------------------------- # | Traceback (most recent call last): # | File "/home/buildbot/buildbot-root/llvm-clang-x86_64-sie-ubuntu-fast/build/utils/lit/tests/unit/TestRunner.py", line 135, in test_commands # | self.assertEqual(value[1].command.strip(), "%dbg(MY_RUN: at line 7) foo bar") # | AssertionError: '%dbg(MY_RUN: at line 12) foo bar' != '%dbg(MY_RUN: at line 7) foo bar' # | - %dbg(MY_RUN: at line 12) foo bar # | ? ^^ # | + %dbg(MY_RUN: at line 7) foo bar # | ? ^ 

Apologies for the build break 🙀

[llvmbot]

@llvm/pr-subscribers-testing-tools

Author: Stephan T. Lavavej (StephanTLavavej)

Changes

My #73541 added lines to llvm/utils/lit/tests/Inputs/testrunner-custom-parsers/test.txt so what was previously on line 7 is now on line 12.

Before:

llvm-project/llvm/utils/lit/tests/Inputs/testrunner-custom-parsers/test.txt

Lines 7 to 8 in 28412d1

	// MY_RUN: foo \
	// MY_RUN: bar

After:

llvm-project/llvm/utils/lit/tests/Inputs/testrunner-custom-parsers/test.txt

Lines 12 to 13 in 6fb7c2d

	// MY_RUN: foo \
	// MY_RUN: bar

This didn't show up in the PR checks, but caused a buildbot failure after merging. Apologies for the build break 🙀

---

Full diff: https://github.com/llvm/llvm-project/pull/73996.diff

1 Files Affected:

• (modified) llvm/utils/lit/tests/unit/TestRunner.py (+1-1)

diff --git a/llvm/utils/lit/tests/unit/TestRunner.py b/llvm/utils/lit/tests/unit/TestRunner.py index f1baf51c02f3a11..09470c7b9386ea8 100644 --- a/llvm/utils/lit/tests/unit/TestRunner.py +++ b/llvm/utils/lit/tests/unit/TestRunner.py @@ -132,7 +132,7 @@ def test_commands(self): value = cmd_parser.getValue() self.assertEqual(len(value), 2) # there are only two run lines self.assertEqual(value[0].command.strip(), "%dbg(MY_RUN: at line 4) baz") - self.assertEqual(value[1].command.strip(), "%dbg(MY_RUN: at line 7) foo bar") + self.assertEqual(value[1].command.strip(), "%dbg(MY_RUN: at line 12) foo bar") def test_boolean(self): parsers = self.make_parsers()