Bug 1401958 - Remove browser_webconsole_copy_entire_message_context_menu.js; r=Honza. draft
authorNicolas Chevobbe <nchevobbe@mozilla.com>
Tue, 07 Nov 2017 12:05:57 +0100
changeset 694116 2c690ac0c5e3685d3c05c11e5cb4524df6498a57
parent 694011 fab80f8fc8b5fe322f1ec4a5b9d494d9bf9416c3
child 694120 fa0fd95292c31fe178a94496b671c4e443e8a6d8
push id88042
push userbmo:nchevobbe@mozilla.com
push dateTue, 07 Nov 2017 11:08:05 +0000
reviewersHonza
bugs1401958
milestone58.0a1
Bug 1401958 - Remove browser_webconsole_copy_entire_message_context_menu.js; r=Honza. The test already exists with a different name (browser_webconsole_context_menu_copy_entire_message). MozReview-Commit-ID: 9NmXosnAVAi
devtools/client/webconsole/new-console-output/test/mochitest/browser.ini
devtools/client/webconsole/new-console-output/test/mochitest/browser_webconsole_copy_entire_message_context_menu.js
--- a/devtools/client/webconsole/new-console-output/test/mochitest/browser.ini
+++ b/devtools/client/webconsole/new-console-output/test/mochitest/browser.ini
@@ -263,20 +263,16 @@ subsuite = clipboard
 skip-if = (os == 'linux' && bits == 32 && debug) # bug 1328915, disable linux32 debug devtools for timeouts
 [browser_webconsole_context_menu_copy_link_location.js]
 subsuite = clipboard
 skip-if = (os == 'linux' && bits == 32 && debug) # bug 1328915, disable linux32 debug devtools for timeouts
 [browser_webconsole_context_menu_copy_object.js]
 subsuite = clipboard
 [browser_webconsole_context_menu_open_url.js]
 [browser_webconsole_context_menu_store_as_global.js]
-[browser_webconsole_copy_entire_message_context_menu.js]
-subsuite = clipboard
-skip-if = true #	Bug 1401958
-# old console skip-if = (os == 'linux' && bits == 32 && debug) # bug 1328915, disable linux32 debug devtools for timeouts
 [browser_webconsole_copy_link_location.js]
 skip-if = true #	Bug 1401944
 [browser_webconsole_csp_ignore_reflected_xss_message.js]
 skip-if = true # Bug 1408931
 # old console skip-if = (e10s && debug) || (e10s && os == 'win') # Bug 1221499 enabled these on windows
 [browser_webconsole_cspro.js]
 skip-if = true # Bug 1408932
 # old console skip-if = e10s && (os == 'win' || os == 'mac') # Bug 1243967
deleted file mode 100644
--- a/devtools/client/webconsole/new-console-output/test/mochitest/browser_webconsole_copy_entire_message_context_menu.js
+++ /dev/null
@@ -1,97 +0,0 @@
-/* -*- indent-tabs-mode: nil; js-indent-level: 2 -*- */
-/* vim: set ft=javascript ts=2 et sw=2 tw=80: */
-/* Any copyright is dedicated to the Public Domain.
- * http://creativecommons.org/publicdomain/zero/1.0/ */
-
-/* globals goDoCommand */
-
-"use strict";
-
-// Test copying of the entire console message when right-clicked
-// with no other text selected. See Bug 1100562.
-
-add_task(function* () {
-  let hud;
-  let outputNode;
-  let contextMenu;
-
-  const TEST_URI = "http://example.com/browser/devtools/client/webconsole/test/test-console.html";
-
-  const { tab, browser } = yield loadTab(TEST_URI);
-  hud = yield openConsole(tab);
-  outputNode = hud.outputNode;
-  contextMenu = hud.iframeWindow.document.getElementById("output-contextmenu");
-
-  registerCleanupFunction(() => {
-    hud = outputNode = contextMenu = null;
-  });
-
-  hud.jsterm.clearOutput();
-
-  yield ContentTask.spawn(browser, {}, function* () {
-    let button = content.document.getElementById("testTrace");
-    button.click();
-  });
-
-  let results = yield waitForMessages({
-    webconsole: hud,
-    messages: [
-      {
-        text: "bug 1100562",
-        category: CATEGORY_WEBDEV,
-        severity: SEVERITY_LOG,
-        lines: 1,
-      },
-      {
-        name: "console.trace output",
-        consoleTrace: true,
-        lines: 3,
-      },
-    ]
-  });
-
-  outputNode.focus();
-
-  for (let result of results) {
-    let message = [...result.matched][0];
-
-    yield waitForContextMenu(contextMenu, message, () => {
-      let copyItem = contextMenu.querySelector("#cMenu_copy");
-      copyItem.doCommand();
-
-      let controller = top.document.commandDispatcher
-                                   .getControllerForCommand("cmd_copy");
-      is(controller.isCommandEnabled("cmd_copy"), true, "cmd_copy is enabled");
-    });
-
-    let clipboardText;
-
-    yield waitForClipboardPromise(
-      () => goDoCommand("cmd_copy"),
-      (str) => {
-        clipboardText = str;
-        return message.textContent == clipboardText;
-      }
-    );
-
-    ok(clipboardText, "Clipboard text was found and saved");
-
-    let lines = clipboardText.split("\n");
-    ok(lines.length > 0, "There is at least one newline in the message");
-    is(lines.pop(), "", "There is a newline at the end");
-    is(lines.length, result.lines, `There are ${result.lines} lines in the message`);
-
-    // Test the first line for "timestamp message repeat file:line"
-    let firstLine = lines.shift();
-    ok(/^[\d:.]+ .+ \d+ .+:\d+$/.test(firstLine),
-      "The message's first line has the right format");
-
-    // Test the remaining lines (stack trace) for "TABfunctionName sourceURL:line:col"
-    for (let line of lines) {
-      ok(/^\t.+ .+:\d+:\d+$/.test(line), "The stack trace line has the right format");
-    }
-  }
-
-  yield closeConsole(tab);
-  yield finishTest();
-});