Reduce nesting of language tests (#83)

Author: ErikSchierboom

test/languages/abap_test.rb

@@ -1,53 +1,49 @@
 require "test_helper"
 
-module SnippetExtractor
-  module Languages
-    class AbapTest < Minitest::Test
-      def test_simple_example
-        code = <<~CODE
-          method run.
-          * delete me
-            write 'Hello World!'.
-            "another comment
-          endmethod.
-        CODE
+class SnippetExtractor::Languages::AbapTest < Minitest::Test
+  def test_simple_example
+    code = <<~CODE
+      method run.
+      * delete me
+        write 'Hello World!'.
+        "another comment
+      endmethod.
+    CODE
 
-        expected = <<~CODE
-          method run.
-            write 'Hello World!'.
-          endmethod.
-        CODE
+    expected = <<~CODE
+      method run.
+        write 'Hello World!'.
+      endmethod.
+    CODE
 
-        assert_equal expected, ExtractSnippet.(code, :abap)
-      end
+    assert_equal expected, SnippetExtractor::ExtractSnippet.(code, :abap)
+  end
 
-      def test_strip_definition
-        code = <<~CODE
-          class foo definition.
-            public section.
-              methods run.
-            private section.
-              data stuff type standard table of string.
-          endclass.
+  def test_strip_definition
+    code = <<~CODE
+      class foo definition.
+        public section.
+          methods run.
+        private section.
+          data stuff type standard table of string.
+      endclass.
 
-          class foo implementation.
-            method run.
-          * comment1
-              write 'Hello world!'.
-              "comment2
-            endmethod.
-          endclass.
-        CODE
+      class foo implementation.
+        method run.
+      * comment1
+          write 'Hello world!'.
+          "comment2
+        endmethod.
+      endclass.
+    CODE
 
-        expected = <<~CODE
-            method run.
-              write 'Hello world!'.
-            endmethod.
-          endclass.
-        CODE
+    expected = <<~CODE
+        method run.
+          write 'Hello world!'.
+        endmethod.
+      endclass.
+    CODE
 
-        assert_equal expected, ExtractSnippet.(code, :abap)
-      end
-    end
+    assert_equal expected, SnippetExtractor::ExtractSnippet.(code, :abap)
   end
 end

test/languages/awk_test.rb

@@ -1,56 +1,49 @@
 require "test_helper"
 
-module SnippetExtractor
-  module Languages
-    class AwkTest < Minitest::Test
-      def test_full_example
-        # rubocop:disable Layout/TrailingWhitespace
-        code = <<~CODE
-          #!/usr/bin/env gawk -f
+class SnippetExtractor::Languages::AwkTest < Minitest::Test
+  def test_full_example
+    code = <<~CODE
+      #!/usr/bin/env gawk -f
 
-          BEGIN {
-              # As discussed at:
-              #   https://www.gnu.org/software/gawk/manual/html_node/gawk-split-records.html 
-              # for the context of the `RS` variable, `^` and `$` match at the
-              # beginning and end of the __file__! So the record separator "^$" can
-              # only match for empty files. Therefore, for non-empty files, no record
-              # separator is found and the whole file is a single record.
-              RS = "^$"
-          }
+      BEGIN {
+          # As discussed at:
+          #   https://www.gnu.org/software/gawk/manual/html_node/gawk-split-records.html 
+          # for the context of the `RS` variable, `^` and `$` match at the
+          # beginning and end of the __file__! So the record separator "^$" can
+          # only match for empty files. Therefore, for non-empty files, no record
+          # separator is found and the whole file is a single record.
+          RS = "^$"
+      }
 
-          END {
-              gsub(/[[:space:]]/, "")
-              if ($0 == "")
-                  print "Fine. Be that way!"
-              else {
-                  yelling = /[A-Z]/ && !/[a-z]/
-                  asking  = /\?$/
-                  switch (yelling "" asking) {
-                      case "11": print "Calm down, I know what I'm doing!"; break
-                      case "10": print "Whoa, chill out!"; break
-                      case "01": print "Sure."; break
-                      case "00": print "Whatever."; break
-                  }
+      END {
+          gsub(/[[:space:]]/, "")
+          if ($0 == "")
+              print "Fine. Be that way!"
+          else {
+              yelling = /[A-Z]/ && !/[a-z]/
+              asking  = /\?$/
+              switch (yelling "" asking) {
+                  case "11": print "Calm down, I know what I'm doing!"; break
+                  case "10": print "Whoa, chill out!"; break
+                  case "01": print "Sure."; break
+                  case "00": print "Whatever."; break
               }
           }
-        CODE
-
-        expected = <<~CODE
-          BEGIN {
-              RS = "^$"
-          }
+      }
+    CODE
 
-          END {
-              gsub(/[[:space:]]/, "")
-              if ($0 == "")
-                  print "Fine. Be that way!"
-              else {
-                  yelling = /[A-Z]/ && !/[a-z]/
-        CODE
-        # rubocop:enable Layout/TrailingWhitespace
+    expected = <<~CODE
+      BEGIN {
+          RS = "^$"
+      }
 
-        assert_equal expected, ExtractSnippet.(code, :awk)
-      end
-    end
+      END {
+          gsub(/[[:space:]]/, "")
+          if ($0 == "")
+              print "Fine. Be that way!"
+          else {
+              yelling = /[A-Z]/ && !/[a-z]/
+    CODE
+    assert_equal expected, SnippetExtractor::ExtractSnippet.(code, :awk)
   end
 end

test/languages/ballerina_test.rb

@@ -1,60 +1,56 @@
 require "test_helper"
 
-module SnippetExtractor
-  module Languages
-    class BallerinaTest < Minitest::Test
-      def test_full_example
-        code = <<~CODE
-          import ballerina/io
-
-          // Package gigasecond constains functionality to add a gigasecond
-          // to a period of time
-
-          # Add one billion seconds to a time
-          #
-          # + timestamp - a string representing the starting time
-          # + return - a string representing the end time
-          function addGigasecond(string timestamp) returns string {
-              Utc epoch = check time:utcFromString(timestamp);
-              return epoch.utcAddSeconds(1000000000).utcToString();
-          }
-        CODE
-
-        expected = <<~CODE
-          function addGigasecond(string timestamp) returns string {
-              Utc epoch = check time:utcFromString(timestamp);
-              return epoch.utcAddSeconds(1000000000).utcToString();
-          }
-        CODE
-
-        assert_equal expected, ExtractSnippet.(code, :ballerina)
-      end
-
-      def test_stop_at_first_loc
-        code = <<~CODE
-          import ballerina/io
-
-          // Package gigasecond constains functionality to add a gigasecond
-          // to a period of time
-
-          # a Doc Comment
-          function addGigasecond(string timestamp) returns string {
-              // expects input in RFC 3339 format (e.g., 2007-12-03T10:15:30.00Z)
-              Utc epoch = check time:utcFromString(timestamp);
-              return epoch.utcAddSeconds(1000000000).utcToString();
-          }
-        CODE
-
-        expected = <<~CODE
-          function addGigasecond(string timestamp) returns string {
-              // expects input in RFC 3339 format (e.g., 2007-12-03T10:15:30.00Z)
-              Utc epoch = check time:utcFromString(timestamp);
-              return epoch.utcAddSeconds(1000000000).utcToString();
-          }
-        CODE
-
-        assert_equal expected, ExtractSnippet.(code, :ballerina)
-      end
-    end
+class SnippetExtractor::Languages::BallerinaTest < Minitest::Test
+  def test_full_example
+    code = <<~CODE
+      import ballerina/io
+
+      // Package gigasecond constains functionality to add a gigasecond
+      // to a period of time
+
+      # Add one billion seconds to a time
+      #
+      # + timestamp - a string representing the starting time
+      # + return - a string representing the end time
+      function addGigasecond(string timestamp) returns string {
+          Utc epoch = check time:utcFromString(timestamp);
+          return epoch.utcAddSeconds(1000000000).utcToString();
+      }
+    CODE
+
+    expected = <<~CODE
+      function addGigasecond(string timestamp) returns string {
+          Utc epoch = check time:utcFromString(timestamp);
+          return epoch.utcAddSeconds(1000000000).utcToString();
+      }
+    CODE
+
+    assert_equal expected, SnippetExtractor::ExtractSnippet.(code, :ballerina)
+  end
+
+  def test_stop_at_first_loc
+    code = <<~CODE
+      import ballerina/io
+
+      // Package gigasecond constains functionality to add a gigasecond
+      // to a period of time
+
+      # a Doc Comment
+      function addGigasecond(string timestamp) returns string {
+          // expects input in RFC 3339 format (e.g., 2007-12-03T10:15:30.00Z)
+          Utc epoch = check time:utcFromString(timestamp);
+          return epoch.utcAddSeconds(1000000000).utcToString();
+      }
+    CODE
+
+    expected = <<~CODE
+      function addGigasecond(string timestamp) returns string {
+          // expects input in RFC 3339 format (e.g., 2007-12-03T10:15:30.00Z)
+          Utc epoch = check time:utcFromString(timestamp);
+          return epoch.utcAddSeconds(1000000000).utcToString();
+      }
+    CODE
+
+    assert_equal expected, SnippetExtractor::ExtractSnippet.(code, :ballerina)
   end
 end