techhub/mastodon
2
0
Fork 0
Code Activity

Remove unneeded to_s on Link header comparison in statuses controller spec (#31941)

Browse Source
This commit is contained in:
Matt Jankowski
2024-09-18 04:21:31 -04:00
committed by GitHub
parent 6f3d7516dc
commit 943738671c
1 changed files with 15 additions and 15 deletions
Download Patch File Download Diff File Expand all files Collapse all files

30
spec/controllers/statuses_controller_spec.rb
View File

@@ -63,7 +63,7 @@ RSpec.describe StatusesController do
expect(response.headers).to include( expect(response.headers).to include(
'Vary' => 'Accept, Accept-Language, Cookie', 'Vary' => 'Accept, Accept-Language, Cookie',
'Cache-Control' => include('public'), 'Cache-Control' => include('public'),
'Link' => satisfy { |header| header.to_s.include?('activity+json') } 'Link' => include('activity+json')
) )
expect(response.body).to include status.text expect(response.body).to include status.text
end end
@@ -79,7 +79,7 @@ RSpec.describe StatusesController do
expect(response.headers).to include( expect(response.headers).to include(
'Content-Type' => include('application/activity+json'), 'Content-Type' => include('application/activity+json'),
'Link' => satisfy { |header| header.to_s.include?('activity+json') } 'Link' => include('activity+json')
) )
expect(response.parsed_body) expect(response.parsed_body)
.to include(content: include(status.text)) .to include(content: include(status.text))
@@ -168,7 +168,7 @@ RSpec.describe StatusesController do
expect(response.headers).to include( expect(response.headers).to include(
'Vary' => 'Accept, Accept-Language, Cookie', 'Vary' => 'Accept, Accept-Language, Cookie',
'Cache-Control' => include('private'), 'Cache-Control' => include('private'),
'Link' => satisfy { |header| header.to_s.include?('activity+json') } 'Link' => include('activity+json')
) )
expect(response.body).to include status.text expect(response.body).to include status.text
end end
@@ -184,7 +184,7 @@ RSpec.describe StatusesController do
'Vary' => 'Accept, Accept-Language, Cookie', 'Vary' => 'Accept, Accept-Language, Cookie',
'Cache-Control' => include('private'), 'Cache-Control' => include('private'),
'Content-Type' => include('application/activity+json'), 'Content-Type' => include('application/activity+json'),
'Link' => satisfy { |header| header.to_s.include?('activity+json') } 'Link' => include('activity+json')
) )
expect(response.parsed_body) expect(response.parsed_body)
.to include(content: include(status.text)) .to include(content: include(status.text))
@@ -212,7 +212,7 @@ RSpec.describe StatusesController do
expect(response.headers).to include( expect(response.headers).to include(
'Vary' => 'Accept, Accept-Language, Cookie', 'Vary' => 'Accept, Accept-Language, Cookie',
'Cache-Control' => include('private'), 'Cache-Control' => include('private'),
'Link' => satisfy { |header| header.to_s.include?('activity+json') } 'Link' => include('activity+json')
) )
expect(response.body).to include status.text expect(response.body).to include status.text
end end
@@ -228,7 +228,7 @@ RSpec.describe StatusesController do
'Vary' => 'Accept, Accept-Language, Cookie', 'Vary' => 'Accept, Accept-Language, Cookie',
'Cache-Control' => include('private'), 'Cache-Control' => include('private'),
'Content-Type' => include('application/activity+json'), 'Content-Type' => include('application/activity+json'),
'Link' => satisfy { |header| header.to_s.include?('activity+json') } 'Link' => include('activity+json')
) )
expect(response.parsed_body) expect(response.parsed_body)
.to include(content: include(status.text)) .to include(content: include(status.text))
@@ -278,7 +278,7 @@ RSpec.describe StatusesController do
expect(response.headers).to include( expect(response.headers).to include(
'Vary' => 'Accept, Accept-Language, Cookie', 'Vary' => 'Accept, Accept-Language, Cookie',
'Cache-Control' => include('private'), 'Cache-Control' => include('private'),
'Link' => satisfy { |header| header.to_s.include?('activity+json') } 'Link' => include('activity+json')
) )
expect(response.body).to include status.text expect(response.body).to include status.text
end end
@@ -294,7 +294,7 @@ RSpec.describe StatusesController do
'Vary' => 'Accept, Accept-Language, Cookie', 'Vary' => 'Accept, Accept-Language, Cookie',
'Cache-Control' => include('private'), 'Cache-Control' => include('private'),
'Content-Type' => include('application/activity+json'), 'Content-Type' => include('application/activity+json'),
'Link' => satisfy { |header| header.to_s.include?('activity+json') } 'Link' => include('activity+json')
) )
expect(response.parsed_body) expect(response.parsed_body)
.to include(content: include(status.text)) .to include(content: include(status.text))
@@ -370,7 +370,7 @@ RSpec.describe StatusesController do
expect(response.headers).to include( expect(response.headers).to include(
'Vary' => 'Accept, Accept-Language, Cookie', 'Vary' => 'Accept, Accept-Language, Cookie',
'Cache-Control' => include('private'), 'Cache-Control' => include('private'),
'Link' => satisfy { |header| header.to_s.include?('activity+json') } 'Link' => include('activity+json')
) )
expect(response.body).to include status.text expect(response.body).to include status.text
end end
@@ -385,7 +385,7 @@ RSpec.describe StatusesController do
.and have_cacheable_headers.with_vary('Accept, Accept-Language, Cookie') .and have_cacheable_headers.with_vary('Accept, Accept-Language, Cookie')
expect(response.headers).to include( expect(response.headers).to include(
'Content-Type' => include('application/activity+json'), 'Content-Type' => include('application/activity+json'),
'Link' => satisfy { |header| header.to_s.include?('activity+json') } 'Link' => include('activity+json')
) )
expect(response.parsed_body) expect(response.parsed_body)
.to include(content: include(status.text)) .to include(content: include(status.text))
@@ -412,7 +412,7 @@ RSpec.describe StatusesController do
expect(response.headers).to include( expect(response.headers).to include(
'Vary' => 'Accept, Accept-Language, Cookie', 'Vary' => 'Accept, Accept-Language, Cookie',
'Cache-Control' => include('private'), 'Cache-Control' => include('private'),
'Link' => satisfy { |header| header.to_s.include?('activity+json') } 'Link' => include('activity+json')
) )
expect(response.body).to include status.text expect(response.body).to include status.text
end end
@@ -428,7 +428,7 @@ RSpec.describe StatusesController do
'Vary' => 'Accept, Accept-Language, Cookie', 'Vary' => 'Accept, Accept-Language, Cookie',
'Cache-Control' => include('private'), 'Cache-Control' => include('private'),
'Content-Type' => include('application/activity+json'), 'Content-Type' => include('application/activity+json'),
'Link' => satisfy { |header| header.to_s.include?('activity+json') } 'Link' => include('activity+json')
) )
expect(response.parsed_body) expect(response.parsed_body)
@@ -479,7 +479,7 @@ RSpec.describe StatusesController do
expect(response.headers).to include( expect(response.headers).to include(
'Vary' => 'Accept, Accept-Language, Cookie', 'Vary' => 'Accept, Accept-Language, Cookie',
'Cache-Control' => include('private'), 'Cache-Control' => include('private'),
'Link' => satisfy { |header| header.to_s.include?('activity+json') } 'Link' => include('activity+json')
) )
expect(response.body).to include status.text expect(response.body).to include status.text
end end
@@ -495,7 +495,7 @@ RSpec.describe StatusesController do
'Vary' => 'Accept, Accept-Language, Cookie', 'Vary' => 'Accept, Accept-Language, Cookie',
'Cache-Control' => include('private'), 'Cache-Control' => include('private'),
'Content-Type' => include('application/activity+json'), 'Content-Type' => include('application/activity+json'),
'Link' => satisfy { |header| header.to_s.include?('activity+json') } 'Link' => include('activity+json')
) )
expect(response.parsed_body) expect(response.parsed_body)
.to include(content: include(status.text)) .to include(content: include(status.text))
@@ -779,7 +779,7 @@ RSpec.describe StatusesController do
expect(response.headers).to include( expect(response.headers).to include(
'Vary' => 'Accept, Accept-Language, Cookie', 'Vary' => 'Accept, Accept-Language, Cookie',
'Cache-Control' => include('public'), 'Cache-Control' => include('public'),
'Link' => satisfy { |header| header.to_s.include?('activity+json') } 'Link' => include('activity+json')
) )
end end
end end