ninoxe 1.1.4 → 1.1.5

data/spec/models/chouette/transport_mode_spec.rb

@@ -0,0 +1,64 @@
+require 'spec_helper'
+
+describe Chouette::TransportMode do
+  
+  def mode(text_code = "test", numerical_code = nil)
+    numerical_code ||= 1 if text_code == "test"
+    Chouette::TransportMode.new(text_code, numerical_code)
+  end
+
+  describe "#to_i" do
+    
+    it "should return numerical code" do
+      mode("test", 1).to_i.should == 1
+    end
+
+  end
+
+  it "should return true to #test? when text code is 'test'" do
+    mode("test").should be_test
+  end
+
+  it "should be equal when text codes are identical" do
+    mode("test",1).should == mode("test", 2)
+  end
+
+  describe ".new" do
+
+    it "should find numerical code from text code" do
+      mode("unknown").to_i.should == 0
+    end
+
+    it "should find text code from numerical code" do
+      mode(0).should be_unknown
+    end
+
+    it "should accept another mode" do
+      Chouette::TransportMode.new(mode("test")).should == mode("test")
+    end
+    
+  end
+
+  describe "#public_transport?" do
+    
+    it "should return false for interchange" do
+      mode("interchange").should_not be_public_transport
+    end
+
+    it "should return true for other modes" do
+      mode("unknown").should be_public_transport
+    end
+
+  end
+
+  describe ".all" do
+    
+    Chouette::TransportMode.definitions.each do |text_code, numerical_code|
+      it "should include a TransportMode #{text_code}" do
+        Chouette::TransportMode.all.should include(Chouette::TransportMode.new(text_code))
+      end
+    end
+
+  end
+
+end

data/spec/models/chouette/trident_active_record_spec.rb

@@ -0,0 +1,115 @@
+require 'spec_helper'
+
+describe Chouette::TridentActiveRecord do
+
+  it { Chouette::TridentActiveRecord.ancestors.should include(Chouette::ActiveRecord) }
+
+  subject { Factory(:time_table) }
+
+  describe "#uniq_objectid" do
+
+    it "should rebuild objectid" do
+      tm = Factory(:time_table)
+      tm.objectid = subject.objectid
+      tm.uniq_objectid
+      tm.objectid.should == subject.objectid+"_1"
+    end
+
+    it "should rebuild objectid" do
+      tm = Factory(:time_table)
+      tm.objectid = subject.objectid
+      tm.uniq_objectid
+      tm.save
+      tm = Factory(:time_table)
+      tm.objectid = subject.objectid
+      tm.uniq_objectid
+      tm.objectid.should == subject.objectid+"_2"
+    end
+
+  end
+
+  describe "#prepare_auto_columns" do
+
+    it "should left objectid" do
+      tm = Chouette::TimeTable.new :comment => "merge1" , :objectid => "NINOXE:Timetable:merge1"
+      tm.prepare_auto_columns
+      tm.objectid.should == "NINOXE:Timetable:merge1"
+    end
+
+    it "should add pending_id to objectid" do
+      tm = Chouette::TimeTable.new :comment => "merge1"
+      tm.prepare_auto_columns
+      tm.objectid.start_with?("NINOXE:Timetable:__pending_id__").should be_true
+    end
+
+    it "should set id to objectid" do
+      tm = Chouette::TimeTable.new :comment => "merge1"
+      tm.save
+      tm.objectid.should == "NINOXE:Timetable:"+tm.id.to_s
+    end
+
+    it "should detect objectid conflicts" do
+      tm = Chouette::TimeTable.new :comment => "merge1"
+      tm.save
+      tm.objectid = "NINOXE:Timetable:"+(tm.id+1).to_s
+      tm.save
+      tm = Chouette::TimeTable.new :comment => "merge1"
+      tm.save
+      tm.objectid.should == "NINOXE:Timetable:"+tm.id.to_s+"_1"
+    end
+
+  end
+
+  describe "objectid" do
+
+    it "should build automatic objectid when empty" do
+      g1 = Chouette::GroupOfLine.new( :name => "g1")
+      g1.save
+      g1.objectid.should == "NINOXE:GroupOfLine:"+g1.id.to_s
+    end
+
+    it "should build automatic objectid with fixed when only suffix given" do
+      g1 = Chouette::GroupOfLine.new( :name => "g1")
+      g1.objectid = "toto"
+      g1.save
+      g1.objectid.should == "NINOXE:GroupOfLine:toto"
+    end
+    
+    it "should build automatic objectid with extension when already exists" do
+      g1 = Chouette::GroupOfLine.new( :name => "g1")
+      g1.save
+      cnt = g1.id + 1
+      g1.objectid = "NINOXE:GroupOfLine:"+cnt.to_s
+      g1.save
+      g2 = Chouette::GroupOfLine.new( :name => "g2")
+      g2.save
+      g2.objectid.should == "NINOXE:GroupOfLine:"+g2.id.to_s+"_1"
+    end
+    
+    it "should build automatic objectid with extension when already exists" do
+      g1 = Chouette::GroupOfLine.new( :name => "g1")
+      g1.save
+      cnt = g1.id + 2
+      g1.objectid = "NINOXE:GroupOfLine:"+cnt.to_s
+      g1.save
+      g2 = Chouette::GroupOfLine.new( :name => "g2")
+      g2.objectid = "NINOXE:GroupOfLine:"+cnt.to_s+"_1"
+      g2.save
+      g3 = Chouette::GroupOfLine.new( :name => "g3")
+      g3.save
+      g3.objectid.should == "NINOXE:GroupOfLine:"+g3.id.to_s+"_2"
+    end
+    
+    it "should build automatic objectid when id cleared" do
+      g1 = Chouette::GroupOfLine.new( :name => "g1")
+      g1.objectid = "NINOXE:GroupOfLine:xxxx"
+      g1.save
+      g1.objectid = nil
+      g1.save
+      g1.objectid.should == "NINOXE:GroupOfLine:"+g1.id.to_s
+    end
+  end
+
+end
+
+

data/spec/models/chouette/vehicle_journey_at_stop_spec.rb

@@ -0,0 +1,46 @@
+require 'spec_helper'
+
+describe Chouette::VehicleJourneyAtStop do
+  let!(:vehicle_journey){ Factory(:vehicle_journey_odd)}
+  subject { vehicle_journey.vehicle_journey_at_stops.first }
+
+  describe "#exceeds_gap?" do
+    it "should return false if gap < 1.hour" do
+      t1 = 1.minutes.ago
+      t2 = 1.minutes.ago + 3.hour
+      subject.exceeds_gap?(t1, t2).should be_true
+    end
+    it "should return false if gap > 2.hour" do
+      t1 = 1.minutes.ago
+      t2 = 1.minutes.ago + 3.minutes
+      subject.exceeds_gap?(t1, t2).should be_false
+    end
+  end
+
+  describe "#increasing_times_validate" do
+    let(:vjas1){ vehicle_journey.vehicle_journey_at_stops[0]}
+    let(:vjas2){ vehicle_journey.vehicle_journey_at_stops[1]}
+    context "when vjas#arrival_time exceeds gap" do
+      it "should add errors on arrival_time" do
+        vjas1.arrival_time = vjas2.arrival_time - 3.hour
+        vjas2.increasing_times_validate(vjas1).should be_false
+        vjas2.errors.should_not be_empty
+        vjas2.errors[:arrival_time].should_not be_blank
+      end
+    end
+    context "when vjas#departure_time exceeds gap" do
+      it "should add errors on departure_time" do
+        vjas1.departure_time = vjas2.departure_time - 3.hour
+        vjas2.increasing_times_validate(vjas1).should be_false
+        vjas2.errors.should_not be_empty
+        vjas2.errors[:departure_time].should_not be_blank
+      end
+    end
+    context "when vjas does'nt exceed gap" do
+      it "should not add errors" do
+        vjas2.increasing_times_validate(vjas1).should be_true
+        vjas2.errors.should be_empty
+      end
+    end
+  end
+end 

data/spec/models/chouette/vehicle_journey_spec.rb

@@ -0,0 +1,223 @@
+require 'spec_helper'
+
+describe Chouette::VehicleJourney do
+  subject { Factory(:vehicle_journey_odd) }
+
+  describe "in_relation_to_a_journey_pattern methods" do
+    let!(:route) { Factory(:route)}
+    let!(:journey_pattern) { Factory(:journey_pattern, :route => route)}
+    let!(:journey_pattern_odd) { Factory(:journey_pattern_odd, :route => route)}
+    let!(:journey_pattern_even) { Factory(:journey_pattern_even, :route => route)}
+
+    context "when vehicle_journey is on odd stop whereas selected journey_pattern is on all stops" do
+      subject { Factory(:vehicle_journey, :route => route, :journey_pattern => journey_pattern_odd)}
+      describe "#extra_stops_in_relation_to_a_journey_pattern" do
+        it "should be empty" do
+          subject.extra_stops_in_relation_to_a_journey_pattern( journey_pattern).should be_empty
+        end
+      end
+      describe "#extra_vjas_in_relation_to_a_journey_pattern" do
+        it "should be empty" do
+          subject.extra_vjas_in_relation_to_a_journey_pattern( journey_pattern).should be_empty
+        end
+      end
+      describe "#missing_stops_in_relation_to_a_journey_pattern" do
+        it "should return even stops" do
+          result = subject.missing_stops_in_relation_to_a_journey_pattern( journey_pattern)
+          result.should == journey_pattern_even.stop_points
+        end
+      end
+      describe "#update_journey_pattern" do
+        it "should new_record for added vjas" do
+          subject.update_journey_pattern( journey_pattern)
+          subject.vehicle_journey_at_stops.select{ |vjas| vjas.new_record? }.each do |vjas|
+            journey_pattern_even.stop_points.should include( vjas.stop_point)
+          end
+        end
+        it "should add vjas on each even stops" do
+          subject.update_journey_pattern( journey_pattern)
+          vehicle_stops = subject.vehicle_journey_at_stops.map(&:stop_point)
+          journey_pattern_even.stop_points.each do |sp|
+            vehicle_stops.should include(sp)
+          end
+        end
+        it "should not mark any vjas as _destroy" do
+          subject.update_journey_pattern( journey_pattern)
+          subject.vehicle_journey_at_stops.any?{ |vjas| vjas._destroy }.should be_false
+        end
+      end
+    end
+    context "when vehicle_journey is on all stops whereas selected journey_pattern is on odd stops" do
+      subject { Factory(:vehicle_journey, :route => route, :journey_pattern => journey_pattern)}
+      describe "#missing_stops_in_relation_to_a_journey_pattern" do
+        it "should be empty" do
+          subject.missing_stops_in_relation_to_a_journey_pattern( journey_pattern_odd).should be_empty
+        end
+      end
+      describe "#extra_stops_in_relation_to_a_journey_pattern" do
+        it "should return even stops" do
+          result = subject.extra_stops_in_relation_to_a_journey_pattern( journey_pattern_odd)
+          result.should == journey_pattern_even.stop_points
+        end
+      end
+      describe "#extra_vjas_in_relation_to_a_journey_pattern" do
+        it "should return vjas on even stops" do
+          result = subject.extra_vjas_in_relation_to_a_journey_pattern( journey_pattern_odd)
+          result.map(&:stop_point).should == journey_pattern_even.stop_points
+        end
+      end
+      describe "#update_journey_pattern" do
+        it "should add no new vjas" do
+          subject.update_journey_pattern( journey_pattern_odd)
+          subject.vehicle_journey_at_stops.any?{ |vjas| vjas.new_record? }.should be_false
+        end
+        it "should mark vehicle_journey_at_stops as _destroy on even stops" do
+          subject.update_journey_pattern( journey_pattern_odd)
+          subject.vehicle_journey_at_stops.each { |vjas|
+            vjas._destroy.should == journey_pattern_even.stop_points.include?(vjas.stop_point)
+          }
+        end
+      end
+    end
+
+  end
+  context "when following departure times exceeds gap" do
+    describe "#increasing_times" do
+      before(:each) do
+        subject.vehicle_journey_at_stops[0].departure_time = subject.vehicle_journey_at_stops[1].departure_time - 2.hour
+        subject.vehicle_journey_at_stops[0].arrival_time = subject.vehicle_journey_at_stops[0].departure_time
+        subject.vehicle_journey_at_stops[1].arrival_time = subject.vehicle_journey_at_stops[1].departure_time
+      end
+      it "should make instance invalid" do
+        subject.increasing_times
+        subject.vehicle_journey_at_stops[1].errors[:departure_time].should_not be_blank
+        subject.should_not be_valid
+      end
+    end
+    describe "#update_attributes" do
+      let!(:params){ {"vehicle_journey_at_stops_attributes" => {
+            "0"=>{"id" => subject.vehicle_journey_at_stops[0].id ,"arrival_time" => 1.minutes.ago,"departure_time" => 1.minutes.ago},
+            "1"=>{"id" => subject.vehicle_journey_at_stops[1].id, "arrival_time" => (1.minutes.ago + 2.hour),"departure_time" => (1.minutes.ago + 2.hour)}
+         }}}
+      it "should return false" do
+        subject.update_attributes(params).should be_false
+      end
+      it "should make instance invalid" do
+        subject.update_attributes(params)
+        subject.should_not be_valid
+      end
+      it "should let first vjas without any errors" do
+        subject.update_attributes(params)
+        subject.vehicle_journey_at_stops[0].errors.should be_empty
+      end
+      it "should add an error on second vjas" do
+        subject.update_attributes(params)
+        subject.vehicle_journey_at_stops[1].errors[:departure_time].should_not be_blank
+      end
+    end
+  end
+
+  context "#time_table_tokens=" do
+    let!(:tm1){Factory(:time_table, :comment => "TM1")}
+    let!(:tm2){Factory(:time_table, :comment => "TM2")}
+
+    it "should return associated time table ids" do
+      subject.update_attributes :time_table_tokens => [tm1.id, tm2.id].join(',')
+      subject.time_tables.should include( tm1)
+      subject.time_tables.should include( tm2)
+    end
+  end
+  describe "#bounding_dates" do
+    before(:each) do
+      tm1 = Factory.build(:time_table, :dates =>
+        [ Factory.build(:time_table_date, :date => 1.days.ago.to_date, :in_out => true),
+          Factory.build(:time_table_date, :date => 2.days.ago.to_date, :in_out => true)])
+      tm2 = Factory.build(:time_table, :periods =>
+        [ Factory.build(:time_table_period, :period_start => 4.days.ago.to_date, :period_end => 3.days.ago.to_date)])
+      tm3 = Factory.build(:time_table)
+      subject.time_tables = [ tm1, tm2, tm3]
+    end
+    it "should return min date from associated calendars" do
+      subject.bounding_dates.min.should == 4.days.ago.to_date
+    end
+    it "should return max date from associated calendars" do
+      subject.bounding_dates.max.should == 1.days.ago.to_date
+    end
+  end
+  context "#vehicle_journey_at_stops" do
+    it "should be ordered like stop_points on route" do
+      route = subject.route
+      vj_stop_ids = subject.vehicle_journey_at_stops.map(&:stop_point_id)
+      expected_order = route.stop_points.map(&:id).select {|s_id| vj_stop_ids.include?(s_id)}
+
+      vj_stop_ids.should == expected_order
+    end
+
+  end
+
+    describe "#transport_mode_name" do
+
+    def self.legacy_transport_modes
+      %w{Air Train LongDistanceTrain LocalTrain RapidTransit Metro Tramway Coach Bus Ferry Waterborne PrivateVehicle Walk Trolleybus Bicycle Shuttle Taxi VAL Other}
+    end
+
+    legacy_transport_modes.each do |transport_mode|
+      context "when transport_mode is #{transport_mode}" do
+        transport_mode_name = Chouette::TransportMode.new(transport_mode.underscore)
+        it "should be #{transport_mode_name}" do
+          subject.transport_mode = transport_mode
+          subject.transport_mode_name.should == transport_mode_name
+        end
+      end
+    end
+    context "when transport_mode is nil" do
+      it "should be nil" do
+        subject.transport_mode = nil
+        subject.transport_mode_name.should be_nil
+      end
+    end
+
+  end
+
+  describe "#transport_mode_name=" do
+
+    it "should change transport_mode with TransportMode#name" do
+      subject.transport_mode_name = "Test"
+      subject.transport_mode.should == "Test"
+    end
+
+  end
+
+  describe ".transport_mode_names" do
+
+    it "should not include unknown transport_mode_name" do
+      Chouette::VehicleJourney.transport_mode_names.should_not include(Chouette::TransportMode.new("unknown"))
+    end
+
+    it "should not include interchange transport_mode" do
+      Chouette::VehicleJourney.transport_mode_names.should_not include(Chouette::TransportMode.new("interchange"))
+    end
+
+  end
+
+  describe "#footnote_ids=" do
+    context "when line have footnotes, " do
+      let!( :route) { Factory( :route ) }
+      let!( :line) { route.line }
+      let!( :footnote_first) {Factory( :footnote, :code => "1", :label => "dummy 1", :line => route.line)}
+      let!( :footnote_second) {Factory( :footnote, :code => "2", :label => "dummy 2", :line => route.line)}
+
+
+      it "should update vehicle's footnotes" do
+        Chouette::VehicleJourney.find(subject.id).footnotes.should be_empty
+        subject.footnote_ids = [ footnote_first.id ]
+        subject.save
+        Chouette::VehicleJourney.find(subject.id).footnotes.count.should == 1
+      end
+
+    end
+
+  end
+
+end
+

data/spec/presenters/chouette/geometry/general_presenter.rb

@@ -0,0 +1 @@
+

data/spec/presenters/chouette/geometry/line_presenter_spec.rb

@@ -0,0 +1,13 @@
+require 'spec_helper'
+
+describe Chouette::Geometry::LinePresenter do
+  let!(:line) { Factory(:line_with_stop_areas_having_parent) }
+  subject { Chouette::Geometry::LinePresenter.new(line)}
+
+  describe "#routes_localized_commercials" do
+    it "should return 3 stop_areas" do
+      subject.routes_localized_commercials(line.routes.first).size.should == 5
+    end
+  end
+end
+

data/spec/spec_helper.rb

@@ -0,0 +1,45 @@
+# Configure Rails Envinronment
+ENV["RAILS_ENV"] = "test"
+require File.expand_path("../dummy/config/environment.rb",  __FILE__)
+
+require 'rspec/rails'
+require 'rspec/autorun'
+
+require 'shoulda-matchers'
+require 'factory_girl_rails'
+
+require 'geo_ruby'
+
+ENGINE_RAILS_ROOT=File.join(File.dirname(__FILE__), '../')
+
+# Requires supporting ruby files with custom matchers and macros, etc,
+# in spec/support/ and its subdirectories.
+Dir[File.join(ENGINE_RAILS_ROOT, "spec/support/**/*.rb")].each {|f| require f }
+Dir[File.join(ENGINE_RAILS_ROOT, "lib/factories/**/*.rb")].each {|f| require f }
+
+RSpec.configure do |config|
+  # == Mock Framework
+  #
+  # If you prefer to use mocha, flexmock or RR, uncomment the appropriate line:
+  #
+  # config.mock_with :mocha
+  # config.mock_with :flexmock
+  # config.mock_with :rr
+  config.mock_with :rspec
+
+  # Remove this line if you're not using ActiveRecord or ActiveRecord fixtures
+  config.fixture_path = "#{::Rails.root}/spec/fixtures"
+
+  # If you're not using ActiveRecord, or you'd prefer not to run each of your
+  # examples within a transaction, remove the following line or assign false
+  # instead of true.
+  config.use_transactional_fixtures = true
+
+  # If true, the base class of anonymous controllers will be inferred
+  # automatically. This will be the default behavior in future versions of
+  # rspec-rails.
+  config.infer_base_class_for_anonymous_controllers = false
+
+end
+
+