Skip to content

Commit

Permalink
Bug 1674773 - Disable a few more tests under tsan. r=decoder
Browse files Browse the repository at this point in the history
These already-flakey tests seem to get pushed over the edge to "almost always
busted" with more instrumented Rust code. Disabling seems like the right approach
to get more coverage elsewhere (and the other similarly-flakey tests suggest
we're probably still hitting the root cause).

Depends on D95950

Differential Revision: https://phabricator.services.mozilla.com/D96627
  • Loading branch information
Gankra committed Nov 13, 2020
1 parent ae8e6d4 commit d78ef29
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 1 deletion.
2 changes: 2 additions & 0 deletions browser/components/ssb/tests/xpcshell/xpcshell.ini
Original file line number Diff line number Diff line change
Expand Up @@ -7,4 +7,6 @@ firefox-appdir = browser
[test_install.js]
[test_manifest.js]
[test_too_new.js]
skip-if = tsan # Times out, bug 1674773
[test_uninstall.js]
skip-if = tsan # Times out, bug 1674773
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,8 @@ prefs =
[include:xpcshell-content.ini]

[test_ext_webRequest_filterResponseData.js]
skip-if = os == "android" && debug
# tsan failure is for test_filter_301 timing out, bug 1674773
skip-if = tsan || os == "android" && debug
# streamFilter is supported with e10s only
[test_ext_webRequest_responseBody.js]
skip-if = os == "android" && debug
Expand Down
1 change: 1 addition & 0 deletions toolkit/mozapps/extensions/test/xpcshell/xpcshell.ini
Original file line number Diff line number Diff line change
Expand Up @@ -61,6 +61,7 @@ skip-if = true
[test_nodisable_hidden.js]
[test_onPropertyChanged_appDisabled.js]
head = head_addons.js head_compat.js
skip-if = tsan # Times out, bug 1674773
[test_permissions.js]
[test_permissions_prefs.js]
[test_pluginchange.js]
Expand Down

0 comments on commit d78ef29

Please sign in to comment.