Merge pull request #3191 from rubygems/simplify-cc

Use CC GitHub Action at CI.
rubygems · Aug 23, 2022 · 1ce102d · 1ce102d
2 parents fbd9883 + 56bde33
commit 1ce102d
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 61 deletions.
diff --git a/.github/workflows/codeclimate.yml b/.github/workflows/codeclimate.yml
diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
@@ -18,9 +18,6 @@ jobs:
     runs-on: ubuntu-18.04
     env:
       RUBYGEMS_VERSION: ${{ matrix.rubygems_version }}
-      # If we don't supply a CC_TEST_REPORTER_ID simplecov won't output coverage in a way that
-      # code climate understands
-      CC_TEST_REPORTER_ID: aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa
       # Fail hard when Toxiproxy is not running to ensure all tests (even Toxiproxy optional ones) are passing
       REQUIRE_TOXIPROXY: true
     steps:
@@ -50,10 +47,8 @@ jobs:
         bundle exec rake db:setup
     - name: Tests
       run: bin/rails test
-    - name: archive coverage
-      if: matrix.rubygems_version == '3.3.11'
-      uses: actions/upload-artifact@v3
-      with:
-        name: coverage
-        path: |
-          coverage
+    - name: Publish code coverage
+      if: matrix.rubygems_version == '3.3.11' && (success() || failure())
+      uses: paambaati/codeclimate-action@v3.0.0
+      env:
+        CC_TEST_REPORTER_ID: ${{ secrets.CC_TEST_REPORTER_ID }}
diff --git a/test/test_helper.rb b/test/test_helper.rb
@@ -2,6 +2,11 @@
 SimpleCov.start "rails" do
   add_filter "lib/tasks"
   add_filter "lib/lograge"
+
+  if ENV["CI"]
+    require "simplecov_json_formatter"
+    formatter SimpleCov::Formatter::JSONFormatter
+  end
 end
 
 ENV["RAILS_ENV"] ||= "test"