DEV: use Qunit.skip to skip a test

DEV: use Qunit.skip to skip a test

commented out code tends to get forgotten, at least with skip we can alert on it.

diff --git a/test/javascripts/acceptance/category-chooser-test.js.es6 b/test/javascripts/acceptance/category-chooser-test.js.es6
index 0a1423a..4c09552 100644
--- a/test/javascripts/acceptance/category-chooser-test.js.es6
+++ b/test/javascripts/acceptance/category-chooser-test.js.es6
@@ -18,15 +18,15 @@ QUnit.test("does not display uncategorized if not allowed", async assert => {
   assert.ok(categoryChooser.rowByIndex(0).name() !== "uncategorized");
 });
 
-// TO-DO: fix the test to work with new code to land on category page
+// TODO: fix the test to work with new code to land on category page
 // (https://github.com/discourse/discourse/commit/7d9c97d66141d35d00258fe544211d9fd7f79a76)
-// QUnit.test("prefill category when category_id is set", async assert => {
-//   await visit("/new-topic?category_id=1");
+QUnit.skip("prefill category when category_id is set", async assert => {
+  await visit("/new-topic?category_id=1");
 
-//   assert.equal(
-//     selectKit(".category-chooser")
-//       .header()
-//       .value(),
-//     1
-//   );
-// });
+  assert.equal(
+    selectKit(".category-chooser")
+      .header()
+      .value(),
+    1
+  );
+});

GitHub sha: 2f8f8019

1 Like