summaryrefslogtreecommitdiff
path: root/tools
diff options
context:
space:
mode:
authorMauro Carvalho Chehab <mchehab+huawei@kernel.org>2026-03-18 12:11:01 +0300
committerJonathan Corbet <corbet@lwn.net>2026-03-23 00:10:39 +0300
commit14b7775ef7471fbb9380048aabb3e96faa1e9123 (patch)
tree7ef1dad5af96b7a95c66e84728274c756fe3c63b /tools
parent781171bec0650c00c642564afcb5cce57abda5bf (diff)
downloadlinux-14b7775ef7471fbb9380048aabb3e96faa1e9123.tar.xz
unittests: test_tokenizer: better handle mismatch error
The current logic is too sensitive to how c_lex name is placed. Also, it doesn't really check the log. Change it to check if the expected message will be reported after a call to C tokenizer with an invalid source. Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org> Signed-off-by: Jonathan Corbet <corbet@lwn.net> Message-ID: <6e19578bc1ffa96e536dc31997ff658017f60173.1773823995.git.mchehab+huawei@kernel.org>
Diffstat (limited to 'tools')
-rwxr-xr-xtools/unittests/test_tokenizer.py13
1 files changed, 10 insertions, 3 deletions
diff --git a/tools/unittests/test_tokenizer.py b/tools/unittests/test_tokenizer.py
index 5634b4a7283e..d1f3c565b9cf 100755
--- a/tools/unittests/test_tokenizer.py
+++ b/tools/unittests/test_tokenizer.py
@@ -46,10 +46,17 @@ def make_tokenizer_test(name, data):
#
# Check if logger is working
#
- if "log_level" in data:
- with self.assertLogs('kdoc.c_lex', level='ERROR') as cm:
+ if "log_msg" in data:
+ with self.assertLogs() as cm:
tokenizer = CTokenizer(data["source"])
+ msg_found = False
+ for result in cm.output:
+ if data["log_msg"] in result:
+ msg_found = True
+
+ self.assertTrue(msg_found, f"Missing log {data['log_msg']}")
+
return
#
@@ -124,7 +131,7 @@ TESTS_TOKENIZER = {
"mismatch_error": {
"source": "int a$ = 5;", # $ is illegal
- "log_level": "ERROR",
+ "log_msg": "Unexpected token",
},
}