Fix explain

lanwin · Dec 14, 2010 · 2ffb790 · 2ffb790
1 parent dcb7912
commit 2ffb790
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 4 deletions.
diff --git a/source/MongoDB.Tests/IntegrationTests/TestCursor.cs b/source/MongoDB.Tests/IntegrationTests/TestCursor.cs
@@ -114,12 +114,21 @@ public void TestCanReadSmall()
         }
 
         [Test]
-        public void TestExplain()
+        public void TestExplainWithSort()
         {
             var exp = DB["reads"].FindAll().Limit(5).Skip(5).Sort("x").Explain();
             Assert.IsTrue(exp.ContainsKey("cursor"));
-            Assert.IsTrue(exp.ContainsKey("n"));
-            Assert.IsTrue(exp.ContainsKey("nscanned"));
+            Assert.AreEqual(9999, exp.Get("nscanned"));
+            Assert.AreEqual(10,exp.Get("n"));
+        }
+
+        [Test]
+        public void TestExplain()
+        {
+            var exp = DB["reads"].FindAll().Limit(5).Skip(5).Explain();
+            Assert.IsTrue(exp.ContainsKey("cursor"));
+            Assert.AreEqual(10, exp.Get("nscanned"));
+            Assert.AreEqual(5, exp.Get("n"));
         }
 
         [Test]

diff --git a/source/MongoDB/Cursor_1.cs b/source/MongoDB/Cursor_1.cs
@@ -207,8 +207,11 @@ public ICursor<T> KeepCursor(bool value)
         /// <returns></returns>
         public Document Explain(){
             TryModify();
-            _specOpts["$explain"] = true;
 
+            var savedLimit = _limit;
+
+            _specOpts["$explain"] = true;
+            _limit = _limit * -1;
             var explainResult = RetrieveData<Document>();
             try
             {
@@ -223,6 +226,8 @@ public ICursor<T> KeepCursor(bool value)
             {
                 if(explainResult.CursorId > 0)
                     KillCursor(explainResult.CursorId);
+
+                _limit = savedLimit;
             }
         }