auto_service_cli 0.1.0 → 0.1.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 355c1cc539e58fa1e332f92898aa552df877911f
-  data.tar.gz: 5ece5d2c31b308bc8679059beb6e22069f9f317a
+  metadata.gz: 460899cc76d351b7302046fd7208b3e0a8e96250
+  data.tar.gz: 5c1cade52f7afc886541c85e4bf5701aec5af1e1
 SHA512:
-  metadata.gz: 26feeebf485c5edd172f49769f1e71e04621f71e44219ff9b32bb626a68d909c653ba9fb23c26aed4db5f7dd016c256c76365a7197fc33d1c523a6c88b7826f3
-  data.tar.gz: ac20eca2ca3f7894b75cb722bc8fa99ad9af6a146c8ed97eda6b441f739b92842124b9b042c500faea12dd94804dd2b0f753045b34c9e6663a1da2b77b63e208
+  metadata.gz: c90f28f29bc86d03fcc0f5b90fb8dbb044a5b72980069e0ddcf8b8647ca375ca6cce43be8dc54178b93c2e4ecea590302b2d99068bdaa10fc761691cbd971ff1
+  data.tar.gz: 03dc7dbecb932690a5b4443b6574dbbe4e45f6bc5dcd13dc586c1ab2b2388906d61af4eef89c7da25de3d800569a66d97291029eed2c489b6304d1937356c354

data/auto_service_cli.gemspec

@@ -10,8 +10,8 @@ Gem::Specification.new do |spec|
   spec.authors       = ["Aleksandr Rogachev"]
   spec.email         = ["aleksandr.rogachev1994@gmail.com"]
 
-  spec.summary       = %q{CLI for searching of auto service centers near your location.}
-  spec.description   = %q{This CLI allows to find service centers near desired zip code (with sorting) and see details about each center.}
+  spec.summary       = %q{CLI for searching auto service centers near your location}
+  spec.description   = %q{This CLI allows you to find auto service centers near desired zip code, sort them, and fetch details about each of them}
   spec.homepage      = "https://github.com/AleksandrRogachev94/auto_service_cli"
   spec.license       = "MIT"
 
@@ -22,11 +22,11 @@ Gem::Specification.new do |spec|
   spec.executables << "auto_service"
   spec.require_paths = ["lib"]
 
-  spec.add_dependency "nokogiri", "~> 1.7"
+  spec.add_dependency "nokogiri", "~> 1.8"
   spec.add_dependency "colorize", "~> 0.8"
 
-  spec.add_development_dependency "bundler", "~> 1.13"
-  spec.add_development_dependency "rake", "~> 10.0"
+  spec.add_development_dependency "bundler", "~> 1.16"
+  spec.add_development_dependency "rake", "~> 10.5"
   spec.add_development_dependency "rspec", "~> 3.0"
-  spec.add_development_dependency "pry", "~> 0.10"
+  spec.add_development_dependency "pry", "~> 0.11"
 end

data/bin/auto_service

@@ -1,6 +1,6 @@
-#!/usr/bin/env ruby
-
-require "bundler/setup"
-require_relative "../lib/auto_service_cli"
-
-AutoServiceCLI::CLI.new.call
+#!/usr/bin/env ruby
+
+require "bundler/setup"
+require_relative "../lib/auto_service_cli"
+
+AutoServiceCLI::CLI.new.call

data/lib/auto_service_cli/cli.rb

@@ -1,162 +1,150 @@
-class AutoServiceCLI::CLI
-
-  attr_reader :zip, :cur_sort_type
-  attr_reader :scraper
-
-  def initialize
-    self.cur_sort_type = "default"
-  end
-
-  def call
-    puts "\n\tWelcome to auto service centers searching CLI!".green
-    prompt_zip
-    scrape_main_page
-    list_centers
-    menu
-  end
-
-  def prompt_zip
-    begin
-      puts "\tEnter you zip code:".magenta
-      zip = gets.strip
-    end until AutoServiceCLI::Scraper.valid_zip?(zip)
-    self.zip = zip
-  end
-
-  # Writers and Readers
-
-  def zip=(zip)
-    @zip = zip if AutoServiceCLI::Scraper.valid_zip?(zip)
-  end
-
-  def cur_sort_type=(type)
-    @cur_sort_type = type if AutoServiceCLI::Scraper.valid_sort_type?(type)
-  end
-
-  #-----------------------------------------------------------------------------------
-  # Menu methods
-
-  def menu
-    loop do
-      help_menu
-      puts "\nMake a choise:".magenta
-      input = gets.strip
-      case input
-      when "1"
-        list_centers
-      when "2"
-        sort
-      when "3"
-        get_details
-      when "4"
-        scrape_main_page
-        list_centers
-      when "10"
-        goodbye
-        break
-      end
-    end
-  end
-
-  def list_centers
-    puts "-----------------------------------------------------------------------------"
-    puts "\tZIP: #{self.scraper.zip}\n\tSorting type: #{self.scraper.sort_type}\n".cyan
-    AutoServiceCLI::ServiceCenter.all.each.with_index(1) do |center,i|
-      print "#{i}.".cyan; print " #{center.name}"
-      puts center.rating.nil? ? "" : ", rating: #{center.rating}"
-    end
-    puts "-----------------------------------------------------------------------------"
-  end
-
-  def sort
-    help_sort
-    puts "\n\tChoose type of sorting:".magenta
-    input = gets.strip
-    case input
-    when "1"
-      self.cur_sort_type = "default"
-    when "2"
-      self.cur_sort_type = "distance"
-    when "3"
-      self.cur_sort_type = "average_rating"
-    when "4"
-      self.cur_sort_type = "name"
-    end
-    scrape_main_page
-    list_centers
-  end
-
-  def get_details
-    puts "\n\tEnter the number of center:".green
-    input = gets.strip
-
-    if input.to_i >= 1 && input.to_i <= AutoServiceCLI::ServiceCenter.all.size
-      center = AutoServiceCLI::ServiceCenter.all[input.to_i - 1]
-      unless center.int_url.nil?
-        puts "\nObtaining data..."
-        self.scraper.scrape_center_details(center)
-        puts "Done"
-      end
-
-      puts "----------------------------------------------------------------------------------------------------------------"
-      puts "\n\t#{center.name.upcase}\n".red
-      puts "\tRating:\n#{center.rating}\n".cyan unless center.rating.nil?
-      puts "\tCategory:\n#{center.main_category}\n".cyan unless center.main_category.nil?
-      puts "\tAddress:\n#{center.address}\n".cyan unless center.address.nil?
-      puts"\tPhone number:\n#{center.phone_number}\n".cyan unless center.phone_number.nil?
-
-      unless center.int_url.nil?
-        puts "\tStatus:\n#{center.open_status}\n".cyan unless center.open_status.nil?
-        puts "\tSlogan:\n#{center.slogan}\n".cyan unless center.slogan.nil?
-        puts "\tWorking hours:\n#{center.working_hours}".cyan unless center.working_hours.nil?
-        puts "\tDescription:\n#{center.description}\n".cyan unless center.description.nil?
-        puts "\tServices:\n#{center.services}\n".cyan unless center.services.nil?
-        puts "\tBrands:\n#{center.brands}\n".cyan unless center.brands.nil?
-        puts "\tPayment methods:\n#{center.payment}\n".cyan unless center.payment.nil?
-      end
-
-      puts "\tSee more at:\n#{center.ext_url}\n".cyan unless center.ext_url.nil?
-      puts "----------------------------------------------------------------------------------------------------------------"
-    end
-  end
-
-  def scrape_main_page
-    puts "Obtaining data..."
-    AutoServiceCLI::ServiceCenter.reset_all!
-    case self.cur_sort_type
-    when "default"
-      @scraper = AutoServiceCLI::Scraper.new(self.zip, self.cur_sort_type)
-    when "distance"
-      @scraper =  AutoServiceCLI::Scraper.new(self.zip, self.cur_sort_type)
-    when "average_rating"
-      @scraper =  AutoServiceCLI::Scraper.new(self.zip, self.cur_sort_type)
-    when "name"
-      @scraper =  AutoServiceCLI::Scraper.new(self.zip, self.cur_sort_type)
-      puts "name"
-    end
-    self.scraper.scrape_centers
-    puts "Done"
-  end
-
-  #-----------------------------------------------------------------------------------
-  # Helper methods
-
-  def help_menu
-    puts "\n1. List centers".green
-    puts "2. Change sorting type".green
-    puts "3. Show details about service center".green
-    puts "4. Reload centers".green
-    puts "10. Exit".green
-  end
-
-  def help_sort
-    puts "\n\t1. Default".green
-    puts "\t2. Sort by distance".green
-    puts "\t3. Sort by rating".green
-    puts "\t4. Sort by name".green
-  end
-
-  def goodbye
-    puts "\n\tThank you for using this application!".blue
-  end
-end
+# TODO cur_sort_type string => data type
+
+
+class AutoServiceCLI::CLI
+  attr_accessor :scraper, :scraper
+  private :scraper, :scraper=
+
+  def initialize
+    self.scraper = AutoServiceCLI::Scraper.new
+  end
+
+  def call
+    welcome
+    prompt_zip
+    scrape_main_page
+    list_centers
+    menu
+  end
+
+  def prompt_zip
+    begin
+      puts "\tEnter you zip code:".magenta
+      zip = gets.strip
+    end until AutoServiceCLI::Scraper.valid_zip?(zip)
+    scraper.zip = zip
+  end
+
+  #-----------------------------------------------------------------------------------
+  # Menu methods
+
+  def menu
+    loop do
+      help_menu
+      puts "\nMake a choice:".magenta
+      input = gets.strip
+      case input
+      when "1"
+        list_centers
+      when "2"
+        get_details
+      when "3"
+        sort
+      when "4"
+        scrape_main_page
+        list_centers
+      when "10"
+        goodbye
+        break
+      end
+    end
+  end
+
+  def list_centers
+    puts "-----------------------------------------------------------------------------"
+    puts "\tZIP: #{scraper.zip}\n\tSorting type: #{scraper.sort_type}\n".cyan
+    AutoServiceCLI::ServiceCenter.all.each.with_index(1) do |center,i|
+      print "#{i}.".cyan; print " #{center.name}"
+      puts center.rating.nil? ? "" : ", rating: #{center.rating}"
+    end
+    puts "-----------------------------------------------------------------------------"
+  end
+
+  def sort
+    help_sort
+    puts "\n\tChoose type of sorting:".magenta
+    input = gets.strip
+    case input
+    when "1"
+      scraper.sort_type = AutoServiceCLI::Scraper::SORT_TYPES[:DAFAULT]
+    when "2"
+      scraper.sort_type = AutoServiceCLI::Scraper::SORT_TYPES[:DISTANCE]
+    when "3"
+      scraper.sort_type = AutoServiceCLI::Scraper::SORT_TYPES[:AVERAGE_RATING]
+    when "4"
+      scraper.sort_type = AutoServiceCLI::Scraper::SORT_TYPES[:NAME]
+    end
+    scrape_main_page
+    list_centers
+  end
+
+  def get_details
+    puts "\n\tEnter the number of center:".green
+    input = gets.strip
+
+    if input.to_i >= 1 && input.to_i <= AutoServiceCLI::ServiceCenter.all.size
+      center = AutoServiceCLI::ServiceCenter.all[input.to_i - 1]
+      unless center.int_url.nil?
+        puts "\nObtaining data..."
+        scraper.scrape_center_details(center)
+        puts "Done"
+      end
+
+      puts "----------------------------------------------------------------------------------------------------------------"
+      puts "\n\t#{center.name.upcase}\n".red
+      puts "\tRating:\n#{center.rating}\n".cyan unless center.rating.nil?
+      puts "\tCategory:\n#{center.main_category}\n".cyan unless center.main_category.nil?
+      puts "\tAddress:\n#{center.address}\n".cyan unless center.address.nil?
+      puts"\tPhone number:\n#{center.phone_number}\n".cyan unless center.phone_number.nil?
+
+      unless center.int_url.nil?
+        puts "\tStatus:\n#{center.open_status}\n".cyan unless center.open_status.nil?
+        puts "\tSlogan:\n#{center.slogan}\n".cyan unless center.slogan.nil?
+        puts "\tWorking hours:\n#{center.working_hours}".cyan unless center.working_hours.nil?
+        puts "\tDescription:\n#{center.description}\n".cyan unless center.description.nil?
+        puts "\tServices:\n#{center.services}\n".cyan unless center.services.nil?
+        puts "\tBrands:\n#{center.brands}\n".cyan unless center.brands.nil?
+        puts "\tPayment methods:\n#{center.payment}\n".cyan unless center.payment.nil?
+      end
+
+      puts "\tSee more at:\n#{center.ext_url}\n".cyan unless center.ext_url.nil?
+      puts "----------------------------------------------------------------------------------------------------------------"
+    end
+  end
+
+  def scrape_main_page
+    puts "Obtaining data for you... It will take a few seconds"
+
+    # clean up all records.
+    AutoServiceCLI::ServiceCenter.reset_all!
+
+    scraper.scrape_centers
+    puts "Done"
+  end
+
+  #-----------------------------------------------------------------------------------
+  # Helper methods
+
+  def welcome
+    puts "\n\tWelcome to auto service centers searching CLI!\n\tby Aleksandr Rogachev © #{Time.new.year}\n".green
+  end
+
+  def help_menu
+    puts "\n1. List centers".green
+    puts "2. Show details about service center".green
+    puts "3. Change sorting type".green
+    puts "4. Reload centers".green
+    puts "10. Exit".green
+  end
+
+  def help_sort
+    puts "\n\t1. Default".green
+    puts "\t2. Sort by distance".green
+    puts "\t3. Sort by rating".green
+    puts "\t4. Sort by name".green
+  end
+
+  def goodbye
+    puts "\n\tThank you for using this application!".blue
+  end
+end

data/lib/auto_service_cli/constants.rb

@@ -1,4 +1,4 @@
-module AutoServiceCLI
-  URL_BASE = "http://www.yellowpages.com"
-  URL_TEMPLATE = "http://www.yellowpages.com/search?search_terms=auto%20service&geo_location_terms="
-end
+module AutoServiceCLI
+  URL_BASE = "https://www.yellowpages.com"
+  URL_TEMPLATE = "https://www.yellowpages.com/search?search_terms=auto%20service&geo_location_terms="
+end

data/lib/auto_service_cli/scraper.rb

@@ -1,163 +1,190 @@
-class AutoServiceCLI::Scraper
-  attr_reader :doc, :sort_type, :zip
-
-  # Constructors and class methods.
-
-  def initialize(zip, sort_type = "default")
-    @sort_type = sort_type
-    @zip = zip
-    @doc = Nokogiri::HTML(open(get_url))
-  end
-
-  def self.valid_zip?(zip)
-      zip.to_i > 0
-  end
-
-  def self.valid_sort_type?(type)
-    type == "default" || type == "distance" || type == "average_rating" || type == "name"
-  end
-
-  # Helper methods
-
-  def get_url
-    raise InvalidURLData, "Invalid input data: zip - #{self.zip}, sort type - #{self.sort_type}" unless valid_url_data?
-    AutoServiceCLI::URL_TEMPLATE + "#{self.zip}&s=#{self.sort_type}"
-  end
-
-  def valid_url_data?
-    self.class.valid_zip?(self.zip) && self.class.valid_sort_type?(self.sort_type)
-  end
-
-  def external_link?(url)
-    url.include?("www.") || url.include?("http://") || url.include?("https://")
-  end
-
-  #------------------------------------------------------------------------------------------------
-  #Scrape from the main page.
-
-  def scrape_centers
-    raise InvalidPage, "Invalid page!!!" if self.doc == nil
-    centers = self.doc.css(".organic .result .info")
-
-    centers.each do |center|
-
-      main_details = {}
-      obj_center = AutoServiceCLI::ServiceCenter.create(center.css(".n a").text)
-
-      url = scrape_internal_url(center); main_details[:int_url] = url unless url.nil?
-      url = scrape_external_url(center); main_details[:ext_url] = url unless url.nil?
-      rating = scrape_rating(center); main_details[:rating] = rating unless rating.nil?
-      address = scrape_address(center); main_details[:address] = address unless address.nil?
-      phone = scrape_phone_number(center); main_details[:phone_number] = phone unless phone.nil?
-      category = scrape_category(center); main_details[:main_category] = category unless category.nil?
-
-      obj_center.details_from_hash(main_details) # setting all details to center.
-    end
-  end
-
-  #------------------------------------------------------------------------------------------------
-  # Scrape center details from the internal url (if available)
-
-  def scrape_center_details(center)
-    raise InvalidPage, "Invalid page!!!" if !center.is_a? AutoServiceCLI::ServiceCenter || center.int_url.nil?
-    doc = Nokogiri::HTML(open(center.int_url))
-    raise InvalidPage "Invalid page!!!" if doc == nil
-
-    details = {}
-
-    status = scrape_status(doc); details[:open_status] = status unless status.nil?
-    slogan = scrape_slogan(doc); details[:slogan] = slogan unless slogan.nil?
-    hours = scrape_hours(doc); details[:working_hours] = hours unless hours.nil?
-    description = scrape_description(doc); details[:description] = description unless description.nil?
-    services = scrape_services(doc); details[:services] = services unless services.nil?
-    brands = scrape_brands(doc); details[:brands] = brands unless brands.nil?
-    payment = scrape_payment(doc); details[:payment] = payment unless payment.nil?
-
-    center.details_from_hash(details)
-  end
-
-  #---------------------------------------------------------------------------------------
-  # Scraping helpers
-
-private
-
-  def scrape_internal_url(center)
-    url = center.css(".n a").attr("href").value
-    (!url.empty? && !external_link?(url)) ? AutoServiceCLI::URL_BASE + url : nil
-  end
-
-  def scrape_external_url(center)
-    url = center.css(".links a.track-visit-website")
-    url.empty? ? nil : url.attr("href").value
-  end
-
-  def scrape_rating(center)
-    rate = center.css(".info-primary .result-rating")
-    unless rate.empty?
-      attributes = rate.attr("class").value.split(" "); attributes.slice(1, attributes.size-1)
-    else
-      nil
-    end
-  end
-
-  def scrape_address(center)
-    address_full = center.css(".info-primary .adr span").collect {|el| el.text.split(",").first}
-    address_full.flatten.empty? ? nil : address_full.join(", ")
-  end
-
-  def scrape_phone_number(center)
-    phone_number = center.css(".info-primary .phone.primary")
-    phone_number.empty? ? nil : phone_number.text
-  end
-
-  def scrape_category(center)
-    category = center.css(".info-secondary .categories a")
-    category.empty? ? nil : category.text
-  end
-end
-
-def scrape_status(doc)
-  open_status = doc.css(".business-card-wrapper .status-text")
-  open_status.empty? ? nil : open_status.text
-end
-
-def scrape_slogan(doc)
-  slogan = doc.css("#business-details .slogan")
-  slogan.empty? ? nil : slogan.text
-end
-
-def scrape_hours(doc)
-  working_hours = ""
-  doc.css("#business-details .open-hours time").each do |time|
-    working_hours << time.css(".day-label").text
-    working_hours << ", " + time.css(".day-hours").text + "\n"
-  end
-  working_hours.empty? ? nil : working_hours
-end
-
-def scrape_description(doc)
-  description = doc.css("#business-details p.description").last
-  description = doc.css("#business-details dd.description").last if description.nil?
-  (!description.nil? && description.css("a").empty?) ? description.text : nil #exclude case of facebook and twitter links, without decription
-end
-
-def scrape_services(doc)
-  for_services = doc.css("#business-details dl")[1]
-  unless for_services.nil?
-    if for_services.css("dt").first.text == "Services/Products:"
-      services = for_services.css("dd").first.text
-      return services.empty? ? nil : services
-    end
-  end
-  nil
-end
-
-def scrape_brands(doc)
-  brands = doc.css("#business-details .brands")
-  brands.empty? ? nil : brands.text
-end
-
-def scrape_payment(doc)
-  payment = doc.css("#business-details .payment")
-  payment.empty? ? nil : payment.text
-end
+class AutoServiceCLI::Scraper
+  attr_reader :zip, :sort_type
+  attr_writer :zip, :sort_type
+
+  SORT_TYPES = {
+    DEFAULT: "default",
+    DISTANCE: "distance",
+    AVERAGE_RATING: "average_rating",
+    NAME: "name"
+  }
+
+  DEFAULT_SORT_TYPE = SORT_TYPES[:default]
+
+  # Constructors and class methods.
+
+  def initialize(zip = nil, sort_type = nil)
+    self.zip = zip if zip
+    self.sort_type = sort_type ? sort_type : self.class::SORT_TYPES[:DEFAULT]
+  end
+
+  def zip=(zip)
+    raise InvalidURLData unless self.class.valid_zip?(zip)
+    @zip = zip
+  end
+  def self.valid_zip?(zip)
+    zip.to_i > 0
+  end
+
+  def sort_type=(type)
+    raise InvalidURLData unless self.class.valid_sort_type?(type)
+    @sort_type = type
+  end
+  def self.valid_sort_type?(type)
+    SORT_TYPES.any? { |key, value| value == type }
+  end
+
+  # def valid_url_data?
+  #   self.class.valid_zip?(self.zip) && self.class.valid_sort_type?(self.sort_type)
+  # end
+
+  #------------------------------------------------------------------------------------------------
+  # Scrape from the main page.
+  # Index action.
+
+  def scrape_centers
+    doc = Nokogiri::HTML(open(get_url))
+    raise InvalidPage "Invalid page!!!" if doc.nil?
+    centers = doc.css(".organic .result .info")
+
+    # form centers hash.
+    centers.each do |center|
+
+      main_details = {}
+      obj_center = AutoServiceCLI::ServiceCenter.create(scrape_name(center))
+
+      url = scrape_internal_url(center); main_details[:int_url] = url unless url.nil?
+      url = scrape_external_url(center); main_details[:ext_url] = url unless url.nil?
+      rating = scrape_rating(center); main_details[:rating] = rating unless rating.nil?
+      address = scrape_address(center); main_details[:address] = address unless address.nil?
+      phone = scrape_phone_number(center); main_details[:phone_number] = phone unless phone.nil?
+      category = scrape_category(center); main_details[:main_category] = category unless category.nil?
+
+      obj_center.details_from_hash(main_details) # setting all details to center.
+    end
+  end
+
+  #------------------------------------------------------------------------------------------------
+  # Scrape center details from the internal url (if available)
+  # Show action.
+
+  def scrape_center_details(center)
+    raise InvalidPage, "Invalid page!!!" if !center.is_a? AutoServiceCLI::ServiceCenter || center.int_url.nil?
+    doc = Nokogiri::HTML(open(center.int_url))
+    raise InvalidPage "Invalid page!!!" if doc.nil?
+
+    details = {}
+
+    status = scrape_status(doc); details[:open_status] = status unless status.nil?
+    slogan = scrape_slogan(doc); details[:slogan] = slogan unless slogan.nil?
+    hours = scrape_hours(doc); details[:working_hours] = hours unless hours.nil?
+    description = scrape_description(doc); details[:description] = description unless description.nil?
+    services = scrape_services(doc); details[:services] = services unless services.nil?
+    brands = scrape_brands(doc); details[:brands] = brands unless brands.nil?
+    payment = scrape_payment(doc); details[:payment] = payment unless payment.nil?
+
+    center.details_from_hash(details)
+  end
+
+  #---------------------------------------------------------------------------------------
+  # Scraping helpers
+
+  private
+
+  def get_url
+    raise InvalidURLData if !self.zip || !self.sort_type
+    AutoServiceCLI::URL_TEMPLATE + "#{self.zip}&s=#{self.sort_type}"
+  end
+
+  def external_link?(url)
+    url.include?("www.") || url.include?("http://") || url.include?("https://")
+  end
+
+  def scrape_name(center)
+    center.css(".n a").text
+  end
+
+  def scrape_internal_url(center)
+    url = center.css(".n a").attr("href").value
+    (!url.empty? && !external_link?(url)) ? AutoServiceCLI::URL_BASE + url : nil
+  end
+
+  def scrape_external_url(center)
+    url = center.css(".links a.track-visit-website")
+    url.empty? ? nil : url.attr("href").value
+  end
+
+  def scrape_rating(center)
+    rate = center.css(".info-primary .result-rating")
+    unless rate.empty?
+      attributes = rate.attr("class").value.split(" ");
+      attributes.slice(1, attributes.size - 1)
+    else
+      nil
+    end
+  end
+
+  def scrape_address(center)
+    address_full = center.css(".info-primary .adr span").collect {|el| el.text.split(",").first}
+    address_full.flatten.empty? ? nil : address_full.join(", ")
+  end
+
+  def scrape_phone_number(center)
+    phone_number = center.css(".info-primary .phone.primary")
+    phone_number.empty? ? nil : phone_number.text
+  end
+
+  def scrape_category(center)
+    category = center.css(".info-secondary .categories a")
+    category.empty? ? nil : category.text
+  end
+
+  def scrape_status(doc)
+    open_status = doc.css(".business-card-wrapper .status-text")
+    open_status.empty? ? nil : open_status.text
+  end
+
+  def scrape_slogan(doc)
+    slogan = doc.css("#business-info .slogan")
+    puts slogan
+    slogan.empty? ? nil : slogan.text
+  end
+
+  def scrape_hours(doc)
+    working_hours = ""
+    doc.css("#business-info .open-hours time").each do |time|
+      working_hours << time.css(".day-label").text
+      working_hours << ", " + time.css(".day-hours").text + "\n"
+    end
+    working_hours.empty? ? nil : working_hours
+  end
+
+
+  def scrape_description(doc)
+    description = doc.css("#business-info .general-info p").last
+    # description = doc.css("#business-info dd.description").last if description.nil?
+    (!description.nil? && description.css("a").empty?) ? description.text : nil #exclude case of facebook and twitter links, without decription
+  end
+
+  def scrape_services(doc)
+    for_services = doc.css("#business-info dl")[1]
+    unless for_services.nil?
+      if for_services.css("dt").first.text == "Services/Products:"
+        services = for_services.css("dd").first.text
+        return services.empty? ? nil : services
+      end
+    end
+    nil
+  end
+
+  def scrape_brands(doc)
+    brands = doc.css("#business-info .brands")
+    brands.empty? ? nil : brands.text
+  end
+
+  def scrape_payment(doc)
+    payment = doc.css("#business-info .payment")
+    payment.empty? ? nil : payment.text
+  end
+
+end

data/lib/auto_service_cli/service_center.rb

@@ -1,68 +1,77 @@
-class AutoServiceCLI::ServiceCenter
-  attr_accessor :name,
-                :int_url,
-                :ext_url,
-                :rating,
-                :main_category,
-                :address,
-                :phone_number,
-                :open_status,
-                :slogan,
-                :working_hours,
-                :description,
-                :services,
-                :brands,
-                :payment
-  @@all = []
-
-  # Constructors
-
-  def initialize(name)
-    @name = name
-  end
-
-  def self.create(name)
-    self.new(name).tap { |center| center.save }
-  end
-
-  # Instance methods.
-
-  def save
-    @@all << self
-  end
-
-  def details_from_hash(details)
-    details.each do |detail, value|
-      self.send("#{detail}=", value)
-    end
-    format_rating if details.include?(:rating)
-  end
-
-  def format_rating
-    case self.rating.first
-    when "one"
-      new_rating = "1"
-    when "two"
-      new_rating = "2"
-    when "three"
-      new_rating = "3"
-    when "four"
-      new_rating = "4"
-    when "five"
-      new_rating = "5"
-    end
-
-    new_rating << ".5" if self.rating.size == 2 && self.rating.last == "half"
-    self.rating = new_rating + " star(s)"
-  end
-
-  # Class mehods
-
-  def self.reset_all!
-    @@all.clear
-  end
-
-  def self.all
-    @@all.dup.freeze
-  end
-end
+class AutoServiceCLI::ServiceCenter
+  attr_accessor :name,
+                :int_url,
+                :ext_url,
+                :rating,
+                :main_category,
+                :address,
+                :phone_number,
+                :open_status,
+                :slogan,
+                :working_hours,
+                :description,
+                :services,
+                :brands,
+                :payment
+  @@all = []
+
+  # Constructors
+
+  def initialize(name)
+    @name = name
+  end
+
+  def self.create(name)
+    self.new(name).tap { |center| center.save }
+  end
+
+  # Instance methods.
+
+  def save
+    @@all << self
+  end
+
+  def details_from_hash(details)
+    # modify hash's rating
+    if details.include?(:rating)
+      details[:rating] = format_rating(details[:rating])
+    end
+
+    details.each do |detail, value|
+      self.send("#{detail}=", value)
+    end
+  end
+
+  # ex ["two", "half"] or ["two"]
+  def format_rating(rating)
+    case rating[0]
+      when "one"
+        new_rating = "1"
+      when "two"
+        new_rating = "2"
+      when "three"
+        new_rating = "3"
+      when "four"
+        new_rating = "4"
+      when "five"
+        new_rating = "5"
+      else
+        # assume it's already formatted
+        return rating
+    end
+
+    # TODO
+    new_rating << ".5" if rating.size == 2 && rating.last == "half"
+    new_rating + " star(s)"
+  end
+
+  # Class mehods
+
+  def self.reset_all!
+    @@all.clear
+  end
+
+  def self.all
+    @@all.dup.freeze
+  end
+end

data/lib/auto_service_cli/version.rb

@@ -1,3 +1,3 @@
 module AutoServiceCLI
-  VERSION = "0.1.0"
+  VERSION = "0.1.2"
 end

data/spec.md

@@ -1,12 +1,12 @@
-# Specifications for the CLI Assessment
-
-Specs:
-- [x] Have a CLI for interfacing with the application  
-The CLI class was implemented to show users the interface. It has a #call method which is a starting point of CLI. A user should enter a zip code to get info about service centers near him. CLI class has menu method which gives to user an opportunity to make choices: list centers, choose sorting type, show details about center, reload centers, and exit.
-
-- [x] Pull data from an external source  
-yellowpages.com is used to scrape the data about 30 service centers near entered zip code. Gem 'nokogiri' is used to scrape the data. The main page about service centers is used to scrape main data. When a user chooses to see the data about specific center, program scrapes data from the additional url (if available) obtained from the main page.
-
-- [x] Implement both list and detail views  
-In CLI class #list_centers method was implemented. It shows 30 centers near entered zip code.
-In CLI class #get_details method was implemented. It invokes the method to scrape the data from the specific url and lists the results.
+# Specifications for the CLI Assessment
+
+Specs:
+- [x] Have a CLI for interfacing with the application  
+The CLI class was implemented to show users the interface. It has a #call method which is a starting point of CLI. A user should enter a zip code to get info about service centers near him. CLI class has menu method which gives to user an opportunity to make choices: list centers, choose sorting type, show details about center, reload centers, and exit.
+
+- [x] Pull data from an external source  
+yellowpages.com is used to scrape the data about 30 service centers near entered zip code. Gem 'nokogiri' is used to scrape the data. The main page about service centers is used to scrape main data. When a user chooses to see the data about specific center, program scrapes data from the additional url (if available) obtained from the main page.
+
+- [x] Implement both list and detail views  
+In CLI class #list_centers method was implemented. It shows 30 centers near entered zip code.
+In CLI class #get_details method was implemented. It invokes the method to scrape the data from the specific url and lists the results.

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: auto_service_cli
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.1.2
 platform: ruby
 authors:
 - Aleksandr Rogachev
@@ -16,14 +16,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.7'
+        version: '1.8'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.7'
+        version: '1.8'
 - !ruby/object:Gem::Dependency
   name: colorize
   requirement: !ruby/object:Gem::Requirement
@@ -44,28 +44,28 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.13'
+        version: '1.16'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.13'
+        version: '1.16'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '10.0'
+        version: '10.5'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '10.0'
+        version: '10.5'
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
@@ -86,16 +86,16 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.10'
+        version: '0.11'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.10'
-description: This CLI allows to find service centers near desired zip code (with sorting)
-  and see details about each center.
+        version: '0.11'
+description: This CLI allows you to find auto service centers near desired zip code,
+  sort them, and fetch details about each of them
 email:
 - aleksandr.rogachev1994@gmail.com
 executables:
@@ -110,6 +110,7 @@ files:
 - LICENSE.txt
 - README.md
 - Rakefile
+- auto_service_cli-0.1.1.gem
 - auto_service_cli.gemspec
 - bin/auto_service
 - bin/console
@@ -141,8 +142,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.6.8
+rubygems_version: 2.4.8
 signing_key: 
 specification_version: 4
-summary: CLI for searching of auto service centers near your location.
+summary: CLI for searching auto service centers near your location
 test_files: []