active_force 0.23.0 → 0.25.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 36b5f3173ab1e4adb37ab96a731723843304a96b658c41c946c65fb9d5d44b30
-  data.tar.gz: 9b2a90628f126b3df5232480282e931a3518a85d8d7a321fefd53e7eccb91e74
+  metadata.gz: 2717b024f19c2c0dbf7530d606797a5880dfe629a154e6eebd083da381b917b1
+  data.tar.gz: ab29915f960873acd523a12f73af63f8c13b200e8cddbe4465ef28038cdebf2e
 SHA512:
-  metadata.gz: bf3392ac7ee848d2b4634002b9d6f7ac3983efff47aa30ecbaf4328c386a258220695251399d3974c1d53f3145b1e73dd31b9d29bad874730f7f67a346cea804
-  data.tar.gz: fa747a0140dd738b13e8956e58fac620af22b37fb896aa503273e8dec73a7c00bb7f1ea8cb87325286ea6c1afa52d139211e083178b5a406863901936ba31a99
+  metadata.gz: dc4aa9e5110884da1008add3a996387d66af6c337328d56e773457fb24e61410a63d5bd3a734ab5bf79a44c4b5962a55aca1c00314af41d1275df663fa133f16
+  data.tar.gz: 6702930b7f9d47c2290c4e00300160f5e3261ed9c22834ab4bef55a8438f6ec778f97e51aab197b23d10b1d2b6c6d20dc3e2991b728a847529cf16e7121d6983

data/.circleci/config.yml

@@ -1,4 +1,7 @@
-version: 2
+version: 2.1
+
+orbs:
+  codecov: codecov/codecov@4.0.1
 
 references:
   default_docker_ruby_executor: &default_docker_ruby_executor
@@ -51,49 +54,16 @@ jobs:
           key: active_force-{{ checksum "active_force.gemspec" }}
           paths:
             - ~/active_force/active_force_index/vendor/bundle
-      - run:
-          name: Install Code Climate Test Reporter
-          command: |
-            curl -L https://codeclimate.com/downloads/test-reporter/test-reporter-latest-linux-amd64 > ./cc-test-reporter
-            chmod +x ./cc-test-reporter
       - run:
           name: Run RSpec
           command: |
             mkdir /tmp/test-results
-            ./cc-test-reporter before-build
             TESTFILES=$(circleci tests glob "spec/**/*_spec.rb" | circleci tests split --split-by=timings)
             bundle exec rspec $TESTFILES --profile 10 --format RspecJunitFormatter --out /tmp/test-results/rspec.xml --format progress
-      - run:
-          name: Code Climate Test Coverage
-          command: |
-            ./cc-test-reporter format-coverage -t simplecov -o "coverage/codeclimate.$CIRCLE_NODE_INDEX.json"
-      - persist_to_workspace:
-          root: coverage
-          paths:
-            - codeclimate.*.json
-      - store_test_results:
-          path: /tmp/test-results
-      - store_artifacts:
-          path: /tmp/test-results
-          destination: test-results
+      - codecov/upload
       - store_artifacts:
           path: coverage
-  upload-coverage:
-    working_directory: ~/active_force/active_force_index
-    docker:
-      - *default_docker_ruby_executor
-    steps:
-      - attach_workspace:
-          at: ~/active_force/active_force_index
-      - run:
-          name: Install Code Climate Test Reporter
-          command: |
-            curl -L https://codeclimate.com/downloads/test-reporter/test-reporter-latest-linux-amd64 > ./cc-test-reporter
-            chmod +x ./cc-test-reporter
-      - run:
-          name: Combine and Upload Coverage
-          command: |
-            ./cc-test-reporter sum-coverage --output - codeclimate.*.json | ./cc-test-reporter upload-coverage --debug --input -
+
 workflows:
   version: 2
   build_and_test:
@@ -102,6 +72,3 @@ workflows:
       - rspec-test:
           requires:
             - build
-      - upload-coverage:
-          requires:
-             - rspec-test

data/.travis.yml

@@ -9,8 +9,6 @@ matrix:
     - rvm:
       - rbx-2
   fast_finish: true
-env:
-  - CODECLIMATE_REPO_TOKEN=1c5ed259429e0bfc5412974d69cfba2780962a87ca8c543d684e33fea37f7e71
 notifications:
   webhooks:
     urls:

data/CHANGELOG.md

@@ -2,6 +2,12 @@
 
 ## Not released
 
+## 0.25.0
+- Add support for Rails 8.x (https://github.com/Beyond-Finance/active_force/pull/108)
+
+## 0.24.0
+- Add support for nested select statements that are used in conjuction with nested includes (https://github.com/Beyond-Finance/active_force/pull/102)
+
 ## 0.23.0
 - Partially addresses #90. `#select` accepts a block and returns an array of filtered SObjects. (https://github.com/Beyond-Finance/active_force/pull/99)
 

data/LICENSE.txt

@@ -1,3 +1,11 @@
+[Copyright © 2024 Beyond Finance, LLC.]
+
+This library is available as open source under the terms of the
+MIT License (https://opensource.org/license/mit). This library is
+based on the open source code library "https://github.com/heroku/active_force" which was in turn forked from
+"https://github.com/ionia-corporation/active_force" and licensed
+under the MIT License.
+
 Copyright (c) 2013 Eloy Espinaco
 
 MIT License

data/README.md

@@ -218,6 +218,17 @@ Comment.includes(post: :owner)
 Comment.includes({post: {owner: :account}})
 ```
 
+You can also use #select with a multi level #includes.
+
+Examples:
+
+```ruby
+Comment.select(:body, post: [:title, :is_active]).includes(post: :owner)
+Comment.select(:body, account: :owner_id).includes({post: {owner: :account}})
+```
+
+The Sobject name in the #select must match the Sobject in the #includes for the fields to be filtered.
+
 ### Aggregates
 
 Summing the values of a column:

data/active_force.gemspec

@@ -25,12 +25,13 @@ Gem::Specification.new do |spec|
 
   spec.required_ruby_version = '>= 1.9.3'
 
-  spec.add_dependency 'activemodel', '~> 7.0'
-  spec.add_dependency 'activesupport', '~> 7.0'
+  spec.add_dependency 'activemodel', '>= 7.0'
+  spec.add_dependency 'activesupport', '>= 7.0'
   spec.add_dependency 'restforce',   '>= 5'
   spec.add_development_dependency 'rake', '>= 0'
   spec.add_development_dependency 'rspec', '>= 0'
   spec.add_development_dependency 'rspec_junit_formatter'
   spec.add_development_dependency 'pry', '>= 0'
-  spec.add_development_dependency 'simplecov', '>= 0'
+  spec.add_development_dependency 'simplecov'
+  spec.add_development_dependency 'simplecov-cobertura'
 end

data/lib/active_force/active_query.rb

@@ -1,5 +1,6 @@
 require 'active_support/all'
 require 'active_force/query'
+require 'active_force/select_builder'
 require 'forwardable'
 
 module ActiveForce
@@ -25,7 +26,7 @@ module ActiveForce
   class ActiveQuery < Query
     extend Forwardable
 
-    attr_reader :sobject, :association_mapping, :belongs_to_association_mapping
+    attr_reader :sobject, :association_mapping, :belongs_to_association_mapping, :nested_query_fields
 
     def_delegators :sobject, :sfdc_client, :build, :table_name, :mappings
     def_delegators :to_a, :blank?, :present?, :any?, :each, :map, :inspect, :pluck, :each_with_object
@@ -36,6 +37,7 @@ module ActiveForce
       @belongs_to_association_mapping = {}
       super custom_table_name || table_name
       fields sobject.fields
+      @nested_query_fields = []
     end
 
     def to_a
@@ -86,8 +88,11 @@ module ActiveForce
         end
         result
       else
-        selected_fields.map! { |field| mappings[field] }
-        super *selected_fields
+        fields_collection = ActiveForce::SelectBuilder.new(selected_fields, self).parse
+        nested_query_fields.concat(fields_collection[:nested_query_fields]) if fields_collection[:nested_query_fields]
+        return self if fields_collection[:non_nested_query_fields].blank?
+  
+        super *fields_collection[:non_nested_query_fields]
       end
     end
 
@@ -114,7 +119,7 @@ module ActiveForce
     end
 
     def includes(*relations)
-      includes_query = Association::EagerLoadBuilderForNestedIncludes.build(relations, sobject)
+      includes_query = Association::EagerLoadBuilderForNestedIncludes.build(relations, sobject, nil, nested_query_fields) 
       fields includes_query[:fields]
       association_mapping.merge!(includes_query[:association_mapping])
       self

data/lib/active_force/association/eager_load_builder_for_nested_includes.rb

@@ -6,18 +6,19 @@ module ActiveForce
     class EagerLoadBuilderForNestedIncludes
 
       class << self
-        def build(relations, current_sobject, parent_association_field = nil)
-          new(relations, current_sobject, parent_association_field).projections
+        def build(relations, current_sobject, parent_association_field = nil, query_fields = nil)
+          new(relations, current_sobject, parent_association_field, query_fields).projections
         end
       end
 
-      attr_reader :relations, :current_sobject, :association_mapping, :parent_association_field, :fields
+      attr_reader :relations, :current_sobject, :association_mapping, :parent_association_field, :fields, :query_fields
 
-      def initialize(relations, current_sobject, parent_association_field = nil)
+      def initialize(relations, current_sobject, parent_association_field = nil, query_fields = nil)
         @relations = [relations].flatten
         @current_sobject = current_sobject
         @association_mapping = {}
         @parent_association_field = parent_association_field
+        @query_fields = query_fields
         @fields = []
       end
 
@@ -37,10 +38,17 @@ module ActiveForce
       end
 
       def build_includes(association)
-        fields.concat(EagerLoadProjectionBuilder.build(association, parent_association_field))
+        fields.concat(EagerLoadProjectionBuilder.build(association, parent_association_field, query_fields_for(association)))
         association_mapping[association.sfdc_association_field.downcase] = association.relation_name
       end
 
+      def query_fields_for(association)
+        return nil if query_fields.blank?
+        query_fields_with_association = query_fields.find { |nested_field| nested_field[association.relation_name].present? }
+        return nil if query_fields_with_association.blank?
+        query_fields_with_association[association.relation_name].map { |field| association.relation_model.mappings[field] }
+      end
+
       def build_hash_includes(relation, model = current_sobject, parent_association_field = nil)
         relation.each do |key, value|
           association = model.associations[key]
@@ -63,10 +71,10 @@ module ActiveForce
 
       def build_relation(association, nested_includes)
         builder_class = ActiveForce::Association::EagerLoadProjectionBuilder.projection_builder_class(association)
-        projection_builder = builder_class.new(association)
+        projection_builder = builder_class.new(association, nil, query_fields_for(association))
         sub_query = projection_builder.query_with_association_fields
         association_mapping[association.sfdc_association_field.downcase] = association.relation_name
-        nested_includes_query = self.class.build(nested_includes, association.relation_model)
+        nested_includes_query = self.class.build(nested_includes, association.relation_model, nil, query_fields)
         sub_query.fields nested_includes_query[:fields]
         { fields: ["(#{sub_query})"], association_mapping: nested_includes_query[:association_mapping] }
       end
@@ -78,7 +86,7 @@ module ActiveForce
         else
           current_parent_association_field = association.sfdc_association_field
         end
-        self.class.build(nested_includes, association.relation_model, current_parent_association_field)
+        self.class.build(nested_includes, association.relation_model, current_parent_association_field, query_fields)
       end
     end
   end

data/lib/active_force/association/eager_load_projection_builder.rb

@@ -3,8 +3,8 @@ module ActiveForce
     class InvalidEagerLoadAssociation < StandardError; end
     class EagerLoadProjectionBuilder
       class << self
-        def build(association, parent_association_field = nil)
-          new(association, parent_association_field).projections
+        def build(association, parent_association_field = nil, query_fields = nil)
+          new(association, parent_association_field, query_fields).projections
         end
 
         def projection_builder_class(association)
@@ -15,26 +15,27 @@ module ActiveForce
         end
       end
 
-      attr_reader :association, :parent_association_field
+      attr_reader :association, :parent_association_field, :query_fields
 
-      def initialize(association, parent_association_field = nil)
+      def initialize(association, parent_association_field = nil, query_fields = nil)
         @association = association
         @parent_association_field = parent_association_field
+        @query_fields = query_fields
       end
 
       def projections
         builder_class = self.class.projection_builder_class(association)
-        builder_class.new(association, parent_association_field).projections
+        builder_class.new(association, parent_association_field, query_fields).projections
       end
-
     end
 
     class AbstractProjectionBuilder
-      attr_reader :association, :parent_association_field
+      attr_reader :association, :parent_association_field, :query_fields
 
-      def initialize(association, parent_association_field = nil)
+      def initialize(association, parent_association_field = nil, query_fields = nil)
         @association = association
         @parent_association_field = parent_association_field
+        @query_fields = query_fields
       end
 
       def projections
@@ -54,8 +55,8 @@ module ActiveForce
       # to be pluralized
       def query_with_association_fields
         relationship_name = association.sfdc_association_field
-        query = ActiveQuery.new(association.relation_model, relationship_name)
-        query.fields association.relation_model.fields
+        selected_fields = query_fields || association.relation_model.fields
+        query = ActiveQuery.new(association.relation_model, relationship_name).select(*selected_fields)
         apply_association_scope(query)
       end
     end
@@ -79,7 +80,8 @@ module ActiveForce
                             else
                               association.sfdc_association_field
                             end
-        association.relation_model.fields.map do |field|
+        selected_fields = query_fields || association.relation_model.fields
+        selected_fields.map do |field|
           "#{ association_field }.#{ field }"
         end
       end

data/lib/active_force/query.rb

@@ -1,6 +1,6 @@
 module ActiveForce
   class Query
-    attr_reader :table
+    attr_reader :table, :query_fields
 
     def initialize table
       @table = table

data/lib/active_force/select_builder.rb

@@ -0,0 +1,41 @@
+module ActiveForce
+  class SelectBuilder
+
+    attr_reader :selected_fields, :nested_query_fields, :non_nested_query_fields, :query
+
+    def initialize(selected_fields, query)
+      @query = query
+      @selected_fields = selected_fields
+      @non_nested_query_fields = []
+      @nested_query_fields = []
+    end
+
+    def parse
+      selected_fields.each do |field|
+        case field
+        when Symbol
+          non_nested_query_fields << query.mappings[field]
+        when Hash
+          populate_nested_query_fields(field)
+        when String
+          non_nested_query_fields << field
+        end
+      end
+      {non_nested_query_fields: non_nested_query_fields, nested_query_fields: nested_query_fields}
+    end
+
+    private
+
+    def populate_nested_query_fields(field)
+      field.each do |key, value|
+        case value
+        when Symbol
+          field[key] = [value]
+        when Hash
+          raise ArgumentError, 'Nested Hash is not supported in select statement, you may wish to use an Array'
+        end
+      end
+      nested_query_fields << field
+    end
+  end
+end

data/lib/active_force/sobject.rb

@@ -26,14 +26,15 @@ module ActiveForce
 
     define_model_callbacks :build, :create, :update, :save, :destroy
 
-    class_attribute :mappings, :table_name
+    class_attribute :mappings, :table_name_store
 
     attr_accessor :id, :title
 
     class << self
       extend Forwardable
       def_delegators :query, :not, :or, :where, :first, :last, :all, :find, :find!, :find_by, :find_by!, :sum, :count, :includes, :limit, :order, :select, :none
-      def_delegators :mapping, :table, :table_name, :custom_table?, :mappings
+      def_delegators :mapping, :table, :custom_table?, :mappings
+      alias_method :table_name=, :table_name_store=
 
       def update(id, attributes)
         prepare_for_update(id, attributes).update
@@ -43,6 +44,10 @@ module ActiveForce
         prepare_for_update(id, attributes).update!
       end
 
+      def table_name
+        table_name_store || mapping.table_name
+      end
+
       private
 
       def prepare_for_update(id, attributes)
@@ -65,6 +70,10 @@ module ActiveForce
       @mapping ||= ActiveForce::Mapping.new name
     end
 
+    def table_name
+      table_name_store || self.class.mapping.table_name
+    end
+
     def self.fields
       mapping.sfdc_names
     end

data/lib/active_force/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module ActiveForce
-  VERSION = '0.23.0'
+  VERSION = '0.25.0'
 end

data/spec/active_force/sobject/includes_spec.rb

@@ -43,6 +43,25 @@ module ActiveForce
           expect(territory.quota.id).to eq "321"
         end
 
+        context 'when nested select statement' do
+          it 'formulates the correct SOQL query' do
+            soql = Salesforce::Territory.select(:id, :quota_id, quota: :id).includes(:quota).where(id: '123').to_s
+            expect(soql).to eq "SELECT Id, QuotaId, QuotaId.Id FROM Territory WHERE (Id = '123')"
+          end
+
+          it 'errors when correct format is not followed' do
+            expect{Salesforce::Territory.select(:id, :quota_id, quota: {id: :quote}).includes(:quota).where(id: '123').to_s}.to raise_error ArgumentError
+          end
+
+          context 'when nested includes statement' do
+            it 'formulates the correct SOQL query' do
+              soql = Comment.select(:post_id, :body, post: [:title, :is_active], blog: :name).includes(post: :blog).to_s
+
+              expect(soql).to eq "SELECT PostId, Body__c, PostId.Title__c, PostId.IsActive, PostId.BlogId.Name FROM Comment__c"
+            end
+          end
+        end
+
         context 'with namespaced SObjects' do
           it 'queries the API for the associated record' do
             soql = Salesforce::Territory.includes(:quota).where(id: '123').to_s
@@ -156,6 +175,20 @@ module ActiveForce
       end
 
       context 'has_many' do
+        context 'when nested select statement' do
+          it 'formulates the correct SOQL query' do
+            soql = Account.select(opportunities: :id).includes(:opportunities).where(id: '123').to_s
+            expect(soql).to eq "SELECT Id, OwnerId, (SELECT Id FROM Opportunities) FROM Account WHERE (Id = '123')"
+          end
+        end
+
+        context 'when normal select with nested includes' do
+          it 'formulates the correct SOQL query' do
+            soql = Blog.select(:id, :link).includes(posts: :comments).to_s
+            expect(soql).to eq "SELECT Id, Link__c, (SELECT Id, Title__c, BlogId, IsActive, (SELECT Id, PostId, PosterId__c, FancyPostId, Body__c FROM Comments__r) FROM Posts__r) FROM Blog__c"
+          end
+        end
+
         context 'with standard objects' do
           it 'formulates the correct SOQL query' do
             soql = Account.includes(:opportunities).where(id: '123').to_s
@@ -294,6 +327,13 @@ module ActiveForce
       end
 
       context 'has_one' do
+        context 'when nested select statement is present' do
+          it 'formulates the correct SOQL query' do
+            soql = ClubMember.select(:name, :email, membership: :type).includes(:membership).to_s
+            expect(soql).to eq "SELECT Name, Email, (SELECT Type FROM Membership__r) FROM ClubMember__c"
+          end
+        end
+
         context 'when assocation has a scope' do
           it 'formulates the correct SOQL query with the scope applied' do
             soql = Post.includes(:last_comment).where(id: '1234').to_s

data/spec/active_force/sobject_spec.rb

@@ -375,6 +375,20 @@ describe ActiveForce::SObject do
     end
   end
 
+  describe '.table_name' do
+
+    context 'when no explicit table name is set' do
+      it 'derives the table name from the class name' do
+        expect(Whizbang.table_name).to eq('Whizbang__c')
+      end
+    end
+    context 'when explicit table name is set' do
+      it 'returns the explicit table name' do
+        expect(Whizbang2.table_name).to eq('Whiz_bang2__c')
+      end
+    end
+  end
+
   describe '.sum' do
     let(:response) { [Restforce::Mash.new(expr0: 22)] }
 

data/spec/spec_helper.rb

@@ -1,7 +1,10 @@
 require 'simplecov'
+require 'simplecov-cobertura'
 SimpleCov.start do
   add_filter '/vendor'
   add_filter '/spec'
+
+  formatter SimpleCov::Formatter::CoberturaFormatter
 end
 
 $LOAD_PATH.unshift File.expand_path('../../lib', __FILE__)

data/spec/support/whizbang.rb

@@ -28,3 +28,7 @@ class Whizbang < ActiveForce::SObject
   end
 
 end
+
+class Whizbang2 < Whizbang
+  self.table_name = 'Whiz_bang2__c'
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: active_force
 version: !ruby/object:Gem::Version
-  version: 0.23.0
+  version: 0.25.0
 platform: ruby
 authors:
 - Eloy Espinaco
@@ -11,34 +11,34 @@ authors:
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-09-27 00:00:00.000000000 Z
+date: 2025-08-07 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activemodel
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
         version: '7.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
         version: '7.0'
 - !ruby/object:Gem::Dependency
   name: activesupport
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
         version: '7.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
         version: '7.0'
 - !ruby/object:Gem::Dependency
@@ -125,6 +125,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: simplecov-cobertura
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 description: Use SalesForce as an ActiveModel
 email: eloyesp@gmail.com
 executables: []
@@ -134,7 +148,6 @@ files:
 - ".circleci/config.yml"
 - ".github/ISSUE_TEMPLATE/bug_report.md"
 - ".github/ISSUE_TEMPLATE/feature_request.md"
-- ".github/workflows"
 - ".gitignore"
 - ".mailmap"
 - ".rspec"
@@ -163,6 +176,7 @@ files:
 - lib/active_force/field.rb
 - lib/active_force/mapping.rb
 - lib/active_force/query.rb
+- lib/active_force/select_builder.rb
 - lib/active_force/sobject.rb
 - lib/active_force/standard_types.rb
 - lib/active_force/table.rb
@@ -219,7 +233,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.5.6
+rubygems_version: 3.4.10
 signing_key:
 specification_version: 4
 summary: Help you implement models persisting on Sales Force within Rails using RESTForce

data/.github/workflows

@@ -1,51 +0,0 @@
-on:
-  pull_request:
-    types: [opened, reopened, synchronize]
-  pull_request_review_comment:
-    types: [created, edited, deleted]
-
-name: Metomic Scan
-
-jobs:
-  scan-secrets:
-    name: Scan For Secrets
-    runs-on: ubuntu-latest
-    steps:
-      - name: checkout-repo
-        uses: actions/checkout@v3
-        with:
-          ref: ${{ github.event.pull_request.head.sha }}
-
-      - name: authenticate-with-metomic
-        id: authenticate
-        continue-on-error: ${{ vars.METOMIC_FAIL_ON_CONNECTION_ERROR != 'TRUE' }}
-        run: |
-          curl -s --location --request GET '${{ vars.METOMIC_INTEGRATION_ENDPOINT }}/token' --header 'X-API-Key: ${{ secrets.METOMIC_API_TOKEN }}' -o .metomic_token.json
-          echo METOMIC_ACCESS_TOKEN=$(cat .metomic_token.json | jq -r -c '.accessToken') >> $GITHUB_OUTPUT
-          PASS=$(cat .metomic_token.json | jq -r -c '.accessToken // empty')
-          if [ -z "$PASS" ]; then
-            cat .metomic_token.json
-            echo ""
-            echo "FAIL: failed to fetch auth token from Metomic. Ensure required organisation variables / secrets are set correctly. METOMIC_INTEGRATION_ENDPOINT, METOMIC_API_TOKEN"
-            echo "The correct values for these variables are available from the installations page of the Metomic dashboard"
-            exit 1
-          fi
-
-      - name: checkout-metomic-action
-        id: checkout
-        uses: actions/checkout@v3
-        if: ${{ steps.authenticate.outcome == 'success' }}
-        continue-on-error: ${{ vars.METOMIC_FAIL_ON_CONNECTION_ERROR != 'TRUE' }}
-        with:
-          repository: metomic/metomic-github-integration-action.git
-          path: "./.metomic"
-          ref: "main"
-          token: ${{ steps.authenticate.outputs.METOMIC_ACCESS_TOKEN }}
-
-      - name: scan
-        uses: ./.metomic/.github/actions/scan-secrets
-        if: ${{ steps.authenticate.outputs.METOMIC_ACCESS_TOKEN && steps.checkout.outcome == 'success' }}
-        with:
-          metomic_endpoint: ${{ vars.METOMIC_INTEGRATION_ENDPOINT }}
-          metomic_api_token: ${{ secrets.METOMIC_API_TOKEN }}
-          head_ref: ${{ github.event.pull_request.head.sha }}