126 'keywords-for-nonparts', |
126 'keywords-for-nonparts', |
127 bad_object = model.find_first_header_object('keywords'), |
127 bad_object = model.find_first_header_object('keywords'), |
128 type = model.header.effective_filetype, |
128 type = model.header.effective_filetype, |
129 ) |
129 ) |
130 |
130 |
|
131 @problem_type('bad-colour-24-nonline', |
|
132 severity = 'hold', |
|
133 message = 'Colour 24 used on non-lines', |
|
134 ) |
|
135 @problem_type('bad-colour-24-line', |
|
136 severity = 'hold', |
|
137 message = 'Line with colour other than 24', |
|
138 ) |
|
139 def colour_24_test(model): |
|
140 for element in model.body: |
|
141 if hasattr(element, 'colour'): |
|
142 is_line = isinstance(element, linetypes.LineSegment) |
|
143 if not is_line and element.colour.index == 24: |
|
144 yield report_problem('bad-colour-24-nonline', bad_object = element) |
|
145 if is_line and element.colour.index != 24: |
|
146 yield report_problem('bad-colour-24-line', bad_object = element) |
|
147 |
131 manifest = { |
148 manifest = { |
132 'tests': [ |
149 'tests': [ |
133 colours_test, |
150 colours_test, |
134 syntax_errors, |
151 syntax_errors, |
135 bad_header, |
152 bad_header, |
136 nocertify_test, |
153 nocertify_test, |
137 physical_colours_test, |
154 physical_colours_test, |
138 unofficiality_test, |
155 unofficiality_test, |
139 header_bfc_test, |
156 header_bfc_test, |
140 keywords_tests, |
157 keywords_tests, |
|
158 colour_24_test, |
141 ], |
159 ], |
142 } |
160 } |