Merge pull request #8 from DFE-Digital/ci

DFE-Digital · Jun 7, 2022 · 52acbc5 · 52acbc5
2 parents 66846b1 + 2a7db37
commit 52acbc5
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 9 deletions.
diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
@@ -1,15 +1,15 @@
-name: build
+name: Build
 
 on:
   workflow_dispatch:
-  push:
-    branches: [ $default-branch ]
   pull_request:
-    branches: [ $default-branch ]
+    branches:
+      - main
 
 jobs:
   test:
     runs-on: ubuntu-latest
+    name: Run specs
     strategy:
       matrix:
         ruby-version: ['2.7']

diff --git a/spec/requests/analytics_spec.rb b/spec/requests/analytics_spec.rb
@@ -1,14 +1,16 @@
 RSpec.describe 'Analytics flow', type: :request do
   before do
+    model = Class.new(Candidate) do
+      include DfE::Analytics::Entities
+    end
+
+    stub_const('Candidate', model)
+
     controller = Class.new(ApplicationController) do
       include DfE::Analytics::Requests
 
       def index
-        model = Class.new(Candidate) do
-          include DfE::Analytics::Entities
-        end
-
-        model.create(
+        Candidate.create(
           email_address: 'a@b.com',
           first_name: 'Mr',
           last_name: 'Knox'