Skip to content

Commit

Permalink
Migrate to request specs in /api/v1/polls (#25596)
Browse files Browse the repository at this point in the history
  • Loading branch information
danielmbrasil committed Jul 17, 2023
1 parent 1aea938 commit 4859958
Show file tree
Hide file tree
Showing 2 changed files with 47 additions and 37 deletions.
37 changes: 0 additions & 37 deletions spec/controllers/api/v1/polls_controller_spec.rb

This file was deleted.

47 changes: 47 additions & 0 deletions spec/requests/api/v1/polls_spec.rb
@@ -0,0 +1,47 @@
# frozen_string_literal: true

require 'rails_helper'

RSpec.describe 'Polls' do
let(:user) { Fabricate(:user) }
let(:scopes) { 'read:statuses' }
let(:token) { Fabricate(:accessible_access_token, resource_owner_id: user.id, scopes: scopes) }
let(:headers) { { 'Authorization' => "Bearer #{token.token}" } }

describe 'GET /api/v1/polls/:id' do
subject do
get "/api/v1/polls/#{poll.id}", headers: headers
end

let(:poll) { Fabricate(:poll, status: Fabricate(:status, visibility: visibility)) }
let(:visibility) { 'public' }

it_behaves_like 'forbidden for wrong scope', 'write write:statuses'

context 'when parent status is public' do
it 'returns the poll data successfully', :aggregate_failures do
subject

expect(response).to have_http_status(200)
expect(body_as_json).to match(
a_hash_including(
id: poll.id.to_s,
voted: false,
voters_count: poll.voters_count,
votes_count: poll.votes_count
)
)
end
end

context 'when parent status is private' do
let(:visibility) { 'private' }

it 'returns http not found' do
subject

expect(response).to have_http_status(404)
end
end
end
end

0 comments on commit 4859958

Please sign in to comment.