@@ -4,38 +4,32 @@ module Ambition #:nodoc: # Allows you to sort queries by particular fields. # It is unsure how well this is supported by Ultrasphinx. class Sort < Base - # >> sort_by { |u| u.age } - # => #sort_by(:age) + # Not implemented yet. def sort_by(method) raise "Not implemented yet." end - # >> sort_by { |u| -u.age } - # => #reverse_sort_by(:age) + # Not implemented yet. def reverse_sort_by(method) raise "Not implemented yet." end - # >> sort_by { |u| u.profile.name } - # => #chained_sort_by(:profile, :name) + # Not implemented yet. def chained_sort_by(receiver, method) raise "Not implemented yet." end - # >> sort_by { |u| -u.profile.name } - # => #chained_reverse_sort_by(:profile, :name) + # Not implemented yet. def chained_reverse_sort_by(receiver, method) raise "Not implemented yet." end - # >> sort_by(&:name) - # => #to_proc(:name) + # Not implemented yet. def to_proc(symbol) raise "Not implemented yet." end - # >> sort_by { rand } - # => #rand + # Not implemented yet. def rand raise "Not implemented yet." end lib/ambition/adapters/ambitious_sphinx/sort.rb @@ -6,48 +6,21 @@ context "AmbitiousSphinx Adapter :: Sort" do @block = @klass.select { |m| m.name =~ 'jon' } end - xspecify "order" do - string = @block.sort_by { |m| m.name }.to_s - string.should == "foo" + specify "order" do + should.raise do + @block.sort_by {|m| m.name} + end end - xspecify "combined order" do - string = @block.sort_by { |m| [ m.name, m.age ] }.to_s - string.should == "foo" + specify "reverse order with -" do + should.raise do + @block.sort_by { |m| -m.age } + end end - xspecify "combined order with single reverse" do - string = @block.sort_by { |m| [ m.name, -m.age ] }.to_s - string.should == "foo" - end - - xspecify "combined order with two reverses" do - string = @block.sort_by { |m| [ -m.name, -m.age ] }.to_s - string.should == "foo" - end - - xspecify "reverse order with -" do - string = @block.sort_by { |m| -m.age }.to_s - string.should == "foo" - end - - xspecify "reverse order with #reverse" do - # TODO: not implemented - string = @block.sort_by { |m| m.age }.reverse.to_s - string.should == "foo" - end - - xspecify "random order" do - string = @block.sort_by { rand }.to_s - string.should == "foo" - end - - xspecify "non-existent method to sort by" do - should.raise(NoMethodError) { @block.sort_by { foo }.to_s } - end - - xspecify "Symbol#to_proc" do - string = @klass.sort_by(&:name).to_s - string.should == "foo" + specify "random order" do + should.raise do + @block.sort_by { rand } + end end end test/sort_test.rb 20708230060d8cdf330af50a38bd1b50136802c6 Josh Nichols josh@technicalpickles.com http://github.com/technicalpickles/ambitious-sphinx/commit/235b70df3e2e54e0fe9b1977a0760b48c3b2f5a0 235b70df3e2e54e0fe9b1977a0760b48c3b2f5a0 2008-02-18T16:42:41-08:00 2008-02-18T16:42:41-08:00 More rdoc. Added specs that sorting should fail. ac2040bae4a9a4f7a7543c085ec612f2a2545aa6 Josh Nichols josh@technicalpickles.com