fix specs
This commit is contained in:
@@ -15,33 +15,33 @@ RSpec.describe BroadcastRecordingsController, type: :controller do
|
|||||||
describe "#update" do
|
describe "#update" do
|
||||||
let(:broadcast) { create(:broadcast, project: project, name: "New Broadcast") }
|
let(:broadcast) { create(:broadcast, project: project, name: "New Broadcast") }
|
||||||
let(:recordings) { create_list(:broadcast_recording, 5, :with_random_asset_uid, broadcast: broadcast) }
|
let(:recordings) { create_list(:broadcast_recording, 5, :with_random_asset_uid, broadcast: broadcast) }
|
||||||
let(:starred_recordings) { create_list(:broadcast_recording, 5, :with_random_asset_uid, broadcast: broadcast, star: true) }
|
let(:starred_recordings) { create_list(:broadcast_recording, 5, :with_random_asset_uid, broadcast: broadcast, starred: true) }
|
||||||
|
|
||||||
it "sets star property to true when recording is starred" do
|
it "sets star property to true when recording is starred" do
|
||||||
recordings.each do |recording|
|
recordings.each do |recording|
|
||||||
expect(recording.star).to be_falsey
|
expect(recording.starred).to be_falsey
|
||||||
end
|
end
|
||||||
|
|
||||||
put :update, params: { project_id: project, broadcast_id: broadcast, id: recordings.first.id }, xhr: true
|
put :update, params: { project_id: project, broadcast_id: broadcast, id: recordings.first.id }, xhr: true
|
||||||
|
|
||||||
expect(recordings.first.reload.star).to eq true
|
expect(recordings.first.reload.starred).to eq true
|
||||||
|
|
||||||
recordings[1..5].each do |recording|
|
recordings[1..5].each do |recording|
|
||||||
expect(recording.reload.star).to be_falsey
|
expect(recording.reload.starred).to be_falsey
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
it "sets star property to false when recording is unstarred" do
|
it "sets star property to false when recording is unstarred" do
|
||||||
starred_recordings.each do |recording|
|
starred_recordings.each do |recording|
|
||||||
expect(recording.star).to be_truthy
|
expect(recording.starred).to be_truthy
|
||||||
end
|
end
|
||||||
|
|
||||||
put :update, params: { project_id: project, broadcast_id: broadcast, id: starred_recordings.first.id }, xhr: true
|
put :update, params: { project_id: project, broadcast_id: broadcast, id: starred_recordings.first.id }, xhr: true
|
||||||
|
|
||||||
expect(starred_recordings.first.reload.star).to eq false
|
expect(starred_recordings.first.reload.starred).to eq false
|
||||||
|
|
||||||
starred_recordings[1..5].each do |recording|
|
starred_recordings[1..5].each do |recording|
|
||||||
expect(recording.reload.star).to eq true
|
expect(recording.reload.starred).to eq true
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|||||||
@@ -243,7 +243,7 @@ feature 'User managing broadcasts' do
|
|||||||
|
|
||||||
scenario 'project manager can unstar broadcast recordings', js: true do
|
scenario 'project manager can unstar broadcast recordings', js: true do
|
||||||
broadcast = create(:broadcast, :with_stream, :with_files, project: project)
|
broadcast = create(:broadcast, :with_stream, :with_files, project: project)
|
||||||
recording = create(:broadcast_recording, broadcast: broadcast, asset_uid: "another_asset_uid", star: true)
|
recording = create(:broadcast_recording, broadcast: broadcast, asset_uid: "another_asset_uid", starred: true)
|
||||||
|
|
||||||
visit project_broadcast_path(project, broadcast)
|
visit project_broadcast_path(project, broadcast)
|
||||||
|
|
||||||
@@ -290,7 +290,7 @@ feature 'User managing broadcasts' do
|
|||||||
|
|
||||||
scenario 'associate does not see broadcast recordings star', js: true do
|
scenario 'associate does not see broadcast recordings star', js: true do
|
||||||
broadcast = create(:broadcast, :with_stream, :with_files, project: project)
|
broadcast = create(:broadcast, :with_stream, :with_files, project: project)
|
||||||
recording = create(:broadcast_recording, broadcast: broadcast, asset_uid: "another_asset_uid", star: true)
|
recording = create(:broadcast_recording, broadcast: broadcast, asset_uid: "another_asset_uid", starred: true)
|
||||||
|
|
||||||
visit project_broadcast_path(project, broadcast)
|
visit project_broadcast_path(project, broadcast)
|
||||||
|
|
||||||
@@ -365,7 +365,7 @@ feature 'User managing broadcasts' do
|
|||||||
|
|
||||||
scenario 'account manager can unstar broadcast recordings', js: true do
|
scenario 'account manager can unstar broadcast recordings', js: true do
|
||||||
broadcast = create(:broadcast, :with_stream, :with_files, project: project)
|
broadcast = create(:broadcast, :with_stream, :with_files, project: project)
|
||||||
recording = create(:broadcast_recording, broadcast: broadcast, asset_uid: "another_asset_uid", star: true)
|
recording = create(:broadcast_recording, broadcast: broadcast, asset_uid: "another_asset_uid", starred: true)
|
||||||
|
|
||||||
visit project_broadcast_path(project, broadcast)
|
visit project_broadcast_path(project, broadcast)
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user