Merge branch 'with_factory_girl'

Author: igal

Gemfile

@@ -35,6 +35,7 @@ group :test do
   gem 'cucumber',         '~> 0.6.2', :require => false
   gem 'cucumber-rails',   '~> 0.2.4', :require => false
   gem 'database_cleaner', '~> 0.4.3', :require => false
+  gem 'factory_girl',     '~> 1.2.4', :require => false
   gem 'rcov',             '~> 0.9.7', :require => false
   gem 'rspec',            '~> 1.3.0', :require => false
   gem 'rspec-rails',      '~> 1.3.0', :require => false

spec/controllers/proposals_controller_spec.rb

@@ -534,9 +534,111 @@ def assert_show(opts={}, &block)
           proposal = assigns(:proposal)
           proposal.presenter.should == user.fullname
         end
+
+        describe "when event has tracks" do
+          describe "when there are no tracks" do
+            it "should display new event form for admin user" do
+              event = Factory(:event, :tracks => [])
+              user = Factory(:admin)
+              login_as(user)
+
+              get :new, :event_id => event.slug
+
+              flash[:failure].should =~ /no tracks/i
+              response.should redirect_to(new_event_track_path(event))
+            end
+
+            it "should display error for non-admin user" do
+              event = Factory(:event, :tracks => [])
+              user = Factory(:user)
+              login_as(user)
+
+              get :new, :event_id => event.slug
+
+              flash[:failure].should =~ /no tracks/i
+              response.should redirect_to(event_proposals_path(event))
+            end
+          end
+
+          it "should assign a track if there's only one" do
+            event = Factory(:event, :tracks => [])
+            track = Factory(:track, :event => event)
+            user = Factory(:user)
+            login_as(user)
+
+            get :new, :event_id => event.slug
+
+            flash[:failure].should be_nil
+            assigns[:proposal].track.should == track
+          end
+
+          it "should not assign a track if there's more than one" do
+            event = Factory(:event, :tracks => [])
+            track1 = Factory(:track, :event => event)
+            track2 = Factory(:track, :event => event)
+            user = Factory(:user)
+            login_as(user)
+
+            get :new, :event_id => event.slug
+
+            flash[:failure].should be_nil
+            assigns[:proposal].track.should be_nil
+          end
+        end
+
+        describe "when event has session types" do
+          describe "when there are no session types" do
+            it "should display new event form for admin user" do
+              event = Factory(:event, :session_types => [])
+              user = Factory(:admin)
+              login_as(user)
+
+              get :new, :event_id => event.slug
+
+              response.should redirect_to(new_event_session_type_path(event))
+              flash[:failure].should =~ /no session types/i
+            end
+
+            it "should display error for non-admin user" do
+              event = Factory(:event, :session_types => [])
+              user = Factory(:user)
+              login_as(user)
+
+              get :new, :event_id => event.slug
+
+              response.should redirect_to(event_proposals_path(event))
+              flash[:failure].should =~ /no session types/i
+            end
+          end
+
+          it "should assign a session type if there's only one" do
+            event = Factory(:event, :session_types => [])
+            session_type = Factory(:session_type, :event => event)
+            user = Factory(:user)
+            login_as(user)
+
+            get :new, :event_id => event.slug
+
+            assigns[:proposal].session_type.should == session_type
+          end
+
+          it "should not assign a session type if there's more than one" do
+            event = Factory(:event, :session_types => [])
+            session_type1 = Factory(:session_type, :event => event)
+            session_type2 = Factory(:session_type, :event => event)
+            user = Factory(:user)
+            login_as(user)
+
+            get :new, :event_id => event.slug
+
+            assigns[:proposal].session_type.should be_nil
+          end
+        end
       end
+    end
 
-      it "should not display form for closed events" do
+    describe "with closed event" do
+      it "should not display form" do
         login_as(users(:quentin))
         event = events(:closed)
         get :new, :event_id => event.slug

spec/factories/common_factory.rb

@@ -0,0 +1,3 @@
+Factory.sequence(:email) { |n| "user#{n}@provider.com" }
+
+Factory.sequence(:website) { |n| "http://provider.com/~user#{n}" }

spec/factories/event_factory.rb

@@ -0,0 +1,21 @@
+Factory.define :event do |f|
+  f.sequence(:title) { |n| "Event #{n}" }
+  f.deadline { Date.today.to_time + 1.day }
+  f.open_text "We're accepting proposals"
+  f.closed_text "We're not accepting proposals"
+  f.proposal_status_published false
+  f.session_text "We have sessions"
+  f.tracks_text "We have tracks"
+  f.start_date { Date.today.to_time + 2.days }
+  f.end_date { Date.today.to_time + 3.days }
+  f.accept_proposal_comments_after_deadline false
+  f.slug { |record| record.title.downcase.gsub(/[^\w]/, '') }
+  f.schedule_published false
+  f.parent_id nil
+  f.proposal_titles_locked false
+
+  # :has_many associations
+  f.tracks { |record| [record.association(:track, :event => record.result)] }
+  f.rooms { |record| [record.association(:room, :event => record.result)] }
+  f.session_types { |record| [record.association(:session_type, :event => record.result)] }
+end

spec/factories/proposal_factory.rb

@@ -0,0 +1,26 @@
+# NOTE: A proposal's relationship to it's user profile information is
+# configurable. If `SETTINGS.have_user_profiles` is false, then the proposal
+# contains the user profile information (e.g., presenter's name). However,
+# if false, the proposal has many users that each have their own profile as
+# part of their user record.
+Factory.define :proposal do |f|
+  f.user_id nil
+  f.presenter "Presenter name"
+  f.affiliation "My affiliation"
+  f.email { Factory.next(:email) }
+  f.website { Factory.next(:website) }
+  f.biography "My biography"
+  f.sequence(:title) { |n| "Proposal #{n}" }
+  f.description "My proposal description"
+  f.agreement true
+  f.note_to_organizers "My note to organizers"
+  f.excerpt "My excerpt"
+  f.status "proposed"
+  f.start_time nil
+
+  # :belongs_to associations
+  f.association :event
+  f.association :track
+  f.association :session_type
+  f.association :room
+end

spec/factories/proposal_user_factory.rb

@@ -0,0 +1,5 @@
+Factory.define :proposal_user do |f|
+  # :belongs_to associations
+  f.association :proposal
+  f.association :user
+end

spec/factories/room_factory.rb

@@ -0,0 +1,9 @@
+Factory.define :room do |f|
+  f.name { |n| "Room #{n}" }
+  f.capacity 40
+  f.size "12 x 20 feet"
+  f.seating_configuration "Theater"
+
+  # :belongs_to association
+  f.association :event
+end

spec/factories/session_types_factory.rb

@@ -0,0 +1,8 @@
+Factory.define :session_type do |f|
+  f.sequence(:title) { |n| "Session Type #{n}" }
+  f.description { |record| "#{record.title} description" }
+  f.duration "45"
+
+  # :belongs_to association
+  f.association :event
+end

spec/factories/track_factory.rb

@@ -0,0 +1,9 @@
+Factory.define :track do |f|
+  f.sequence(:title) { |n| "Track #{n}" }
+  f.description { |record| "#{record.title} description" }
+  f.color "#000000"
+  f.excerpt "A track"
+
+  # :belongs_to association
+  f.association :event
+end

spec/factories/user_factory.rb

@@ -0,0 +1,21 @@
+Factory.define :user do |f|
+  f.sequence(:login) { |n| "user#{n}" }
+  f.email { Factory.next(:email) }
+  f.admin false
+  f.remember_token nil
+  f.remember_token_expires_at nil
+  f.using_openid true
+  f.affiliation "My affiliation"
+  f.biography "My biography"
+  f.website { Factory.next(:website) }
+  f.complete_profile true
+  f.sequence(:first_name) { |n| "user_first_name-#{n}" }
+  f.sequence(:last_name) { |n|  "user_last_name-#{n}" }
+  f.blog_url { Factory.next(:website) }
+  f.identica { |record| record.fullname.gsub(/[^\w]/, '_').downcase }
+  f.twitter { |record| record.fullname.gsub(/[^\w]/, '_').downcase }
+end
+
+Factory.define :admin, :parent => :user do |f|
+  f.admin true
+end