Merge pull request #16 from betacraft/RUB-14

RUB-14 Add rspec workflow for github actions
betacraft · Oct 12, 2023 · 0dd1491 · 0dd1491
2 parents 3db395f + fc5f8b6
commit 0dd1491
Show file tree

Hide file tree

Showing 12 changed files with 64 additions and 12 deletions.
diff --git a/.github/workflows/rspec.yml b/.github/workflows/rspec.yml
@@ -0,0 +1,15 @@
+name: Run tests
+on: [push]
+jobs:
+  test:
+    runs-on: ubuntu-latest
+    steps:
+    - uses: actions/checkout@v4
+    - uses: ruby/setup-ruby@v1
+      with:
+        ruby-version: '2.7'
+        bundler-cache: true # runs 'bundle install' and caches installed gems automatically
+    - run: bundle exec rake
+    # Follow steps to debug => https://til.simonwillison.net/github-actions/debug-tmate
+    # - name: Setup tmate session
+    #   uses: mxschmitt/action-tmate@v3
diff --git a/Gemfile.lock b/Gemfile.lock
@@ -32,6 +32,7 @@ GEM
     google-cloud-translate-v3 (0.9.0)
       gapic-common (>= 0.20.0, < 2.a)
       google-cloud-errors (~> 1.0)
+    google-protobuf (3.24.3)
     google-protobuf (3.24.3-x86_64-linux)
     googleapis-common-protos (1.4.0)
       google-protobuf (~> 3.14)
@@ -45,6 +46,9 @@ GEM
       multi_json (~> 1.11)
       os (>= 0.9, < 2.0)
       signet (>= 0.16, < 2.a)
+    grpc (1.58.0)
+      google-protobuf (~> 3.23)
+      googleapis-common-protos-types (~> 1.0)
     grpc (1.58.0-x86_64-linux)
       google-protobuf (~> 3.23)
       googleapis-common-protos-types (~> 1.0)
@@ -81,6 +85,7 @@ GEM
     yaml (0.2.0)
 
 PLATFORMS
+  ruby
   x86_64-linux
 
 DEPENDENCIES

diff --git a/ruby-vernac-parser.gemspec b/ruby-vernac-parser.gemspec
@@ -11,7 +11,7 @@ Gem::Specification.new do |spec|
   spec.summary = "Write a short summary, because RubyGems requires one."
   spec.homepage = "https://github.com/betacraft/ruby-vernac-parser"
   spec.license = "MIT"
-  spec.required_ruby_version = ">= 2.6.0"
+  spec.required_ruby_version = ">= 2.7"
 
   spec.metadata["homepage_uri"] = "https://github.com/betacraft/ruby-vernac-parser"
   spec.metadata["source_code_uri"] = "https://github.com/betacraft/ruby-vernac-parser"

diff --git a/spec/generators/language_classes_generator_spec.rb b/spec/generators/language_classes_generator_spec.rb
@@ -16,7 +16,19 @@
       subject.generate_class_files
 
       ['array.yml', 'class.yml', 'integer.yml', 'math.yml', 'module.yml', 'object.yml'].each do |file|
-        expect(FileUtils.compare_file(Dir.pwd + '/spec/stubs/gem_path/lib/translations/classes/' + file, Dir.pwd + '/spec/stubs/language_classes_generator/' + file)).to be_truthy
+        generated = YAML.load_file(Dir.pwd + '/spec/stubs/gem_path/lib/translations/classes/' + file)
+        orig = YAML.load_file(Dir.pwd + '/spec/stubs/language_classes_generator/' + file)
+
+        klass = file.split('.')[0]
+        expect(generated[klass]['cname']).to eq(orig[klass]['cname'])
+
+        orig[klass].keys.each do |method_type|
+          next if method_type == 'cname'
+
+          orig[klass][method_type].keys do |method|
+            expect(generated[klass][method_type][method]).to eq(orig[klass][method_type][method])
+          end
+        end
       end
     end
   end

diff --git a/spec/stubs/language_classes_generator/array.yml b/spec/stubs/language_classes_generator/array.yml
@@ -129,7 +129,6 @@ array:
     singleton_methods: singleton_methods
     stub: stub
     stub_chain: stub_chain
-    subclasses: subclasses
     superclass: superclass
     taint: taint
     tainted?: tainted?
@@ -237,6 +236,7 @@ array:
     system: system
     test: test
     throw: throw
+    timeout: timeout
     trace_var: trace_var
     trap: trap
     untrace_var: untrace_var
@@ -339,7 +339,6 @@ array:
     instance_variable_get: instance_variable_get
     instance_variable_set: instance_variable_set
     instance_variables: instance_variables
-    intersect?: intersect?
     intersection: intersection
     is_a?: is_a?
     itself: itself
@@ -532,6 +531,7 @@ array:
     system: system
     test: test
     throw: throw
+    timeout: timeout
     trace_var: trace_var
     trap: trap
     untrace_var: untrace_var

diff --git a/spec/stubs/language_classes_generator/class.yml b/spec/stubs/language_classes_generator/class.yml
@@ -129,7 +129,6 @@ class:
     singleton_methods: singleton_methods
     stub: stub
     stub_chain: stub_chain
-    subclasses: subclasses
     superclass: superclass
     taint: taint
     tainted?: tainted?
@@ -237,6 +236,7 @@ class:
     system: system
     test: test
     throw: throw
+    timeout: timeout
     trace_var: trace_var
     trap: trap
     untrace_var: untrace_var
@@ -369,7 +369,6 @@ class:
     singleton_methods: singleton_methods
     stub: stub
     stub_chain: stub_chain
-    subclasses: subclasses
     superclass: superclass
     taint: taint
     tainted?: tainted?
@@ -475,6 +474,7 @@ class:
     system: system
     test: test
     throw: throw
+    timeout: timeout
     trace_var: trace_var
     trap: trap
     untrace_var: untrace_var

diff --git a/spec/stubs/language_classes_generator/integer.yml b/spec/stubs/language_classes_generator/integer.yml
@@ -128,7 +128,6 @@ integer:
     sqrt: sqrt
     stub: stub
     stub_chain: stub_chain
-    subclasses: subclasses
     superclass: superclass
     taint: taint
     tainted?: tainted?
@@ -139,7 +138,6 @@ integer:
     to_s: to_s
     to_yaml: to_yaml
     trust: trust
-    try_convert: try_convert
     undef_method: undef_method
     unstub: unstub
     untaint: untaint
@@ -236,6 +234,7 @@ integer:
     system: system
     test: test
     throw: throw
+    timeout: timeout
     trace_var: trace_var
     trap: trap
     untrace_var: untrace_var
@@ -482,6 +481,7 @@ integer:
     system: system
     test: test
     throw: throw
+    timeout: timeout
     trace_var: trace_var
     trap: trap
     untrace_var: untrace_var

diff --git a/spec/stubs/language_classes_generator/math.yml b/spec/stubs/language_classes_generator/math.yml
@@ -258,6 +258,7 @@ math:
     system: system
     test: test
     throw: throw
+    timeout: timeout
     trace_var: trace_var
     trap: trap
     untrace_var: untrace_var

diff --git a/spec/stubs/language_classes_generator/module.yml b/spec/stubs/language_classes_generator/module.yml
@@ -17,6 +17,7 @@ module:
     __id__: __id__
     __send__: __send__
     alias_method: alias_method
+    allocate: allocate
     ancestors: ancestors
     any_instance: any_instance
     as_null_object: as_null_object
@@ -128,7 +129,6 @@ module:
     singleton_methods: singleton_methods
     stub: stub
     stub_chain: stub_chain
-    subclasses: subclasses
     superclass: superclass
     taint: taint
     tainted?: tainted?
@@ -236,6 +236,7 @@ module:
     system: system
     test: test
     throw: throw
+    timeout: timeout
     trace_var: trace_var
     trap: trap
     untrace_var: untrace_var
@@ -472,6 +473,7 @@ module:
     system: system
     test: test
     throw: throw
+    timeout: timeout
     trace_var: trace_var
     trap: trap
     untrace_var: untrace_var

diff --git a/spec/stubs/language_classes_generator/object.yml b/spec/stubs/language_classes_generator/object.yml
@@ -128,7 +128,6 @@ object:
     singleton_methods: singleton_methods
     stub: stub
     stub_chain: stub_chain
-    subclasses: subclasses
     superclass: superclass
     taint: taint
     tainted?: tainted?
@@ -235,6 +234,7 @@ object:
     system: system
     test: test
     throw: throw
+    timeout: timeout
     trace_var: trace_var
     trap: trap
     untrace_var: untrace_var
@@ -396,6 +396,7 @@ object:
     system: system
     test: test
     throw: throw
+    timeout: timeout
     trace_var: trace_var
     trap: trap
     untrace_var: untrace_var

diff --git a/spec/templates/translator/language_based_translator_spec.rb b/spec/templates/translator/language_based_translator_spec.rb
@@ -21,7 +21,23 @@
 
         translated_class_path = Dir.pwd + "/spec/stubs/translated/classes/#{filename}"
 
-        expect(FileUtils.compare_file(translated_class_path, target_class_path)).to be_truthy
+        compare_translations(translated_class_path, target_class_path, filename)
+      end
+    end
+  end
+
+  def compare_translations(orig, generated, filename)
+    orig = YAML.load_file(orig)
+    generated = YAML.load_file(generated)
+
+    klass = filename.split('.')[0]
+    expect(generated[klass]['cname']).to eq(orig[klass]['cname'])
+
+    orig[klass].keys.each do |method_type|
+      next if method_type == 'cname'
+
+      orig[klass][method_type].keys do |method|
+        expect(generated[klass][method_type][method]).to eq(orig[klass][method_type][method])
       end
     end
   end

diff --git a/spec/utils/file_handler_spec.rb b/spec/utils/file_handler_spec.rb
@@ -70,7 +70,7 @@
     end
 
     it "returns files in a directory" do
-      expect(subject.list_files(@file_dir)).to eq(@paths)
+      expect(subject.list_files(@file_dir).sort).to eq(@paths)
     end
 
     after :all do