Rev 418: Test more than one target. Each one we stop on should stop loading in http://bazaar.launchpad.net/~jameinel/loggerhead/page_loading

[John Arbash Meinel]

At http://bazaar.launchpad.net/~jameinel/loggerhead/page_loading

------------------------------------------------------------
revno: 418
revision-id: john at arbash-meinel.com-20110314144914-j67mzkwdyrszv4yx
parent: john at arbash-meinel.com-20110314144748-blc0zub6x7u8nf2m
committer: John Arbash Meinel <john at arbash-meinel.com>
branch nick: page_loading
timestamp: Mon 2011-03-14 15:49:14 +0100
message:
  Test more than one target. Each one we stop on should stop loading
  extra history. That way if we are looking for 100 revs, but only displaying 10,
  we don't have to load all 100.
-------------- next part --------------
=== modified file 'loggerhead/tests/test_history.py'
--- a/loggerhead/tests/test_history.py	2011-03-14 14:47:48 +0000
+++ b/loggerhead/tests/test_history.py	2011-03-14 14:49:14 +0000
@@ -179,14 +179,16 @@
         # answers for the first few revisions.
         his = self.make_long_linear_ancestry()
         accessed = track_rev_info_accesses(his)
-        result = his.get_revids_from(['X'], 'Z')
+        result = his.get_revids_from(['X', 'V'], 'Z')
         self.assertEqual(set(), accessed)
         self.assertEqual('X', result.next())
         # We access 'W' because we are checking that W wasn't merged into X.
         # The important bit is that we aren't getting the whole ancestry.
         self.assertEqual(set([his._rev_indices[x] for x in 'ZYXW']), accessed)
+        self.assertEqual('V', result.next())
+        self.assertEqual(set([his._rev_indices[x] for x in 'ZYXWVU']), accessed)
         self.assertRaises(StopIteration, result.next)
-        self.assertEqual(set([his._rev_indices[x] for x in 'ZYXW']), accessed)
+        self.assertEqual(set([his._rev_indices[x] for x in 'ZYXWVU']), accessed)