zillow_demographics 0.0.1 → 0.0.2

data/lib/zillow_demographics/people.rb

@@ -35,7 +35,7 @@ module ZillowApi
 
       def mode_of_transit(location)
         key = parsed_response(location).search("uniqueness").search("category").last.attributes['type'].value.downcase
-        value = parsed_response(location).search("uniqueness").search("category").last.text
+        value = parsed_response(location).search("uniqueness").search("category").last.children.map {|s| s.text}
         { key => value }
       end
 

data/lib/zillow_demographics/version.rb

@@ -1,3 +1,3 @@
 module ZillowDemographics
-  VERSION = "0.0.1"
+  VERSION = "0.0.2"
 end

data/spec/client_spec.rb

@@ -0,0 +1,29 @@
+require 'spec_helper'
+
+describe ZillowApi::Client do
+  let(:client) { ZillowApi::Client.new }
+  VALID_CITY = ({city: 'chicago', state: 'il'})
+  INVALID_CITY = ({city: 'denver'})
+  REQUEST = {
+             '0' => "Request successfully processed",
+             '1' => "Error: missing parameters"
+            }
+
+  describe "#get_city_data" do
+    context "when given a hash including a valid city and state" do
+      it "returns an xml package of data for the specified city" do
+        response = Nokogiri::HTML(client.get_city_data(VALID_CITY))
+        message = response.search("message").children.first.text
+        message.should == REQUEST['0']
+      end
+    end
+
+    context "when given an invalid hash" do
+      it "returns an error message" do
+        response = Nokogiri::HTML(client.get_city_data(INVALID_CITY))
+        message = response.search("message").children.first.text
+        message.should == REQUEST['1']
+      end
+    end
+  end
+end

data/spec/people_spec.rb

@@ -0,0 +1,32 @@
+require 'spec_helper'
+
+class MockClient
+  def get_city_data
+
+  end
+end
+
+describe ZillowApi::People do
+  describe ".demo_attributes" do
+  end
+
+  describe ".lives_here_attributes" do
+  end
+
+  describe '.mode_of_transit' do
+  end
+
+  describe '.home_value' do
+  end
+
+  describe '.find_by_city' do
+  end
+
+  describe '.parse_all' do
+  end
+
+  describe '.parsed_response' do
+  end
+
+
+end

data/spec/spec_helper.rb

@@ -0,0 +1,13 @@
+require 'rspec'
+require 'faraday'
+require 'nokogiri'
+require './lib/zillow_demographics'
+require 'client_spec'
+# require 'people_spec'
+# require 'national_spec'
+
+RSpec.configure do |config|
+  config.treat_symbols_as_metadata_keys_with_true_values = true
+  config.run_all_when_everything_filtered = true
+  config.filter_run :focus
+end

data/zillow_demographics.gemspec

@@ -4,8 +4,8 @@ require File.expand_path('../lib/zillow_demographics/version', __FILE__)
 Gem::Specification.new do |gem|
   gem.authors       = ["michael verdi"]
   gem.email         = ["michael.v.verdi@gmail.com"]
-  gem.description   = %q{wrapper for the zillow zillow_demographics api}
-  gem.summary       = %q{wrapper for the zillow zillow_demographics api}
+  gem.description   = %q{wrapper for the zillow_demographics api}
+  gem.summary       = %q{wrapper for the zillow_demographics api}
   gem.homepage      = ""
 
   gem.files         = `git ls-files`.split($\)
@@ -17,4 +17,5 @@ Gem::Specification.new do |gem|
 
   gem.add_runtime_dependency('faraday')
   gem.add_runtime_dependency('nokogiri')
+  gem.add_development_dependency('rspec')
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: zillow_demographics
 version: !ruby/object:Gem::Version
-  version: 0.0.1
+  version: 0.0.2
   prerelease: 
 platform: ruby
 authors:
@@ -9,11 +9,11 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-06-22 00:00:00.000000000 Z
+date: 2012-06-26 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: faraday
-  requirement: &70308717122000 !ruby/object:Gem::Requirement
+  requirement: &70342049238340 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -21,10 +21,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *70308717122000
+  version_requirements: *70342049238340
 - !ruby/object:Gem::Dependency
   name: nokogiri
-  requirement: &70308717121460 !ruby/object:Gem::Requirement
+  requirement: &70342049237720 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -32,8 +32,19 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *70308717121460
-description: wrapper for the zillow zillow_demographics api
+  version_requirements: *70342049237720
+- !ruby/object:Gem::Dependency
+  name: rspec
+  requirement: &70342049237020 !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: *70342049237020
+description: wrapper for the zillow_demographics api
 email:
 - michael.v.verdi@gmail.com
 executables: []
@@ -45,14 +56,14 @@ files:
 - LICENSE
 - README.md
 - Rakefile
-- client.rb
 - lib/zillow_demographics.rb
 - lib/zillow_demographics/client.rb
 - lib/zillow_demographics/national.rb
 - lib/zillow_demographics/people.rb
 - lib/zillow_demographics/version.rb
-- national.rb
-- people.rb
+- spec/client_spec.rb
+- spec/people_spec.rb
+- spec/spec_helper.rb
 - version.rb
 - zillow_demographics.gemspec
 homepage: ''
@@ -78,5 +89,8 @@ rubyforge_project:
 rubygems_version: 1.8.17
 signing_key: 
 specification_version: 3
-summary: wrapper for the zillow zillow_demographics api
-test_files: []
+summary: wrapper for the zillow_demographics api
+test_files:
+- spec/client_spec.rb
+- spec/people_spec.rb
+- spec/spec_helper.rb

data/client.rb

@@ -1,23 +0,0 @@
-require 'faraday'
-require 'nokogiri'
-
-module ZillowApi
-  class Client
-    BASE_URL = 'http://www.zillow.com'
-
-    def initialize
-      @connection = Faraday.new(url: BASE_URL)
-    end
-
-    def get_city_data(location)
-      response = @connection.get do |req|
-        req.url "/webservice/GetDemographics.htm"
-        req.headers['Accepts'] = 'application/xml'
-        req.params['zws-id']   = 'X1-ZWz1bn1je20hzf_1wtus'
-        req.params['state']    = location[:state]
-        req.params['city']     = location[:city]
-      end
-      response.body
-    end
-  end
-end

data/national.rb

@@ -1,48 +0,0 @@
-require 'nokogiri'
-
-module ZillowApi
-  class National
-    attr_accessor :single_males, :single_females, :median_age, :average_commute_time, :home_value
-
-    def initialize(attributes)
-      self.average_commute_time = attributes['average commute time (minutes)']
-      self.single_females       = attributes['single females']
-      self.single_males         = attributes['single males']
-      self.median_age           = attributes['median age']
-      self.home_value           = attributes['zillow home value index']
-    end
-
-    class << self
-
-      def client
-        ZillowApi::Client.new
-      end
-
-      def demo_attributes(location)
-        keys = parsed_response(location).search("page")[-1].search("table")[0].search("attribute").map {|key| key.child.text.downcase }
-        values = parsed_response(location).search("page")[-1].search("table")[0].search("attribute").search("nation").map { |values| values.child.text }
-        Hash[keys.zip(values)]
-      end
-
-      def home_value(location)
-        key = parsed_response(location).search("page").first.search("data").first.search("attribute").first.child.text.downcase
-        value = parsed_response(location).search("page").first.search("data").first.search("nation").first.text
-        { key => value }
-      end
-
-      def find_by_city(location)
-        attributes = demo_attributes(location).merge(home_value(location))
-        ZillowApi::National.new(attributes)
-      end
-
-      def parsed_response(location)
-        parse_all(client.get_city_data(location))
-      end
-
-      def parse_all(xml_package)
-        Nokogiri::HTML(xml_package)
-      end
-
-    end
-  end
-end

data/people.rb

@@ -1,63 +0,0 @@
-require 'nokogiri'
-
-module ZillowApi
-  class People
-    attr_accessor :single_males, :single_females, :median_age, :average_commute_time
-    attr_accessor :transportation, :home_value, :who_live_here
-
-    def initialize(attributes)
-      self.average_commute_time = attributes['average commute time (minutes)']
-      self.single_females       = attributes['single females']
-      self.single_males         = attributes['single males']
-      self.median_age           = attributes['median age']
-      self.who_live_here        = attributes['lives here']
-      self.transportation       = attributes['transportation']
-      self.home_value           = attributes['zillow home value index']
-    end
-
-    class << self
-
-      def client
-        ZillowApi::Client.new
-      end
-
-      def demo_attributes(location)
-        keys = parsed_response(location).search("page")[-1].search("table")[0].search("attribute").map {|key| key.child.text.downcase }
-        values = parsed_response(location).search("page")[-1].search("table")[0].search("attribute").search("city").map { |values| values.child.text }
-        Hash[keys.zip(values)]
-      end
-
-      def lives_here_attributes(location)
-        keys = parsed_response(location).search("page")[-1].search("liveshere").search("title").map { |value| value.text.downcase }
-        values = parsed_response(location).search("page")[-1].search("liveshere").search("name").map { |key| key.text }
-        { 'lives here' => Hash[keys.zip(values)] }
-      end
-
-      def mode_of_transit(location)
-        key = parsed_response(location).search("uniqueness").search("category").last.attributes['type'].value.downcase
-        value = parsed_response(location).search("uniqueness").search("category").last.text
-        { key => value }
-      end
-
-      def home_value(location)
-        key = parsed_response(location).search("page").first.search("data").first.search("attribute").first.child.text.downcase
-        value = parsed_response(location).search("page").first.search("data").first.search("city").first.text
-        { key => value }
-      end
-
-      def find_by_city(location)
-        attributes = lives_here_attributes(location).merge(demo_attributes(location)).merge(home_value(location)).merge(mode_of_transit(location))
-        ZillowApi::People.new(attributes)
-      end
-
-      def parsed_response(location)
-        parse_all(client.get_city_data(location))
-      end
-
-      def parse_all(xml_package)
-        Nokogiri::HTML(xml_package)
-      end
-
-    end
-  end
-end