fix spec on problem with mongoid 2.2.x

Cyril Mougel
1 parent 489586f1
Showing 2 changed files with 13 additions and 0 deletions Show diff stats
app/models/problem.rb
spec/models/problem_spec.rb
@@ -2,6 +2,14 @@
 # reported as various Errs, but the user has grouped the
 # Errs together as belonging to the same problem.
+## Add methode nan? in Time because needed by #max(:created_at)
+#
+# Fix on  Mongoid > 2.3.x with commit :
+# https://github.com/mongoid/mongoid/commit/5481556e24480f0a1783f85d6b5b343b0cef7192
+class Time
+  def nan?; false ;end
+end
+
 class Problem
   include Mongoid::Document
   include Mongoid::Timestamps
@@ -91,6 +91,11 @@ describe Problem do
       problem = Fabricate(:problem)
       problem.should_not be_resolved
       problem.stub!(:valid?).and_return(false)
+      ## update_attributes not test #valid? but #errors.any?
+      # https://github.com/mongoid/mongoid/blob/master/lib/mongoid/persistence.rb#L137
+      er = ActiveModel::Errors.new(problem)
+      er.add_on_blank(:resolved)
+      problem.stub!(:errors).and_return(er)
       problem.should_not be_valid
       lambda {
         problem.resolve!
	@@ -2,6 +2,14 @@		@@ -2,6 +2,14 @@
2	# reported as various Errs, but the user has grouped the	2	# reported as various Errs, but the user has grouped the
3	# Errs together as belonging to the same problem.	3	# Errs together as belonging to the same problem.
4		4
		5	+## Add methode nan? in Time because needed by #max(:created_at)
		6	+#
		7	+# Fix on Mongoid > 2.3.x with commit :
		8	+# https://github.com/mongoid/mongoid/commit/5481556e24480f0a1783f85d6b5b343b0cef7192
		9	+class Time
		10	+ def nan?; false ;end
		11	+end
		12	+
5	class Problem	13	class Problem
6	include Mongoid::Document	14	include Mongoid::Document
7	include Mongoid::Timestamps	15	include Mongoid::Timestamps