From 3c78905f4782ea9a3fe88f2d17dee89ef153766e Mon Sep 17 00:00:00 2001 From: Tom Marble Date: Mon, 29 Jan 2024 17:30:48 +0000 Subject: [PATCH] Bug 1868866 - adds ManifestParser TOML linter r=jmaher,linter-reviewers,ahal,webdriver-reviewers,settings-reviewers,search-reviewers,devtools-reviewers,fxview-reviewers,cookie-reviewers,sessionstore-reviewers,places-reviewers,dom-storage-reviewers,translations-reviewers,valentin,whimboo,Standard8,sclements,jesup Differential Revision: https://phabricator.services.mozilla.com/D199054 --- accessible/tests/mochitest/a11y.toml | 14 +- accessible/tests/mochitest/actions/a11y.toml | 23 ++- accessible/tests/mochitest/aom/a11y.toml | 3 +- .../tests/mochitest/attributes/a11y.toml | 20 +-- accessible/tests/mochitest/bounds/a11y.toml | 2 +- accessible/tests/mochitest/elm/a11y.toml | 13 +- accessible/tests/mochitest/events/a11y.toml | 106 +++++++------- .../tests/mochitest/events/docload/a11y.toml | 12 +- accessible/tests/mochitest/focus/a11y.toml | 8 +- accessible/tests/mochitest/hittest/a11y.toml | 15 +- .../tests/mochitest/hyperlink/a11y.toml | 5 +- .../tests/mochitest/hypertext/a11y.toml | 5 +- accessible/tests/mochitest/name/a11y.toml | 20 +-- .../tests/mochitest/relations/a11y.toml | 18 +-- accessible/tests/mochitest/role/a11y.toml | 15 +- accessible/tests/mochitest/scroll/a11y.toml | 2 +- .../tests/mochitest/selectable/a11y.toml | 11 +- accessible/tests/mochitest/states/a11y.toml | 46 +++--- accessible/tests/mochitest/table/a11y.toml | 40 +++--- accessible/tests/mochitest/text/a11y.toml | 31 ++-- .../tests/mochitest/textattrs/a11y.toml | 12 +- .../tests/mochitest/textcaret/a11y.toml | 2 +- .../tests/mochitest/textrange/a11y.toml | 5 +- .../tests/mochitest/textselection/a11y.toml | 4 +- accessible/tests/mochitest/tree/a11y.toml | 92 ++++++------ .../tests/mochitest/treeupdate/a11y.toml | 87 ++++++----- accessible/tests/mochitest/value/a11y.toml | 14 +- browser/base/content/test/forms/browser.toml | 4 +- .../base/content/test/general/browser.toml | 24 ++-- .../content/test/siteIdentity/browser.toml | 14 +- browser/base/content/test/tabs/browser.toml | 4 +- .../test/browser/browser.toml | 4 +- .../components/doh/test/browser/browser.toml | 4 +- .../newtab/test/browser/browser.toml | 4 +- .../newtab/test/xpcshell/xpcshell.toml | 2 +- .../places/tests/chrome/chrome.toml | 4 +- .../components/preferences/tests/browser.toml | 16 +-- .../test/browser/browser.toml | 16 +-- .../components/sessionstore/test/browser.toml | 2 +- .../translations/tests/browser/browser.toml | 10 +- browser/components/uitour/test/browser.toml | 3 +- .../urlbar/tests/browser/browser.toml | 12 +- .../tests/quicksuggest/browser/browser.toml | 6 +- .../formautofill/test/unit/xpcshell.toml | 6 +- .../screenshots/test/browser/browser.toml | 3 +- devtools/client/netmonitor/test/browser.toml | 6 +- devtools/client/styleeditor/test/browser.toml | 14 +- .../server/tracer/tests/browser/browser.toml | 4 +- .../test/browser/browser.toml | 7 +- docs/code-quality/lint/create.rst | 5 +- .../lint/linters/test-manifest-toml.rst | 35 +++++ docshell/test/browser/browser.toml | 118 +++++++-------- docshell/test/chrome/chrome.toml | 18 +-- docshell/test/navigation/browser.toml | 4 +- dom/base/test/browser.toml | 10 +- dom/base/test/chrome.toml | 12 +- dom/base/test/chrome/chrome.toml | 28 ++-- dom/base/test/jsmodules/chrome.toml | 4 +- .../test/jsmodules/importmaps/mochitest.toml | 4 +- dom/bindings/test/chrome.toml | 4 +- dom/events/test/chrome.toml | 16 +-- dom/html/test/browser.toml | 10 +- .../test/unit/xpcshell-parent-process.toml | 8 +- dom/media/test/mochitest_compat.toml | 3 +- .../webrtc/tests/mochitests/mochitest.toml | 12 +- dom/power/tests/mochitest.toml | 6 +- dom/push/test/mochitest.toml | 6 +- dom/security/test/general/browser.toml | 13 +- dom/tests/browser/browser.toml | 4 +- dom/tests/mochitest/chrome/chrome.toml | 8 +- dom/xul/test/chrome.toml | 14 +- editor/composer/test/chrome.toml | 4 +- editor/libeditor/tests/chrome.toml | 4 +- extensions/universalchardet/tests/chrome.toml | 16 +-- gfx/layers/apz/test/mochitest/browser.toml | 8 +- image/test/browser/browser.toml | 6 +- js/xpconnect/tests/chrome/chrome.toml | 56 ++++---- layout/base/tests/browser.toml | 123 ++++++++-------- layout/base/tests/chrome/chrome.toml | 18 +-- layout/inspector/tests/chrome/chrome.toml | 8 +- layout/style/test/chrome/chrome.toml | 10 +- layout/xul/test/chrome.toml | 4 +- netwerk/cookie/test/browser/browser.toml | 4 +- netwerk/test/mochitests/mochitest.toml | 10 +- testing/cppunittest.toml | 5 +- .../browser_chrome/browser.toml | 4 +- .../baselinecoverage/chrome/chrome.toml | 2 +- .../baselinecoverage/plain/mochitest.toml | 2 +- testing/mochitest/chrome/chrome.toml | 30 ++-- testing/mochitest/manifests/emulator-jb.toml | 4 +- .../tests/Harness_sanity/mochitest.toml | 64 ++++----- .../tests/Harness_sanity/xpcshell.toml | 6 +- .../manifestparser/manifestparser/toml.py | 52 ++++--- testing/mozbase/mozhttpd/tests/manifest.toml | 4 +- .../moztest/tests/data/srcdir/apple/a11y.toml | 4 +- .../tests/xpcshell/xpcshell.toml | 6 +- .../cookiebanners/test/browser/browser.toml | 12 +- .../passwordmgr/test/mochitest/mochitest.toml | 8 +- .../components/printing/tests/browser.toml | 4 +- .../tests/browser/browser.toml | 8 +- .../translations/tests/browser/browser.toml | 4 +- tools/lint/test-manifest-toml.yml | 32 +++++ tools/lint/test-manifest-toml/__init__.py | 135 ++++++++++++++++++ .../test-manifest-toml/comment-section.toml | 5 + .../files/test-manifest-toml/invalid.toml | 2 + .../files/test-manifest-toml/no-default.toml | 1 + .../non-double-quote-sections.toml | 6 + .../skip-if-explicit-or.toml | 7 + .../test-manifest-toml/skip-if-not-array.toml | 4 + .../files/test-manifest-toml/unsorted.toml | 10 ++ .../test/files/test-manifest-toml/valid.toml | 2 + tools/lint/test/python.toml | 2 + tools/lint/test/test_manifest_toml.py | 78 ++++++++++ widget/tests/browser/browser.toml | 4 +- 114 files changed, 1125 insertions(+), 789 deletions(-) create mode 100644 docs/code-quality/lint/linters/test-manifest-toml.rst create mode 100644 tools/lint/test-manifest-toml.yml create mode 100644 tools/lint/test-manifest-toml/__init__.py create mode 100644 tools/lint/test/files/test-manifest-toml/comment-section.toml create mode 100644 tools/lint/test/files/test-manifest-toml/invalid.toml create mode 100644 tools/lint/test/files/test-manifest-toml/no-default.toml create mode 100644 tools/lint/test/files/test-manifest-toml/non-double-quote-sections.toml create mode 100644 tools/lint/test/files/test-manifest-toml/skip-if-explicit-or.toml create mode 100644 tools/lint/test/files/test-manifest-toml/skip-if-not-array.toml create mode 100644 tools/lint/test/files/test-manifest-toml/unsorted.toml create mode 100644 tools/lint/test/files/test-manifest-toml/valid.toml create mode 100644 tools/lint/test/test_manifest_toml.py diff --git a/accessible/tests/mochitest/a11y.toml b/accessible/tests/mochitest/a11y.toml index e90ca4b4eb7a..9125e244d217 100644 --- a/accessible/tests/mochitest/a11y.toml +++ b/accessible/tests/mochitest/a11y.toml @@ -12,16 +12,16 @@ support-files = [ "*.js", "treeview.css"] -['test_OuterDocAccessible.html'] +["test_OuterDocAccessible.html"] -['test_aria_token_attrs.html'] +["test_aria_token_attrs.html"] -['test_bug420863.html'] +["test_bug420863.html"] -['test_custom_element_accessibility_defaults.html'] +["test_custom_element_accessibility_defaults.html"] -['test_descr.html'] +["test_descr.html"] -['test_nsIAccessibleDocument.html'] +["test_nsIAccessibleDocument.html"] -['test_nsIAccessibleImage.html'] +["test_nsIAccessibleImage.html"] diff --git a/accessible/tests/mochitest/actions/a11y.toml b/accessible/tests/mochitest/actions/a11y.toml index 05e631883eb0..30a98d40ac42 100644 --- a/accessible/tests/mochitest/actions/a11y.toml +++ b/accessible/tests/mochitest/actions/a11y.toml @@ -3,25 +3,24 @@ support-files = [ "!/accessible/tests/mochitest/*.js", "!/dom/media/test/bug461281.ogg"] -['test_anchors.html'] +["test_anchors.html"] -['test_aria.html'] +["test_aria.html"] -['test_controls.html'] +["test_controls.html"] -['test_general.html'] +["test_general.html"] -['test_general.xhtml'] +["test_general.xhtml"] -['test_keys.html'] +["test_keys.html"] -['test_keys.xhtml'] +["test_keys.xhtml"] -['test_media.html'] +["test_media.html"] -['test_select.html'] +["test_select.html"] -['test_tree.xhtml'] - -['test_treegrid.xhtml'] +["test_tree.xhtml"] +["test_treegrid.xhtml"] diff --git a/accessible/tests/mochitest/aom/a11y.toml b/accessible/tests/mochitest/aom/a11y.toml index bfdecfc356c6..08ee9e476eec 100644 --- a/accessible/tests/mochitest/aom/a11y.toml +++ b/accessible/tests/mochitest/aom/a11y.toml @@ -1,4 +1,3 @@ [DEFAULT] -['test_general.html'] - +["test_general.html"] diff --git a/accessible/tests/mochitest/attributes/a11y.toml b/accessible/tests/mochitest/attributes/a11y.toml index 3faeaf6851e2..7ea2d1824f4e 100644 --- a/accessible/tests/mochitest/attributes/a11y.toml +++ b/accessible/tests/mochitest/attributes/a11y.toml @@ -1,22 +1,22 @@ [DEFAULT] support-files = "!/accessible/tests/mochitest/*.js" -['test_dpub_aria_xml-roles.html'] +["test_dpub_aria_xml-roles.html"] -['test_graphics_aria_xml-roles.html'] +["test_graphics_aria_xml-roles.html"] -['test_listbox.html'] +["test_listbox.html"] -['test_obj.html'] +["test_obj.html"] -['test_obj_css.html'] +["test_obj_css.html"] -['test_obj_group.html'] +["test_obj_group.html"] -['test_obj_group.xhtml'] +["test_obj_group.xhtml"] -['test_obj_group_tree.xhtml'] +["test_obj_group_tree.xhtml"] -['test_tag.html'] +["test_tag.html"] -['test_xml-roles.html'] +["test_xml-roles.html"] diff --git a/accessible/tests/mochitest/bounds/a11y.toml b/accessible/tests/mochitest/bounds/a11y.toml index 494833c1885c..9e8fc860453f 100644 --- a/accessible/tests/mochitest/bounds/a11y.toml +++ b/accessible/tests/mochitest/bounds/a11y.toml @@ -1,4 +1,4 @@ [DEFAULT] support-files = "!/accessible/tests/mochitest/*.js" -['test_list.html'] +["test_list.html"] diff --git a/accessible/tests/mochitest/elm/a11y.toml b/accessible/tests/mochitest/elm/a11y.toml index 55c6efc82096..0559fbf55bd4 100644 --- a/accessible/tests/mochitest/elm/a11y.toml +++ b/accessible/tests/mochitest/elm/a11y.toml @@ -5,16 +5,15 @@ support-files = [ "!/dom/media/test/bug461281.ogg", "!/dom/security/test/csp/dummy.pdf"] -['test_HTMLSpec.html'] +["test_HTMLSpec.html"] -['test_MathMLSpec.html'] +["test_MathMLSpec.html"] -['test_figure.html'] +["test_figure.html"] -['test_listbox.xhtml'] +["test_listbox.xhtml"] -['test_nsApplicationAcc.html'] +["test_nsApplicationAcc.html"] -['test_shadowroot.html'] +["test_shadowroot.html"] support-files = "test_shadowroot_subframe.html" - diff --git a/accessible/tests/mochitest/events/a11y.toml b/accessible/tests/mochitest/events/a11y.toml index 72003176484b..501b59f7b715 100644 --- a/accessible/tests/mochitest/events/a11y.toml +++ b/accessible/tests/mochitest/events/a11y.toml @@ -8,121 +8,121 @@ support-files = [ "!/image/test/mochitest/animated-gif-finalframe.gif", "!/image/test/mochitest/animated-gif.gif"] -['test_announcement.html'] +["test_announcement.html"] -['test_aria_alert.html'] +["test_aria_alert.html"] -['test_aria_menu.html'] +["test_aria_menu.html"] -['test_aria_objattr.html'] +["test_aria_objattr.html"] -['test_aria_owns.html'] +["test_aria_owns.html"] -['test_aria_statechange.html'] +["test_aria_statechange.html"] -['test_attrchange.html'] +["test_attrchange.html"] -['test_attrs.html'] +["test_attrs.html"] -['test_bug1322593-2.html'] +["test_bug1322593-2.html"] -['test_bug1322593.html'] +["test_bug1322593.html"] -['test_caretmove.html'] +["test_caretmove.html"] -['test_coalescence.html'] +["test_coalescence.html"] -['test_contextmenu.html'] +["test_contextmenu.html"] -['test_descrchange.html'] +["test_descrchange.html"] -['test_dragndrop.html'] +["test_dragndrop.html"] -['test_flush.html'] +["test_flush.html"] -['test_focus_aria_activedescendant.html'] +["test_focus_aria_activedescendant.html"] -['test_focus_autocomplete.html'] +["test_focus_autocomplete.html"] -['test_focus_autocomplete.xhtml'] +["test_focus_autocomplete.xhtml"] # Disabled on Linux and Windows due to frequent failures - bug 695019, bug 890795 skip-if = [ "os == 'win'", "os == 'linux'", ] -['test_focus_canvas.html'] +["test_focus_canvas.html"] -['test_focus_contextmenu.xhtml'] +["test_focus_contextmenu.xhtml"] -['test_focus_controls.html'] +["test_focus_controls.html"] -['test_focus_doc.html'] +["test_focus_doc.html"] -['test_focus_general.html'] +["test_focus_general.html"] -['test_focus_general.xhtml'] +["test_focus_general.xhtml"] -['test_focus_listcontrols.xhtml'] +["test_focus_listcontrols.xhtml"] -['test_focus_menu.xhtml'] +["test_focus_menu.xhtml"] -['test_focus_name.html'] +["test_focus_name.html"] -['test_focus_removal.html'] +["test_focus_removal.html"] -['test_focus_selects.html'] +["test_focus_selects.html"] -['test_focus_tabbox.xhtml'] -skip-if = true +["test_focus_tabbox.xhtml"] +skip-if = ["true"] -['test_focus_tree.xhtml'] +["test_focus_tree.xhtml"] -['test_focusable_statechange.html'] +["test_focusable_statechange.html"] -['test_fromUserInput.html'] +["test_fromUserInput.html"] -['test_label.xhtml'] +["test_label.xhtml"] -['test_menu.xhtml'] +["test_menu.xhtml"] -['test_mutation.html'] +["test_mutation.html"] -['test_namechange.html'] +["test_namechange.html"] -['test_namechange.xhtml'] +["test_namechange.xhtml"] -['test_scroll.xhtml'] +["test_scroll.xhtml"] -['test_scroll_caret.xhtml'] +["test_scroll_caret.xhtml"] -['test_selection.html'] +["test_selection.html"] skip-if = [ "os == 'mac'", ] -['test_selection.xhtml'] +["test_selection.xhtml"] skip-if = [ "os == 'mac'", ] -['test_selection_aria.html'] +["test_selection_aria.html"] -['test_statechange.html'] +["test_statechange.html"] -['test_statechange.xhtml'] +["test_statechange.xhtml"] -['test_text.html'] +["test_text.html"] -['test_text_alg.html'] +["test_text_alg.html"] -['test_textattrchange.html'] +["test_textattrchange.html"] -['test_textselchange.html'] +["test_textselchange.html"] -['test_tree.xhtml'] +["test_tree.xhtml"] -['test_valuechange.html'] +["test_valuechange.html"] skip-if = [ "os == 'mac'", ] diff --git a/accessible/tests/mochitest/events/docload/a11y.toml b/accessible/tests/mochitest/events/docload/a11y.toml index 77121a037be2..dc718177252c 100644 --- a/accessible/tests/mochitest/events/docload/a11y.toml +++ b/accessible/tests/mochitest/events/docload/a11y.toml @@ -3,18 +3,18 @@ support-files = [ "docload_wnd.html", "!/accessible/tests/mochitest/*.js"] -['test_docload_aria.html'] +["test_docload_aria.html"] -['test_docload_busy.html'] +["test_docload_busy.html"] -['test_docload_embedded.html'] +["test_docload_embedded.html"] -['test_docload_iframe.html'] +["test_docload_iframe.html"] -['test_docload_root.html'] +["test_docload_root.html"] skip-if = ["os == 'mac'"] # bug 1456997 -['test_docload_shutdown.html'] +["test_docload_shutdown.html"] skip-if = [ "os == 'mac'", # bug 1456997 "display == 'wayland'", # bug 1850412 diff --git a/accessible/tests/mochitest/focus/a11y.toml b/accessible/tests/mochitest/focus/a11y.toml index ded877a7f7ce..fd33c080e715 100644 --- a/accessible/tests/mochitest/focus/a11y.toml +++ b/accessible/tests/mochitest/focus/a11y.toml @@ -1,10 +1,10 @@ [DEFAULT] support-files = "!/accessible/tests/mochitest/*.js" -['test_focus_radio.xhtml'] +["test_focus_radio.xhtml"] -['test_focusedChild.html'] +["test_focusedChild.html"] -['test_takeFocus.html'] +["test_takeFocus.html"] -['test_takeFocus.xhtml'] +["test_takeFocus.xhtml"] diff --git a/accessible/tests/mochitest/hittest/a11y.toml b/accessible/tests/mochitest/hittest/a11y.toml index bfab9ab28c5a..b17ffbc28904 100644 --- a/accessible/tests/mochitest/hittest/a11y.toml +++ b/accessible/tests/mochitest/hittest/a11y.toml @@ -3,18 +3,17 @@ support-files = [ "zoom_tree.xhtml", "!/accessible/tests/mochitest/*.js", "!/accessible/tests/mochitest/letters.gif"] -['test_browser.html'] +["test_browser.html"] -['test_general.html'] +["test_general.html"] -['test_menu.xhtml'] +["test_menu.xhtml"] -['test_shadowroot.html'] +["test_shadowroot.html"] support-files = "test_shadowroot_subframe.html" -['test_zoom.html'] +["test_zoom.html"] -['test_zoom_text.html'] - -['test_zoom_tree.xhtml'] +["test_zoom_text.html"] +["test_zoom_tree.xhtml"] diff --git a/accessible/tests/mochitest/hyperlink/a11y.toml b/accessible/tests/mochitest/hyperlink/a11y.toml index 165fa8058782..77f551ee5d72 100644 --- a/accessible/tests/mochitest/hyperlink/a11y.toml +++ b/accessible/tests/mochitest/hyperlink/a11y.toml @@ -3,7 +3,6 @@ support-files = [ "hyperlink.js", "!/accessible/tests/mochitest/*.js", "!/accessible/tests/mochitest/letters.gif"] -['test_general.html'] - -['test_general.xhtml'] +["test_general.html"] +["test_general.xhtml"] diff --git a/accessible/tests/mochitest/hypertext/a11y.toml b/accessible/tests/mochitest/hypertext/a11y.toml index 5322a196377e..04370669ae2b 100644 --- a/accessible/tests/mochitest/hypertext/a11y.toml +++ b/accessible/tests/mochitest/hypertext/a11y.toml @@ -3,7 +3,6 @@ support-files = [ "!/accessible/tests/mochitest/*.js", "!/accessible/tests/mochitest/letters.gif"] -['test_general.html'] - -['test_update.html'] +["test_general.html"] +["test_update.html"] diff --git a/accessible/tests/mochitest/name/a11y.toml b/accessible/tests/mochitest/name/a11y.toml index c49be6735cf6..8faf1b1c45df 100644 --- a/accessible/tests/mochitest/name/a11y.toml +++ b/accessible/tests/mochitest/name/a11y.toml @@ -5,25 +5,25 @@ support-files = [ "!/accessible/tests/mochitest/*.js", "!/accessible/tests/mochitest/moz.png"] -['test_ARIACore_examples.html'] +["test_ARIACore_examples.html"] -['test_browserui.xhtml'] +["test_browserui.xhtml"] -['test_counterstyle.html'] +["test_counterstyle.html"] -['test_general.html'] +["test_general.html"] -['test_general.xhtml'] +["test_general.xhtml"] -['test_link.html'] +["test_link.html"] -['test_list.html'] +["test_list.html"] -['test_markup.html'] +["test_markup.html"] skip-if = [ "win11_2009 && debug", # Bug 1296784 ] -['test_svg.html'] +["test_svg.html"] -['test_tree.xhtml'] +["test_tree.xhtml"] diff --git a/accessible/tests/mochitest/relations/a11y.toml b/accessible/tests/mochitest/relations/a11y.toml index f7d3c446ca1f..ae41bf91a593 100644 --- a/accessible/tests/mochitest/relations/a11y.toml +++ b/accessible/tests/mochitest/relations/a11y.toml @@ -1,23 +1,23 @@ [DEFAULT] support-files = "!/accessible/tests/mochitest/*.js" -['test_embeds.xhtml'] +["test_embeds.xhtml"] skip-if = [ "os == 'linux' && !debug", # bug 1411145 ] -['test_general.html'] +["test_general.html"] -['test_general.xhtml'] +["test_general.xhtml"] -['test_groupInfoUpdate.html'] +["test_groupInfoUpdate.html"] -['test_shadowdom.html'] +["test_shadowdom.html"] -['test_tabbrowser.xhtml'] +["test_tabbrowser.xhtml"] -['test_tree.xhtml'] +["test_tree.xhtml"] -['test_ui_modalprompt.html'] +["test_ui_modalprompt.html"] -['test_update.html'] +["test_update.html"] diff --git a/accessible/tests/mochitest/role/a11y.toml b/accessible/tests/mochitest/role/a11y.toml index d93841959874..8febfc3bd71e 100644 --- a/accessible/tests/mochitest/role/a11y.toml +++ b/accessible/tests/mochitest/role/a11y.toml @@ -4,17 +4,16 @@ support-files = [ "!/accessible/tests/mochitest/*.js", "!/accessible/tests/mochitest/moz.png"] -['test_aria.html'] +["test_aria.html"] -['test_aria.xhtml'] +["test_aria.xhtml"] -['test_dpub_aria.html'] +["test_dpub_aria.html"] -['test_general.html'] +["test_general.html"] -['test_general.xhtml'] +["test_general.xhtml"] -['test_graphics_aria.html'] - -['test_svg.html'] +["test_graphics_aria.html"] +["test_svg.html"] diff --git a/accessible/tests/mochitest/scroll/a11y.toml b/accessible/tests/mochitest/scroll/a11y.toml index 89849050fdfe..66a0aadf28ed 100644 --- a/accessible/tests/mochitest/scroll/a11y.toml +++ b/accessible/tests/mochitest/scroll/a11y.toml @@ -1,4 +1,4 @@ [DEFAULT] support-files = "!/accessible/tests/mochitest/*.js" -['test_zoom.html'] +["test_zoom.html"] diff --git a/accessible/tests/mochitest/selectable/a11y.toml b/accessible/tests/mochitest/selectable/a11y.toml index b6d7cd40d524..4cb700ba7364 100644 --- a/accessible/tests/mochitest/selectable/a11y.toml +++ b/accessible/tests/mochitest/selectable/a11y.toml @@ -3,13 +3,12 @@ support-files = [ "!/accessible/tests/mochitest/*.js", "!/accessible/tests/mochitest/treeview.css"] -['test_listbox.xhtml'] +["test_listbox.xhtml"] -['test_menu.xhtml'] +["test_menu.xhtml"] -['test_menulist.xhtml'] +["test_menulist.xhtml"] -['test_tabs.xhtml'] - -['test_tree.xhtml'] +["test_tabs.xhtml"] +["test_tree.xhtml"] diff --git a/accessible/tests/mochitest/states/a11y.toml b/accessible/tests/mochitest/states/a11y.toml index b1a39be9eb21..f92526226d66 100644 --- a/accessible/tests/mochitest/states/a11y.toml +++ b/accessible/tests/mochitest/states/a11y.toml @@ -10,51 +10,51 @@ support-files = [ "!/accessible/tests/mochitest/formimage.png", "!/accessible/tests/mochitest/treeview.css"] -['test_aria.html'] +["test_aria.html"] -['test_aria.xhtml'] +["test_aria.xhtml"] -['test_aria_imgmap.html'] +["test_aria_imgmap.html"] -['test_aria_widgetitems.html'] +["test_aria_widgetitems.html"] -['test_buttons.html'] +["test_buttons.html"] -['test_controls.html'] +["test_controls.html"] -['test_controls.xhtml'] +["test_controls.xhtml"] -['test_doc.html'] +["test_doc.html"] -['test_doc_busy.html'] +["test_doc_busy.html"] -['test_docarticle.html'] +["test_docarticle.html"] -['test_editablebody.html'] +["test_editablebody.html"] -['test_expandable.xhtml'] +["test_expandable.xhtml"] -['test_frames.html'] +["test_frames.html"] -['test_inputs.html'] +["test_inputs.html"] -['test_link.html'] +["test_link.html"] -['test_popup.xhtml'] +["test_popup.xhtml"] -['test_selects.html'] +["test_selects.html"] -['test_stale.html'] +["test_stale.html"] -['test_tabs.xhtml'] +["test_tabs.xhtml"] -['test_textbox.xhtml'] +["test_textbox.xhtml"] -['test_tree.xhtml'] +["test_tree.xhtml"] -['test_visibility.html'] +["test_visibility.html"] -['test_visibility.xhtml'] +["test_visibility.xhtml"] skip-if = [ "asan", # Bug 1199631 ] diff --git a/accessible/tests/mochitest/table/a11y.toml b/accessible/tests/mochitest/table/a11y.toml index ed3495f5f94a..2250bf85c915 100644 --- a/accessible/tests/mochitest/table/a11y.toml +++ b/accessible/tests/mochitest/table/a11y.toml @@ -1,42 +1,42 @@ [DEFAULT] support-files = "!/accessible/tests/mochitest/*.js" -['test_css_tables.html'] +["test_css_tables.html"] -['test_headers_ariagrid.html'] +["test_headers_ariagrid.html"] -['test_headers_ariatable.html'] +["test_headers_ariatable.html"] -['test_headers_table.html'] +["test_headers_table.html"] -['test_headers_tree.xhtml'] +["test_headers_tree.xhtml"] -['test_indexes_ariagrid.html'] +["test_indexes_ariagrid.html"] -['test_indexes_table.html'] +["test_indexes_table.html"] -['test_indexes_tree.xhtml'] +["test_indexes_tree.xhtml"] -['test_layoutguess.html'] +["test_layoutguess.html"] -['test_mtable.html'] +["test_mtable.html"] -['test_sels_ariagrid.html'] +["test_sels_ariagrid.html"] -['test_sels_table.html'] +["test_sels_table.html"] -['test_sels_tree.xhtml'] +["test_sels_tree.xhtml"] -['test_struct_ariagrid.html'] +["test_struct_ariagrid.html"] -['test_struct_ariatreegrid.html'] +["test_struct_ariatreegrid.html"] -['test_struct_table.html'] +["test_struct_table.html"] -['test_struct_tree.xhtml'] +["test_struct_tree.xhtml"] -['test_table_1.html'] +["test_table_1.html"] -['test_table_2.html'] +["test_table_2.html"] -['test_table_mutation.html'] +["test_table_mutation.html"] diff --git a/accessible/tests/mochitest/text/a11y.toml b/accessible/tests/mochitest/text/a11y.toml index ba73dbce8b7e..740073611bc5 100644 --- a/accessible/tests/mochitest/text/a11y.toml +++ b/accessible/tests/mochitest/text/a11y.toml @@ -2,33 +2,32 @@ support-files = [ "doc.html", "!/accessible/tests/mochitest/*.js"] -['test_atcaretoffset.html'] +["test_atcaretoffset.html"] -['test_charboundary.html'] +["test_charboundary.html"] -['test_doc.html'] +["test_doc.html"] -['test_dynamic.html'] +["test_dynamic.html"] -['test_general.xhtml'] +["test_general.xhtml"] -['test_gettext.html'] +["test_gettext.html"] -['test_hypertext.html'] +["test_hypertext.html"] -['test_lineboundary.html'] +["test_lineboundary.html"] -['test_paragraphboundary.html'] +["test_paragraphboundary.html"] -['test_passwords.html'] +["test_passwords.html"] -['test_selection.html'] +["test_selection.html"] -['test_settext_input_event.html'] +["test_settext_input_event.html"] -['test_textBounds.html'] +["test_textBounds.html"] -['test_wordboundary.html'] - -['test_words.html'] +["test_wordboundary.html"] +["test_words.html"] diff --git a/accessible/tests/mochitest/textattrs/a11y.toml b/accessible/tests/mochitest/textattrs/a11y.toml index ffde26d6b2ab..795f1fa9b7eb 100644 --- a/accessible/tests/mochitest/textattrs/a11y.toml +++ b/accessible/tests/mochitest/textattrs/a11y.toml @@ -4,14 +4,14 @@ support-files = [ "!/accessible/tests/mochitest/*.js", "!/accessible/tests/mochitest/moz.png"] -['test_general.html'] +["test_general.html"] -['test_general.xhtml'] +["test_general.xhtml"] -['test_invalid.html'] +["test_invalid.html"] -['test_mathml.html'] +["test_mathml.html"] -['test_spelling.html'] +["test_spelling.html"] -['test_svg.html'] +["test_svg.html"] diff --git a/accessible/tests/mochitest/textcaret/a11y.toml b/accessible/tests/mochitest/textcaret/a11y.toml index 1b7dba2e0bd5..d89b87e1cf7d 100644 --- a/accessible/tests/mochitest/textcaret/a11y.toml +++ b/accessible/tests/mochitest/textcaret/a11y.toml @@ -1,4 +1,4 @@ [DEFAULT] support-files = "!/accessible/tests/mochitest/*.js" -['test_general.html'] +["test_general.html"] diff --git a/accessible/tests/mochitest/textrange/a11y.toml b/accessible/tests/mochitest/textrange/a11y.toml index 5cdfa9f5c075..c7d4f270e406 100644 --- a/accessible/tests/mochitest/textrange/a11y.toml +++ b/accessible/tests/mochitest/textrange/a11y.toml @@ -3,7 +3,6 @@ support-files = [ "!/accessible/tests/mochitest/*.js", "!/accessible/tests/mochitest/moz.png"] -['test_general.html'] - -['test_selection.html'] +["test_general.html"] +["test_selection.html"] diff --git a/accessible/tests/mochitest/textselection/a11y.toml b/accessible/tests/mochitest/textselection/a11y.toml index f4d57ab26732..b87962bc4133 100644 --- a/accessible/tests/mochitest/textselection/a11y.toml +++ b/accessible/tests/mochitest/textselection/a11y.toml @@ -1,6 +1,6 @@ [DEFAULT] support-files = "!/accessible/tests/mochitest/*.js" -['test_general.html'] +["test_general.html"] -['test_userinput.html'] +["test_userinput.html"] diff --git a/accessible/tests/mochitest/tree/a11y.toml b/accessible/tests/mochitest/tree/a11y.toml index f364e2ec2c8d..e9181c752e9b 100644 --- a/accessible/tests/mochitest/tree/a11y.toml +++ b/accessible/tests/mochitest/tree/a11y.toml @@ -9,97 +9,97 @@ support-files = [ "!/accessible/tests/mochitest/tree/wnd.xhtml", "!/dom/media/test/bug461281.ogg"] -['test_applicationacc.xhtml'] -skip-if = true # Bug 561508 +["test_applicationacc.xhtml"] +skip-if = ["true"] # Bug 561508 -['test_aria_display_contents.html'] +["test_aria_display_contents.html"] -['test_aria_globals.html'] +["test_aria_globals.html"] -['test_aria_grid.html'] +["test_aria_grid.html"] -['test_aria_imgmap.html'] +["test_aria_imgmap.html"] -['test_aria_list.html'] +["test_aria_list.html"] -['test_aria_menu.html'] +["test_aria_menu.html"] -['test_aria_owns.html'] +["test_aria_owns.html"] -['test_aria_presentation.html'] +["test_aria_presentation.html"] -['test_aria_table.html'] +["test_aria_table.html"] -['test_brokencontext.html'] +["test_brokencontext.html"] -['test_button.xhtml'] +["test_button.xhtml"] -['test_canvas.html'] +["test_canvas.html"] -['test_combobox.xhtml'] +["test_combobox.xhtml"] -['test_cssflexbox.html'] +["test_cssflexbox.html"] -['test_cssoverflow.html'] +["test_cssoverflow.html"] -['test_display_contents.html'] +["test_display_contents.html"] -['test_divs.html'] +["test_divs.html"] -['test_dochierarchy.html'] +["test_dochierarchy.html"] -['test_dockids.html'] +["test_dockids.html"] -['test_filectrl.html'] +["test_filectrl.html"] -['test_formctrl.html'] +["test_formctrl.html"] -['test_formctrl.xhtml'] +["test_formctrl.xhtml"] -['test_gencontent.html'] +["test_gencontent.html"] -['test_groupbox.xhtml'] +["test_groupbox.xhtml"] -['test_html_in_mathml.html'] +["test_html_in_mathml.html"] -['test_iframe.html'] +["test_iframe.html"] -['test_image.xhtml'] +["test_image.xhtml"] -['test_img.html'] +["test_img.html"] -['test_invalid_img.xhtml'] +["test_invalid_img.xhtml"] -['test_invalidationlist.html'] +["test_invalidationlist.html"] -['test_list.html'] +["test_list.html"] -['test_map.html'] +["test_map.html"] -['test_media.html'] +["test_media.html"] -['test_select.html'] +["test_select.html"] -['test_svg.html'] +["test_svg.html"] -['test_tabbox.xhtml'] +["test_tabbox.xhtml"] -['test_tabbrowser.xhtml'] +["test_tabbrowser.xhtml"] skip-if = [ "os == 'linux' && debug", # Bug 1389365 "os == 'win' && ccov", # Bug 1423218 ] -['test_table.html'] +["test_table.html"] -['test_table_2.html'] +["test_table_2.html"] -['test_table_3.html'] +["test_table_3.html"] -['test_tree.xhtml'] +["test_tree.xhtml"] -['test_txtcntr.html'] +["test_txtcntr.html"] -['test_txtctrl.html'] +["test_txtctrl.html"] -['test_txtctrl.xhtml'] +["test_txtctrl.xhtml"] diff --git a/accessible/tests/mochitest/treeupdate/a11y.toml b/accessible/tests/mochitest/treeupdate/a11y.toml index 4eead6d23a8f..b98af5388e77 100644 --- a/accessible/tests/mochitest/treeupdate/a11y.toml +++ b/accessible/tests/mochitest/treeupdate/a11y.toml @@ -4,82 +4,81 @@ support-files = [ "!/accessible/tests/mochitest/letters.gif", "!/accessible/tests/mochitest/moz.png"] -['test_ariadialog.html'] +["test_ariadialog.html"] -['test_ariahidden.html'] +["test_ariahidden.html"] -['test_ariaowns.html'] +["test_ariaowns.html"] -['test_bug1040735.html'] +["test_bug852150.xhtml"] -['test_bug1175913.html'] +["test_bug883708.xhtml"] -['test_bug1189277.html'] +["test_bug884251.xhtml"] -['test_bug1276857.html'] +["test_bug895082.html"] + +["test_bug1040735.html"] + +["test_bug1175913.html"] + +["test_bug1189277.html"] + +["test_bug1276857.html"] support-files = "test_bug1276857_subframe.html" -['test_bug852150.xhtml'] +["test_canvas.html"] -['test_bug883708.xhtml'] +["test_contextmenu.xhtml"] -['test_bug884251.xhtml'] +["test_cssoverflow.html"] -['test_bug895082.html'] +["test_deck.xhtml"] -['test_canvas.html'] +["test_delayed_removal.html"] -['test_contextmenu.xhtml'] +["test_doc.html"] -['test_cssoverflow.html'] +["test_gencontent.html"] -['test_deck.xhtml'] +["test_general.html"] -['test_delayed_removal.html'] +["test_hidden.html"] -['test_doc.html'] +["test_imagemap.html"] -['test_gencontent.html'] +["test_inert.html"] -['test_general.html'] +["test_inner_reorder.html"] -['test_hidden.html'] +["test_list.html"] -['test_imagemap.html'] +["test_list_editabledoc.html"] -['test_inert.html'] +["test_list_style.html"] -['test_inner_reorder.html'] +["test_listbox.xhtml"] -['test_list.html'] +["test_menu.xhtml"] -['test_list_editabledoc.html'] +["test_menubutton.xhtml"] -['test_list_style.html'] +["test_optgroup.html"] -['test_listbox.xhtml'] +["test_recreation.html"] -['test_menu.xhtml'] +["test_select.html"] -['test_menubutton.xhtml'] +["test_shadow_slots.html"] -['test_optgroup.html'] +["test_shutdown.xhtml"] -['test_recreation.html'] +["test_table.html"] -['test_select.html'] +["test_textleaf.html"] -['test_shadow_slots.html'] +["test_tooltip.xhtml"] -['test_shutdown.xhtml'] - -['test_table.html'] - -['test_textleaf.html'] - -['test_tooltip.xhtml'] - -['test_visibility.html'] - -['test_whitespace.html'] +["test_visibility.html"] +["test_whitespace.html"] diff --git a/accessible/tests/mochitest/value/a11y.toml b/accessible/tests/mochitest/value/a11y.toml index 9406b2e31e8e..c64d23d8cb44 100644 --- a/accessible/tests/mochitest/value/a11y.toml +++ b/accessible/tests/mochitest/value/a11y.toml @@ -1,16 +1,16 @@ [DEFAULT] support-files = "!/accessible/tests/mochitest/*.js" -['test_ariavalue.html'] +["test_ariavalue.html"] -['test_datetime.html'] +["test_datetime.html"] -['test_general.html'] +["test_general.html"] -['test_meter.html'] +["test_meter.html"] -['test_number.html'] +["test_number.html"] -['test_progress.html'] +["test_progress.html"] -['test_range.html'] +["test_range.html"] diff --git a/browser/base/content/test/forms/browser.toml b/browser/base/content/test/forms/browser.toml index 562525f2d4e8..33d73ba8bf5b 100644 --- a/browser/base/content/test/forms/browser.toml +++ b/browser/base/content/test/forms/browser.toml @@ -14,6 +14,8 @@ skip-if = ["os == 'linux'"] # Bug 1329991 - test fails intermittently on Linux b ["browser_selectpopup_dir.js"] +["browser_selectpopup_hr.js"] + ["browser_selectpopup_large.js"] ["browser_selectpopup_searchfocus.js"] @@ -30,5 +32,3 @@ fail-if = ["a11y_checks"] # Bug 1854233 input may not be labeled ["browser_selectpopup_width.js"] ["browser_selectpopup_xhtml.js"] - -["browser_selectpopup_hr.js"] diff --git a/browser/base/content/test/general/browser.toml b/browser/base/content/test/general/browser.toml index ab8ba9567f4c..6928ba2d4ba0 100644 --- a/browser/base/content/test/general/browser.toml +++ b/browser/base/content/test/general/browser.toml @@ -64,18 +64,6 @@ https_first_disabled = true https_first_disabled = true # DO NOT ADD MORE TESTS HERE. USE A TOPICAL DIRECTORY INSTEAD. -["browser_bug1261299.js"] -skip-if = ["os != 'mac'"] # Because of tests for supporting Service Menu of macOS, bug 1261299 -# DO NOT ADD MORE TESTS HERE. USE A TOPICAL DIRECTORY INSTEAD. - -["browser_bug1297539.js"] -skip-if = ["os != 'mac'"] # Because of tests for supporting pasting from Service Menu of macOS, bug 1297539 -# DO NOT ADD MORE TESTS HERE. USE A TOPICAL DIRECTORY INSTEAD. - -["browser_bug1299667.js"] -https_first_disabled = true -# DO NOT ADD MORE TESTS HERE. USE A TOPICAL DIRECTORY INSTEAD. - ["browser_bug321000.js"] skip-if = ["true"] # browser_bug321000.js is disabled because newline handling is shaky (bug 592528) # DO NOT ADD MORE TESTS HERE. USE A TOPICAL DIRECTORY INSTEAD. @@ -258,6 +246,18 @@ skip-if = ["os == 'linux' && !debug"] # Bug 1556066 ["browser_bug963945.js"] # DO NOT ADD MORE TESTS HERE. USE A TOPICAL DIRECTORY INSTEAD. +["browser_bug1261299.js"] +skip-if = ["os != 'mac'"] # Because of tests for supporting Service Menu of macOS, bug 1261299 +# DO NOT ADD MORE TESTS HERE. USE A TOPICAL DIRECTORY INSTEAD. + +["browser_bug1297539.js"] +skip-if = ["os != 'mac'"] # Because of tests for supporting pasting from Service Menu of macOS, bug 1297539 +# DO NOT ADD MORE TESTS HERE. USE A TOPICAL DIRECTORY INSTEAD. + +["browser_bug1299667.js"] +https_first_disabled = true +# DO NOT ADD MORE TESTS HERE. USE A TOPICAL DIRECTORY INSTEAD. + ["browser_clipboard.js"] https_first_disabled = true # DO NOT ADD MORE TESTS HERE. USE A TOPICAL DIRECTORY INSTEAD. diff --git a/browser/base/content/test/siteIdentity/browser.toml b/browser/base/content/test/siteIdentity/browser.toml index aff2b0f68e14..f0b6191302c5 100644 --- a/browser/base/content/test/siteIdentity/browser.toml +++ b/browser/base/content/test/siteIdentity/browser.toml @@ -9,13 +9,6 @@ support-files = [ https_first_disabled = true support-files = ["open-self-from-frame.html"] -["browser_bug1045809.js"] -tags = "mcb" -support-files = [ - "file_bug1045809_1.html", - "file_bug1045809_2.html", -] - ["browser_bug822367.js"] tags = "mcb" support-files = [ @@ -50,6 +43,13 @@ support-files = [ "file_bug906190.sjs", ] +["browser_bug1045809.js"] +tags = "mcb" +support-files = [ + "file_bug1045809_1.html", + "file_bug1045809_2.html", +] + ["browser_check_identity_state.js"] https_first_disabled = true diff --git a/browser/base/content/test/tabs/browser.toml b/browser/base/content/test/tabs/browser.toml index b395d5edacde..b810b27ee344 100644 --- a/browser/base/content/test/tabs/browser.toml +++ b/browser/base/content/test/tabs/browser.toml @@ -214,10 +214,10 @@ support-files = ["file_new_tab_page.html"] ["browser_newwindow_tabstrip_overflow.js"] -["browser_open_newtab_start_observer_notification.js"] - ["browser_openURI_background.js"] +["browser_open_newtab_start_observer_notification.js"] + ["browser_opened_file_tab_navigated_to_web.js"] https_first_disabled = true diff --git a/browser/components/contextualidentity/test/browser/browser.toml b/browser/components/contextualidentity/test/browser/browser.toml index f18fba599445..242466b10d22 100644 --- a/browser/components/contextualidentity/test/browser/browser.toml +++ b/browser/components/contextualidentity/test/browser/browser.toml @@ -34,11 +34,11 @@ skip-if = ["verify"] ["browser_forgetaboutsite.js"] skip-if = ["true"] # Bug 1541885 +["browser_guessusercontext.js"] + ["browser_imageCache.js"] skip-if = ["verify && debug && os == 'win'"] -["browser_guessusercontext.js"] - ["browser_middleClick.js"] skip-if = [ "verify && debug && os == 'linux'", diff --git a/browser/components/doh/test/browser/browser.toml b/browser/components/doh/test/browser/browser.toml index 3c0605cb9ea9..0cf2d4937b89 100644 --- a/browser/components/doh/test/browser/browser.toml +++ b/browser/components/doh/test/browser/browser.toml @@ -6,6 +6,8 @@ skip-if = ["socketprocess_networking"] ["browser_dirtyEnable.js"] +["browser_doh_region.js"] + ["browser_doorhangerUserReject.js"] ["browser_platformDetection.js"] @@ -29,5 +31,3 @@ skip-if = ["win11_2009 && bits == 32"] # Bug 1713464 ["browser_trrSelection_disable.js"] ["browser_userInterference.js"] - -["browser_doh_region.js"] diff --git a/browser/components/newtab/test/browser/browser.toml b/browser/components/newtab/test/browser/browser.toml index 1c63dc9185e9..02e4ce73d89c 100644 --- a/browser/components/newtab/test/browser/browser.toml +++ b/browser/components/newtab/test/browser/browser.toml @@ -72,10 +72,10 @@ skip-if = [ "win11_2009", # Bug 1804349 ] -["browser_foxdoodle_set_default.js"] - ["browser_feature_callout_panel.js"] +["browser_foxdoodle_set_default.js"] + ["browser_getScreenshots.js"] ["browser_highlights_section.js"] diff --git a/browser/components/newtab/test/xpcshell/xpcshell.toml b/browser/components/newtab/test/xpcshell/xpcshell.toml index 22f4f4095d8a..dc1174186520 100644 --- a/browser/components/newtab/test/xpcshell/xpcshell.toml +++ b/browser/components/newtab/test/xpcshell/xpcshell.toml @@ -4,7 +4,7 @@ firefox-appdir = "browser" skip-if = ["os == 'android'"] # bug 1730213 prefs = [ "browser.startup.homepage.abouthome_cache.enabled=true", - "browser.startup.homepage.abouthome_cache.testing=true", + "browser.startup.homepage.abouthome_cache.testing=true", ] ["test_ASRouterTargeting_attribution.js"] diff --git a/browser/components/places/tests/chrome/chrome.toml b/browser/components/places/tests/chrome/chrome.toml index 349b8055cc2c..ca953fe89823 100644 --- a/browser/components/places/tests/chrome/chrome.toml +++ b/browser/components/places/tests/chrome/chrome.toml @@ -3,12 +3,12 @@ support-files = ["head.js"] ["test_0_bug510634.xhtml"] -["test_bug1163447_selectItems_through_shortcut.xhtml"] - ["test_bug549192.xhtml"] ["test_bug549491.xhtml"] +["test_bug1163447_selectItems_through_shortcut.xhtml"] + ["test_selectItems_on_nested_tree.xhtml"] ["test_treeview_date.xhtml"] diff --git a/browser/components/preferences/tests/browser.toml b/browser/components/preferences/tests/browser.toml index 17d5a6cfb30c..fe523bde94fe 100644 --- a/browser/components/preferences/tests/browser.toml +++ b/browser/components/preferences/tests/browser.toml @@ -26,6 +26,12 @@ skip-if = [ "!debug && os == 'win'", # Bug 1518370 ] +["browser_bug410900.js"] + +["browser_bug731866.js"] + +["browser_bug795764_cachedisabled.js"] + ["browser_bug1018066_resetScrollPosition.js"] ["browser_bug1020245_openPreferences_to_paneContent.js"] @@ -40,12 +46,6 @@ support-files = ["browser_bug1184989_prevent_scrolling_when_preferences_flipped. ["browser_bug1579418.js"] -["browser_bug410900.js"] - -["browser_bug731866.js"] - -["browser_bug795764_cachedisabled.js"] - ["browser_cert_export.js"] ["browser_change_app_handler.js"] @@ -55,6 +55,8 @@ run-if = ["os == 'win'"] # Windows-specific handler application selection dialog ["browser_connection.js"] +["browser_connection_bug388287.js"] + ["browser_connection_bug1445991.js"] ["browser_connection_bug1505330.js"] @@ -63,8 +65,6 @@ skip-if = [ "verify && debug && os == 'mac'", ] -["browser_connection_bug388287.js"] - ["browser_connection_valid_hostname.js"] ["browser_containers_name_input.js"] diff --git a/browser/components/reportbrokensite/test/browser/browser.toml b/browser/components/reportbrokensite/test/browser/browser.toml index 1bd1a6f78540..09e4b7207942 100644 --- a/browser/components/reportbrokensite/test/browser/browser.toml +++ b/browser/components/reportbrokensite/test/browser/browser.toml @@ -9,12 +9,16 @@ support-files = [ ["browser_antitracking_data_sent.js"] support-files = [ "send_more_info.js" ] +["browser_back_buttons.js"] + +["browser_error_messages.js"] + ["browser_keyboard_navigation.js"] -["browser_tab_key_order.js"] - ["browser_parent_menuitems.js"] +["browser_prefers_contrast.js"] + ["browser_reason_dropdown.js"] ["browser_report_send.js"] @@ -22,18 +26,14 @@ support-files = [ "send.js" ] ["browser_report_site_issue_fallback.js"] -["browser_site_not_working_fallback.js"] - ["browser_send_more_info.js"] support-files = [ "send_more_info.js", "../../../../../toolkit/components/gfx/content/videotest.mp4", ] -["browser_back_buttons.js"] +["browser_site_not_working_fallback.js"] -["browser_error_messages.js"] +["browser_tab_key_order.js"] ["browser_tab_switch_handling.js"] - -["browser_prefers_contrast.js"] diff --git a/browser/components/sessionstore/test/browser.toml b/browser/components/sessionstore/test/browser.toml index 4eb9f46eefbf..26fb4b4550d1 100644 --- a/browser/components/sessionstore/test/browser.toml +++ b/browser/components/sessionstore/test/browser.toml @@ -320,7 +320,7 @@ skip-if = ["!sessionHistoryInParent"] ["browser_async_window_flushing.js"] https_first_disabled = true -skip-if = true # Bug 1775616 +skip-if = ["true"] # Bug 1775616 ["browser_attributes.js"] diff --git a/browser/components/translations/tests/browser/browser.toml b/browser/components/translations/tests/browser/browser.toml index ff978e5e038c..bc617bf2fdb7 100644 --- a/browser/components/translations/tests/browser/browser.toml +++ b/browser/components/translations/tests/browser/browser.toml @@ -39,16 +39,16 @@ support-files = [ ["browser_translations_panel_cancel.js"] -["browser_translations_panel_engine_destroy.js"] - -["browser_translations_panel_engine_destroy_pending.js"] - ["browser_translations_panel_close_panel_never_translate_language_with_translations_active.js"] ["browser_translations_panel_close_panel_never_translate_language_with_translations_inactive.js"] ["browser_translations_panel_close_panel_never_translate_site.js"] +["browser_translations_panel_engine_destroy.js"] + +["browser_translations_panel_engine_destroy_pending.js"] + ["browser_translations_panel_engine_unsupported.js"] ["browser_translations_panel_engine_unsupported_lang.js"] @@ -58,7 +58,7 @@ support-files = [ ["browser_translations_panel_firstrun_revisit.js"] ["browser_translations_panel_fuzzing.js"] -skip-if = true +skip-if = ["true"] ["browser_translations_panel_gear.js"] diff --git a/browser/components/uitour/test/browser.toml b/browser/components/uitour/test/browser.toml index eb268b4c499b..2fb9cfa02573 100644 --- a/browser/components/uitour/test/browser.toml +++ b/browser/components/uitour/test/browser.toml @@ -9,7 +9,8 @@ support-files = [ ["browser_UITour.js"] skip-if = [ - "os == 'linux' || verify", # Intermittent failures, bug 951965 + "os == 'linux'", # Intermittent failures, bug 951965 + "verify", ] ["browser_UITour2.js"] diff --git a/browser/components/urlbar/tests/browser/browser.toml b/browser/components/urlbar/tests/browser/browser.toml index 0fdc0dbb9e1e..aac2a6e50a4b 100644 --- a/browser/components/urlbar/tests/browser/browser.toml +++ b/browser/components/urlbar/tests/browser/browser.toml @@ -14,17 +14,15 @@ prefs = [ ["browser_UrlbarInput_formatValue.js"] -["browser_UrlbarInput_formatValue_strikeout.js"] -support-files = [ - "mixed_active.html", -] - ["browser_UrlbarInput_formatValue_detachedTab.js"] skip-if = [ "apple_catalina", # Bug 1756585 "os == 'win'", # Bug 1756585 ] +["browser_UrlbarInput_formatValue_strikeout.js"] +support-files = ["mixed_active.html"] + ["browser_UrlbarInput_hiddenFocus.js"] ["browser_UrlbarInput_overflow.js"] @@ -297,6 +295,8 @@ fail-if = ["a11y_checks"] # Bug 1854660 clicked element may not be focusable and ["browser_new_tab_urlbar_reset.js"] +["browser_observers_for_strip_on_share.js"] + ["browser_oneOffs.js"] support-files = [ "searchSuggestionEngine.xml", @@ -685,5 +685,3 @@ skip-if = [ https_first_disabled = true ["browser_whereToOpen.js"] - -["browser_observers_for_strip_on_share.js"] diff --git a/browser/components/urlbar/tests/quicksuggest/browser/browser.toml b/browser/components/urlbar/tests/quicksuggest/browser/browser.toml index aed6a4070153..8eb782630a87 100644 --- a/browser/components/urlbar/tests/quicksuggest/browser/browser.toml +++ b/browser/components/urlbar/tests/quicksuggest/browser/browser.toml @@ -36,6 +36,9 @@ tags = "search-telemetry" ["browser_telemetry_dynamicWikipedia.js"] tags = "search-telemetry" +["browser_telemetry_gleanEmptyStrings.js"] +tags = "search-telemetry" + ["browser_telemetry_impressionEdgeCases.js"] tags = "search-telemetry" @@ -54,7 +57,4 @@ tags = "search-telemetry" ["browser_telemetry_weather.js"] tags = "search-telemetry" -["browser_telemetry_gleanEmptyStrings.js"] -tags = "search-telemetry" - ["browser_weather.js"] diff --git a/browser/extensions/formautofill/test/unit/xpcshell.toml b/browser/extensions/formautofill/test/unit/xpcshell.toml index ff5e02459263..9401a963dfb3 100644 --- a/browser/extensions/formautofill/test/unit/xpcshell.toml +++ b/browser/extensions/formautofill/test/unit/xpcshell.toml @@ -138,6 +138,9 @@ skip-if = [ head = "head.js ../../../../../services/sync/tests/unit/head_appinfo.js ../../../../../services/common/tests/unit/head_helpers.js ../../../../../services/sync/tests/unit/head_helpers.js ../../../../../services/sync/tests/unit/head_http_server.js" skip-if = ["tsan"] # Times out, bug 1612707 +["test_sync_deprecate_address_x_name_fields.js"] +head = "head.js ../../../../../services/sync/tests/unit/head_appinfo.js ../../../../../services/common/tests/unit/head_helpers.js ../../../../../services/sync/tests/unit/head_helpers.js ../../../../../services/sync/tests/unit/head_http_server.js" + ["test_sync_deprecate_credit_card_v4.js"] head = "head.js ../../../../../services/sync/tests/unit/head_appinfo.js ../../../../../services/common/tests/unit/head_helpers.js ../../../../../services/sync/tests/unit/head_helpers.js ../../../../../services/sync/tests/unit/head_http_server.js" skip-if = [ @@ -145,9 +148,6 @@ skip-if = [ "apple_silicon", # bug 1729554 ] -["test_sync_deprecate_address_x_name_fields.js"] -head = "head.js ../../../../../services/sync/tests/unit/head_appinfo.js ../../../../../services/common/tests/unit/head_helpers.js ../../../../../services/sync/tests/unit/head_helpers.js ../../../../../services/sync/tests/unit/head_http_server.js" - ["test_toOneLineAddress.js"] ["test_transformFields.js"] diff --git a/browser/extensions/screenshots/test/browser/browser.toml b/browser/extensions/screenshots/test/browser/browser.toml index acea7c9b1b28..240471cc7d65 100644 --- a/browser/extensions/screenshots/test/browser/browser.toml +++ b/browser/extensions/screenshots/test/browser/browser.toml @@ -8,7 +8,8 @@ https_first_disabled = true # Bug 1714237 Disabled on tsan due to timeouts interacting with the UI # Bug 1714210 Disabled on headless which doesnt support image data on the clipboard skip-if = [ - "headless || tsan", + "headless", + "tsan", "os == 'win'", # Bug 1714295 "os == 'linux' && bits == 64", # Bug 1714295 ] diff --git a/devtools/client/netmonitor/test/browser.toml b/devtools/client/netmonitor/test/browser.toml index 2d0ae7195813..00f6ac206893 100644 --- a/devtools/client/netmonitor/test/browser.toml +++ b/devtools/client/netmonitor/test/browser.toml @@ -329,9 +329,6 @@ fail-if = ["a11y_checks"] # Bug 1849028 clicked element may not be focusable and ["browser_net_new_request_panel.js"] -["browser_net_offline_mode.js"] -skip-if = ["!fission"] - ["browser_net_new_request_panel_clear_button.js"] ["browser_net_new_request_panel_content-length.js"] @@ -346,6 +343,9 @@ skip-if = ["a11y_checks"] # Bug 1858037 to investigate intermittent a11y_checks ["browser_net_new_request_panel_sync_url_params.js"] fail-if = ["a11y_checks"] # Bug 1849028 clicked element may not be focusable and/or labeled +["browser_net_offline_mode.js"] +skip-if = ["!fission"] + ["browser_net_open_in_debugger.js"] ["browser_net_open_in_style_editor.js"] diff --git a/devtools/client/styleeditor/test/browser.toml b/devtools/client/styleeditor/test/browser.toml index 9c091a9b4942..34cf13082657 100644 --- a/devtools/client/styleeditor/test/browser.toml +++ b/devtools/client/styleeditor/test/browser.toml @@ -90,6 +90,13 @@ fail-if = ["a11y_checks"] # Bug 1849028 clicked element may not be focusable and ["browser_styleeditor_bom.js"] +["browser_styleeditor_bug_740541_iframes.js"] + +["browser_styleeditor_bug_851132_middle_click.js"] +fail-if = ["a11y_checks"] # Bug 1849028 clicked element may not be focusable and/or labeled + +["browser_styleeditor_bug_870339.js"] + ["browser_styleeditor_bug_1247083_inline_stylesheet_numbering.js"] fail-if = ["a11y_checks"] # Bug 1849028 clicked element may not be focusable and/or labeled @@ -99,13 +106,6 @@ skip-if = [ "os == 'linux' && os_version == '18.04'", #bug 1424914 ] -["browser_styleeditor_bug_740541_iframes.js"] - -["browser_styleeditor_bug_851132_middle_click.js"] -fail-if = ["a11y_checks"] # Bug 1849028 clicked element may not be focusable and/or labeled - -["browser_styleeditor_bug_870339.js"] - ["browser_styleeditor_copyurl.js"] ["browser_styleeditor_enabled.js"] diff --git a/devtools/server/tracer/tests/browser/browser.toml b/devtools/server/tracer/tests/browser/browser.toml index c65099f7c289..61423b42b959 100644 --- a/devtools/server/tracer/tests/browser/browser.toml +++ b/devtools/server/tracer/tests/browser/browser.toml @@ -2,10 +2,10 @@ tags = "devtools" subsuite = "devtools" +["browser_document_tracer.js"] + ["browser_worker_tracer.js"] support-files = [ "Worker.tracer.js", "WorkerDebugger.tracer.js", ] - -["browser_document_tracer.js"] diff --git a/devtools/shared/network-observer/test/browser/browser.toml b/devtools/shared/network-observer/test/browser/browser.toml index d02005f151f4..3b3b44aaae97 100644 --- a/devtools/shared/network-observer/test/browser/browser.toml +++ b/devtools/shared/network-observer/test/browser/browser.toml @@ -20,11 +20,14 @@ skip-if = [ ] ["browser_networkobserver_auth_listener.js"] -skip-if = ["debug || asan"] # Disabled for frequent leaks in Bug 1873571. +skip-if = [ + "debug", # Disabled for frequent leaks in Bug 1873571. + "asan", +] ["browser_networkobserver_invalid_constructor.js"] ["browser_networkobserver_override.js"] ["browser_networkobserver_serviceworker.js"] -fail-if = true # Disabled until Bug 1267119 and Bug 1246289 +fail-if = ["true"] # Disabled until Bug 1267119 and Bug 1246289 diff --git a/docs/code-quality/lint/create.rst b/docs/code-quality/lint/create.rst index 4b2754175329..c4d8b15480ac 100644 --- a/docs/code-quality/lint/create.rst +++ b/docs/code-quality/lint/create.rst @@ -211,7 +211,10 @@ When generating the list of results, the following values are available. Automated testing ----------------- -Every new checker must have tests associated. +Every new checker must have associated tests. If your linter is ``mylinter`` then the +test file should be named ``tools/lint/test/test_mylinter.py`` and any example files +named like ``tools/lint/test/files/mylinter/my-example-file``. Be sure that your test +has been added as a section ``["test_mylinter.py"]`` in the manifest ``tools/lint/test/python.toml``. They should be pretty easy to write as most of the work is managed by the Mozlint framework. The key declaration is the ``LINTER`` variable which must match diff --git a/docs/code-quality/lint/linters/test-manifest-toml.rst b/docs/code-quality/lint/linters/test-manifest-toml.rst new file mode 100644 index 000000000000..1060b2756b8d --- /dev/null +++ b/docs/code-quality/lint/linters/test-manifest-toml.rst @@ -0,0 +1,35 @@ +Test Manifest TOML +================== + +This linter verifies syntax for ManifestParser TOML files. + +Run Locally +----------- + +This mozlint linter can be run using mach: + +.. parsed-literal:: + + $ mach lint --linter test-manifest-toml + + +Configuration +------------- + +The configuration excludes all non-ManifestParser TOML files (as well as +generated TOML manifests). + +Sources +------- + +* `Configuration (YAML) `_ +* `Source `_ + +Errors Detected +--------------- +* Invalid TOML +* Missing DEFAULT section (fixable) +* Sections not in alphabetical order (fixable) +* Section name not double quoted (fixable) +* Disabling a path by commenting out the section +* Conditional contains explicit || diff --git a/docshell/test/browser/browser.toml b/docshell/test/browser/browser.toml index 9d15c9c31537..bcda46fd2e09 100644 --- a/docshell/test/browser/browser.toml +++ b/docshell/test/browser/browser.toml @@ -113,6 +113,65 @@ https_first_disabled = true ["browser_browsing_context_discarded.js"] https_first_disabled = true +["browser_bug92473.js"] + +["browser_bug134911.js"] + +["browser_bug234628-1.js"] + +["browser_bug234628-10.js"] + +["browser_bug234628-11.js"] + +["browser_bug234628-2.js"] + +["browser_bug234628-3.js"] + +["browser_bug234628-4.js"] + +["browser_bug234628-5.js"] + +["browser_bug234628-6.js"] + +["browser_bug234628-8.js"] + +["browser_bug234628-9.js"] + +["browser_bug349769.js"] + +["browser_bug388121-1.js"] + +["browser_bug388121-2.js"] + +["browser_bug420605.js"] +skip-if = ["verify"] + +["browser_bug422543.js"] +https_first_disabled = true + +["browser_bug441169.js"] + +["browser_bug503832.js"] +skip-if = ["verify"] + +["browser_bug554155.js"] + +["browser_bug655270.js"] + +["browser_bug655273.js"] + +["browser_bug670318.js"] +skip-if = ["os == 'linux' && (debug || asan || tsan)"] # Bug 1717403 + +["browser_bug673087-1.js"] + +["browser_bug673087-2.js"] + +["browser_bug673467.js"] + +["browser_bug852909.js"] +skip-if = ["verify && debug && os == 'win'"] + ["browser_bug1206879.js"] https_first_disabled = true @@ -124,8 +183,6 @@ https_first_disabled = true ["browser_bug1347823.js"] -["browser_bug134911.js"] - ["browser_bug1415918_beforeunload_options.js"] https_first_disabled = true @@ -184,63 +241,6 @@ https_first_disabled = true ["browser_bug1798780.js"] -["browser_bug234628-1.js"] - -["browser_bug234628-10.js"] - -["browser_bug234628-11.js"] - -["browser_bug234628-2.js"] - -["browser_bug234628-3.js"] - -["browser_bug234628-4.js"] - -["browser_bug234628-5.js"] - -["browser_bug234628-6.js"] - -["browser_bug234628-8.js"] - -["browser_bug234628-9.js"] - -["browser_bug349769.js"] - -["browser_bug388121-1.js"] - -["browser_bug388121-2.js"] - -["browser_bug420605.js"] -skip-if = ["verify"] - -["browser_bug422543.js"] -https_first_disabled = true - -["browser_bug441169.js"] - -["browser_bug503832.js"] -skip-if = ["verify"] - -["browser_bug554155.js"] - -["browser_bug655270.js"] - -["browser_bug655273.js"] - -["browser_bug670318.js"] -skip-if = ["os == 'linux' && (debug || asan || tsan)"] # Bug 1717403 - -["browser_bug673087-1.js"] - -["browser_bug673087-2.js"] - -["browser_bug673467.js"] - -["browser_bug852909.js"] -skip-if = ["verify && debug && os == 'win'"] - -["browser_bug92473.js"] - ["browser_click_link_within_view_source.js"] ["browser_cross_process_csp_inheritance.js"] diff --git a/docshell/test/chrome/chrome.toml b/docshell/test/chrome/chrome.toml index 9d382a5f865b..ee4d2872470b 100644 --- a/docshell/test/chrome/chrome.toml +++ b/docshell/test/chrome/chrome.toml @@ -48,6 +48,15 @@ support-files = [ ["test_allowContentRetargeting.html"] +["test_bug89419.xhtml"] + +["test_bug92598.xhtml"] +support-files = [ + "92598_nostore.html", + "92598_nostore.html^headers^", + "bug92598_window.xhtml", +] + ["test_bug112564.xhtml"] support-files = [ "bug112564_window.xhtml", @@ -117,17 +126,8 @@ support-files = [ ["test_bug846906.xhtml"] -["test_bug89419.xhtml"] - ["test_bug909218.html"] -["test_bug92598.xhtml"] -support-files = [ - "92598_nostore.html", - "92598_nostore.html^headers^", - "bug92598_window.xhtml", -] - ["test_docRedirect.xhtml"] ["test_mozFrameType.xhtml"] diff --git a/docshell/test/navigation/browser.toml b/docshell/test/navigation/browser.toml index e58d656bb694..f56b1dea1278 100644 --- a/docshell/test/navigation/browser.toml +++ b/docshell/test/navigation/browser.toml @@ -10,10 +10,10 @@ support-files = [ "redirect_to_blank.sjs", ] -["browser_bug1757458.js"] - ["browser_bug343515.js"] +["browser_bug1757458.js"] + ["browser_ghistorymaxsize_is_0.js"] ["browser_test-content-chromeflags.js"] diff --git a/dom/base/test/browser.toml b/dom/base/test/browser.toml index 53a5a19f1c52..1deeb88d89bf 100644 --- a/dom/base/test/browser.toml +++ b/dom/base/test/browser.toml @@ -45,6 +45,9 @@ skip-if = [ ["browser_blocking_image.js"] +["browser_bug902350.js"] +tags = "mcb" + ["browser_bug1011748.js"] ["browser_bug1058164.js"] @@ -73,15 +76,14 @@ support-files = [ ["browser_bug1703472.js"] support-files = ["file_bug1703472.html"] -["browser_bug902350.js"] -tags = "mcb" - ["browser_chromeutils_getalldomprocesses.js"] ["browser_chromeutils_isdomobject.js"] ["browser_data_documents_aboutmemory.js"] +["browser_event_source_reconnect_after_disconnect.js"] + ["browser_force_process_selector.js"] skip-if = [ "verify", @@ -146,5 +148,3 @@ skip-if = ["verify"] ["browser_user_input_handling_delay_reload_ticks.js"] ["browser_xml_toggle.js"] - -["browser_event_source_reconnect_after_disconnect.js"] diff --git a/dom/base/test/chrome.toml b/dom/base/test/chrome.toml index eeb01f344da9..cddcf395783a 100644 --- a/dom/base/test/chrome.toml +++ b/dom/base/test/chrome.toml @@ -25,18 +25,18 @@ support-files = [ ["test_blocking_image.html"] -["test_bug1008126.html"] - -["test_bug1016960.html"] - -["test_bug1120222.html"] - ["test_bug419527.xhtml"] ["test_bug564863-2.xhtml"] ["test_bug945152.html"] +["test_bug1008126.html"] + +["test_bug1016960.html"] + +["test_bug1120222.html"] + ["test_domrequesthelper.xhtml"] ["test_fragment_sanitization.xhtml"] diff --git a/dom/base/test/chrome/chrome.toml b/dom/base/test/chrome/chrome.toml index f8c1111f55cb..b8439a2d2eec 100644 --- a/dom/base/test/chrome/chrome.toml +++ b/dom/base/test/chrome/chrome.toml @@ -22,20 +22,8 @@ support-files = [ ] prefs = ["gfx.font_rendering.fallback.async=false"] -["test_bug1063837.xhtml"] - -["test_bug1098074_throw_from_ReceiveMessage.xhtml"] - -["test_bug1139964.xhtml"] - ["test_bug120684.xhtml"] -["test_bug1209621.xhtml"] - -["test_bug1339722.html"] - -["test_bug1346936.html"] - ["test_bug206691.xhtml"] ["test_bug289714.xhtml"] @@ -47,8 +35,6 @@ support-files = ["../file_bug357450.js"] ["test_bug380418.html"] -["test_bug380418.html^headers^"] - ["test_bug383430.html"] ["test_bug418986-1.xhtml"] @@ -96,6 +82,20 @@ skip-if = ["verify"] ["test_bug990812.xhtml"] +["test_bug1063837.xhtml"] + +["test_bug1098074_throw_from_ReceiveMessage.xhtml"] + +["test_bug1139964.xhtml"] + +["test_bug1209621.xhtml"] + +["test_bug1339722.html"] + +["test_bug1346936.html"] + +["test_bug380418.html^headers^"] + ["test_chromeOuterWindowID.xhtml"] support-files = ["window_chromeOuterWindowID.xhtml"] diff --git a/dom/base/test/jsmodules/chrome.toml b/dom/base/test/jsmodules/chrome.toml index ddba1e82a86c..82d02ad4df8b 100644 --- a/dom/base/test/jsmodules/chrome.toml +++ b/dom/base/test/jsmodules/chrome.toml @@ -39,14 +39,14 @@ support-files = [ ["test_dynamicImportErrorMessage.html"] -["test_import_errorMessage.html"] - ["test_importIntroType.html"] ["test_importNotFound.html"] ["test_importResolveFailed.html"] +["test_import_errorMessage.html"] + ["test_import_meta_resolve.html"] ["test_importedModuleMemoization.html"] diff --git a/dom/base/test/jsmodules/importmaps/mochitest.toml b/dom/base/test/jsmodules/importmaps/mochitest.toml index b3c7dd532dbd..422945572292 100644 --- a/dom/base/test/jsmodules/importmaps/mochitest.toml +++ b/dom/base/test/jsmodules/importmaps/mochitest.toml @@ -26,8 +26,8 @@ support-files = [ "circular_dependency.mjs", ] -["test_importMap_with_external_script.html"] - ["test_bug_1865410.html"] ["test_bug_1873417.html"] + +["test_importMap_with_external_script.html"] diff --git a/dom/bindings/test/chrome.toml b/dom/bindings/test/chrome.toml index 84a68a640287..0986974ece6b 100644 --- a/dom/bindings/test/chrome.toml +++ b/dom/bindings/test/chrome.toml @@ -6,6 +6,8 @@ support-files = [ "!/dom/bindings/test/file_proxies_via_xray.html", ] +["test_bug775543.html"] + ["test_bug1123516_maplikesetlikechrome.xhtml"] skip-if = ["!debug"] # TestFunctions is only available in debug builds @@ -13,8 +15,6 @@ skip-if = ["!debug"] # TestFunctions is only available in debug builds ["test_bug1457051.html"] -["test_bug775543.html"] - ["test_document_location_set_via_xray.html"] ["test_document_location_via_xray_cached.html"] diff --git a/dom/events/test/chrome.toml b/dom/events/test/chrome.toml index 066065ec9f2c..4a1221d664c2 100644 --- a/dom/events/test/chrome.toml +++ b/dom/events/test/chrome.toml @@ -15,14 +15,6 @@ support-files = [ ["test_DataTransferItemList.html"] skip-if = ["!debug && (os == 'linux')"] #Bug 1421150 -["test_bug1128787-1.html"] - -["test_bug1128787-2.html"] - -["test_bug1128787-3.html"] - -["test_bug1412775.xhtml"] - ["test_bug336682_2.xhtml"] ["test_bug415498.xhtml"] @@ -45,4 +37,12 @@ skip-if = ["!debug && (os == 'linux')"] #Bug 1421150 ["test_bug930374-chrome.html"] +["test_bug1128787-1.html"] + +["test_bug1128787-2.html"] + +["test_bug1128787-3.html"] + +["test_bug1412775.xhtml"] + ["test_eventctors.xhtml"] diff --git a/dom/html/test/browser.toml b/dom/html/test/browser.toml index 94c329b04ad8..13775f607b28 100644 --- a/dom/html/test/browser.toml +++ b/dom/html/test/browser.toml @@ -16,6 +16,11 @@ skip-if = ["bits == 64 && (os == 'mac' || os == 'linux')"] #Bug 1646862 ["browser_ImageDocument_svg_zoom.js"] +["browser_bug436200.js"] +support-files = ["bug436200.html"] + +["browser_bug592641.js"] + ["browser_bug1081537.js"] ["browser_bug1108547.js"] @@ -25,11 +30,6 @@ support-files = [ "file_bug1108547-3.html", ] -["browser_bug436200.js"] -support-files = ["bug436200.html"] - -["browser_bug592641.js"] - ["browser_containerLoadingContent.js"] ["browser_form_post_from_file_to_http.js"] diff --git a/dom/indexedDB/test/unit/xpcshell-parent-process.toml b/dom/indexedDB/test/unit/xpcshell-parent-process.toml index 2a1b32973ae1..41451d6a4679 100644 --- a/dom/indexedDB/test/unit/xpcshell-parent-process.toml +++ b/dom/indexedDB/test/unit/xpcshell-parent-process.toml @@ -34,6 +34,10 @@ support-files = [ ["test_cleanup_transaction.js"] +["test_clear_object_store_with_indexes.js"] +requesttimeoutfactor = 2 +skip-if = ["tsan"] + ["test_database_close_without_onclose.js"] ["test_database_onclose.js"] @@ -92,7 +96,3 @@ skip-if = ["os == 'android' && processor == 'x86'"] ["test_wasm_get_values.js"] ["test_wasm_put_get_values.js"] - -["test_clear_object_store_with_indexes.js"] -requesttimeoutfactor = 2 -skip-if = ["tsan"] diff --git a/dom/media/test/mochitest_compat.toml b/dom/media/test/mochitest_compat.toml index 8c5661cc1764..c477661a1986 100644 --- a/dom/media/test/mochitest_compat.toml +++ b/dom/media/test/mochitest_compat.toml @@ -860,7 +860,8 @@ skip-if = ["(android_version == '25' && debug)"] # android(bug 1232305) ["test_fragment_play.html"] ["test_hevc_playback.html"] -skip-if = ["!mda_gpu || os != 'win'"] +run-if = ["os == 'win'"] +skip-if = ["!mda_gpu"] scheme = "https" ["test_hevc_support.html"] diff --git a/dom/media/webrtc/tests/mochitests/mochitest.toml b/dom/media/webrtc/tests/mochitests/mochitest.toml index 1ddbe458330f..cf6381c63c26 100644 --- a/dom/media/webrtc/tests/mochitests/mochitest.toml +++ b/dom/media/webrtc/tests/mochitests/mochitest.toml @@ -69,17 +69,17 @@ skip-if = ["os != 'linux'"] # the only platform with real devices ["test_ondevicechange.html"] run-sequentially = "sets prefs that may disrupt other tests" +["test_setSinkId-echoCancellation.html"] +skip-if = ["os == 'android'"] # bug 1473346 - no setSinkId() + +["test_setSinkId-stream-source.html"] +skip-if = ["os == 'android'"] # bug 1473346 - no setSinkId() + ["test_setSinkId.html"] skip-if = ["os != 'linux'"] # the only platform with real devices ["test_setSinkId_default_addTrack.html"] -["test_setSinkId-echoCancellation.html"] -skip-if = ["os == 'android'"] # bug 1473346 - no setSinkId() - ["test_setSinkId_preMutedElement.html"] -["test_setSinkId-stream-source.html"] -skip-if = ["os == 'android'"] # bug 1473346 - no setSinkId() - ["test_unfocused_pref.html"] diff --git a/dom/power/tests/mochitest.toml b/dom/power/tests/mochitest.toml index 543a8efc0bfd..0b57119a8af1 100644 --- a/dom/power/tests/mochitest.toml +++ b/dom/power/tests/mochitest.toml @@ -2,8 +2,8 @@ prefs = ["dom.screenwakelock.enabled=true"] scheme = "https" -["test_wakelock_default_permission.html"] -fail-if = ["xorigin"] # cross-origin use requires permissions policy - ["test_dynamic_pref_change.html"] fail-if = ["xorigin"] # cross-origin use requires permissions policy + +["test_wakelock_default_permission.html"] +fail-if = ["xorigin"] # cross-origin use requires permissions policy diff --git a/dom/push/test/mochitest.toml b/dom/push/test/mochitest.toml index 0d7475e8b798..d95b8ef8e16b 100644 --- a/dom/push/test/mochitest.toml +++ b/dom/push/test/mochitest.toml @@ -41,12 +41,12 @@ skip-if = [ "os == 'mac'", #Bug 1274773 ] -["test_permissions.html"] -skip-if = ["os == 'android'"] - ["test_permission_granted.html"] scheme = "https" +["test_permissions.html"] +skip-if = ["os == 'android'"] + ["test_register.html"] skip-if = [ "os == 'android'", diff --git a/dom/security/test/general/browser.toml b/dom/security/test/general/browser.toml index 3141d3f5bb37..0f4ec5b22435 100644 --- a/dom/security/test/general/browser.toml +++ b/dom/security/test/general/browser.toml @@ -42,6 +42,12 @@ support-files = [ "file_framing_error_pages.sjs", ] +["browser_test_gpc_privateBrowsingMode.js"] +support-files = [ + "file_empty.html", + "file_gpc_server.sjs", +] + ["browser_test_referrer_loadInOtherProcess.js"] ["browser_test_report_blocking.js"] @@ -73,10 +79,3 @@ support-files = [ "file_framing_xfo_object.html", "file_framing_xfo_embed_object.sjs", ] - -["browser_test_gpc_privateBrowsingMode.js"] -support-files = [ - "file_empty.html", - "file_gpc_server.sjs", -] - diff --git a/dom/tests/browser/browser.toml b/dom/tests/browser/browser.toml index 772df67bbc06..bc6be07c2035 100644 --- a/dom/tests/browser/browser.toml +++ b/dom/tests/browser/browser.toml @@ -31,6 +31,8 @@ support-files = [ ["browser_beforeunload_between_chrome_content.js"] https_first_disabled = true +["browser_bug396843.js"] + ["browser_bug1004814.js"] ["browser_bug1008941_dismissGeolocationHanger.js"] @@ -55,8 +57,6 @@ support-files = [ "file_empty_cross_site_frame.html", ] -["browser_bug396843.js"] - ["browser_bytecode_cache_asm_js.js"] support-files = [ "page_bytecode_cache_asm_js.html", diff --git a/dom/tests/mochitest/chrome/chrome.toml b/dom/tests/mochitest/chrome/chrome.toml index 17a734da0e74..186b381e7d5a 100644 --- a/dom/tests/mochitest/chrome/chrome.toml +++ b/dom/tests/mochitest/chrome/chrome.toml @@ -42,6 +42,10 @@ support-files = [ ["test_activation.xhtml"] tags = "fullscreen" +["test_bug800817.xhtml"] + +["test_bug830858.xhtml"] + ["test_bug1224790-1.xhtml"] tags = "openwindow" skip-if = ["os != 'mac'"] @@ -50,10 +54,6 @@ skip-if = ["os != 'mac'"] tags = "openwindow" skip-if = ["os != 'mac'"] -["test_bug800817.xhtml"] - -["test_bug830858.xhtml"] - ["test_callback_wrapping.xhtml"] ["test_clipboard_events_chrome.html"] diff --git a/dom/xul/test/chrome.toml b/dom/xul/test/chrome.toml index e0d87a4e31ea..3e239fe1d97f 100644 --- a/dom/xul/test/chrome.toml +++ b/dom/xul/test/chrome.toml @@ -6,13 +6,6 @@ support-files = [ ["test_accesskey.xhtml"] -["test_bug1070049_throw_from_script.xhtml"] - -["test_bug1290965.xhtml"] - -["test_bug1686822.xhtml"] -support-files = ["window_bug1686822.xhtml"] - ["test_bug199692.xhtml"] ["test_bug311681.xhtml"] @@ -35,6 +28,13 @@ support-files = ["window_bug1686822.xhtml"] ["test_bug775972.xhtml"] +["test_bug1070049_throw_from_script.xhtml"] + +["test_bug1290965.xhtml"] + +["test_bug1686822.xhtml"] +support-files = ["window_bug1686822.xhtml"] + ["test_html_template.xhtml"] ["test_import_xul_to_content.xhtml"] diff --git a/editor/composer/test/chrome.toml b/editor/composer/test/chrome.toml index df88d8aaeb95..a7427cf51049 100644 --- a/editor/composer/test/chrome.toml +++ b/editor/composer/test/chrome.toml @@ -1,6 +1,6 @@ [DEFAULT] skip-if = ["os == 'android'"] -["test_bug1266815.html"] - ["test_bug434998.xhtml"] + +["test_bug1266815.html"] diff --git a/editor/libeditor/tests/chrome.toml b/editor/libeditor/tests/chrome.toml index 3f709dea3e4d..a12dd64a763b 100644 --- a/editor/libeditor/tests/chrome.toml +++ b/editor/libeditor/tests/chrome.toml @@ -1,8 +1,6 @@ [DEFAULT] skip-if = ["os == 'android'"] -["test_bug1397412.xhtml"] - ["test_bug489202.xhtml"] ["test_bug599983.xhtml"] @@ -13,6 +11,8 @@ skip-if = ["os == 'android'"] ["test_bug780908.xhtml"] +["test_bug1397412.xhtml"] + ["test_can_undo_after_setting_value.xhtml"] ["test_contenteditable_text_input_handling.html"] diff --git a/extensions/universalchardet/tests/chrome.toml b/extensions/universalchardet/tests/chrome.toml index 49825ed07cb3..767be95fda22 100644 --- a/extensions/universalchardet/tests/chrome.toml +++ b/extensions/universalchardet/tests/chrome.toml @@ -25,14 +25,6 @@ support-files = [ "bug1071816-4_text.html", ] -["test_bug1071816-1.html"] - -["test_bug1071816-2.html"] - -["test_bug1071816-3.html"] - -["test_bug1071816-4.html"] - ["test_bug306272.html"] ["test_bug426271-euc-jp.html"] @@ -70,3 +62,11 @@ support-files = [ ["test_bug811363-2-8.html"] ["test_bug811363-2-9.html"] + +["test_bug1071816-1.html"] + +["test_bug1071816-2.html"] + +["test_bug1071816-3.html"] + +["test_bug1071816-4.html"] diff --git a/gfx/layers/apz/test/mochitest/browser.toml b/gfx/layers/apz/test/mochitest/browser.toml index ef1a66b842e3..5432aa0ae11b 100644 --- a/gfx/layers/apz/test/mochitest/browser.toml +++ b/gfx/layers/apz/test/mochitest/browser.toml @@ -69,12 +69,12 @@ support-files = [ skip-if = ["os == 'win'"] # bug 1495580 support-files = ["helper_test_select_zoom.html"] -["browser_test_tab_drag_zoom.js"] -skip-if = ["os == 'win'"] # Our Windows touch injection test code doesn't support pinch gestures (bug 1495580) -support-files = ["helper_test_tab_drag_zoom.html"] - ["browser_test_tab_drag_event_counts.js"] skip-if = ["os == 'linux'"] # No native key event support on Linux at this time (bug 1770143) support-files = [ "helper_test_tab_drag_event_counts.html" ] + +["browser_test_tab_drag_zoom.js"] +skip-if = ["os == 'win'"] # Our Windows touch injection test code doesn't support pinch gestures (bug 1495580) +support-files = ["helper_test_tab_drag_zoom.html"] diff --git a/image/test/browser/browser.toml b/image/test/browser/browser.toml index 2aed8918a3c3..f264e1f6ab45 100644 --- a/image/test/browser/browser.toml +++ b/image/test/browser/browser.toml @@ -12,6 +12,9 @@ support-files = [ ["browser_bug666317.js"] skip-if = ["true"] # Bug 1207012 - Permaorange from an uncaught exception that isn't actually turning the suite orange until it hits beta, Bug 948194 - Decoded Images seem to not be discarded on memory-pressure notification +["browser_bug1869938.js"] +support-files = ["helper1869938.html"] + ["browser_docshell_type_editor.js"] ["browser_image.js"] @@ -22,6 +25,3 @@ skip-if = ["true"] # Bug 987616 ["browser_offscreen_image_in_out_of_process_iframe.js"] https_first_disabled = true support-files = ["empty.html"] - -["browser_bug1869938.js"] -support-files = ["helper1869938.html"] diff --git a/js/xpconnect/tests/chrome/chrome.toml b/js/xpconnect/tests/chrome/chrome.toml index 2a60282fcdb9..6aec4b549bbe 100644 --- a/js/xpconnect/tests/chrome/chrome.toml +++ b/js/xpconnect/tests/chrome/chrome.toml @@ -36,34 +36,6 @@ prefs = ["javascript.options.large_arraybuffers=true"] ["test_APIExposer.xhtml"] -["test_bug1041626.xhtml"] - -["test_bug1042436.xhtml"] - -["test_bug1065185.html"] - -["test_bug1074863.html"] - -["test_bug1092477.xhtml"] - -["test_bug1124898.html"] - -["test_bug1126911.html"] - -["test_bug1281071.xhtml"] - -["test_bug1390159.xhtml"] - -["test_bug1430164.html"] - -["test_bug1516237.html"] - -["test_bug1530146.html"] -support-files = [ - "file_bug1530146.html", - "file_bug1530146_inner.html", -] - ["test_bug361111.xhtml"] ["test_bug448587.xhtml"] @@ -152,6 +124,34 @@ allow_xul_xbl = true ["test_bug996069.xhtml"] +["test_bug1041626.xhtml"] + +["test_bug1042436.xhtml"] + +["test_bug1065185.html"] + +["test_bug1074863.html"] + +["test_bug1092477.xhtml"] + +["test_bug1124898.html"] + +["test_bug1126911.html"] + +["test_bug1281071.xhtml"] + +["test_bug1390159.xhtml"] + +["test_bug1430164.html"] + +["test_bug1516237.html"] + +["test_bug1530146.html"] +support-files = [ + "file_bug1530146.html", + "file_bug1530146_inner.html", +] + ["test_chrometoSource.xhtml"] ["test_cloneInto.xhtml"] diff --git a/layout/base/tests/browser.toml b/layout/base/tests/browser.toml index 428e0a8eeda3..a5b279145f09 100644 --- a/layout/base/tests/browser.toml +++ b/layout/base/tests/browser.toml @@ -1,62 +1,61 @@ -[DEFAULT] -prefs = [ - "layout.css.properties-and-values.enabled=true", -] - -["browser_bug1701027-1.js"] -support-files = ["helper_bug1701027-1.html"] - -["browser_bug1701027-2.js"] -support-files = ["helper_bug1701027-2.html"] - -["browser_bug1757410.js"] -run-if = [ - "os == 'mac' && debug", - "os == 'win' && processor == 'x86_64' && debug" -] - -["browser_bug1787079.js"] -run-if = ["os == 'win' && processor == 'x86_64' && debug"] - -["browser_bug1791083.js"] -skip-if = ["!sessionHistoryInParent"] - -["browser_bug617076.js"] - -["browser_css_registered_property.js"] - -["browser_disableDialogs_onbeforeunload.js"] - -["browser_onbeforeunload_only_after_interaction.js"] - -["browser_onbeforeunload_only_after_interaction_in_frame.js"] - -["browser_scroll_into_view_in_out_of_process_iframe.js"] -support-files = [ - "test_scroll_into_view_in_oopif.html", - "scroll_into_view_in_child.html" -] - -["browser_select_popup_position_in_out_of_process_iframe.js"] -skip-if = [ - "verify && (os == 'mac')", # bug 1627874 - "apple_silicon", # Disabled due to bleedover with other tests when run in regular suites; passes in "failures" jobs - "os == 'linux' && socketprocess_networking && fission && !debug", -] # high frequency intermittent -support-files = [ - "!/gfx/layers/apz/test/mochitest/apz_test_native_event_utils.js", - "!/browser/base/content/test/forms/head.js" -] - -["browser_stylesheet_change_events.js"] -support-files = [ - "file_stylesheet_change_events.html", - "stylesheet_change_events.css" -] - -["browser_visual_viewport_iframe.js"] -support-files = [ - "test_visual_viewport_in_oopif.html", - "visual_viewport_in_child.html" -] - +[DEFAULT] +prefs = [ + "layout.css.properties-and-values.enabled=true", +] + +["browser_bug617076.js"] + +["browser_bug1701027-1.js"] +support-files = ["helper_bug1701027-1.html"] + +["browser_bug1701027-2.js"] +support-files = ["helper_bug1701027-2.html"] + +["browser_bug1757410.js"] +run-if = [ + "os == 'mac' && debug", + "os == 'win' && processor == 'x86_64' && debug" +] + +["browser_bug1787079.js"] +run-if = ["os == 'win' && processor == 'x86_64' && debug"] + +["browser_bug1791083.js"] +skip-if = ["!sessionHistoryInParent"] + +["browser_css_registered_property.js"] + +["browser_disableDialogs_onbeforeunload.js"] + +["browser_onbeforeunload_only_after_interaction.js"] + +["browser_onbeforeunload_only_after_interaction_in_frame.js"] + +["browser_scroll_into_view_in_out_of_process_iframe.js"] +support-files = [ + "test_scroll_into_view_in_oopif.html", + "scroll_into_view_in_child.html" +] + +["browser_select_popup_position_in_out_of_process_iframe.js"] +skip-if = [ + "verify && (os == 'mac')", # bug 1627874 + "apple_silicon", # Disabled due to bleedover with other tests when run in regular suites; passes in "failures" jobs + "os == 'linux' && socketprocess_networking && fission && !debug", +] # high frequency intermittent +support-files = [ + "!/gfx/layers/apz/test/mochitest/apz_test_native_event_utils.js", + "!/browser/base/content/test/forms/head.js" +] + +["browser_stylesheet_change_events.js"] +support-files = [ + "file_stylesheet_change_events.html", + "stylesheet_change_events.css" +] + +["browser_visual_viewport_iframe.js"] +support-files = [ + "test_visual_viewport_in_oopif.html", + "visual_viewport_in_child.html" +] diff --git a/layout/base/tests/chrome/chrome.toml b/layout/base/tests/chrome/chrome.toml index 358de3d5cc9a..6636b224e647 100644 --- a/layout/base/tests/chrome/chrome.toml +++ b/layout/base/tests/chrome/chrome.toml @@ -89,15 +89,6 @@ support-files = [ "markB.ttf", ] -["test_bug1018265.xhtml"] - -["test_bug1041200.xhtml"] -skip-if = ["os == 'win' && bits == 64"] # Bug 1272321 -support-files = [ - "bug1041200_frame.html", - "bug1041200_window.html", -] - ["test_bug396367-1.html"] ["test_bug396367-2.html"] @@ -120,6 +111,15 @@ skip-if = ["os == 'linux' && !debug"] # Bug 1208197 ["test_bug812817.xhtml"] +["test_bug1018265.xhtml"] + +["test_bug1041200.xhtml"] +skip-if = ["os == 'win' && bits == 64"] # Bug 1272321 +support-files = [ + "bug1041200_frame.html", + "bug1041200_window.html", +] + ["test_chrome_content_integration.xhtml"] ["test_color_scheme_browser.xhtml"] diff --git a/layout/inspector/tests/chrome/chrome.toml b/layout/inspector/tests/chrome/chrome.toml index b254fa7ed69a..193be351cb77 100644 --- a/layout/inspector/tests/chrome/chrome.toml +++ b/layout/inspector/tests/chrome/chrome.toml @@ -32,12 +32,12 @@ skip-if = [ ] support-files = ["test_fontVariationsAPI.css"] +["test_getBlockLineCounts.html"] + +["test_getSelectorWarnings.html"] + ["test_ua_rule_modification.html"] ["test_ua_sheet_disable.html"] ["test_visited_style.html"] - -["test_getBlockLineCounts.html"] - -["test_getSelectorWarnings.html"] diff --git a/layout/style/test/chrome/chrome.toml b/layout/style/test/chrome/chrome.toml index 6220e41170f0..8c4c6045d85c 100644 --- a/layout/style/test/chrome/chrome.toml +++ b/layout/style/test/chrome/chrome.toml @@ -10,18 +10,18 @@ support-files = [ "mismatch.png", ] -["test_bug1157097.html"] +["test_bug418986-2.xhtml"] ["test_bug511909.html"] +["test_bug535806.xhtml"] + +["test_bug1157097.html"] + ["test_bug1346623.html"] ["test_bug1371453.html"] -["test_bug418986-2.xhtml"] - -["test_bug535806.xhtml"] - ["test_chrome_only_media_queries.html"] support-files = ["chrome-only-media-queries.js"] diff --git a/layout/xul/test/chrome.toml b/layout/xul/test/chrome.toml index 983f87c74634..6588db2ea4bb 100644 --- a/layout/xul/test/chrome.toml +++ b/layout/xul/test/chrome.toml @@ -14,8 +14,6 @@ support-files = [ "titledpanelwindow.xhtml", ] -["test_bug1197913.xhtml"] - ["test_bug159346.xhtml"] ["test_bug381167.xhtml"] @@ -33,6 +31,8 @@ support-files = [ ["test_bug987230.xhtml"] skip-if = ["os == 'linux'"] # No native mousedown event on Linux +["test_bug1197913.xhtml"] + ["test_menuitem_ctrl_click.xhtml"] ["test_popupReflowPos.xhtml"] diff --git a/netwerk/cookie/test/browser/browser.toml b/netwerk/cookie/test/browser/browser.toml index b43acf2e63ac..05a302ddbdf3 100644 --- a/netwerk/cookie/test/browser/browser.toml +++ b/netwerk/cookie/test/browser/browser.toml @@ -26,10 +26,10 @@ support-files = ["server.sjs"] ["browser_oversize.js"] support-files = ["oversize.sjs"] -["browser_partitioned_telemetry.js"] +["browser_partitionedConsole.js"] support-files = ["partitioned.sjs"] -["browser_partitionedConsole.js"] +["browser_partitioned_telemetry.js"] support-files = ["partitioned.sjs"] ["browser_sameSiteConsole.js"] diff --git a/netwerk/test/mochitests/mochitest.toml b/netwerk/test/mochitests/mochitest.toml index a636e059c215..8d5cb8ff9261 100644 --- a/netwerk/test/mochitests/mochitest.toml +++ b/netwerk/test/mochitests/mochitest.toml @@ -206,12 +206,12 @@ skip-if = [ ["test_uri_scheme.html"] skip-if = ["verify && debug && os == 'mac'"] -["test_viewsource_unlinkable.html"] - -["test_xhr_method_case.html"] -skip-if = ["http2"] - ["test_url_perf.html"] # test to check the performance of URL parsing. # No need to run in CI. skip-if = ["true"] + +["test_viewsource_unlinkable.html"] + +["test_xhr_method_case.html"] +skip-if = ["http2"] diff --git a/testing/cppunittest.toml b/testing/cppunittest.toml index 62791f2b004b..13b3966ac49d 100644 --- a/testing/cppunittest.toml +++ b/testing/cppunittest.toml @@ -108,7 +108,10 @@ run-if = ["os == 'win'"] run-if = ["os == 'win'"] ["TestPoisonArea"] -skip-if = ["asan || tsan"] # Not built on sanitizer builds +skip-if = [ + "asan", # Not built on sanitizer builds + "tsan", +] ["TestPrintf"] diff --git a/testing/mochitest/baselinecoverage/browser_chrome/browser.toml b/testing/mochitest/baselinecoverage/browser_chrome/browser.toml index cc6d67955266..3c95127f1c60 100644 --- a/testing/mochitest/baselinecoverage/browser_chrome/browser.toml +++ b/testing/mochitest/baselinecoverage/browser_chrome/browser.toml @@ -1,11 +1,11 @@ [DEFAULT] -['browser_baselinecoverage.js'] +["browser_baselinecoverage.js"] run-if = [ "ccov && verify", ] -['browser_baselinecoverage_browser-chrome.js'] +["browser_baselinecoverage_browser-chrome.js"] run-if = [ "ccov && verify", ] diff --git a/testing/mochitest/baselinecoverage/chrome/chrome.toml b/testing/mochitest/baselinecoverage/chrome/chrome.toml index 16961989235b..9a5b148439f0 100644 --- a/testing/mochitest/baselinecoverage/chrome/chrome.toml +++ b/testing/mochitest/baselinecoverage/chrome/chrome.toml @@ -1,6 +1,6 @@ [DEFAULT] -['test_baselinecoverage.xhtml'] +["test_baselinecoverage.xhtml"] run-if = [ "ccov && verify", ] diff --git a/testing/mochitest/baselinecoverage/plain/mochitest.toml b/testing/mochitest/baselinecoverage/plain/mochitest.toml index 0944d68e7389..0fa3cf2ccae5 100644 --- a/testing/mochitest/baselinecoverage/plain/mochitest.toml +++ b/testing/mochitest/baselinecoverage/plain/mochitest.toml @@ -1,6 +1,6 @@ [DEFAULT] -['test_baselinecoverage.html'] +["test_baselinecoverage.html"] run-if = [ "ccov && verify", ] diff --git a/testing/mochitest/chrome/chrome.toml b/testing/mochitest/chrome/chrome.toml index 54b947bcbfe2..944db9e3c0dc 100644 --- a/testing/mochitest/chrome/chrome.toml +++ b/testing/mochitest/chrome/chrome.toml @@ -1,28 +1,24 @@ [DEFAULT] -skip-if = [ - "os == 'android'", -] +skip-if = ["os == 'android'"] support-files = "test-dir/test-file" -['test_chromeGetTestFile.xhtml'] +["test_chromeGetTestFile.xhtml"] -['test_sample.xhtml'] +["test_sample.xhtml"] -['test_sanityEventUtils.xhtml'] -fail-if = [ - "headless", # Bug 1678303 -] +["test_sanityEventUtils.xhtml"] +fail-if = ["headless"] # Bug 1678303 -['test_sanityException.xhtml'] +["test_sanityException.xhtml"] -['test_sanityException2.xhtml'] +["test_sanityException2.xhtml"] -['test_sanityManifest.xhtml'] -fail-if = true +["test_sanityManifest.xhtml"] +fail-if = ["true"] -['test_sanityManifest_pf.xhtml'] -fail-if = true +["test_sanityManifest_pf.xhtml"] +fail-if = ["true"] -['test_tasks_skip.xhtml'] +["test_tasks_skip.xhtml"] -['test_tasks_skipall.xhtml'] +["test_tasks_skipall.xhtml"] diff --git a/testing/mochitest/manifests/emulator-jb.toml b/testing/mochitest/manifests/emulator-jb.toml index b122d4732990..3481d5f7d130 100644 --- a/testing/mochitest/manifests/emulator-jb.toml +++ b/testing/mochitest/manifests/emulator-jb.toml @@ -1,2 +1,4 @@ -['include:../tests/dom/media/test/mochitest.toml'] +[DEFAULT] + +# 'include:../tests/dom/media/test/mochitest.toml' # NOTE: translated from INI: included file does not exist diff --git a/testing/mochitest/tests/Harness_sanity/mochitest.toml b/testing/mochitest/tests/Harness_sanity/mochitest.toml index fbd58e3fea88..a910407dc8a5 100644 --- a/testing/mochitest/tests/Harness_sanity/mochitest.toml +++ b/testing/mochitest/tests/Harness_sanity/mochitest.toml @@ -1,75 +1,75 @@ [DEFAULT] -['test_SimpletestGetTestFileURL.html'] +["test_SimpletestGetTestFileURL.html"] -['test_SpecialPowersExtension.html'] +["test_SpecialPowersExtension.html"] -['test_SpecialPowersExtension2.html'] +["test_SpecialPowersExtension2.html"] support-files = "file_SpecialPowersFrame1.html" -['test_SpecialPowersLoadChromeScript.html'] +["test_SpecialPowersLoadChromeScript.html"] support-files = "SpecialPowersLoadChromeScript.js" -['test_SpecialPowersLoadChromeScript_function.html'] +["test_SpecialPowersLoadChromeScript_function.html"] -['test_SpecialPowersLoadPrivilegedScript.html'] +["test_SpecialPowersLoadPrivilegedScript.html"] -['test_SpecialPowersPushPermissions.html'] +["test_SpecialPowersPushPermissions.html"] support-files = "specialPowers_framescript.js" -['test_SpecialPowersPushPrefEnv.html'] +["test_SpecialPowersPushPrefEnv.html"] -['test_SpecialPowersSandbox.html'] +["test_SpecialPowersSandbox.html"] -['test_SpecialPowersSpawn.html'] +["test_SpecialPowersSpawn.html"] support-files = "file_spawn.html" -['test_SpecialPowersSpawnChrome.html'] +["test_SpecialPowersSpawnChrome.html"] -['test_TestsRunningAfterSimpleTestFinish.html'] -skip-if = true #depends on fix for bug 1048446 +["test_TestsRunningAfterSimpleTestFinish.html"] +skip-if = ["true"] #depends on fix for bug 1048446 -['test_bug649012.html'] +["test_bug649012.html"] -['test_createFiles.html'] +["test_createFiles.html"] -['test_getweakmapkeys.html'] +["test_getweakmapkeys.html"] -['test_sanity.html'] +["test_sanity.html"] -['test_sanityEventUtils.html'] +["test_sanityEventUtils.html"] skip-if = [ "verify && (os == 'win')", # bug 688052 "fission && xorigin", # Bug 1716411 - New fission platform triage ] -['test_sanityException.html'] +["test_sanityException.html"] -['test_sanityException2.html'] +["test_sanityException2.html"] -['test_sanityParams.html'] +["test_sanityParams.html"] -['test_sanityRegisteredServiceWorker.html'] +["test_sanityRegisteredServiceWorker.html"] support-files = "empty.js" -['test_sanityRegisteredServiceWorker2.html'] +["test_sanityRegisteredServiceWorker2.html"] skip-if = [ "verify", ] support-files = "empty.js" -['test_sanitySimpletest.html'] +["test_sanitySimpletest.html"] -['test_sanityWindowSnapshot.html'] +["test_sanityWindowSnapshot.html"] -['test_sanity_cleanup.html'] +["test_sanity_cleanup.html"] -['test_sanity_cleanup2.html'] +["test_sanity_cleanup2.html"] -['test_sanity_manifest.html'] -fail-if = true +["test_sanity_manifest.html"] +fail-if = ["true"] -['test_sanity_manifest_pf.html'] -fail-if = true +["test_sanity_manifest_pf.html"] +fail-if = ["true"] -['test_sanity_waitForCondition.html'] +["test_sanity_waitForCondition.html"] diff --git a/testing/mochitest/tests/Harness_sanity/xpcshell.toml b/testing/mochitest/tests/Harness_sanity/xpcshell.toml index 97b219d12ea7..f203c1ef9f52 100644 --- a/testing/mochitest/tests/Harness_sanity/xpcshell.toml +++ b/testing/mochitest/tests/Harness_sanity/xpcshell.toml @@ -1,3 +1,5 @@ -['test_SpecialPowersSandbox.js'] +[DEFAULT] -['test_SpecialPowersSpawn.js'] +["test_SpecialPowersSandbox.js"] + +["test_SpecialPowersSpawn.js"] diff --git a/testing/mozbase/manifestparser/manifestparser/toml.py b/testing/mozbase/manifestparser/manifestparser/toml.py index 6e611d9fc0bb..e028a4b0d77b 100644 --- a/testing/mozbase/manifestparser/manifestparser/toml.py +++ b/testing/mozbase/manifestparser/manifestparser/toml.py @@ -8,9 +8,10 @@ import re from .ini import combine_fields -__all__ = ["read_toml", "alphabetize_toml_str", "add_skip_if"] +__all__ = ["read_toml", "alphabetize_toml_str", "add_skip_if", "sort_paths"] FILENAME_REGEX = r"^([A-Za-z0-9_./-]*)([Bb][Uu][Gg])([-_]*)([0-9]+)([A-Za-z0-9_./-]*)$" +DEFAULT_SECTION = "DEFAULT" def sort_paths_keyfn(k): @@ -18,7 +19,7 @@ def sort_paths_keyfn(k): if sort_paths_keyfn.rx is None: sort_paths_keyfn.rx = re.compile(FILENAME_REGEX) name = str(k) - if name == "DEFAULT": + if name == DEFAULT_SECTION: return "" m = sort_paths_keyfn.rx.findall(name) if len(m) == 1 and len(m[0]) == 5: @@ -75,7 +76,7 @@ def parse_tomlkit_str(contents): def read_toml( fp, defaults=None, - default="DEFAULT", + default=DEFAULT_SECTION, _comments=None, _separators=None, strict=True, @@ -176,26 +177,43 @@ def alphabetize_toml_str(manifest): """ from tomlkit import document, dumps, table - from tomlkit.items import KeyType, SingleKey + from tomlkit.items import Table preamble = "" + new_manifest = document() + first_section = False + sections = {} + for k, v in manifest.body: if k is None: preamble += v.as_string() + continue + if not isinstance(v, Table): + raise Exception(f"MP TOML illegal keyval in preamble: {k} = {v}") + section = None + if not first_section: + if k == DEFAULT_SECTION: + new_manifest.add(k, v) + else: + new_manifest.add(DEFAULT_SECTION, table()) + first_section = True else: - break - new_manifest = document() - if "DEFAULT" in manifest: - new_manifest.add("DEFAULT", manifest["DEFAULT"]) - else: - new_manifest.add("DEFAULT", table()) - sections = sort_paths([k for k in manifest.keys() if k != "DEFAULT"]) - for k in sections: - if k.find('"') >= 0: - section = k - else: - section = SingleKey(k, KeyType.Basic) - new_manifest.add(section, manifest[k]) + values = v.items() + if len(values) == 1: + for kk, vv in values: + if isinstance(vv, Table): # unquoted, dotted key + section = f"{k}.{kk}" + sections[section] = vv + if section is None: + section = str(k).strip("'\"") + sections[section] = v + + if not first_section: + new_manifest.add(DEFAULT_SECTION, table()) + + for section in sort_paths([k for k in sections.keys() if k != DEFAULT_SECTION]): + new_manifest.add(section, sections[section]) + manifest_str = dumps(new_manifest) # tomlkit fixups manifest_str = preamble + manifest_str.replace('"",]', "]") diff --git a/testing/mozbase/mozhttpd/tests/manifest.toml b/testing/mozbase/mozhttpd/tests/manifest.toml index fcc902c73940..59c9be5ed0e3 100644 --- a/testing/mozbase/mozhttpd/tests/manifest.toml +++ b/testing/mozbase/mozhttpd/tests/manifest.toml @@ -2,14 +2,14 @@ subsuite = "mozbase" ["api.py"] -skip-if = true +skip-if = ["true"] ["baseurl.py"] ["basic.py"] ["filelisting.py"] -skip-if = true +skip-if = ["true"] ["paths.py"] diff --git a/testing/mozbase/moztest/tests/data/srcdir/apple/a11y.toml b/testing/mozbase/moztest/tests/data/srcdir/apple/a11y.toml index 69e130fcd84c..6cd19840bc62 100644 --- a/testing/mozbase/moztest/tests/data/srcdir/apple/a11y.toml +++ b/testing/mozbase/moztest/tests/data/srcdir/apple/a11y.toml @@ -1 +1,3 @@ -['test_a11y.html'] +[DEFAULT] + +["test_a11y.html"] diff --git a/toolkit/components/backgroundtasks/tests/xpcshell/xpcshell.toml b/toolkit/components/backgroundtasks/tests/xpcshell/xpcshell.toml index 0d1ec995a4c3..229f929c6b97 100644 --- a/toolkit/components/backgroundtasks/tests/xpcshell/xpcshell.toml +++ b/toolkit/components/backgroundtasks/tests/xpcshell/xpcshell.toml @@ -7,6 +7,9 @@ support-files = [ "experiment.json", ] +["test_backgroundtask_automaticrestart.js"] +run-if = ["os == 'win'"] + ["test_backgroundtask_deletes_profile.js"] ["test_backgroundtask_exitcodes.js"] @@ -16,9 +19,6 @@ tags = "remote-settings" run-if = ["buildapp == 'browser'"] reason = "ASRouter is Firefox-only." -["test_backgroundtask_automaticrestart.js"] -run-if = ["os == 'win'"] - ["test_backgroundtask_help.js"] ["test_backgroundtask_localization.js"] diff --git a/toolkit/components/cookiebanners/test/browser/browser.toml b/toolkit/components/cookiebanners/test/browser/browser.toml index 5a370a262944..69728540d988 100644 --- a/toolkit/components/cookiebanners/test/browser/browser.toml +++ b/toolkit/components/cookiebanners/test/browser/browser.toml @@ -13,12 +13,10 @@ support-files = [ ["browser_bannerClicking_events.js"] support-files = ["file_banner.html"] -["browser_bannerClicking_stopExecuteAfterSeveralAttempt.js"] +["browser_bannerClicking_globalRules.js"] ["browser_bannerClicking_globalRules_telemetry.js"] -["browser_bannerClicking_globalRules.js"] - ["browser_bannerClicking_runContext.js"] support-files = ["file_banner.html"] @@ -28,6 +26,8 @@ support-files = [ "slowSubresource.sjs", ] +["browser_bannerClicking_stopExecuteAfterSeveralAttempt.js"] + ["browser_bannerClicking_telemetry_querySelector.js"] support-files = [ "file_banner.html", @@ -36,14 +36,14 @@ support-files = [ ["browser_bannerClicking_visibilityOverride.js"] support-files = ["file_banner_invisible.html"] +["browser_cookiebanner_telemetry.js"] +support-files = ["file_iframe_banner.html"] + ["browser_cookiebanner_webconsole.js"] support-files = [ "file_banner.html", ] -["browser_cookiebanner_telemetry.js"] -support-files = ["file_iframe_banner.html"] - ["browser_cookiebannerservice.js"] ["browser_cookiebannerservice_domainPrefs.js"] diff --git a/toolkit/components/passwordmgr/test/mochitest/mochitest.toml b/toolkit/components/passwordmgr/test/mochitest/mochitest.toml index b47e3f29e70d..86d5d83912bc 100644 --- a/toolkit/components/passwordmgr/test/mochitest/mochitest.toml +++ b/toolkit/components/passwordmgr/test/mochitest/mochitest.toml @@ -125,6 +125,10 @@ skip-if = [ "os == 'android'", # autocomplete ] +["test_autocomplete_password_generation_confirm.html"] +scheme = "https" +skip-if = ["os == 'android'"] # autocomplete + ["test_autocomplete_password_generation_telemetry.html"] scheme = "https" skip-if = [ @@ -132,10 +136,6 @@ skip-if = [ "os == 'android'", # autocomplete ] -["test_autocomplete_password_generation_confirm.html"] -scheme = "https" -skip-if = ["os == 'android'"] # autocomplete - ["test_autocomplete_password_open.html"] scheme = "https" skip-if = [ diff --git a/toolkit/components/printing/tests/browser.toml b/toolkit/components/printing/tests/browser.toml index 683caba9a231..8028c0f22ff1 100644 --- a/toolkit/components/printing/tests/browser.toml +++ b/toolkit/components/printing/tests/browser.toml @@ -46,10 +46,10 @@ support-files = ["file_link_modulepreload.html"] ["browser_print_context_menu.js"] -["browser_print_disabled.js"] - ["browser_print_copies.js"] +["browser_print_disabled.js"] + ["browser_print_duplex.js"] skip-if = ["verify && os == 'mac'"] # bug 1675609 diff --git a/toolkit/components/resistfingerprinting/tests/browser/browser.toml b/toolkit/components/resistfingerprinting/tests/browser/browser.toml index 0d2e5a715b01..213d6d4287a7 100644 --- a/toolkit/components/resistfingerprinting/tests/browser/browser.toml +++ b/toolkit/components/resistfingerprinting/tests/browser/browser.toml @@ -19,15 +19,15 @@ support-files = [ ["browser_fingerprintingRemoteOverrides.js"] -["browser_fingerprinting_randomization_key.js"] - ["browser_fingerprintingWebCompat.js"] +["browser_fingerprinting_randomization_key.js"] + ["browser_font_fingerprinter_telemetry.js"] ["browser_fpiServiceWorkers_fingerprinting.js"] -["browser_serviceWorker_fingerprinting_webcompat.js"] - ["browser_rfp_canvasplaceholder_pdfjs.js"] support-files = ["file_pdf.pdf"] + +["browser_serviceWorker_fingerprinting_webcompat.js"] diff --git a/toolkit/components/translations/tests/browser/browser.toml b/toolkit/components/translations/tests/browser/browser.toml index c80542b6aa03..50a1be715032 100644 --- a/toolkit/components/translations/tests/browser/browser.toml +++ b/toolkit/components/translations/tests/browser/browser.toml @@ -42,10 +42,10 @@ skip-if = ["os == 'linux'"] # Bug 1821461 ["browser_translations_lang_tags.js"] -["browser_translations_remote_settings.js"] - ["browser_translations_pdf_is_disabled.js"] +["browser_translations_remote_settings.js"] + ["browser_translations_shadow_dom.js"] ["browser_translations_shadow_dom_mutation.js"] diff --git a/tools/lint/test-manifest-toml.yml b/tools/lint/test-manifest-toml.yml new file mode 100644 index 000000000000..d1e4b7df9421 --- /dev/null +++ b/tools/lint/test-manifest-toml.yml @@ -0,0 +1,32 @@ +--- +test-manifest-toml: + description: ManifestParser TOML linter. + exclude: + - 'intl/icu/source/test/testdata/codepointtrie/' + - 'python/mozbuild/mozbuild/test/' + - 'testing/marionette/harness/marionette_harness/tests/unit-tests.toml' + - 'testing/mozbase/manifestparser/tests/' + - 'third_party/rust/' + - 'toolkit/components/featuregates/test/python/data/' + - '**/.*ruff.toml' + - '**/Cargo.toml' + - '**/Cross.toml' + - '**/Features.toml' + - '**/ServoBindings.toml' + - '**/askama.toml' + - '**/audits.toml' + - '**/cbindgen.toml' + - '**/clippy.toml' + - '**/config-lock.toml' + - '**/config.toml' + - '**/cram.toml' + - '**/empty.toml' + - '**/generated-mochitest.toml' + - '**/l10n.toml' + - '**/labels.toml' + - '**/pyproject.toml' + - '**/rustfmt.toml' + - '**/uniffi.toml' + extensions: ['toml'] + type: external + payload: test-manifest-toml:lint diff --git a/tools/lint/test-manifest-toml/__init__.py b/tools/lint/test-manifest-toml/__init__.py new file mode 100644 index 000000000000..08f0e4ed9366 --- /dev/null +++ b/tools/lint/test-manifest-toml/__init__.py @@ -0,0 +1,135 @@ +# This Source Code Form is subject to the terms of the Mozilla Public +# License, v. 2.0. If a copy of the MPL was not distributed with this +# file, You can obtain one at http://mozilla.org/MPL/2.0/. + +import io +import os +import re + +from manifestparser import TestManifest +from manifestparser.toml import DEFAULT_SECTION, alphabetize_toml_str, sort_paths +from mozlint import result +from mozlint.pathutils import expand_exclusions +from mozpack import path as mozpath +from tomlkit.items import Array, Table + +SECTION_REGEX = r"^\[.*\]$" +DISABLE_REGEX = r"^[ \t]*#[ \t]*\[.*\]" + + +def make_result(path, message, is_error=False): + if is_error: + level = "error" + else: + level = "warning" + result = { + "path": path, + "lineno": 0, # tomlkit does not report lineno/column + "column": 0, + "message": message, + "level": level, + } + return result + + +def lint(paths, config, fix=None, **lintargs): + results = [] + fixed = 0 + topsrcdir = lintargs["root"] + file_names = list(expand_exclusions(paths, config, topsrcdir)) + file_names = [os.path.normpath(f) for f in file_names] + section_rx = re.compile(SECTION_REGEX, flags=re.M) + disable_rx = re.compile(DISABLE_REGEX, flags=re.M) + + for file_name in file_names: + path = mozpath.relpath(file_name, topsrcdir) + os.path.basename(file_name) + parser = TestManifest(use_toml=True, document=True) + + try: + parser.read(file_name) + except Exception: + r = make_result(path, "The manifest is not valid TOML.", True) + results.append(result.from_config(config, **r)) + continue + + manifest = parser.source_documents[file_name] + manifest_str = io.open(file_name, "r", encoding="utf-8").read() + + if not DEFAULT_SECTION in manifest: + r = make_result( + path, f"The manifest does not start with a [{DEFAULT_SECTION}] section." + ) + if fix: + fixed += 1 + results.append(result.from_config(config, **r)) + + sections = [k for k in manifest.keys() if k != DEFAULT_SECTION] + sorted_sections = sort_paths(sections) + if sections != sorted_sections: + r = make_result( + path, "The manifest sections are not in alphabetical order." + ) + if fix: + fixed += 1 + results.append(result.from_config(config, **r)) + + m = section_rx.findall(manifest_str) + if len(m) > 0: + for section_match in m: + section = section_match[1:-1] + if section == DEFAULT_SECTION: + continue + if not section.startswith('"'): + r = make_result( + path, f"The section name must be double quoted: [{section}]" + ) + if fix: + fixed += 1 + results.append(result.from_config(config, **r)) + + m = disable_rx.findall(manifest_str) + if len(m) > 0: + for disabled_section in m: + r = make_result( + path, + f"Use 'disabled = \"\"' to disable a test instead of a comment: {disabled_section}", + True, + ) + results.append(result.from_config(config, **r)) + + for section, keyvals in manifest.body: + if section is None: + continue + if not isinstance(keyvals, Table): + r = make_result( + path, f"Bad assignment in preamble: {section} = {keyvals}", True + ) + results.append(result.from_config(config, **r)) + else: + for k, v in keyvals.items(): + if k.endswith("-if"): + if not isinstance(v, Array): + r = make_result( + path, + f'Value for conditional must be an array: {k} = "{v}"', + True, + ) + results.append(result.from_config(config, **r)) + else: + for e in v: + if e.find("||") > 0 and e.find("&&") < 0: + r = make_result( + path, + f'Value for conditional must not include explicit ||, instead put on multiple lines: {k} = [ ... "{e}" ... ]', + True, + ) + results.append(result.from_config(config, **r)) + + if fix: + manifest_str = alphabetize_toml_str(manifest) + fp = io.open(file_name, "w", encoding="utf-8", newline="\n") + fp.write(manifest_str) + fp.close() + + return {"results": results, "fixed": fixed} diff --git a/tools/lint/test/files/test-manifest-toml/comment-section.toml b/tools/lint/test/files/test-manifest-toml/comment-section.toml new file mode 100644 index 000000000000..fa7289ee906e --- /dev/null +++ b/tools/lint/test/files/test-manifest-toml/comment-section.toml @@ -0,0 +1,5 @@ +[DEFAULT] + +# ["aaa.js"] + + # ["bbb.js"] diff --git a/tools/lint/test/files/test-manifest-toml/invalid.toml b/tools/lint/test/files/test-manifest-toml/invalid.toml new file mode 100644 index 000000000000..d9e8ef6463cf --- /dev/null +++ b/tools/lint/test/files/test-manifest-toml/invalid.toml @@ -0,0 +1,2 @@ +# Invalid TOML +& = @ diff --git a/tools/lint/test/files/test-manifest-toml/no-default.toml b/tools/lint/test/files/test-manifest-toml/no-default.toml new file mode 100644 index 000000000000..b891d4ffb894 --- /dev/null +++ b/tools/lint/test/files/test-manifest-toml/no-default.toml @@ -0,0 +1 @@ +# this Manifest has no DEFAULT section diff --git a/tools/lint/test/files/test-manifest-toml/non-double-quote-sections.toml b/tools/lint/test/files/test-manifest-toml/non-double-quote-sections.toml new file mode 100644 index 000000000000..abfa25eb6875 --- /dev/null +++ b/tools/lint/test/files/test-manifest-toml/non-double-quote-sections.toml @@ -0,0 +1,6 @@ +[DEFAULT] + +[aaa.js] +# Every non DEFAULT section must be double quoted + +['bbb.js'] diff --git a/tools/lint/test/files/test-manifest-toml/skip-if-explicit-or.toml b/tools/lint/test/files/test-manifest-toml/skip-if-explicit-or.toml new file mode 100644 index 000000000000..082baca3c3db --- /dev/null +++ b/tools/lint/test/files/test-manifest-toml/skip-if-explicit-or.toml @@ -0,0 +1,7 @@ +[DEFAULT] + +["aaa.js"] +skip-if = ["asan || verify"] # should be two lines + +["bbb.js"] +skip-if = ["(asan || verify) && os == 'linux'"] # OK diff --git a/tools/lint/test/files/test-manifest-toml/skip-if-not-array.toml b/tools/lint/test/files/test-manifest-toml/skip-if-not-array.toml new file mode 100644 index 000000000000..e6f7460fb96b --- /dev/null +++ b/tools/lint/test/files/test-manifest-toml/skip-if-not-array.toml @@ -0,0 +1,4 @@ +[DEFAULT] + +["aaa.js"] +run-if = "os == 'linux'" diff --git a/tools/lint/test/files/test-manifest-toml/unsorted.toml b/tools/lint/test/files/test-manifest-toml/unsorted.toml new file mode 100644 index 000000000000..5e949daa04a3 --- /dev/null +++ b/tools/lint/test/files/test-manifest-toml/unsorted.toml @@ -0,0 +1,10 @@ +[DEFAULT] +# unsorted sections + +["ccc.js"] + +["aaa.js"] + +["bug_10.js"] + +["bug_2.js"] diff --git a/tools/lint/test/files/test-manifest-toml/valid.toml b/tools/lint/test/files/test-manifest-toml/valid.toml new file mode 100644 index 000000000000..20ff24979d28 --- /dev/null +++ b/tools/lint/test/files/test-manifest-toml/valid.toml @@ -0,0 +1,2 @@ +[DEFAULT] +# a minimal valid ManifestParser TOML file diff --git a/tools/lint/test/python.toml b/tools/lint/test/python.toml index 7ad524c41f8a..65036449a83c 100644 --- a/tools/lint/test/python.toml +++ b/tools/lint/test/python.toml @@ -30,6 +30,8 @@ skip-if = ["os == 'win'"] ["test_manifest_alpha.py"] +["test_manifest_toml.py"] + ["test_perfdocs.py"] ["test_perfdocs_generation.py"] diff --git a/tools/lint/test/test_manifest_toml.py b/tools/lint/test/test_manifest_toml.py new file mode 100644 index 000000000000..f205a1cd1f9d --- /dev/null +++ b/tools/lint/test/test_manifest_toml.py @@ -0,0 +1,78 @@ +# This Source Code Form is subject to the terms of the Mozilla Public +# License, v. 2.0. If a copy of the MPL was not distributed with this +# file, You can obtain one at http://mozilla.org/MPL/2.0/. +import mozunit + +LINTER = "test-manifest-toml" +fixed = 0 + + +def test_valid(lint, paths): + results = lint(paths("valid.toml")) + assert len(results) == 0 + + +def test_invalid(lint, paths): + results = lint(paths("invalid.toml")) + assert len(results) == 1 + assert results[0].message == "The manifest is not valid TOML." + + +def test_no_default(lint, paths): + """Test verifying [DEFAULT] section.""" + results = lint(paths("no-default.toml")) + assert len(results) == 1 + assert results[0].message == "The manifest does not start with a [DEFAULT] section." + + +def test_no_default_fix(lint, paths, create_temp_file): + """Test fixing missing [DEFAULT] section.""" + contents = "# this Manifest has no DEFAULT section\n" + path = create_temp_file(contents, "no-default.toml") + results = lint([path], fix=True) + assert len(results) == 1 + assert results[0].message == "The manifest does not start with a [DEFAULT] section." + assert fixed == 1 + + +def test_non_double_quote_sections(lint, paths): + """Test verifying [DEFAULT] section.""" + results = lint(paths("non-double-quote-sections.toml")) + assert len(results) == 2 + assert results[0].message.startswith("The section name must be double quoted:") + + +def test_unsorted(lint, paths): + """Test sections in alpha order.""" + results = lint(paths("unsorted.toml")) + assert len(results) == 1 + assert results[0].message == "The manifest sections are not in alphabetical order." + + +def test_comment_section(lint, paths): + """Test for commented sections.""" + results = lint(paths("comment-section.toml")) + assert len(results) == 2 + assert results[0].message.startswith( + "Use 'disabled = \"\"' to disable a test instead of a comment:" + ) + + +def test_skip_if_not_array(lint, paths): + """Test for non-array skip-if value.""" + results = lint(paths("skip-if-not-array.toml")) + assert len(results) == 1 + assert results[0].message.startswith("Value for conditional must be an array:") + + +def test_skip_if_explicit_or(lint, paths): + """Test for explicit || in skip-if.""" + results = lint(paths("skip-if-explicit-or.toml")) + assert len(results) == 1 + assert results[0].message.startswith( + "Value for conditional must not include explicit ||, instead put on multiple lines:" + ) + + +if __name__ == "__main__": + mozunit.main() diff --git a/widget/tests/browser/browser.toml b/widget/tests/browser/browser.toml index 3f5fff9c5d48..506fe1a998c4 100644 --- a/widget/tests/browser/browser.toml +++ b/widget/tests/browser/browser.toml @@ -8,14 +8,14 @@ skip-if = ["os == 'linux'"] # Linux build has not implemented sendNativeKeyEvent ["browser_test_InputContextURI.js"] +["browser_test_clipboard_contextmenu.js"] + ["browser_test_clipboardcache.js"] skip-if = [ "os == 'win' && bits == 32 && !debug", # Bug 1759422 "os == 'linux'", # Bug 1792749 ] -["browser_test_clipboard_contextmenu.js"] - ["browser_test_fullscreen_size.js"] ["browser_test_ime_state_in_contenteditable_on_focus_move_in_remote_content.js"]