remote_table 0.2.16 → 0.2.17

data/VERSION

@@ -1 +1 @@
-0.2.16
+0.2.17

data/lib/remote_table/file.rb

@@ -59,13 +59,13 @@ class RemoteTable
     
     def skip_rows!
       return unless skip
-      `cat #{path} | tail -n +#{skip + 1} > #{path}.tmp`
+      RemoteTable.backtick_with_reporting "cat #{path} | tail -n +#{skip + 1} > #{path}.tmp"
       FileUtils.mv "#{path}.tmp", path
     end
     
     def convert_file_to_utf8!
       return if encoding == 'UTF-8' or encoding == 'UTF8'
-      `iconv -c -f #{encoding} -t UTF-8 #{path} > #{path}.tmp`
+      RemoteTable.backtick_with_reporting "iconv -c -f #{encoding} -t UTF-8 #{path} > #{path}.tmp"
       FileUtils.mv "#{path}.tmp", path
     end
     
@@ -75,13 +75,13 @@ class RemoteTable
     
     def cut_columns!
       return unless cut
-      `cat #{path} | cut -c #{cut} > #{path}.tmp`
+      RemoteTable.backtick_with_reporting "cat #{path} | cut -c #{cut} > #{path}.tmp"
       FileUtils.mv "#{path}.tmp", path
     end
     
     def crop_rows!
       return unless crop
-      `cat #{path} | tail -n +#{crop.first} | head -n #{crop.last - crop.first + 1} > #{path}.tmp`
+      RemoteTable.backtick_with_reporting "cat #{path} | tail -n +#{crop.first} | head -n #{crop.last - crop.first + 1} > #{path}.tmp"
       FileUtils.mv "#{path}.tmp", path
     end
     

data/lib/remote_table/package.rb

@@ -47,7 +47,8 @@ class RemoteTable
     end
     
     def move_and_process(path, extname, cmd, args)
-      `mv #{path} #{path}.#{extname} && #{cmd} #{path}.#{extname} #{args}`
+      FileUtils.mv path, "#{path}.#{extname}"
+      RemoteTable.backtick_with_reporting "#{cmd} #{path}.#{extname} #{args}"
     end
 
     # ex. A: 2007-01.csv.gz  (compression not capable of storing multiple files)

data/lib/remote_table/request.rb

@@ -20,16 +20,14 @@ class RemoteTable
       if parsed_url.scheme == 'file'
         parsed_url.path
       else
-        cmd = %{
-          curl \
-          --silent \
-          --header "Expect: " \
-          --location \
-          #{"--data \"#{form_data}\"" if form_data.present?} \
-          "#{parsed_url}" \
+        RemoteTable.backtick_with_reporting %{
+          curl
+          --header "Expect: "
+          --location
+          #{"--data \"#{form_data}\"" if form_data.present?}
+          "#{parsed_url}"
           --output "#{path}"
         }
-        `#{cmd}`
         path
       end
     end

data/lib/remote_table.rb

@@ -56,6 +56,26 @@ class RemoteTable
   def <=>(other)
     raise "Not implemented"
   end
+
+  protected
+  
+  # TODO this should probably live somewhere else
+  def self.backtick_with_reporting(cmd)
+    cmd = cmd.gsub /\s+/m, ' '
+    cmd = cmd + ' 2>&1' unless cmd.include? '2>'
+    output = `#{cmd}`
+    unless $?.success?
+      raise %{
+From the remote_table gem...
+
+Command failed:
+#{cmd}
+
+Output:
+#{output}
+}
+    end
+  end
   
   private
   

data/remote_table.gemspec

@@ -5,11 +5,11 @@
 
 Gem::Specification.new do |s|
   s.name = %q{remote_table}
-  s.version = "0.2.16"
+  s.version = "0.2.17"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["Seamus Abshere", "Andy Rossmeissl"]
-  s.date = %q{2010-04-27}
+  s.date = %q{2010-04-28}
   s.description = %q{Remotely open and parse Excel XLS, ODS, CSV and fixed-width tables.}
   s.email = %q{seamus@abshere.net}
   s.extra_rdoc_files = [

data/test/remote_table_test.rb

@@ -50,21 +50,7 @@ class RemoteTableTest < Test::Unit::TestCase
     ]
   end
   
-  if ENV['NEW'] == 'true'
-  end
-  
-  if ENV['OLD'] == 'true'
-    should "read an HTML table made with frontpage" do
-      t = RemoteTable.new :url => "http://www.faa.gov/air_traffic/publications/atpubs/CNT/5-2-E.htm",
-                          :encoding => 'US-ASCII',
-                          :row_xpath => '//table/tr[2]/td/table/tr',
-                          :column_xpath => 'td'
-      assert_equal 'E110', t.rows.first['Designator']
-      assert_equal 'EMBRAER', t.rows.first['Manufacturer']
-      assert_equal 'EZKC', t.rows.last['Designator']
-      assert_equal 'EZ King Cobra', t.rows.last['Model']
-    end
-    
+  if ENV['ALL'] == 'true' or ENV['SLOW'] == 'true'
     should "open an XLS inside a zip file" do
       t = RemoteTable.new(:url => 'http://www.fueleconomy.gov/FEG/epadata/02data.zip', :filename => 'guide_jan28.xls')
       assert_equal 'ACURA',      t.rows.first['Manufacturer']
@@ -79,6 +65,58 @@ class RemoteTableTest < Test::Unit::TestCase
       assert_equal nil,          t.rows.first[:Manufacturer]
     end
   
+    should "open a CSV inside a zip file" do
+      t = RemoteTable.new(:url => 'http://www.fueleconomy.gov/FEG/epadata/98guide6.zip', :filename => '98guide6.csv')
+      assert_equal 'ACURA',             t.rows.first['Manufacturer']
+      assert_equal 'NSX',               t.rows.first['carline name']
+      assert_equal 'TOYOTA',            t.rows.last['Manufacturer']
+      assert_equal 'RAV4 SOFT TOP 4WD', t.rows.last['carline name']
+    end
+
+    should "open a fixed-width file with an inline schema inside a zip file" do
+      t = RemoteTable.new(:url => 'http://www.fueleconomy.gov/FEG/epadata/00data.zip',
+                          :filename => 'Gd6-dsc.txt',
+                          :format => :fixed_width,
+                          :crop => 21..26, # inclusive
+                          :cut => '2-',
+                          :select => lambda { |row| /\A[A-Z]/.match row['code'] },
+                          :schema => [[ 'code',   2, { :type => :string }  ],
+                                      [ 'spacer', 2 ],
+                                      [ 'name',   52, { :type => :string } ]])
+      assert_equal 'regular grade gasoline (octane number of 87)', t.rows.first['name']
+      assert_equal 'R',                                            t.rows.first['code']
+      assert_equal 'electricity',                                  t.rows.last['name']
+      assert_equal 'El',                                           t.rows.last['code']
+    end
+    
+    should "send form data, follow redirects and use a filename glob" do
+      url = 'http://www.transtats.bts.gov/DownLoad_Table.asp?Table_ID=293&Has_Group=3&Is_Zipped=0'
+      form_data = 'UserTableName=T_100_Segment__All_Carriers&DBShortName=Air_Carriers&RawDataTable=T_T100_SEGMENT_ALL_CARRIER&sqlstr=+SELECT+DEPARTURES_SCHEDULED%2CDEPARTURES_PERFORMED%2CPAYLOAD%2CSEATS%2CPASSENGERS%2CFREIGHT%2CMAIL%2CDISTANCE%2CRAMP_TO_RAMP%2CAIR_TIME%2CUNIQUE_CARRIER%2CAIRLINE_ID%2CUNIQUE_CARRIER_NAME%2CUNIQUE_CARRIER_ENTITY%2CREGION%2CCARRIER%2CCARRIER_NAME%2CCARRIER_GROUP%2CCARRIER_GROUP_NEW%2CORIGIN%2CORIGIN_CITY_NAME%2CORIGIN_CITY_NUM%2CORIGIN_STATE_ABR%2CORIGIN_STATE_FIPS%2CORIGIN_STATE_NM%2CORIGIN_COUNTRY%2CORIGIN_COUNTRY_NAME%2CORIGIN_WAC%2CDEST%2CDEST_CITY_NAME%2CDEST_CITY_NUM%2CDEST_STATE_ABR%2CDEST_STATE_FIPS%2CDEST_STATE_NM%2CDEST_COUNTRY%2CDEST_COUNTRY_NAME%2CDEST_WAC%2CAIRCRAFT_GROUP%2CAIRCRAFT_TYPE%2CAIRCRAFT_CONFIG%2CYEAR%2CQUARTER%2CMONTH%2CDISTANCE_GROUP%2CCLASS%2CDATA_SOURCE+FROM++T_T100_SEGMENT_ALL_CARRIER+WHERE+Month+%3D1+AND+YEAR%3D2008&varlist=DEPARTURES_SCHEDULED%2CDEPARTURES_PERFORMED%2CPAYLOAD%2CSEATS%2CPASSENGERS%2CFREIGHT%2CMAIL%2CDISTANCE%2CRAMP_TO_RAMP%2CAIR_TIME%2CUNIQUE_CARRIER%2CAIRLINE_ID%2CUNIQUE_CARRIER_NAME%2CUNIQUE_CARRIER_ENTITY%2CREGION%2CCARRIER%2CCARRIER_NAME%2CCARRIER_GROUP%2CCARRIER_GROUP_NEW%2CORIGIN%2CORIGIN_CITY_NAME%2CORIGIN_CITY_NUM%2CORIGIN_STATE_ABR%2CORIGIN_STATE_FIPS%2CORIGIN_STATE_NM%2CORIGIN_COUNTRY%2CORIGIN_COUNTRY_NAME%2CORIGIN_WAC%2CDEST%2CDEST_CITY_NAME%2CDEST_CITY_NUM%2CDEST_STATE_ABR%2CDEST_STATE_FIPS%2CDEST_STATE_NM%2CDEST_COUNTRY%2CDEST_COUNTRY_NAME%2CDEST_WAC%2CAIRCRAFT_GROUP%2CAIRCRAFT_TYPE%2CAIRCRAFT_CONFIG%2CYEAR%2CQUARTER%2CMONTH%2CDISTANCE_GROUP%2CCLASS%2CDATA_SOURCE&grouplist=&suml=&sumRegion=&filter1=title%3D&filter2=title%3D&geo=All%A0&time=January&timename=Month&GEOGRAPHY=All&XYEAR=2008&FREQUENCY=1&AllVars=All&VarName=DEPARTURES_SCHEDULED&VarDesc=DepScheduled&VarType=Num&VarName=DEPARTURES_PERFORMED&VarDesc=DepPerformed&VarType=Num&VarName=PAYLOAD&VarDesc=Payload&VarType=Num&VarName=SEATS&VarDesc=Seats&VarType=Num&VarName=PASSENGERS&VarDesc=Passengers&VarType=Num&VarName=FREIGHT&VarDesc=Freight&VarType=Num&VarName=MAIL&VarDesc=Mail&VarType=Num&VarName=DISTANCE&VarDesc=Distance&VarType=Num&VarName=RAMP_TO_RAMP&VarDesc=RampToRamp&VarType=Num&VarName=AIR_TIME&VarDesc=AirTime&VarType=Num&VarName=UNIQUE_CARRIER&VarDesc=UniqueCarrier&VarType=Char&VarName=AIRLINE_ID&VarDesc=AirlineID&VarType=Num&VarName=UNIQUE_CARRIER_NAME&VarDesc=UniqueCarrierName&VarType=Char&VarName=UNIQUE_CARRIER_ENTITY&VarDesc=UniqCarrierEntity&VarType=Char&VarName=REGION&VarDesc=CarrierRegion&VarType=Char&VarName=CARRIER&VarDesc=Carrier&VarType=Char&VarName=CARRIER_NAME&VarDesc=CarrierName&VarType=Char&VarName=CARRIER_GROUP&VarDesc=CarrierGroup&VarType=Num&VarName=CARRIER_GROUP_NEW&VarDesc=CarrierGroupNew&VarType=Num&VarName=ORIGIN&VarDesc=Origin&VarType=Char&VarName=ORIGIN_CITY_NAME&VarDesc=OriginCityName&VarType=Char&VarName=ORIGIN_CITY_NUM&VarDesc=OriginCityNum&VarType=Num&VarName=ORIGIN_STATE_ABR&VarDesc=OriginState&VarType=Char&VarName=ORIGIN_STATE_FIPS&VarDesc=OriginStateFips&VarType=Char&VarName=ORIGIN_STATE_NM&VarDesc=OriginStateName&VarType=Char&VarName=ORIGIN_COUNTRY&VarDesc=OriginCountry&VarType=Char&VarName=ORIGIN_COUNTRY_NAME&VarDesc=OriginCountryName&VarType=Char&VarName=ORIGIN_WAC&VarDesc=OriginWac&VarType=Num&VarName=DEST&VarDesc=Dest&VarType=Char&VarName=DEST_CITY_NAME&VarDesc=DestCityName&VarType=Char&VarName=DEST_CITY_NUM&VarDesc=DestCityNum&VarType=Num&VarName=DEST_STATE_ABR&VarDesc=DestState&VarType=Char&VarName=DEST_STATE_FIPS&VarDesc=DestStateFips&VarType=Char&VarName=DEST_STATE_NM&VarDesc=DestStateName&VarType=Char&VarName=DEST_COUNTRY&VarDesc=DestCountry&VarType=Char&VarName=DEST_COUNTRY_NAME&VarDesc=DestCountryName&VarType=Char&VarName=DEST_WAC&VarDesc=DestWac&VarType=Num&VarName=AIRCRAFT_GROUP&VarDesc=AircraftGroup&VarType=Num&VarName=AIRCRAFT_TYPE&VarDesc=AircraftType&VarType=Char&VarName=AIRCRAFT_CONFIG&VarDesc=AircraftConfig&VarType=Num&VarName=YEAR&VarDesc=Year&VarType=Num&VarName=QUARTER&VarDesc=Quarter&VarType=Num&VarName=MONTH&VarDesc=Month&VarType=Num&VarName=DISTANCE_GROUP&VarDesc=DistanceGroup&VarType=Num&VarName=CLASS&VarDesc=Class&VarType=Char&VarName=DATA_SOURCE&VarDesc=DataSource&VarType=Char'
+      t = RemoteTable.new :url => url, :form_data => form_data, :compression => :zip, :glob => '/*.csv'
+      assert_equal 'United States of America', t.rows.first['DEST_COUNTRY_NAME']
+    end
+  end
+  
+  if ENV['ALL'] == 'true' or ENV['FAST'] == 'true'
+    should "connect using HTTPS if available" do
+      t = RemoteTable.new(:url => 'https://spreadsheets.google.com/pub?key=t5HM1KbaRngmTUbntg8JwPA')
+      assert_equal 'Gulf Coast',     t.rows.first['PAD district name']
+      assert_equal 'AL',             t.rows.first['State']
+      assert_equal 'Rocky Mountain', t.rows.last['PAD district name']
+      assert_equal 'WY',             t.rows.last['State']
+    end
+    
+    should "read an HTML table made with frontpage" do
+      t = RemoteTable.new :url => "http://www.faa.gov/air_traffic/publications/atpubs/CNT/5-2-E.htm",
+                          :encoding => 'US-ASCII',
+                          :row_xpath => '//table/tr[2]/td/table/tr',
+                          :column_xpath => 'td'
+      assert_equal 'E110', t.rows.first['Designator']
+      assert_equal 'EMBRAER', t.rows.first['Manufacturer']
+      assert_equal 'EZKC', t.rows.last['Designator']
+      assert_equal 'EZ King Cobra', t.rows.last['Model']
+    end
+  
     should "hash rows without paying attention to order" do
       x = ActiveSupport::OrderedHash.new
       x[:a] = 1
@@ -116,13 +154,6 @@ class RemoteTableTest < Test::Unit::TestCase
       assert_equal 'Rocky Mountain', t.rows.last[4]
     end
   
-    should "send form data, follow redirects and use a filename glob" do
-      url = 'http://www.transtats.bts.gov/DownLoad_Table.asp?Table_ID=293&Has_Group=3&Is_Zipped=0'
-      form_data = 'UserTableName=T_100_Segment__All_Carriers&DBShortName=Air_Carriers&RawDataTable=T_T100_SEGMENT_ALL_CARRIER&sqlstr=+SELECT+DEPARTURES_SCHEDULED%2CDEPARTURES_PERFORMED%2CPAYLOAD%2CSEATS%2CPASSENGERS%2CFREIGHT%2CMAIL%2CDISTANCE%2CRAMP_TO_RAMP%2CAIR_TIME%2CUNIQUE_CARRIER%2CAIRLINE_ID%2CUNIQUE_CARRIER_NAME%2CUNIQUE_CARRIER_ENTITY%2CREGION%2CCARRIER%2CCARRIER_NAME%2CCARRIER_GROUP%2CCARRIER_GROUP_NEW%2CORIGIN%2CORIGIN_CITY_NAME%2CORIGIN_CITY_NUM%2CORIGIN_STATE_ABR%2CORIGIN_STATE_FIPS%2CORIGIN_STATE_NM%2CORIGIN_COUNTRY%2CORIGIN_COUNTRY_NAME%2CORIGIN_WAC%2CDEST%2CDEST_CITY_NAME%2CDEST_CITY_NUM%2CDEST_STATE_ABR%2CDEST_STATE_FIPS%2CDEST_STATE_NM%2CDEST_COUNTRY%2CDEST_COUNTRY_NAME%2CDEST_WAC%2CAIRCRAFT_GROUP%2CAIRCRAFT_TYPE%2CAIRCRAFT_CONFIG%2CYEAR%2CQUARTER%2CMONTH%2CDISTANCE_GROUP%2CCLASS%2CDATA_SOURCE+FROM++T_T100_SEGMENT_ALL_CARRIER+WHERE+Month+%3D1+AND+YEAR%3D2008&varlist=DEPARTURES_SCHEDULED%2CDEPARTURES_PERFORMED%2CPAYLOAD%2CSEATS%2CPASSENGERS%2CFREIGHT%2CMAIL%2CDISTANCE%2CRAMP_TO_RAMP%2CAIR_TIME%2CUNIQUE_CARRIER%2CAIRLINE_ID%2CUNIQUE_CARRIER_NAME%2CUNIQUE_CARRIER_ENTITY%2CREGION%2CCARRIER%2CCARRIER_NAME%2CCARRIER_GROUP%2CCARRIER_GROUP_NEW%2CORIGIN%2CORIGIN_CITY_NAME%2CORIGIN_CITY_NUM%2CORIGIN_STATE_ABR%2CORIGIN_STATE_FIPS%2CORIGIN_STATE_NM%2CORIGIN_COUNTRY%2CORIGIN_COUNTRY_NAME%2CORIGIN_WAC%2CDEST%2CDEST_CITY_NAME%2CDEST_CITY_NUM%2CDEST_STATE_ABR%2CDEST_STATE_FIPS%2CDEST_STATE_NM%2CDEST_COUNTRY%2CDEST_COUNTRY_NAME%2CDEST_WAC%2CAIRCRAFT_GROUP%2CAIRCRAFT_TYPE%2CAIRCRAFT_CONFIG%2CYEAR%2CQUARTER%2CMONTH%2CDISTANCE_GROUP%2CCLASS%2CDATA_SOURCE&grouplist=&suml=&sumRegion=&filter1=title%3D&filter2=title%3D&geo=All%A0&time=January&timename=Month&GEOGRAPHY=All&XYEAR=2008&FREQUENCY=1&AllVars=All&VarName=DEPARTURES_SCHEDULED&VarDesc=DepScheduled&VarType=Num&VarName=DEPARTURES_PERFORMED&VarDesc=DepPerformed&VarType=Num&VarName=PAYLOAD&VarDesc=Payload&VarType=Num&VarName=SEATS&VarDesc=Seats&VarType=Num&VarName=PASSENGERS&VarDesc=Passengers&VarType=Num&VarName=FREIGHT&VarDesc=Freight&VarType=Num&VarName=MAIL&VarDesc=Mail&VarType=Num&VarName=DISTANCE&VarDesc=Distance&VarType=Num&VarName=RAMP_TO_RAMP&VarDesc=RampToRamp&VarType=Num&VarName=AIR_TIME&VarDesc=AirTime&VarType=Num&VarName=UNIQUE_CARRIER&VarDesc=UniqueCarrier&VarType=Char&VarName=AIRLINE_ID&VarDesc=AirlineID&VarType=Num&VarName=UNIQUE_CARRIER_NAME&VarDesc=UniqueCarrierName&VarType=Char&VarName=UNIQUE_CARRIER_ENTITY&VarDesc=UniqCarrierEntity&VarType=Char&VarName=REGION&VarDesc=CarrierRegion&VarType=Char&VarName=CARRIER&VarDesc=Carrier&VarType=Char&VarName=CARRIER_NAME&VarDesc=CarrierName&VarType=Char&VarName=CARRIER_GROUP&VarDesc=CarrierGroup&VarType=Num&VarName=CARRIER_GROUP_NEW&VarDesc=CarrierGroupNew&VarType=Num&VarName=ORIGIN&VarDesc=Origin&VarType=Char&VarName=ORIGIN_CITY_NAME&VarDesc=OriginCityName&VarType=Char&VarName=ORIGIN_CITY_NUM&VarDesc=OriginCityNum&VarType=Num&VarName=ORIGIN_STATE_ABR&VarDesc=OriginState&VarType=Char&VarName=ORIGIN_STATE_FIPS&VarDesc=OriginStateFips&VarType=Char&VarName=ORIGIN_STATE_NM&VarDesc=OriginStateName&VarType=Char&VarName=ORIGIN_COUNTRY&VarDesc=OriginCountry&VarType=Char&VarName=ORIGIN_COUNTRY_NAME&VarDesc=OriginCountryName&VarType=Char&VarName=ORIGIN_WAC&VarDesc=OriginWac&VarType=Num&VarName=DEST&VarDesc=Dest&VarType=Char&VarName=DEST_CITY_NAME&VarDesc=DestCityName&VarType=Char&VarName=DEST_CITY_NUM&VarDesc=DestCityNum&VarType=Num&VarName=DEST_STATE_ABR&VarDesc=DestState&VarType=Char&VarName=DEST_STATE_FIPS&VarDesc=DestStateFips&VarType=Char&VarName=DEST_STATE_NM&VarDesc=DestStateName&VarType=Char&VarName=DEST_COUNTRY&VarDesc=DestCountry&VarType=Char&VarName=DEST_COUNTRY_NAME&VarDesc=DestCountryName&VarType=Char&VarName=DEST_WAC&VarDesc=DestWac&VarType=Num&VarName=AIRCRAFT_GROUP&VarDesc=AircraftGroup&VarType=Num&VarName=AIRCRAFT_TYPE&VarDesc=AircraftType&VarType=Char&VarName=AIRCRAFT_CONFIG&VarDesc=AircraftConfig&VarType=Num&VarName=YEAR&VarDesc=Year&VarType=Num&VarName=QUARTER&VarDesc=Quarter&VarType=Num&VarName=MONTH&VarDesc=Month&VarType=Num&VarName=DISTANCE_GROUP&VarDesc=DistanceGroup&VarType=Num&VarName=CLASS&VarDesc=Class&VarType=Char&VarName=DATA_SOURCE&VarDesc=DataSource&VarType=Char'
-      t = RemoteTable.new :url => url, :form_data => form_data, :compression => :zip, :glob => '/*.csv'
-      assert_equal 'United States of America', t.rows.first['DEST_COUNTRY_NAME']
-    end
-  
     should "take the last of values if the header is duplicated" do
       t = RemoteTable.new(:url => 'http://spreadsheets.google.com/pub?key=tujrgUOwDSLWb-P4KCt1qBg')
       assert_equal '2', t.rows.first['dup_header']
@@ -252,30 +283,6 @@ class RemoteTableTest < Test::Unit::TestCase
       assert_equal 99,               t.rows[5]['MAP DATA population (millions) 2002'].to_i
     end
     
-    should "open a CSV inside a zip file" do
-      t = RemoteTable.new(:url => 'http://www.fueleconomy.gov/FEG/epadata/98guide6.zip', :filename => '98guide6.csv')
-      assert_equal 'ACURA',             t.rows.first['Manufacturer']
-      assert_equal 'NSX',               t.rows.first['carline name']
-      assert_equal 'TOYOTA',            t.rows.last['Manufacturer']
-      assert_equal 'RAV4 SOFT TOP 4WD', t.rows.last['carline name']
-    end
-  
-    should "open a fixed-width file with an inline schema inside a zip file" do
-      t = RemoteTable.new(:url => 'http://www.fueleconomy.gov/FEG/epadata/00data.zip',
-                          :filename => 'Gd6-dsc.txt',
-                          :format => :fixed_width,
-                          :crop => 21..26, # inclusive
-                          :cut => '2-',
-                          :select => lambda { |row| /\A[A-Z]/.match row['code'] },
-                          :schema => [[ 'code',   2, { :type => :string }  ],
-                                      [ 'spacer', 2 ],
-                                      [ 'name',   52, { :type => :string } ]])
-      assert_equal 'regular grade gasoline (octane number of 87)', t.rows.first['name']
-      assert_equal 'R',                                            t.rows.first['code']
-      assert_equal 'electricity',                                  t.rows.last['name']
-      assert_equal 'El',                                           t.rows.last['code']
-    end
-  
     should "open an XLS with a parser" do
       ma_1990_01 = {"month"=>1, "cost"=>"54.0", "locatable"=>"Massachusetts (State)", "year"=>1990}
       ga_1990_01 = {"month"=>1, "cost"=>"50.7", "locatable"=>"Georgia (State)", "year"=>1990}
@@ -287,17 +294,17 @@ class RemoteTableTest < Test::Unit::TestCase
       assert t.rows.include?(ga_1990_01)
     end
   
-    should "provide a row_hash on demand" do
-      t = RemoteTable.new(:url => 'http://www.fueleconomy.gov/FEG/epadata/00data.zip',
-                          :filename => 'Gd6-dsc.txt',
-                          :format => :fixed_width,
-                          :crop => 21..26, # inclusive
-                          :cut => '2-',
-                          :select => lambda { |row| /\A[A-Z]/.match row['code'] },
-                          :schema => [[ 'code',   2, { :type => :string }  ],
-                                      [ 'spacer', 2 ],
-                                      [ 'name',   52, { :type => :string } ]])
-      assert_equal 'a8a5d7f17b56772723c657eb62b0f238', t.rows.first['row_hash']
-    end
+    # should "provide a row_hash on demand" do
+    #   t = RemoteTable.new(:url => 'http://www.fueleconomy.gov/FEG/epadata/00data.zip',
+    #                       :filename => 'Gd6-dsc.txt',
+    #                       :format => :fixed_width,
+    #                       :crop => 21..26, # inclusive
+    #                       :cut => '2-',
+    #                       :select => lambda { |row| /\A[A-Z]/.match row['code'] },
+    #                       :schema => [[ 'code',   2, { :type => :string }  ],
+    #                                   [ 'spacer', 2 ],
+    #                                   [ 'name',   52, { :type => :string } ]])
+    #   assert_equal 'a8a5d7f17b56772723c657eb62b0f238', t.rows.first['row_hash']
+    # end
   end
 end

metadata

@@ -5,8 +5,8 @@ version: !ruby/object:Gem::Version
   segments: 
   - 0
   - 2
-  - 16
-  version: 0.2.16
+  - 17
+  version: 0.2.17
 platform: ruby
 authors: 
 - Seamus Abshere
@@ -15,7 +15,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2010-04-27 00:00:00 -04:00
+date: 2010-04-28 00:00:00 -04:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency