ninoxe 1.1.4 → 1.1.5

data/spec/models/chouette/route_spec.rb

@@ -0,0 +1,234 @@
+require 'spec_helper'
+
+describe Chouette::Route do
+  subject { Factory(:route) }
+
+  it { should validate_uniqueness_of :objectid }
+  its(:objectid) { should be_kind_of(Chouette::ObjectId) }
+
+  it { should validate_presence_of :name }
+  it { should validate_presence_of :line }
+  it { should validate_presence_of :wayback_code }
+  it { should validate_presence_of :direction_code }
+
+  context "reordering methods" do
+    let( :bad_stop_point_ids){subject.stop_points.map { |sp| sp.id + 1}}
+    let( :ident){subject.stop_points.map(&:id)}
+    let( :first_last_swap){ [ident.last] + ident[1..-2] + [ident.first]}
+
+    describe "#reorder!" do
+      context "invalid stop_point_ids" do
+        let( :new_stop_point_ids) { bad_stop_point_ids}
+        it { subject.reorder!( new_stop_point_ids).should be_false}
+      end
+
+      context "swaped last and first stop_point_ids" do
+        let!( :new_stop_point_ids) { first_last_swap}
+        let!( :old_stop_point_ids) { subject.stop_points.map(&:id) }
+        let!( :old_stop_area_ids) { subject.stop_areas.map(&:id) }
+
+        it "should keep stop_point_ids order unchanged" do
+          subject.reorder!( new_stop_point_ids).should be_true
+          subject.stop_points.map(&:id).should eq( old_stop_point_ids)
+        end
+        it "should have changed stop_area_ids order" do
+          subject.reorder!( new_stop_point_ids).should be_true
+          subject.reload
+          subject.stop_areas.map(&:id).should eq( [old_stop_area_ids.last] + old_stop_area_ids[1..-2] + [old_stop_area_ids.first])
+        end
+      end
+    end
+
+    describe "#stop_point_permutation?" do
+      context "invalid stop_point_ids" do
+        let( :new_stop_point_ids) { bad_stop_point_ids}
+        it { should_not be_stop_point_permutation( new_stop_point_ids)}
+      end
+      context "unchanged stop_point_ids" do
+        let( :new_stop_point_ids) { ident}
+        it { should be_stop_point_permutation( new_stop_point_ids)}
+      end
+      context "swaped last and first stop_point_ids" do
+        let( :new_stop_point_ids) { first_last_swap}
+        it { should be_stop_point_permutation( new_stop_point_ids)}
+      end
+    end
+  end
+
+  describe "#stop_points_attributes=" do
+      let( :journey_pattern) { Factory( :journey_pattern, :route => subject )}
+      let( :vehicle_journey) { Factory( :vehicle_journey, :journey_pattern => journey_pattern)}
+      def subject_stop_points_attributes
+          {}.tap do |hash|
+              subject.stop_points.each_with_index { |sp,index| hash[ index.to_s ] = sp.attributes }
+          end
+      end
+      context "route having swapped a new stop" do
+          let( :new_stop_point ){Factory.build( :stop_point, :route => subject)}
+          def added_stop_hash
+            subject_stop_points_attributes.tap do |h|
+                h["4"] = new_stop_point.attributes.merge( "position" => "4", "_destroy" => "" )
+            end
+          end
+          let!( :new_route_size ){ subject.stop_points.size+1 }
+
+          it "should have added stop_point in route" do
+              subject.update_attributes( :stop_points_attributes => added_stop_hash)
+              Chouette::Route.find( subject.id ).stop_points.size.should == new_route_size
+          end
+          it "should have added stop_point in route's journey pattern" do
+              subject.update_attributes( :stop_points_attributes => added_stop_hash)
+              Chouette::JourneyPattern.find( journey_pattern.id ).stop_points.size.should == new_route_size
+          end
+          it "should have added stop_point in route's vehicle journey at stop" do
+              subject.update_attributes( :stop_points_attributes => added_stop_hash)
+              Chouette::VehicleJourney.find( vehicle_journey.id ).vehicle_journey_at_stops.size.should == new_route_size
+          end
+      end
+      context "route having swapped stop" do
+          def swapped_stop_hash
+            subject_stop_points_attributes.tap do |h|
+                h[ "1" ][ "position" ] = "3"
+                h[ "3" ][ "position" ] = "1"
+            end
+          end
+          let!( :new_stop_id_list ){ subject.stop_points.map(&:id).tap {|array| array.insert( 1, array.delete_at(3)); array.insert( 3, array.delete_at(2) )}.join(",") }
+
+          it "should have swap stop_points from route" do
+              subject.update_attributes( :stop_points_attributes => swapped_stop_hash)
+              Chouette::Route.find( subject.id ).stop_points.map(&:id).join(",").should == new_stop_id_list
+          end
+          it "should have swap stop_points from route's journey pattern" do
+              subject.update_attributes( :stop_points_attributes => swapped_stop_hash)
+              Chouette::JourneyPattern.find( journey_pattern.id ).stop_points.map(&:id).join(",").should == new_stop_id_list
+          end
+          it "should have swap stop_points from route's vehicle journey at stop" do
+              subject.update_attributes( :stop_points_attributes => swapped_stop_hash)
+              Chouette::VehicleJourney.find( vehicle_journey.id ).vehicle_journey_at_stops.map(&:stop_point_id).join(",").should == new_stop_id_list
+          end
+      end
+      context "route having a deleted stop" do
+          def removed_stop_hash
+            subject_stop_points_attributes.tap do |h|
+                h[ "1" ][ "_destroy" ] = "1"
+            end
+          end
+          let!( :new_stop_id_list ){ subject.stop_points.map(&:id).tap {|array| array.delete_at(1) }.join(",") }
+
+          it "should ignore deleted stop_point from route" do
+              subject.update_attributes( :stop_points_attributes => removed_stop_hash)
+              Chouette::Route.find( subject.id ).stop_points.map(&:id).join(",").should == new_stop_id_list
+          end
+          it "should ignore deleted stop_point from route's journey pattern" do
+              subject.update_attributes( :stop_points_attributes => removed_stop_hash)
+              Chouette::JourneyPattern.find( journey_pattern.id ).stop_points.map(&:id).join(",").should == new_stop_id_list
+          end
+          it "should ignore deleted stop_point from route's vehicle journey at stop" do
+              subject.update_attributes( :stop_points_attributes => removed_stop_hash)
+              Chouette::VehicleJourney.find( vehicle_journey.id ).vehicle_journey_at_stops.map(&:stop_point_id).join(",").should == new_stop_id_list
+          end
+      end
+  end
+
+  describe "#stop_points" do
+    context "#find_by_stop_area" do
+      context "when arg is first quay id" do
+        let(:first_stop_point) { subject.stop_points.first}
+        it "should return first quay" do
+          subject.stop_points.find_by_stop_area( first_stop_point.stop_area_id).should eq( first_stop_point)
+        end
+      end
+    end
+  end
+  describe "#stop_areas" do
+    let(:line){ Factory(:line)}
+    let(:route_1){ Factory(:route, :line => line)}
+    let(:route_2){ Factory(:route, :line => line)}
+    it "should retreive all stop_area on route" do
+      route_1.stop_areas.each do |sa|
+        sa.stop_points.map(&:route_id).uniq.should == [route_1.id]
+      end
+    end
+
+    context "when route is looping: last and first stop area are the same" do
+      it "should retreive same stop_area one last and first position" do
+        route_loop = Factory(:route, :line => line)
+        first_stop = Chouette::StopPoint.where( :route_id => route_loop.id, :position => 0).first
+        last_stop = Factory(:stop_point, :route => route_loop, :position => 5, :stop_area => first_stop.stop_area)
+
+        route_loop.stop_areas.size.should == 6
+        route_loop.stop_areas.select {|s| s.id == first_stop.stop_area.id}.size.should == 2
+      end
+    end
+  end
+
+  describe "#direction_code" do
+    def self.legacy_directions
+      %w{A R ClockWise CounterClockWise North NorthWest West SouthWest
+        South SouthEast East NorthEast}
+    end
+    legacy_directions.each do |direction|
+      context "when direction is #{direction}" do
+        direction_code = Chouette::Direction.new( Chouette::Route.direction_binding[ direction])
+        it "should be #{direction_code}" do
+          subject.direction = direction
+          subject.direction_code.should == direction_code
+        end
+      end
+    end
+    context "when direction is nil" do
+      it "should be nil" do
+        subject.direction = nil
+        subject.direction_code.should be_nil
+      end
+    end
+  end
+  describe "#direction_code=" do
+    context "when unknown direction is provided" do
+      it "should change direction to nil" do
+        subject.direction_code = "dummy"
+        subject.direction.should be_nil
+      end
+    end
+    context "when an existing direction (west) is provided" do
+      it "should change direction Direction.west" do
+        subject.direction_code = "west"
+        subject.direction.should == "West"
+      end
+    end
+  end
+  describe "#wayback_code" do
+    def self.legacy_waybacks
+      %w{A R}
+    end
+    legacy_waybacks.each do |wayback|
+      context "when wayback is #{wayback}" do
+        wayback_code = Chouette::Wayback.new( Chouette::Route.wayback_binding[ wayback])
+        it "should be #{wayback_code}" do
+          subject.wayback = wayback
+          subject.wayback_code.should == wayback_code
+        end
+      end
+    end
+    context "when wayback is nil" do
+      it "should be nil" do
+        subject.wayback = nil
+        subject.wayback_code.should be_nil
+      end
+    end
+  end
+  describe "#wayback_code=" do
+    context "when unknown wayback is provided" do
+      it "should change wayback to nil" do
+        subject.wayback_code = "dummy"
+        subject.wayback.should be_nil
+      end
+    end
+    context "when an existing wayback (straight_forward) is provided" do
+      it "should change wayback Wayback.straight_forward" do
+        subject.wayback_code = "straight_forward"
+        subject.wayback.should == "A"
+      end
+    end
+  end
+end

data/spec/models/chouette/stop_area_spec.rb

@@ -0,0 +1,440 @@
+require 'spec_helper'
+
+describe Chouette::StopArea do
+  let!(:quay) { Factory :stop_area, :area_type => "Quay" }
+  let!(:boarding_position) { Factory :stop_area, :area_type => "BoardingPosition" }
+  let!(:commercial_stop_point) { Factory :stop_area, :area_type => "CommercialStopPoint" }
+  let!(:stop_place) { Factory :stop_area, :area_type => "StopPlace" }
+  let!(:itl) { Factory :stop_area, :area_type => "ITL" }
+
+  its(:objectid) { should be_kind_of(Chouette::ObjectId) }
+
+  it { should validate_presence_of :name }
+  it { should validate_presence_of :area_type }
+  it { should validate_numericality_of :latitude }
+  it { should validate_numericality_of :longitude }
+
+  
+  describe ".latitude" do
+    it "should accept -90 value" do
+      subject = Factory :stop_area, :area_type => "BoardingPosition"
+      subject.latitude = -90
+      subject.valid?.should be_true
+    end
+    it "should reject < -90 value" do
+      subject = Factory :stop_area, :area_type => "BoardingPosition"
+      subject.latitude = -90.0001
+      subject.valid?.should be_false
+    end
+    it "should accept 90 value" do
+      subject = Factory :stop_area, :area_type => "BoardingPosition"
+      subject.latitude = 90
+      subject.valid?.should be_true
+    end
+    it "should reject > 90 value" do
+      subject = Factory :stop_area, :area_type => "BoardingPosition"
+      subject.latitude = 90.0001
+      subject.valid?.should be_false
+    end
+  end
+
+  describe ".longitude" do
+    it "should accept -180 value" do
+      subject = Factory :stop_area, :area_type => "BoardingPosition"
+      subject.longitude = -180
+      subject.valid?.should be_true
+    end
+    it "should reject < -180 value" do
+      subject = Factory :stop_area, :area_type => "BoardingPosition"
+      subject.longitude = -180.0001
+      subject.valid?.should be_false
+    end
+    it "should accept 180 value" do
+      subject = Factory :stop_area, :area_type => "BoardingPosition"
+      subject.longitude = 180
+      subject.valid?.should be_true
+    end
+    it "should reject > 180 value" do
+      subject = Factory :stop_area, :area_type => "BoardingPosition"
+      subject.longitude = 180.0001
+      subject.valid?.should be_false
+    end
+  end
+
+  describe ".long_lat" do
+    it "should accept longitude and latitude both as nil" do
+      subject = Factory :stop_area, :area_type => "BoardingPosition"
+      subject.longitude = nil
+      subject.latitude = nil
+      subject.valid?.should be_true
+    end
+    it "should accept longitude and latitude both numerical" do
+      subject = Factory :stop_area, :area_type => "BoardingPosition"
+      subject.longitude = 10
+      subject.latitude = 10
+      subject.valid?.should be_true
+    end
+    it "should reject longitude nil with latitude numerical" do
+      subject = Factory :stop_area, :area_type => "BoardingPosition"
+      subject.longitude = nil
+      subject.latitude = 10
+      subject.valid?.should be_false
+    end
+    it "should reject longitude numerical with latitude nil" do
+      subject = Factory :stop_area, :area_type => "BoardingPosition"
+      subject.longitude = 10
+      subject.latitude = nil
+      subject.valid?.should be_false
+    end
+  end 
+  
+
+  describe ".children_in_depth" do
+    it "should return all the deepest children from stop area" do
+      subject = Factory :stop_area, :area_type => "StopPlace"
+      commercial_stop_point = Factory :stop_area, :area_type => "CommercialStopPoint", :parent => subject 
+      commercial_stop_point2 = Factory :stop_area, :area_type => "CommercialStopPoint", :parent => commercial_stop_point
+      quay = Factory :stop_area, :parent => commercial_stop_point, :area_type => "Quay"
+      subject.children_in_depth.should =~ [commercial_stop_point, commercial_stop_point2, quay]
+    end
+    it "should return only the deepest children from stop area" do
+      subject = Factory :stop_area, :area_type => "StopPlace"
+      commercial_stop_point = Factory :stop_area, :area_type => "CommercialStopPoint", :parent => subject 
+      commercial_stop_point2 = Factory :stop_area, :area_type => "CommercialStopPoint", :parent => commercial_stop_point
+      quay = Factory :stop_area, :parent => commercial_stop_point, :area_type => "Quay"
+      subject.children_at_base.should =~ [quay]
+    end
+  end
+
+  describe ".stop_area_type" do
+    it "should have area_type of BoardingPosition when stop_area_type is set to boarding_position" do
+      subject = Factory :stop_area, :stop_area_type => "boarding_position"
+      subject.area_type.should == "BoardingPosition"
+    end
+    it "should have area_type of Quay when stop_area_type is set to quay" do
+      subject = Factory :stop_area, :stop_area_type => "quay"
+      subject.area_type.should == "Quay"
+    end
+    it "should have area_type of CommercialStopPoint when stop_area_type is set to commercial_stop_point" do
+      subject = Factory :stop_area, :stop_area_type => "commercial_stop_point"
+      subject.area_type.should == "CommercialStopPoint"
+    end
+    it "should have area_type of StopPlace when stop_area_type is set to stop_place" do
+      subject = Factory :stop_area, :stop_area_type => "stop_place"
+      subject.area_type.should == "StopPlace"
+    end
+    it "should have area_type of ITL when stop_area_type is set to itl" do
+      subject = Factory :stop_area, :stop_area_type => "itl"
+      subject.area_type.should == "ITL"
+    end
+  end
+
+  describe ".parent" do
+    it "should check if parent method exists" do
+      subject = Factory :stop_area, :parent_id => commercial_stop_point.id
+      subject.parent.should == commercial_stop_point
+    end
+  end
+
+  describe ".possible_children" do    
+    
+    it "should find no possible descendant for stop area type quay" do
+      subject = Factory :stop_area, :area_type => "Quay"
+      subject.possible_children.should == [] 
+    end
+
+    it "should find no possible descendant for stop area type boarding position" do
+      subject = Factory :stop_area, :area_type => "BoardingPosition"
+      subject.possible_children.should == [] 
+    end
+
+    it "should find descendant of type quay or boarding position for stop area type commercial stop point" do
+      subject = Factory :stop_area, :area_type => "CommercialStopPoint"
+      subject.possible_children.should =~ [quay, boarding_position] 
+    end
+
+    it "should find no children of type stop place or commercial stop point for stop area type stop place" do
+      subject = Factory :stop_area, :area_type => "StopPlace"
+      subject.possible_children.should =~ [stop_place, commercial_stop_point] 
+    end
+
+    it "should find no children of type ITL for stop area type ITL" do
+      subject = Factory :stop_area, :area_type => "ITL"
+      subject.possible_children.should =~ [stop_place, commercial_stop_point, quay, boarding_position] 
+    end
+
+  end
+
+  describe ".possible_parents" do
+
+    it "should find parent type commercial stop point for stop area type boarding position" do
+      subject = Factory :stop_area, :area_type => "BoardingPosition"
+      subject.possible_parents.should == [commercial_stop_point] 
+    end
+
+    it "should find parent type commercial stop point for stop area type quay" do
+      subject = Factory :stop_area, :area_type => "Quay"
+      subject.possible_parents.should == [commercial_stop_point] 
+    end    
+
+    it "should find parent type stop place for stop area type commercial stop point" do
+      subject = Factory :stop_area, :area_type => "CommercialStopPoint"
+      subject.possible_parents.should == [stop_place] 
+    end    
+
+    it "should find parent type stop place for stop area type stop place" do
+      subject = Factory :stop_area, :area_type => "StopPlace"
+      subject.possible_parents.should == [stop_place] 
+    end    
+
+  end
+
+
+  describe ".near" do
+
+    let(:stop_area) { Factory :stop_area, :latitude => 1, :longitude => 1 }
+    let(:stop_area2) { Factory :stop_area, :latitude => 1, :longitude => 1 }
+    
+    it "should find a StopArea at 300m from given origin" do
+      Chouette::StopArea.near(stop_area.to_lat_lng.endpoint(0, 0.250, :units => :kms)).should == [stop_area]
+    end
+
+    it "should not find a StopArea at more than 300m from given origin" do
+      Chouette::StopArea.near(stop_area2.to_lat_lng.endpoint(0, 0.350, :units => :kms)).should be_empty
+    end
+
+  end
+
+  describe "#to_lat_lng" do
+    
+    it "should return nil if latitude is nil" do
+      subject.latitude = nil
+      subject.to_lat_lng.should be_nil
+    end
+
+    it "should return nil if longitude is nil" do
+      subject.longitude = nil
+      subject.to_lat_lng.should be_nil
+    end
+
+  end
+
+  describe "#geometry" do
+    
+    it "should be nil when to_lat_lng is nil" do
+      subject.stub :to_lat_lng => nil
+      subject.geometry.should be_nil
+    end
+
+  end
+
+  describe ".bounds" do
+    
+    it "should return transform coordinates in floats" do
+      Chouette::StopArea.connection.stub :select_rows => [["113.5292500000000000", "22.1127580000000000", "113.5819330000000000", "22.2157050000000000"]]
+      GeoRuby::SimpleFeatures::Envelope.should_receive(:from_coordinates).with([[113.5292500000000000, 22.1127580000000000], [113.5819330000000000, 22.2157050000000000]])
+      Chouette::StopArea.bounds
+    end
+
+  end
+
+  describe "#default_position" do
+    
+    it "should return nil when StopArea.bounds is nil" do
+      Chouette::StopArea.stub :bounds => nil
+      subject.default_position.should be_nil
+    end
+
+    it "should return StopArea.bounds center" do
+      Chouette::StopArea.stub :bounds => double(:center => "center")
+      subject.default_position.should == Chouette::StopArea.bounds.center
+    end
+
+  end
+
+  describe "#children_at_base" do
+    it "should have 2 children_at_base" do
+      subject = Factory :stop_area, :area_type => "StopPlace" 
+      commercial_stop_point = Factory :stop_area, :area_type => "CommercialStopPoint" ,:parent => subject
+      quay1 = Factory :stop_area, :parent => commercial_stop_point, :area_type => "Quay"
+      quay2 = Factory :stop_area, :parent => commercial_stop_point, :area_type => "Quay"
+      subject.children_at_base.size.should == 2
+    end
+   end 
+
+
+  describe "#generic_access_link_matrix" do
+    it "should have no access_links in matrix with no access_point" do
+      subject = Factory :stop_area, :area_type => "StopPlace" 
+      commercial_stop_point = Factory :stop_area, :area_type => "CommercialStopPoint" ,:parent => subject
+      subject.generic_access_link_matrix.size.should == 0
+    end
+    it "should have 4 generic_access_links in matrix with 2 access_points" do
+      subject = Factory :stop_area, :area_type => "StopPlace" 
+      commercial_stop_point = Factory :stop_area, :area_type => "CommercialStopPoint" ,:parent => subject
+      access_point1 = Factory :access_point, :stop_area => subject
+      access_point2 = Factory :access_point, :stop_area => subject
+      subject.generic_access_link_matrix.size.should == 4
+    end
+   end 
+  describe "#detail_access_link_matrix" do
+    it "should have no access_links in matrix with no access_point" do
+      subject = Factory :stop_area, :area_type => "StopPlace" 
+      commercial_stop_point = Factory :stop_area, :area_type => "CommercialStopPoint" ,:parent => subject
+      quay1 = Factory :stop_area, :parent => commercial_stop_point, :area_type => "Quay"
+      quay2 = Factory :stop_area, :parent => commercial_stop_point, :area_type => "Quay"
+      subject.detail_access_link_matrix.size.should == 0
+    end
+    it "should have 8 detail_access_links in matrix with 2 children_at_base and 2 access_points" do
+      subject = Factory :stop_area, :area_type => "StopPlace" 
+      commercial_stop_point = Factory :stop_area, :area_type => "CommercialStopPoint" ,:parent => subject
+      quay1 = Factory :stop_area, :parent => commercial_stop_point, :area_type => "Quay"
+      quay2 = Factory :stop_area, :parent => commercial_stop_point, :area_type => "Quay"
+      access_point1 = Factory :access_point, :stop_area => subject
+      access_point2 = Factory :access_point, :stop_area => subject
+      subject.detail_access_link_matrix.size.should == 8
+    end
+   end 
+  describe "#parents" do
+    it "should return parent hireachy list" do
+      stop_place = Factory :stop_area, :area_type => "StopPlace" 
+      commercial_stop_point = Factory :stop_area, :area_type => "CommercialStopPoint" ,:parent => stop_place
+      subject = Factory :stop_area, :parent => commercial_stop_point, :area_type => "Quay"
+      subject.parents.size.should == 2
+    end
+    it "should return empty parent hireachy list" do
+      subject = Factory :stop_area, :area_type => "Quay"
+      subject.parents.size.should == 0
+    end
+  end
+  
+  describe "#clean_invalid_access_links" do
+    it "should remove invalid access links" do
+      # subject is a CSP with a SP as parent, a quay as child
+      # 2 access_points of SP have access_link, one on subject, one on subject child
+      # when detaching subject from SP, both access_links must be deleted
+      stop_place = Factory :stop_area, :area_type => "StopPlace" 
+      subject = Factory :stop_area, :area_type => "CommercialStopPoint" ,:parent => stop_place
+      access_point1 = Factory :access_point, :stop_area => stop_place
+      access_point2 = Factory :access_point, :stop_area => stop_place
+      quay = Factory :stop_area, :parent => subject, :area_type => "Quay"
+      access_link1 = Factory :access_link, :stop_area => subject, :access_point => access_point1
+      access_link2 = Factory :access_link, :stop_area => quay, :access_point => access_point2
+      subject.save 
+      subject.access_links.size.should == 1
+      quay.access_links.size.should == 1
+      subject.parent=nil
+      subject.save 
+      subject.reload
+      subject.access_links.size.should == 0
+      quay.access_links.size.should == 0
+    end
+    it "should not remove still valid access links" do
+      # subject is a Q of CSP with a SP as parent
+      # 2 access_points, one of SP, one of CSP have access_link on subject
+      # when changing subject CSP to another CSP of same SP
+      # one access_links must be kept
+      stop_place = Factory :stop_area, :area_type => "StopPlace" 
+      commercial_stop_point1 = Factory :stop_area, :area_type => "CommercialStopPoint" ,:parent => stop_place
+      commercial_stop_point2 = Factory :stop_area, :area_type => "CommercialStopPoint" ,:parent => stop_place
+      access_point1 = Factory :access_point, :stop_area => stop_place
+      access_point2 = Factory :access_point, :stop_area => commercial_stop_point1
+      subject = Factory :stop_area, :parent => commercial_stop_point1, :area_type => "Quay"
+      access_link1 = Factory :access_link, :stop_area => subject, :access_point => access_point1
+      access_link2 = Factory :access_link, :stop_area => subject, :access_point => access_point2
+      subject.save 
+      subject.access_links.size.should == 2
+      subject.parent=commercial_stop_point2
+      subject.save 
+      subject.reload
+      subject.access_links.size.should == 1
+    end
+  end
+  
+  describe "#coordinates" do
+    it "should convert coordinates into latitude/longitude" do
+     subject = Factory :stop_area, :area_type => "BoardingPosition", :coordinates => "45.123,120.456"
+     subject.longitude.should be_within(0.001).of(120.456)
+     subject.latitude.should be_within(0.001).of(45.123)
+   end
+    it "should set empty coordinates into nil latitude/longitude" do
+     subject = Factory :stop_area, :area_type => "BoardingPosition", :coordinates => "45.123,120.456"
+     subject.longitude.should be_within(0.001).of(120.456)
+     subject.latitude.should be_within(0.001).of(45.123)
+     subject.coordinates = ""
+     subject.save
+     subject.longitude.should be_nil
+     subject.latitude.should be_nil
+   end
+    it "should convert latitude/longitude into coordinates" do
+     subject = Factory :stop_area, :area_type => "BoardingPosition", :longitude => 120.456, :latitude => 45.123
+     subject.coordinates.should == "45.123,120.456"
+   end
+    it "should convert nil latitude/longitude into empty coordinates" do
+     subject = Factory :stop_area, :area_type => "BoardingPosition", :longitude => nil, :latitude => nil
+     subject.coordinates.should == ""
+   end
+    it "should accept valid coordinates" do
+     subject = Factory :stop_area, :area_type => "BoardingPosition", :coordinates => "45.123,120.456"
+     subject.valid?.should be_true
+     subject.coordinates = "45.123, 120.456"
+     subject.valid?.should be_true
+     subject.longitude.should be_within(0.001).of(120.456)
+     subject.latitude.should be_within(0.001).of(45.123)
+     subject.coordinates = "45.123,  -120.456"
+     subject.valid?.should be_true
+     subject.coordinates = "45.123 ,120.456"
+     subject.valid?.should be_true
+     subject.coordinates = "45.123   ,   120.456"
+     subject.valid?.should be_true
+     subject.coordinates = " 45.123,120.456"
+     subject.valid?.should be_true
+     subject.coordinates = "45.123,120.456  "
+     subject.valid?.should be_true
+    end
+    it "should accept valid coordinates on limits" do
+     subject = Factory :stop_area, :area_type => "BoardingPosition", :coordinates => "90,180"
+     subject.valid?.should be_true
+     subject.coordinates = "-90,-180"
+     subject.valid?.should be_true
+     subject.coordinates = "-90.,180."
+     subject.valid?.should be_true
+     subject.coordinates = "-90.0,180.00"
+     subject.valid?.should be_true
+    end
+    it "should reject invalid coordinates" do
+     subject = Factory :stop_area, :area_type => "BoardingPosition"
+     subject.coordinates = ",12"
+     subject.valid?.should be_false
+     subject.coordinates = "-90"
+     subject.valid?.should be_false
+     subject.coordinates = "-90.1,180."
+     subject.valid?.should be_false
+     subject.coordinates = "-90.0,180.1"
+     subject.valid?.should be_false
+     subject.coordinates = "-91.0,18.1"
+     subject.valid?.should be_false
+    end
+  end
+
+  describe "#duplicate" do
+      it "should be a copy of" do
+        stop_place = Factory :stop_area, :area_type => "StopPlace" 
+        subject = Factory :stop_area, :area_type => "CommercialStopPoint" ,:parent => stop_place, :coordinates => "45.123,120.456"
+        access_point1 = Factory :access_point, :stop_area => subject
+        access_point2 = Factory :access_point, :stop_area => subject
+        quay1 = Factory :stop_area, :parent => subject, :area_type => "Quay"
+        target=subject.duplicate
+        target.id.should be_nil
+        target.name.should == "Copy of "+subject.name
+        target.objectid.should == subject.objectid+"_1"
+        target.area_type.should == subject.area_type
+        target.parent.should be_nil
+        target.children.size.should == 0
+        target.access_points.size.should == 0
+        target.coordinates.should == "45.123,120.456"
+      end
+  end
+
+
+end