Closed Bug 1059686 Opened 7 years ago Closed 6 years ago

Add more tests for get_error_search_term()


(Tree Management :: Treeherder, defect, P2)



(Not tracked)



(Reporter: emorley, Assigned: emorley)




(1 file)

In bug 1057377, the search term blacklist wasn't working.
It's since been fixed, but we should add a test.
The test should also check that the bugs object returned is not just None, but:
{'open_recent': [], 'all_others': []} avoid bug 1059306 happening again (since the UI expects the open_recent and all_others keys to be present).
Depends on: 1059306, 1057377
Blocks: 1072681
As part of doing bug 1057359, it's become apparent we have insufficient test coverage of the search term extraction code in, get_error_search_term().

(In reply to Ed Morley [:edmorley] from comment #0)
> The test should also check that the bugs object returned is not just None,

This is not directly related, and is pretty much covered by bug 1043338, so not doing that here.
Assignee: nobody → emorley
Blocks: 1057359
No longer depends on: 1059306
Priority: P4 → P2
Summary: Add a test for the bug suggestion search term blacklist → Add more tests for get_error_search_term()
Done as part of looking at bug 1057359. Passes on Travis.
For easier to follow diffs, see the individual commits rather than the overall PR diff.
Attachment #8513879 - Flags: review?(mdoglio)
Attachment #8513879 - Flags: review?(mdoglio) → review+
Commits pushed to master at
Bug 1059686 - Remove excess newlines in
Bug 1059686 - Correct the name of the leak error search term test

The current test is checking the extraction of the search term for the
leak case, not the standard test error case.
Bug 1059686 - Add search term tests for pipe-delimited log error lines
Bug 1059686 - Add tests for the search term blacklist
Bug 1059686 - Add tests for the search term full error line fall-back

Test that the full error line is used as a fall-back search term if no
test name was found, or the name found was in the blacklist.
Bug 1059686 - Add test for truncating test name style search terms
Closed: 6 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.