Stub constant in spec rather than defining it

By defining it we may inadvertently overwrite an existing constant for the whole test suite. Use RSpec's `stub_const` to only overwrite it for this one test.
appsignal · Mar 9, 2022 · 2d8c525 · 2d8c525
1 parent 89a6224
commit 2d8c525
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/spec/lib/appsignal/transaction_spec.rb b/spec/lib/appsignal/transaction_spec.rb
@@ -1350,8 +1350,8 @@ def session_exists?(_env)
     end
 
     context "with a PG::UniqueViolation" do
-      module PG
-        class UniqueViolation < StandardError; end
+      before do
+        stub_const("PG::UniqueViolation", Class.new(StandardError))
       end
 
       let(:error) do