Merge master

Gemfile.lock

@@ -249,7 +249,7 @@ GEM
       thor (>= 0.14, < 2.0)
     jquery-ui-rails (4.2.1)
       railties (>= 3.2.16)
-    json (2.2.0)
+    json (2.3.0)
     json-schema (2.8.1)
       addressable (>= 2.4)
     jsonapi-renderer (0.2.2)
@@ -302,7 +302,7 @@ GEM
     nenv (0.3.0)
     netrc (0.11.0)
     nio4r (2.5.2)
-    nokogiri (1.10.5)
+    nokogiri (1.10.8)
       mini_portile2 (~> 2.4.0)
     notiffany (0.1.1)
       nenv (~> 0.1)
@@ -372,7 +372,7 @@ GEM
       method_source (~> 0.8.1)
       slop (~> 3.4)
     public_suffix (3.1.1)
-    puma (3.12.2)
+    puma (3.12.4)
     rack (2.0.8)
     rack-openid (1.3.1)
       rack (>= 1.1.0)
@@ -547,7 +547,7 @@ GEM
       actionpack (>= 4.0)
       activesupport (>= 4.0)
       sprockets (>= 3.0.0)
-    sqlite3 (1.4.1)
+    sqlite3 (1.4.2)
     stripe (5.14.0)
     stripe-ruby-mock (3.0.1)
       dante (>= 0.2.0)

app/models/vote.rb

@@ -4,6 +4,8 @@ class Vote < ApplicationRecord
   belongs_to :user
   belongs_to :event
 
+  validates :user_id, uniqueness: { scope: :event_id }
+
   has_paper_trail ignore: [:updated_at], meta: { conference_id: :conference_id }
 
   delegate :name, to: :user

spec/controllers/admin/roles_controller_spec.rb

@@ -91,7 +91,7 @@ describe Admin::RolesController do
                                      user:          { email: 'user1@osem.io' },
                                      id:            'organizer' }
 
-        expect(user1.roles).to eq [organizer_role, cfp_role]
+        expect(user1.roles).to contain_exactly(cfp_role, organizer_role)
       end
     end
 
@@ -109,7 +109,7 @@ describe Admin::RolesController do
                                      user:          { email: 'user1@osem.io' },
                                      id:            'organizer' }
 
-        expect(user1.roles).to eq [organizer_role, cfp_role]
+        expect(user1.roles).to contain_exactly(cfp_role, organizer_role)
 
         post :toggle_user, params: { conference_id: conference.short_title,
                                      user:          { email: 'user1@osem.io', state: 'false' },

spec/controllers/admin/versions_controller_spec.rb

@@ -18,7 +18,7 @@ describe Admin::VersionsController do
 
       it 'reverts all changes for update actions' do
         conference.update_attributes(short_title: 'testtitle', description: 'Some random text')
-        get :revert_object, params: { id: PaperTrail::Version.last.id }
+        get :revert_object, params: { id: conference.versions.last.id }
         conference.reload
         expect(conference.short_title).to eq 'exampletitle'
         expect(conference.description).to eq 'Example Description'
@@ -32,18 +32,19 @@ describe Admin::VersionsController do
       end
 
       it 'reverting deletion of object  creates it again' do
-        conference.program.event_types.first.destroy
+        event_type = conference.program.event_types.first
+        event_type.destroy
         event_types_count = conference.program.event_types.count
-        get :revert_object, params: { id: PaperTrail::Version.last.id }
+        get :revert_object, params: { id: event_type.versions.last.id }
         conference.reload
-        expect(PaperTrail::Version.last.event).to eq 'create'
+        expect(event_type.versions.last.event).to eq 'create'
         expect(conference.program.event_types.count).to eq(event_types_count + 1)
       end
 
       it 'reverting creation of object deletes it ' do
-        create(:lodging, conference: conference)
-        get :revert_object, params: { id: PaperTrail::Version.last.id }
-        expect(PaperTrail::Version.last.event).to eq 'destroy'
+        lodging = create(:lodging, conference: conference)
+        get :revert_object, params: { id: lodging.versions.last.id }
+        expect(lodging.versions.last.event).to eq 'destroy'
         expect(Lodging.count).to eq 0
       end
 
@@ -62,7 +63,7 @@ describe Admin::VersionsController do
 
       it 'reverts specified change for update actions' do
         conference.update_attributes(short_title: 'testtitle', description: 'Some random text')
-        get :revert_attribute, params: { id: PaperTrail::Version.last.id, attribute: 'short_title' }
+        get :revert_attribute, params: { id: conference.versions.last.id, attribute: 'short_title' }
         conference.reload
         expect(conference.short_title).to eq 'exampletitle'
         expect(conference.description).to eq 'Some random text'
@@ -71,22 +72,23 @@ describe Admin::VersionsController do
       it 'shows correct flash on trying to revert to the current state' do
         conference.update_attributes(short_title: 'testtitle', description: 'Some random text')
         conference.update_attributes(short_title: 'exampletitle')
-        get :revert_attribute, params: { id: PaperTrail::Version.all[-2].id, attribute: 'short_title' }
+        get :revert_attribute, params: { id: conference.versions[-2].id, attribute: 'short_title' }
         expect(flash[:error]).to match('The item is already in the state that you are trying to revert it back to')
         expect(conference.short_title).to eq 'exampletitle'
       end
 
       it 'fails on trying to revert deleted object' do
-        conference.program.event_types.first.update_attributes(title: 'New Event Title')
-        conference.program.event_types.first.destroy
-        get :revert_attribute, params: { id: PaperTrail::Version.all[-2].id, attribute: 'title' }
+        event_type = conference.program.event_types.first
+        event_type.update_attributes(title: 'New Event Title')
+        event_type.destroy
+        get :revert_attribute, params: { id: event_type.versions[-2].id, attribute: 'title' }
         conference.reload
         expect(flash[:alert]).to eq 'You are not authorized to access this page.'
       end
 
       it 'fails on trying to revert creation event' do
-        create(:lodging, conference: conference)
-        get :revert_attribute, params: { id: PaperTrail::Version.last.id, attribute: 'name' }
+        lodging = create(:lodging, conference: conference)
+        get :revert_attribute, params: { id: lodging.versions.last.id, attribute: 'name' }
         expect(flash[:alert]).to eq 'You are not authorized to access this page.'
       end
 
@@ -94,7 +96,7 @@ describe Admin::VersionsController do
         conference.update_attributes(short_title: 'testtitle', description: 'Some random text')
         before_conference_title = conference.title
         # Note: even though title is a valid attribute of conference, it was not updated in the change we are trying to revert
-        get :revert_attribute, params: { id: PaperTrail::Version.last.id, attribute: 'title' }
+        get :revert_attribute, params: { id: conference.versions.last.id, attribute: 'title' }
         conference.reload
         expect(conference.short_title).to eq 'testtitle'
         expect(conference.description).to eq 'Some random text'
@@ -116,12 +118,12 @@ describe Admin::VersionsController do
           @user = create(:user)
 
           conference.update_attributes(short_title: 'testtitle', description: 'Some random text')
-          @version_organizer = PaperTrail::Version.last
-          create(:cfp, program: conference.program)
-          @version_cfp = PaperTrail::Version.last
+          @version_organizer = conference.versions.last
+          cfp = create(:cfp, program: conference.program)
+          @version_cfp = cfp.versions.last
           registration = create(:registration, conference: conference)
           registration.update_attributes(attended: true)
-          @version_info_desk = PaperTrail::Version.last
+          @version_info_desk = registration.versions.last
         end
 
         it 'when user has role cfp' do

spec/controllers/api/v1/conferences_controller_spec.rb

@@ -21,8 +21,7 @@ describe Api::V1::ConferencesController do
     end
 
     it 'returns correct conferences' do
-      expect(@json[0]['short_title']).to eq('conf_one')
-      expect(@json[1]['short_title']).to eq('conf_two')
+      expect(@json.pluck('short_title')).to contain_exactly('conf_one', 'conf_two')
     end
   end
 

spec/controllers/api/v1/events_controller_spec.rb

@@ -15,9 +15,7 @@ describe Api::V1::EventsController do
         json = JSON.parse(response.body)['events']
         expect(response).to be_success
 
-        expect(json.length).to eq(2)
-        expect(json[0]['title']).to eq('Example Event')
-        expect(json[1]['title']).to eq('Conference Event')
+        expect(json.pluck('title')).to contain_exactly('Conference Event', 'Example Event')
       end
     end
 

spec/controllers/api/v1/rooms_controller_spec.rb

@@ -17,9 +17,7 @@ describe Api::V1::RoomsController do
 
         expect(response).to be_success
 
-        expect(json.length).to eq(2)
-        expect(json[0]['name']).to eq('Conference Room')
-        expect(json[1]['name']).to eq('Test Room')
+        expect(json.pluck('name')).to contain_exactly('Conference Room', 'Test Room')
       end
     end
 

spec/controllers/api/v1/speakers_controller_spec.rb

@@ -22,9 +22,7 @@ describe Api::V1::SpeakersController do
         get :index, params: { format: :json }
         json = JSON.parse(response.body)['speakers']
         expect(response).to be_success
-        expect(json.length).to eq(2)
-        expect(json[0]['name']).to eq('Speaker')
-        expect(json[1]['name']).to eq('Conf_Speaker')
+        expect(json.pluck('name')).to contain_exactly('Conf_Speaker', 'Speaker')
       end
     end
 

spec/controllers/api/v1/tracks_controller_spec.rb

@@ -16,9 +16,7 @@ describe Api::V1::TracksController do
 
         expect(response).to be_success
 
-        expect(json.length).to eq(2)
-        expect(json[0]['name']).to eq('Conference Track')
-        expect(json[1]['name']).to eq('Test Track')
+        expect(json.pluck('name')).to contain_exactly('Conference Track', 'Test Track')
       end
     end
 

spec/features/roles_spec.rb

@@ -66,7 +66,7 @@ feature Role do
     scenario "removes role #{role_name}", feature: true, js: true do
       click_link('Users', href: admin_conference_role_path(conference.short_title, role_name))
 
-      bootstrap_switch = first('td').find('.bootstrap-switch-container')
+      bootstrap_switch = find('tr', text: user_with_role.name).find('.bootstrap-switch-container')
       bootstrap_switch.click
 
       expect(page).to have_css('.alert', text: "Successfully removed role #{role_name} from user #{user_with_role.email}")

spec/models/vote_spec.rb

@@ -0,0 +1,23 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+describe Vote do
+  let!(:vote) { create(:vote) }
+
+  describe 'validation' do
+    it 'has a valid factory' do
+      expect(build(:vote)).to be_valid
+    end
+
+    it { is_expected.to validate_uniqueness_of(:user_id).scoped_to(:event_id) }
+
+    # This is testing the relationship instead of using the shoulda-matchers
+    context 'vote with user already exists' do
+      it 'fails when adding vote twice for user and event' do
+        expect { create(:vote, user: vote.user, event: vote.event) }
+          .to raise_error(ActiveRecord::RecordInvalid, 'Validation failed: User has already been taken')
+      end
+    end
+  end
+end