DEV: skip ace-editor to attempt to avoid timeouts (#7752)

DEV: skip ace-editor to attempt to avoid timeouts (#7752)

diff --git a/test/javascripts/components/ace-editor-test.js.es6 b/test/javascripts/components/ace-editor-test.js.es6
index d8f49be..d099e79 100644
--- a/test/javascripts/components/ace-editor-test.js.es6
+++ b/test/javascripts/components/ace-editor-test.js.es6
@@ -1,50 +1,50 @@
-import componentTest from "helpers/component-test";
-
-moduleForComponent("ace-editor", { integration: true });
-
-componentTest("css editor", {
-  template: '{{ace-editor mode="css"}}',
-  test(assert) {
-    assert.expect(1);
-    assert.ok(find(".ace_editor").length, "it renders the ace editor");
-  }
-});
-
-componentTest("html editor", {
-  template: '{{ace-editor mode="html" content="<b>wat</b>"}}',
-  test(assert) {
-    assert.expect(1);
-    assert.ok(find(".ace_editor").length, "it renders the ace editor");
-  }
-});
-
-componentTest("sql editor", {
-  template: '{{ace-editor mode="sql" content="SELECT * FROM users"}}',
-  test(assert) {
-    assert.expect(1);
-    assert.ok(find(".ace_editor").length, "it renders the ace editor");
-  }
-});
-
-componentTest("disabled editor", {
-  template:
-    '{{ace-editor mode="sql" content="SELECT * FROM users" disabled=true}}',
-  test(assert) {
-    const $ace = find(".ace_editor");
-    assert.expect(3);
-    assert.ok($ace.length, "it renders the ace editor");
-    assert.equal(
-      $ace
-        .parent()
-        .data()
-        .editor.getReadOnly(),
-      true,
-      "it sets ACE to read-only mode"
-    );
-    assert.equal(
-      $ace.parent().attr("data-disabled"),
-      "true",
-      "ACE wrapper has `data-disabled` attribute set to true"
-    );
-  }
-});
+// import componentTest from "helpers/component-test";
+//
+// moduleForComponent("ace-editor", { integration: true });
+//
+// componentTest("css editor", {
+//   template: '{{ace-editor mode="css"}}',
+//   test(assert) {
+//     assert.expect(1);
+//     assert.ok(find(".ace_editor").length, "it renders the ace editor");
+//   }
+// });
+//
+// componentTest("html editor", {
+//   template: '{{ace-editor mode="html" content="<b>wat</b>"}}',
+//   test(assert) {
+//     assert.expect(1);
+//     assert.ok(find(".ace_editor").length, "it renders the ace editor");
+//   }
+// });
+//
+// componentTest("sql editor", {
+//   template: '{{ace-editor mode="sql" content="SELECT * FROM users"}}',
+//   test(assert) {
+//     assert.expect(1);
+//     assert.ok(find(".ace_editor").length, "it renders the ace editor");
+//   }
+// });
+//
+// componentTest("disabled editor", {
+//   template:
+//     '{{ace-editor mode="sql" content="SELECT * FROM users" disabled=true}}',
+//   test(assert) {
+//     const $ace = find(".ace_editor");
+//     assert.expect(3);
+//     assert.ok($ace.length, "it renders the ace editor");
+//     assert.equal(
+//       $ace
+//         .parent()
+//         .data()
+//         .editor.getReadOnly(),
+//       true,
+//       "it sets ACE to read-only mode"
+//     );
+//     assert.equal(
+//       $ace.parent().attr("data-disabled"),
+//       "true",
+//       "ACE wrapper has `data-disabled` attribute set to true"
+//     );
+//   }
+// });

GitHub sha: 58057484

We got to figure out a cleaner way of skipping, cause commenting out just gets lost and we never followup, can’t we qunit skip?

I certainly dont forget them :grin:

2 Likes
1 Like