Fix TruffleRuby simplecov. Also move Rubocop into the core test.yml f…

…ile.
SAML-Toolkits · Jul 10, 2024 · 3260b0a · 3260b0a
1 parent 3229214
commit 3260b0a
Show file tree

Hide file tree

Showing 3 changed files with 27 additions and 40 deletions.
diff --git a/.github/workflows/rubocop.yml b/.github/workflows/rubocop.yml
diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
@@ -3,6 +3,23 @@ name: ruby-saml CI
 on: [push, pull_request]
 
 jobs:
+  rubocop:
+    runs-on: ubuntu-latest
+    env:
+      CI: true
+      TESTOPTS: '-v'
+    steps:
+      - uses: actions/checkout@v3
+      - name: Set up Ruby 3.3
+        uses: ruby/setup-ruby@v1
+        with:
+          ruby-version: 3.3
+          bundler-cache: true
+      - name: Install dependencies
+        run: bundle install
+      - name: Run RuboCop
+        run: bundle exec rubocop --parallel
+
   test:
     name: Unit test
     strategy:
@@ -40,9 +57,6 @@ jobs:
 
       - name: Coveralls
         uses: coverallsapp/github-action@master
-        # 2023/03/07 - Simplecov is not working on TruffleRuby.
-        # TruffleRuby tests are otherwise passing.
-        if: ${{ matrix.ruby-version != 'truffleruby' }}
         with:
           github-token: ${{ secrets.github_token }}
           parallel: true

diff --git a/test/test_helper.rb b/test/test_helper.rb
@@ -1,16 +1,13 @@
-# 2023/03/07 - Simplecov is not working on TruffleRuby.
-unless defined?(TruffleRuby)
-  require 'simplecov'
-  require 'simplecov-lcov'
-  SimpleCov::Formatter::LcovFormatter.config.report_with_single_file = true
-  SimpleCov::Formatter::LcovFormatter.config.output_directory = 'coverage'
-  SimpleCov::Formatter::LcovFormatter.config.lcov_file_name = 'lcov.info'
-  SimpleCov.formatter = SimpleCov::Formatter::LcovFormatter
-  SimpleCov.start do
-    add_filter "test/"
-    add_filter "vendor/"
-    add_filter "lib/ruby_saml/logging.rb"
-  end
+require 'simplecov'
+require 'simplecov-lcov'
+SimpleCov::Formatter::LcovFormatter.config.report_with_single_file = true
+SimpleCov::Formatter::LcovFormatter.config.output_directory = 'coverage'
+SimpleCov::Formatter::LcovFormatter.config.lcov_file_name = 'lcov.info'
+SimpleCov.formatter = SimpleCov::Formatter::LcovFormatter
+SimpleCov.start do
+  add_filter "test/"
+  add_filter "vendor/"
+  add_filter "lib/ruby_saml/logging.rb"
 end
 
 require 'stringio'