From 8340114d720dd1ad8505d8d9858cf6644ea37f0f Mon Sep 17 00:00:00 2001
From: Andrew Davis <andrew.affinity@gmail.com>
Date: Fri, 2 Jul 2021 20:31:46 +0800
Subject: [PATCH] verify_claims test shouldnt be within the verify_sub test

---
 spec/jwt/verify_spec.rb | 44 ++++++++++++++++++++---------------------
 1 file changed, 22 insertions(+), 22 deletions(-)

diff --git a/spec/jwt/verify_spec.rb b/spec/jwt/verify_spec.rb
index da645ac8..d4757758 100644
--- a/spec/jwt/verify_spec.rb
+++ b/spec/jwt/verify_spec.rb
@@ -220,31 +220,31 @@
     it 'must allow a matching sub' do
       described_class.verify_sub(base_payload.merge('sub' => sub), options.merge(sub: sub))
     end
+  end
 
-    context '.verify_claims' do
-      let(:fail_verifications_options) { { iss: 'mismatched-issuer', aud: 'no-match', sub: 'some subject' } }
-      let(:fail_verifications_payload) {
-        {
-          'exp' => (Time.now.to_i - 50),
-          'jti' => '   ',
-          'iss' => 'some-issuer',
-          'nbf' => (Time.now.to_i + 50),
-          'iat' => 'not a number',
-          'sub' => 'not-a-match'
-        }
+  context '.verify_claims' do
+    let(:fail_verifications_options) { { iss: 'mismatched-issuer', aud: 'no-match', sub: 'some subject' } }
+    let(:fail_verifications_payload) {
+      {
+        'exp' => (Time.now.to_i - 50),
+        'jti' => '   ',
+        'iss' => 'some-issuer',
+        'nbf' => (Time.now.to_i + 50),
+        'iat' => 'not a number',
+        'sub' => 'not-a-match'
       }
+    }
+
+    %w[verify_aud verify_expiration verify_iat verify_iss verify_jti verify_not_before verify_sub].each do |method|
+      let(:payload) { base_payload.merge(fail_verifications_payload) }
+      it "must skip verification when #{method} option is set to false" do
+        described_class.verify_claims(payload, options.merge(method => false))
+      end
 
-      %w[verify_aud verify_expiration verify_iat verify_iss verify_jti verify_not_before verify_sub].each do |method|
-        let(:payload) { base_payload.merge(fail_verifications_payload) }
-        it "must skip verification when #{method} option is set to false" do
-          described_class.verify_claims(payload, options.merge(method => false))
-        end
-
-        it "must raise error when #{method} option is set to true" do
-          expect do
-            described_class.verify_claims(payload, options.merge(method => true).merge(fail_verifications_options))
-          end.to raise_error JWT::DecodeError
-        end
+      it "must raise error when #{method} option is set to true" do
+        expect do
+          described_class.verify_claims(payload, options.merge(method => true).merge(fail_verifications_options))
+        end.to raise_error JWT::DecodeError
       end
     end
   end