fix specs

This commit is contained in:
Bilal
2020-08-26 15:05:34 +03:00
parent 70a0c77054
commit 99fa8471b2
2 changed files with 10 additions and 10 deletions

View File

@@ -15,33 +15,33 @@ RSpec.describe BroadcastRecordingsController, type: :controller do
describe "#update" do
let(:broadcast) { create(:broadcast, project: project, name: "New 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
recordings.each do |recording|
expect(recording.star).to be_falsey
expect(recording.starred).to be_falsey
end
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|
expect(recording.reload.star).to be_falsey
expect(recording.reload.starred).to be_falsey
end
end
it "sets star property to false when recording is unstarred" do
starred_recordings.each do |recording|
expect(recording.star).to be_truthy
expect(recording.starred).to be_truthy
end
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|
expect(recording.reload.star).to eq true
expect(recording.reload.starred).to eq true
end
end
end

View File

@@ -243,7 +243,7 @@ feature 'User managing broadcasts' do
scenario 'project manager can unstar broadcast recordings', js: true do
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)
@@ -290,7 +290,7 @@ feature 'User managing broadcasts' do
scenario 'associate does not see broadcast recordings star', js: true do
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)
@@ -365,7 +365,7 @@ feature 'User managing broadcasts' do
scenario 'account manager can unstar broadcast recordings', js: true do
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)