Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Side by Side Diff: tests/test_rpy.py

Issue 30022565: Issue 7325 - Let the parser handle filters than begin and end with '%' (Closed) Base URL: https://hg.adblockplus.org/python-abp
Patch Set: Created March 5, 2019, 1:17 a.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View unified diff | Download patch
« no previous file with comments | « tests/test_parser.py ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # This file is part of Adblock Plus <https://adblockplus.org/>, 1 # This file is part of Adblock Plus <https://adblockplus.org/>,
2 # Copyright (C) 2006-present eyeo GmbH 2 # Copyright (C) 2006-present eyeo GmbH
3 # 3 #
4 # Adblock Plus is free software: you can redistribute it and/or modify 4 # Adblock Plus is free software: you can redistribute it and/or modify
5 # it under the terms of the GNU General Public License version 3 as 5 # it under the terms of the GNU General Public License version 3 as
6 # published by the Free Software Foundation. 6 # published by the Free Software Foundation.
7 # 7 #
8 # Adblock Plus is distributed in the hope that it will be useful, 8 # Adblock Plus is distributed in the hope that it will be useful,
9 # but WITHOUT ANY WARRANTY; without even the implied warranty of 9 # but WITHOUT ANY WARRANTY; without even the implied warranty of
10 # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the 10 # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
(...skipping 53 matching lines...) Expand 10 before | Expand all | Expand 10 after
64 'filter_single': { 64 'filter_single': {
65 'in': b'foo.com##div#ad1', 65 'in': b'foo.com##div#ad1',
66 'out': { 66 'out': {
67 b'type': b'Filter', 67 b'type': b'Filter',
68 b'text': b'foo.com##div#ad1', 68 b'text': b'foo.com##div#ad1',
69 b'selector': {b'type': b'css', b'value': b'div#ad1'}, 69 b'selector': {b'type': b'css', b'value': b'div#ad1'},
70 b'action': b'hide', 70 b'action': b'hide',
71 b'options': [(b'domain', [(b'foo.com', True)])], 71 b'options': [(b'domain', [(b'foo.com', True)])],
72 }, 72 },
73 }, 73 },
74 'filter_with_%': {
75 'in': b'%22banner%*%22idzone%',
76 'out': {
77 b'type': b'Filter',
78 b'text': b'%22banner%*%22idzone%',
79 b'selector': {b'type': b'url-pattern',
80 b'value': b'%22banner%*%22idzone%'},
81 b'action': b'block',
82 b'options': [],
83 },
84 },
74 'filter_multiple': { 85 'filter_multiple': {
75 'in': b'foo.com,bar.com##div#ad1', 86 'in': b'foo.com,bar.com##div#ad1',
76 'out': { 87 'out': {
77 b'type': b'Filter', 88 b'type': b'Filter',
78 b'text': b'foo.com,bar.com##div#ad1', 89 b'text': b'foo.com,bar.com##div#ad1',
79 b'selector': {b'type': b'css', b'value': b'div#ad1'}, 90 b'selector': {b'type': b'css', b'value': b'div#ad1'},
80 b'action': b'hide', 91 b'action': b'hide',
81 b'options': [(b'domain', [(b'foo.com', True), (b'bar.com', 92 b'options': [(b'domain', [(b'foo.com', True), (b'bar.com',
82 True)])], 93 True)])],
83 }, 94 },
(...skipping 81 matching lines...) Expand 10 before | Expand all | Expand 10 after
165 """Test that the API returns the correct result in the appropriate format. 176 """Test that the API returns the correct result in the appropriate format.
166 177
167 By default, lines2dicts() does not correctly parse headers and metadata. 178 By default, lines2dicts() does not correctly parse headers and metadata.
168 """ 179 """
169 tests = [t for t in _TEST_EXAMPLES.values() 180 tests = [t for t in _TEST_EXAMPLES.values()
170 if t['out'][b'type'] not in {b'Header', b'Metadata'}] 181 if t['out'][b'type'] not in {b'Header', b'Metadata'}]
171 ins = [ex['in'] for ex in tests] 182 ins = [ex['in'] for ex in tests]
172 outs = [ex['out'] for ex in tests] 183 outs = [ex['out'] for ex in tests]
173 184
174 assert lines2dicts(ins) == outs 185 assert lines2dicts(ins) == outs
OLDNEW
« no previous file with comments | « tests/test_parser.py ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld