refactor(tests): clean tests by removing unecessary describe sections

Author: clementdessoude

packages/java-parser/test/bugs-spec.js

@@ -3,14 +3,7 @@
 const { expect } = require("chai");
 const javaParser = require("../src/index");
 
-// TODO: we don't need to copy the describe section so many times in this file...
-//   `IT` should be nested under describe...
 describe("The Java Parser fixed bugs", () => {
-  it("issue #129 - this.<bar>.baz()", () => {
-    const input = "this.<Number>anyIterableType()";
-    expect(() => javaParser.parse(input, "expression")).to.not.throw();
-  });
-
   it("issue #112 - Special handling of transitive keyword in module requires statement", () => {
     const input = `module foo {
       // Regular requires statement (without modifier)
@@ -23,39 +16,7 @@ describe("The Java Parser fixed bugs", () => {
     `;
     expect(() => javaParser.parse(input)).to.not.throw();
   });
-});
-
-describe("The Java Parser fixed bugs", () => {
-  it("issue #131 - 1.0e-10", () => {
-    const input = "1.0e-10";
-    expect(() => javaParser.parse(input, "expression")).to.not.throw();
-  });
-});
-
-describe("The Java Parser fixed bugs", () => {
-  it("issue #130 - enumConstantList", () => {
-    const input = "enum Foo { BAR, }";
-    expect(() => javaParser.parse(input)).to.not.throw();
-  });
-});
-
-describe("The Java Parser fixed bugs", () => {
-  it("issue #158 - semicolons inside imports", () => {
-    const input = "import Foo;;import Bar;";
-    expect(() =>
-      javaParser.parse(input, "ordinaryCompilationUnit")
-    ).to.not.throw();
-  });
-});
-
-describe("The Java Parser fixed bugs", () => {
-  it("issue #177 - annotation in packageDeclaration", () => {
-    const input = "@Test public class Foo{}";
-    expect(() => javaParser.parse(input)).to.not.throw();
-  });
-});
 
-describe("The Java Parser fixed bugs", () => {
   it("issue #113 - Bit shifting operators must not include whitespace - >>", () => {
     const input = "public class Foo{int hello = 1 >> 5;}";
     expect(() => javaParser.parse(input)).to.not.throw();
@@ -89,4 +50,31 @@ describe("The Java Parser fixed bugs", () => {
     const input = "public class Foo{int hello = 1 < < 5;}";
     expect(() => javaParser.parse(input)).to.throw();
   });
+
+  it("issue #129 - this.<bar>.baz()", () => {
+    const input = "this.<Number>anyIterableType()";
+    expect(() => javaParser.parse(input, "expression")).to.not.throw();
+  });
+
+  it("issue #131 - 1.0e-10", () => {
+    const input = "1.0e-10";
+    expect(() => javaParser.parse(input, "expression")).to.not.throw();
+  });
+
+  it("issue #130 - enumConstantList", () => {
+    const input = "enum Foo { BAR, }";
+    expect(() => javaParser.parse(input)).to.not.throw();
+  });
+
+  it("issue #158 - semicolons inside imports", () => {
+    const input = "import Foo;;import Bar;";
+    expect(() =>
+      javaParser.parse(input, "ordinaryCompilationUnit")
+    ).to.not.throw();
+  });
+
+  it("issue #177 - annotation in packageDeclaration", () => {
+    const input = "@Test public class Foo{}";
+    expect(() => javaParser.parse(input)).to.not.throw();
+  });
 });