Browse Source

Use strings instead of numeric literals and remove `rubocop:disable` in cache spec (#27742)

Matt Jankowski 1 year ago
parent
commit
a688a9ed20
1 changed files with 3 additions and 5 deletions
  1. 3 5
      spec/requests/cache_spec.rb

+ 3 - 5
spec/requests/cache_spec.rb

@@ -171,12 +171,10 @@ describe 'Caching behavior' do
   let(:user)  { Fabricate(:user, role: UserRole.find_by(name: 'Moderator')) }
   let(:user)  { Fabricate(:user, role: UserRole.find_by(name: 'Moderator')) }
 
 
   before do
   before do
-    # rubocop:disable Style/NumericLiterals
-    status = Fabricate(:status, account: alice, id: 110224538612341312)
-    Fabricate(:status, account: alice, id: 110224538643211312, visibility: :private)
+    status = Fabricate(:status, account: alice, id: '110224538612341312')
+    Fabricate(:status, account: alice, id: '110224538643211312', visibility: :private)
     Fabricate(:invite, code: 'abcdef')
     Fabricate(:invite, code: 'abcdef')
-    Fabricate(:poll, status: status, account: alice, id: 12345)
-    # rubocop:enable Style/NumericLiterals
+    Fabricate(:poll, status: status, account: alice, id: '12345')
 
 
     user.account.follow!(alice)
     user.account.follow!(alice)
   end
   end