Skip to content

Commit 929cd96

Browse files
committed
Test as result == expected not the other way
This is the new pytest way Signed-off-by: Philippe Ombredanne <[email protected]>
1 parent c61bf09 commit 929cd96

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

src/scancode/cli_test_utils.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -195,7 +195,7 @@ def check_json_scan(
195195
# TODO: remove sort, this should no longer be needed
196196
expected = json.dumps(expected, indent=2, sort_keys=True, separators=(',', ': '))
197197
results = json.dumps(results, indent=2, sort_keys=True, separators=(',', ': '))
198-
assert expected == results
198+
assert results == expected
199199

200200

201201
def load_json_result(location, remove_file_date=False):
@@ -297,7 +297,7 @@ def check_jsonlines_scan(
297297

298298
expected = json.dumps(expected, indent=2, separators=(',', ': '))
299299
results = json.dumps(results, indent=2, separators=(',', ': '))
300-
assert expected == results
300+
assert results == expected
301301

302302

303303
def streamline_jsonlines_scan(scan_result, remove_file_date=False):

tests/licensedcode/test_detection_validate.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -164,7 +164,7 @@ def check_ignorable_clues(rule, regen=False):
164164
expected = dict([(k, v) for k, v in sorted(expected.items()) if v])
165165

166166
try:
167-
assert expected == results
167+
assert results == expected
168168
except:
169169
# On failure, we compare againto get additional failure details such as
170170
# a clickable text_file path

0 commit comments

Comments
 (0)