Skip to content

Commit 2c48c8c

Browse files
committed
Add invalid_tabbing.ini test
1 parent 57fdee9 commit 2c48c8c

File tree

3 files changed

+14
-17
lines changed

3 files changed

+14
-17
lines changed

Python/ini_converting/ini_parser_tests.py

Lines changed: 6 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -9,23 +9,12 @@ def parser_tests():
99
{ "type": "property", "content": "AddEffect" }, { "type": "extra", "content": " " }, { "type": "extra", "content": "=" }, { "type": "extra", "content": " " }, { "type": "value", "content": "MOPixel" }
1010
]
1111
])
12-
test("comments", [
13-
[
14-
{ "type": "extra", "content": "\n" },
15-
{ "type": "extra", "content": "// foo"}, { "type": "extra", "content": "\n" },
16-
{ "type": "extra", "content": "/*a\nb\nc*/" }, { "type": "extra", "content": "\n" },
17-
],
18-
])
19-
test("nested", [
20-
[
21-
{ "type": "property", "content": "Foo" }, { "type": "extra", "content": " " }, { "type": "extra", "content": "=" }, { "type": "extra", "content": " " }, { "type": "value", "content": "Bar" }, { "type": "extra", "content": "\n" },
22-
{ "type": "lines_tokens", "content": [
23-
[
24-
{ "type": "extra", "content": "\t" }, { "type": "property", "content": "Baz" }, { "type": "extra", "content": " " }, { "type": "extra", "content": "=" }, { "type": "extra", "content": " " }, { "type": "value", "content": "Bee" }, { "type": "extra", "content": "\n" }
25-
]
26-
]}
27-
]
28-
])
12+
# test("invalid_tabbing", [
13+
# [
14+
# { "type": "property", "content": "AddEffect" }, { "type": "extra", "content": " " }, { "type": "extra", "content": "=" }, { "type": "extra", "content": " " }, { "type": "value", "content": "MOPixel" }, { "type": "extra", "content": "\n" },
15+
# { "type": "extra", "content": "\t\t" }, { "type": "property", "content": "Foo" }, { "type": "extra", "content": " " }, { "type": "extra", "content": "=" }, { "type": "extra", "content": " " }, { "type": "value", "content": "Bar" }
16+
# ]
17+
# ])
2918
# test("multiple", [
3019
# [
3120
# { "type": "property", "content": "Foo" }, { "type": "extra", "content": " " }, { "type": "extra", "content": "=" }, { "type": "extra", "content": " " }, { "type": "value", "content": "Bar" }, { "type": "extra", "content": "\n" },
Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,2 @@
1+
AddEffect = MOPixel
2+
Foo = Bar

Python/ini_converting/ini_tokenizer_tests.py

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,12 @@ def tokenizer_tests():
66
test("simple", [
77
{ "type": "WORD", "content": "AddEffect" }, { "type": "EXTRA", "content": " " }, { "type": "EQUALS", "content": "=" }, { "type": "EXTRA", "content": " " }, { "type": "WORD", "content": "MOPixel" },
88
])
9+
# It's fine that the tokenizer doesn't notice that there's invalid tabbing in "invalid_tabbing.ini" and happily outputs this data,
10+
# because checking for invalid tabbing is the parser's responsibility.
11+
test("invalid_tabbing", [
12+
{ "type": "WORD", "content": "AddEffect" }, { "type": "EXTRA", "content": " " }, { "type": "EQUALS", "content": "=" }, { "type": "EXTRA", "content": " " }, { "type": "WORD", "content": "MOPixel" }, { "type": "NEWLINES", "content": "\n" },
13+
{ "type": "TABS", "content": "\t\t" }, { "type": "WORD", "content": "Foo" }, { "type": "EXTRA", "content": " " }, { "type": "EQUALS", "content": "=" }, { "type": "EXTRA", "content": " " }, { "type": "WORD", "content": "Bar" },
14+
])
915
test("comments", [
1016
{ "type": "NEWLINES", "content": "\n" },
1117
{ "type": "EXTRA", "content": "// foo" }, { "type": "NEWLINES", "content": "\n" },

0 commit comments

Comments
 (0)