itstommymorgan / asari

a Ruby wrapper for AWS CloudSearch.
52 stars 54 forks source link

Search Spec failing #21

Closed jwarchol closed 10 years ago

jwarchol commented 10 years ago

I'm getting the following failure when running the specs. Can anyone else confirm they're passing? If so I'll dig in and figure out what's wrong on my side, but I suspect it's just a minor thing related to cleanup of geo

.................................................................F

Failures:

  1) Asari geography searching builds a proper query string
     Failure/Error: HTTParty.should_receive(:get).with("http://search-testdomain.us-east-1.cloudsearch.amazonaws.com/2011-02-01/search?q=&bq=%28and+lat%3A2505771415..2506771417+lng%3A111275735..111322958%29&size=10")
       Double received :get with unexpected arguments
         expected: ("http://search-testdomain.us-east-1.cloudsearch.amazonaws.com/2011-02-01/search?q=&bq=%28and+lat%3A2505771415..2506771417+lng%3A111275735..111322958%29&size=10")
              got: ("http://search-testdomain.us-east-1.cloudsearch.amazonaws.com/2011-02-01/search?q=&bq=%28and+lat%3A2505771415..2506771417+lng%3A2358260777..2359261578%29&size=10")
     # ./spec/search_spec.rb:149:in `block (3 levels) in <top (required)>'

Finished in 0.04652 seconds
66 examples, 1 failure

Failed examples:

rspec ./spec/search_spec.rb:148 # Asari geography searching builds a proper query string
zmoshansky commented 10 years ago

As of commit 45133f694cede9cdd33c2015daec042a85f84b10

Failures:

  1) Asari updating the index converts Time, DateTime, and Date fields to timestamp integers for rankability
     Failure/Error: HTTParty.should_receive(:post).with("http://doc-testdomain.us-east-1.cloudsearch.amazonaws.com/2011-02-01/documents/batch", { :body => [{ "type" => "add", "id" => "1", "version" => 1, "lang" => "en", "fields" => { :time => 1333263600, :datetime => 1333238400, :date => 1333252800 }}].to_json, :headers => { "Content-Type" => "application/json"}})
       Double received :post with unexpected arguments
         expected: ("http://doc-testdomain.us-east-1.cloudsearch.amazonaws.com/2011-02-01/documents/batch", {:body=>"[{\"type\":\"add\",\"id\":\"1\",\"version\":1,\"lang\":\"en\",\"fields\":{\"time\":1333263600,\"datetime\":1333238400,\"date\":1333252800}}]", :headers=>{"Content-Type"=>"application/json"}})
              got: ("http://doc-testdomain.us-east-1.cloudsearch.amazonaws.com/2011-02-01/documents/batch", {:body=>"[{\"type\":\"add\",\"id\":\"1\",\"version\":1,\"lang\":\"en\",\"fields\":{\"time\":1333263600,\"datetime\":1333238400,\"date\":1333263600}}]", :headers=>{"Content-Type"=>"application/json"}})
     # ./spec/documents_spec.rb:32:in `block (3 levels) in <top (required)>'

  2) Asari updating the index converts Time, DateTime, and Date fields to timestamp integers for rankability on update as well
     Failure/Error: HTTParty.should_receive(:post).with("http://doc-testdomain.us-east-1.cloudsearch.amazonaws.com/2011-02-01/documents/batch", { :body => [{ "type" => "add", "id" => "1", "version" => 1, "lang" => "en", "fields" => { :time => 1333263600, :datetime => 1333238400, :date => 1333252800 }}].to_json, :headers => { "Content-Type" => "application/json"}})
       Double received :post with unexpected arguments
         expected: ("http://doc-testdomain.us-east-1.cloudsearch.amazonaws.com/2011-02-01/documents/batch", {:body=>"[{\"type\":\"add\",\"id\":\"1\",\"version\":1,\"lang\":\"en\",\"fields\":{\"time\":1333263600,\"datetime\":1333238400,\"date\":1333252800}}]", :headers=>{"Content-Type"=>"application/json"}})
              got: ("http://doc-testdomain.us-east-1.cloudsearch.amazonaws.com/2011-02-01/documents/batch", {:body=>"[{\"type\":\"add\",\"id\":\"1\",\"version\":1,\"lang\":\"en\",\"fields\":{\"time\":1333263600,\"datetime\":1333238400,\"date\":1333263600}}]", :headers=>{"Content-Type"=>"application/json"}})
     # ./spec/documents_spec.rb:44:in `block (3 levels) in <top (required)>'

  3) Asari geography searching builds a proper query string
     Failure/Error: HTTParty.should_receive(:get).with("http://search-testdomain.us-east-1.cloudsearch.amazonaws.com/2011-02-01/search?q=&bq=%28and+lat%3A2505771415..2506771417+lng%3A111275735..111322958%29&size=10")
       Double received :get with unexpected arguments
         expected: ("http://search-testdomain.us-east-1.cloudsearch.amazonaws.com/2011-02-01/search?q=&bq=%28and+lat%3A2505771415..2506771417+lng%3A111275735..111322958%29&size=10")
              got: ("http://search-testdomain.us-east-1.cloudsearch.amazonaws.com/2011-02-01/search?q=&bq=%28and+lat%3A2505771415..2506771417+lng%3A2358260777..2359261578%29&size=10")
     # ./spec/search_spec.rb:149:in `block (3 levels) in <top (required)>'

Finished in 0.02437 seconds
66 examples, 3 failures

Failed examples:

rspec ./spec/documents_spec.rb:31 # Asari updating the index converts Time, DateTime, and Date fields to timestamp integers for rankability
rspec ./spec/documents_spec.rb:43 # Asari updating the index converts Time, DateTime, and Date fields to timestamp integers for rankability on update as well
rspec ./spec/search_spec.rb:148 # Asari geography searching builds a proper query string
lgleasain commented 10 years ago

I'm not seeing the second one, but am seeing the first.....will have a fix up shortly and will then close it. I'm also going to add this into Travis.

lgleasain commented 10 years ago

Ok, now I'm seeing the second set of errors. Tommy and I are in the same timezone but the ci servers aren't. :)

itstommymorgan commented 10 years ago

Timezones are hard.

On 04/14, Lance Gleason wrote:

Ok, now I'm seeing the second set of errors. Tommy and I are in the same timezone but the ci servers aren't. :)


Reply to this email directly or view it on GitHub: https://github.com/wellbredgrapefruit/asari/issues/21#issuecomment-40436045

Tommy Morgan http://www.wellbredgrapefruit.com

lgleasain commented 10 years ago

fixed