gares 0.1.4 → 1.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: e2ad3bf50a1a740e624ca06ba9ea4de2173b0b22
-  data.tar.gz: 7e04564b13989559a80d666ce3d74717d1863ab9
+  metadata.gz: 371e96298472d334dcfbce42c20bc6a67813139a
+  data.tar.gz: 3dc98f7e626ed7d6092b912a8e3441047bffb0c3
 SHA512:
-  metadata.gz: d1794901b7b8dc65826c3eb3fa07457798eccc6462828cda410c4d14fbff7062b49c6cfa85b8f5ef9cb7cf7b340378c68a688bf26ad66b52298ba834b194f92a
-  data.tar.gz: e8b3bc039ff0b802dab7ee8ff1a81c60915774d275da36110d23db98ba01204cde0fcf8412a5708da63fb559ab8778e47fada48dcf1d97d028190e03f6d1f082
+  metadata.gz: 89a404991c7555b8c65ccfa57987f8e553b03f18c284be6a73536ab0ab3213e8060e32faafe29b2d4b8492c340d0ec8586ffe7cbf0b678ba8f71efe1485b69aa
+  data.tar.gz: a19cc680b00bc2244cf8670e0c60e60c91a92e75a0bade68004abc8adfc6f3c9e90a58000cc25700b0186629cbf0de49969162831c34a5caff3b9c48e40bccb9

data/.gitignore

@@ -6,3 +6,4 @@ rdoc/*
 Gemfile.lock
 .rvmrc
 .gh_pages
+coverage/

data/Guardfile

@@ -0,0 +1,6 @@
+group 'gem' do
+  guard 'rspec', :cmd => "bundle exec rspec", :all_on_start => false, :all_after_pass => false, :failed_mode => :focus do
+    watch(%r{^spec/.+_spec\.rb$})
+    watch(%r{^lib/(.+)\.rb$})                          { |m| "spec/gares/#{m[1]}_spec.rb" }
+  end
+end

data/README.md

@@ -1,23 +1,23 @@
-# Gares (en-mouvement)
+# Gares - A gem to interact with French train Stations
 
 [![Build Status](https://travis-ci.org/paulRbr/gares.svg)](https://travis-ci.org/paulRbr/gares) [![Gem Version](https://badge.fury.io/rb/gares.svg)](http://badge.fury.io/rb/gares) [![Coverage Status](https://coveralls.io/repos/paulRbr/gares/badge.svg)](https://coveralls.io/r/paulRbr/gares)
 
 ## Description
 
-The Gares gem allows you to easy access publicly available data from gares-en-mouvement.com.
+The Gares gem allows you to easily access publicly available data from gares-en-mouvement.com.
 
 ## Features
 
 Gares currently features the following:
 
-* Search for a gare
-* Retrieve gare information
+* Search for a station
+* Retrieve station information
 
 ## Examples
 
-### Gare:
+### Station:
 
-    g = Gares::Gare.new("frlpd")
+    g = Gares::Station.new("frlpd")
 
     g.name
     #=> "Lyon Part Dieu"
@@ -38,41 +38,41 @@ Gares currently features the following:
 
     g = Gares::Search.new("Aix")
 
-    g.gares.size
+    g.stations.size
     #=> 7
 
     # or
 
-    gares = Gares::Gare.search("lyon")
-    gare = gares.last
+    stations = Gares::Station.search("lyon")
+    station = stations.last
 
-    gare.name
+    station.name
     #=> "Paris Gare de Lyon"
 
 ## Installation
 
     gem install gares
 
-Or, if you're using this in a project with Bundler:
+Or if you want to use it in a project add this to your `Gemfile`:
 
-    gem gares
+    gem "gares"
 
 ## Running Tests
 
-As this gem uses content from gare-en-mouvement.com, the test suite uses a set of
-pre-defined fixute files in `spec/fixtures`. These fixtures are
-copies of gares-en-mouvement page used in tests.
+As this gem uses external content from gare-en-mouvement.com and sncf.com, the test suite uses a set of
+pre-defined fixture files in `spec/fixtures`. These fixtures are
+copies of gares-en-mouvement.com and sncf.com pages used in tests.
 
 Run bundle install to install all dependencies, including fakeweb, which
-will serve the fixture files instead of doing actual requests to gares-en-mouvement.com.
+will serve the fixture files instead of doing actual requests to gares-en-mouvement.com or sncf.com.
 
     $ bundle install
 
 Next, simple run `rake` to run the entire test suite.
 
-### Running against actual gare-en-mouvement data
+### Running against real data
 
-It's possible to run the test suite directly against gares-en-mouvement.com. This has
+It's possible to run the test suite directly against gares-en-mouvement.com or sncf.com. This has
 two disadvantages:
 
  1. Tests will be slow
@@ -80,7 +80,7 @@ two disadvantages:
 
     $ LIVE_TEST=true rake
 
-If you want to run against actual gares-en-mouvement data, it's better to just update
+If you want to run against actual data, it's better to just update
 the fixture files once with up-to-date content:
 
     $ rake fixtures:refresh
@@ -92,12 +92,12 @@ When you run the test suite now, it will use the updated fixture files.
 Neither I, nor any developer who contributed to this project, accept any kind of
 liability for your use of this library.
 
-gares-en-mouvement does not permit use of its data by third parties without their consent.
+gares-en-mouvement.com and sncf.com certainly does not permit use of its data by third parties without their consent.
 
 Using this library for anything other than limited personal use may result
-in an IP ban to the gares-en-mouvement website.
+in an IP ban to the gares-en-mouvement.com or sncf.com website.
 
-_This gem is not endorsed or affiliated with gares-en-mouvement.com, or SNCF, Inc._
+_This gem is not endorsed or affiliated with gares-en-mouvement.com, nor SNCF, Inc._
 
 ## License
 

data/lib/gares/base.rb

@@ -6,13 +6,13 @@ module Gares
     GPS_COORD = 'Coordonnées GPS : '
     NAME = 'En direct de '
 
-    # Initialize a new Gare object with it's gare-en-mouvemnt id (as a String)
+    # Initialize a new Station object with it's gare-en-mouvemnt id (as a String)
     #
-    #   gare = Gares::Gare.new("frabt")
+    #   station = Gares::Station.new("frabt")
     #
-    # Gares::Gare objects are lazy loading, meaning that no HTTP request
-    # will be performed when a new object is created. Only when you use an
-    # accessor that needs the remote data, a HTTP request is made (once).
+    # Gares::Station objects are lazy loaded, meaning that no HTTP request
+    # will be performed when a new object is created. An HTTP request is made (once)
+    # Only when you use an accessor that needs the remote data.
     #
     def initialize(slug, name = nil)
       @slug = slug
@@ -76,7 +76,7 @@ module Gares
 
     # Returns a new Nokogiri document for parsing.
     def document
-      @document ||= Nokogiri::HTML(Gares::Gare.find_by_slug(@slug))
+      @document ||= Nokogiri::HTML(Gares::Station.find_by_slug(@slug))
     end
 
     # Use HTTParty to fetch the raw HTML for this gare.
@@ -86,7 +86,7 @@ module Gares
 
     # Convenience method for search
     def self.search(query)
-      Gares::Search.new(query).gares
+      Gares::Search.new(query).stations
     end
-  end # Gare
+  end # Base
 end # Gares

data/lib/gares/sales.rb

@@ -9,7 +9,7 @@ module Gares
     private
 
     def document
-      @document ||= Nokogiri::HTML(Gares::Gare.find_by_slug(
+      @document ||= Nokogiri::HTML(Gares::Station.find_by_slug(
         @slug, :"services-en-gare/vente/"))
     end
   end

data/lib/gares/search.rb

@@ -1,25 +1,26 @@
 module Gares
-  # Search Gares-en-mouvement for a gare name
-  class Search < GareList
+  # Search Gares-en-mouvement for a station name
+  class Search < StationList
     attr_reader :query
 
+    # This is a file containing minimal information (name and slug) of all stations of gares-en-mouvement.com
     GARES_LIST_URL = "https://www.kimonolabs.com/api/7jys32dy?apikey=lsOO4tNm78cH9JxqWg9gAk9l4nYaou9j&kimmodify=1"
 
-    # Initialize a new Gares search with the specified query
+    # Initialize a new Station search with the specified query
     #
     #   search = Gares::Search.new("Aix")
     #
-    # Gares::Search is lazy loading, meaning that unless you access the +gares+
-    # attribute, no query is made to gares-en-mouvement.com.
+    # Gares::Search is lazy loaded, meaning that unless you access the +stations+
+    # attribute, no remomte query is made.
     #
     def initialize(query)
       @query = query
     end
 
-    # Returns an array of Gares::Gare objects for easy search result yielded.
+    # Returns an array of Gares::Station objects in order to easily search result yielded.
     # If the +query+ was an exact match, a single element array will be returned.
-    def gares
-      @gares ||= (exact_match? ? parse_gare : parse_gares)
+    def stations
+      @stations ||= (exact_match? ? parse_station : parse_stations)
     end
 
     private
@@ -29,16 +30,18 @@ module Gares
     end
 
     def result
-      @result ||= document.results.collection1.map(&:station).select { |gare| gare.name.to_ascii =~ /#{@query}/i }
+      @result ||= document.results.collection1.map(&:station).select do |station|
+        station.name.to_ascii =~ /#{@query.split(" ").join(".*")}/i
+      end
     end
 
     def self.query
       open(GARES_LIST_URL)
     end
 
-    def parse_gare
-      gare = result.first
-      [Gares::Gare.new(gare.slug, gare.name)]
+    def parse_station
+      station = result.first
+      [Gares::Station.new(station.slug, station.name)]
     end
 
     def exact_match?

data/lib/gares/services.rb

@@ -9,7 +9,7 @@ module Gares
     private
 
     def document
-      @document ||= Nokogiri::HTML(Gares::Gare.find_by_slug(
+      @document ||= Nokogiri::HTML(Gares::Station.find_by_slug(
         @slug, :"services-en-gare/service/"))
     end
   end

data/lib/gares/station.rb

@@ -0,0 +1,5 @@
+module Gares
+  # Represents a station on gares-en-mouvement.com
+  class Station < Base
+  end # Station
+end # Gares

data/lib/gares/station_list.rb

@@ -0,0 +1,12 @@
+module Gares
+  class StationList
+
+    private
+
+    def parse_stations
+      result.compact.uniq.map do |station|
+        Gares::Station.new(station.slug, station.name)
+      end
+    end
+  end # StationList
+end # Gares

data/lib/gares/version.rb

@@ -1,3 +1,3 @@
 module Gares
-  VERSION = '0.1.4'
+  VERSION = '1.0.0'
 end

data/lib/gares.rb

@@ -9,8 +9,8 @@ require 'hashie'
 require 'unidecoder'
 
 require 'gares/base'
-require 'gares/gare'
-require 'gares/gare_list'
+require 'gares/station'
+require 'gares/station_list'
 require 'gares/sales'
 require 'gares/services'
 require 'gares/search'

data/spec/gares/gare_spec.rb

@@ -6,14 +6,13 @@ require 'spec_helper'
 #
 #     http://www.gares-en-mouvement.com/fr/frlpd/votre-gare/
 #
-
-describe Gares::Gare do
+describe Gares::Station do
 
   describe 'valid gare' do
 
     before(:each) do
       # Get gare de Lyon Part-Dieu
-      @gare = Gares::Gare.new('frlpd')
+      @gare = Gares::Station.new('frlpd')
     end
 
     it 'should find the name' do
@@ -53,7 +52,7 @@ describe Gares::Gare do
     context 'a gare without wifi nor defibrillator' do
       before(:each) do
         # Get gare de Agde
-        @gare = Gares::Gare.new('frxag')
+        @gare = Gares::Station.new('frxag')
       end
 
       it { expect(@gare.wifi?).to be(false) }
@@ -63,7 +62,7 @@ describe Gares::Gare do
     context 'a gare with no sales services' do
       before(:each) do
         # Get gare de Agde
-        @gare = Gares::Gare.new('frxag')
+        @gare = Gares::Station.new('frxag')
       end
 
       it { expect(@gare.has_borne?).to be(false) }
@@ -73,7 +72,7 @@ describe Gares::Gare do
   describe 'with name that has utf-8 characters' do
     # Belleville sur Sâone
     before(:each) do
-      @gare = Gares::Gare.search('Saone').first
+      @gare = Gares::Station.search('Saone').first
     end
 
     it 'should give the proper name' do

data/spec/gares/sales_spec.rb

@@ -1,6 +1,6 @@
 describe Gares::Sales do
 
-  context 'with a valid gare' do
+  context 'with a valid station' do
 
     subject(:sales) { Gares::Sales.new('frqxb') }
 

data/spec/gares/search_spec.rb

@@ -11,32 +11,45 @@ describe Gares::Search do
     end
 
     it 'should find 5 results' do
-      expect(@search.gares.size).to eql(7)
+      expect(@search.stations.size).to eql(7)
     end
 
-    it 'should return Gares::Gare objects only' do
-      expect(@search.gares).to all(be_an(Gares::Gare))
+    it 'should return Gares::Station objects only' do
+      expect(@search.stations).to all(be_an(Gares::Station))
     end
 
     it 'should not return gares with no name' do
-      @search.gares.each { |gare| expect(gare.name).to_not be_blank }
+      @search.stations.each { |gare| expect(gare.name).to_not be_blank }
     end
 
     it 'should return only the name of the result' do
-      expect(@search.gares.first.name).to eql('Aix en Provence')
+      expect(@search.stations.first.name).to eql('Aix en Provence')
     end
   end
 
   context 'with an exact match' do
     it 'should not raise an exception' do
       expect do
-        @search = Gares::Search.new('Paris Austerlitz').gares
+        @search = Gares::Search.new('Paris Austerlitz').stations
       end.not_to raise_error
     end
 
-    it 'should return the movie slug correctly' do
+    it 'should return the gare slug correctly' do
       @search = Gares::Search.new('Paris Austerlitz')
-      expect(@search.gares.first.slug).to eql('frpaz')
+      expect(@search.stations.first.slug).to eql('frpaz')
+    end
+  end
+
+  context 'with a fuzzy match' do
+    it 'should not raise an exception' do
+      expect do
+        @search = Gares::Search.new('CULMONT CHALINDREY').stations
+      end.not_to raise_error
+    end
+
+    it 'should return the gare slug correctly' do
+      @search = Gares::Search.new('CULMONT CHALINDREY')
+      expect(@search.stations.first.slug).to eql('frccy')
     end
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: gares
 version: !ruby/object:Gem::Version
-  version: 0.1.4
+  version: 1.0.0
 platform: ruby
 authors:
 - Paul Bonaud
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-03-11 00:00:00.000000000 Z
+date: 2015-04-25 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: nokogiri
@@ -177,17 +177,18 @@ files:
 - ".rubocop_todo.yml"
 - ".travis.yml"
 - Gemfile
+- Guardfile
 - MIT-LICENSE
 - README.md
 - Rakefile
 - gares.gemspec
 - lib/gares.rb
 - lib/gares/base.rb
-- lib/gares/gare.rb
-- lib/gares/gare_list.rb
 - lib/gares/sales.rb
 - lib/gares/search.rb
 - lib/gares/services.rb
+- lib/gares/station.rb
+- lib/gares/station_list.rb
 - lib/gares/string_extensions.rb
 - lib/gares/version.rb
 - spec/fixtures/frabt

data/lib/gares/gare.rb

@@ -1,5 +0,0 @@
-module Gares
-  # Represents a Gare on gares-en-mouvement.com
-  class Gare < Base
-  end # Gare
-end # Gares

data/lib/gares/gare_list.rb

@@ -1,12 +0,0 @@
-module Gares
-  class GareList
-
-    private
-
-    def parse_gares
-      result.compact.uniq.map do |gare|
-        Gares::Gare.new(gare.slug, gare.name)
-      end
-    end
-  end # GareList
-end # Gares