add response code assertions

DataDog · Jan 13, 2025 · 3f50937 · 3f50937
1 parent e6e4fcf
commit 3f50937
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 3 deletions.
diff --git a/integration/apps/rails-five/spec/integration/di_spec.rb b/integration/apps/rails-five/spec/integration/di_spec.rb
@@ -6,9 +6,12 @@
   di_test
 
   describe 'ActiveRecord integration' do
-    subject { JSON.parse(get('di/ar_serializer').body) }
+    let(:response) { get('di/ar_serializer') }
+    subject { JSON.parse(response.body) }
 
     it 'is loaded' do
+      expect(response.code).to eq '200'
+
       # If AR integration is loaded, this output will be the result of
       # the custom serializer.
       # If AR integration is not loaded, the output here will have a bunch of

diff --git a/integration/apps/rails-seven/spec/integration/di_spec.rb b/integration/apps/rails-seven/spec/integration/di_spec.rb
@@ -6,9 +6,12 @@
   di_test
 
   describe 'ActiveRecord integration' do
-    subject { JSON.parse(get('di/ar_serializer').body) }
+    let(:response) { get('di/ar_serializer') }
+    subject { JSON.parse(response.body) }
 
     it 'is loaded' do
+      expect(response.code).to eq '200'
+
       # If AR integration is loaded, this output will be the result of
       # the custom serializer.
       # If AR integration is not loaded, the output here will have a bunch of

diff --git a/integration/apps/rails-six/spec/integration/di_spec.rb b/integration/apps/rails-six/spec/integration/di_spec.rb
@@ -6,9 +6,12 @@
   di_test
 
   describe 'ActiveRecord integration' do
-    subject { JSON.parse(get('di/ar_serializer').body) }
+    let(:response) { get('di/ar_serializer') }
+    subject { JSON.parse(response.body) }
 
     it 'is loaded' do
+      expect(response.code).to eq '200'
+
       # If AR integration is loaded, this output will be the result of
       # the custom serializer.
       # If AR integration is not loaded, the output here will have a bunch of