Merge pull request #257 from meejah/missing-last-router

Send .done() input to Microdescriptor parser
meejah · Nov 4, 2017 · c294eb6 · c294eb6
2 parents 46c0c39 + 6bd4090
commit c294eb6
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 2 deletions.
diff --git a/docs/releases.rst b/docs/releases.rst
@@ -25,6 +25,7 @@ unreleased
  * add method :func:`txtorcon.Tor.is_ready`
  * add method :func:`txtorcon.Tor.become_ready`
  * fix handling of certain defaults (`*PortLines` and friends)
+ * fix last router (usually) missing with (new) `MicroDescriptorParser`
 
 
 v0.19.3

diff --git a/test/test_torstate.py b/test/test_torstate.py
@@ -229,8 +229,9 @@ def confirm_state(self, x):
         return x
 
     def confirm_consensus(self, x):
-        self.assertEqual(1, len(x.all_routers))
-        self.assertEqual('fake', list(x.routers.values())[0].name)
+        self.assertEqual(2, len(x.all_routers))
+        self.assertIn('fake', x.routers)
+        self.assertIn('ekaf', x.routers)
         return x
 
     def test_build(self):

diff --git a/txtorcon/torstate.py b/txtorcon/torstate.py
@@ -349,6 +349,7 @@ def _bootstrap(self, arg=None):
             'ns/all',
             self._network_status_parser.feed_line,
         )
+        self._network_status_parser.done()
 
         # update list of existing circuits
         cs = yield self.protocol.get_info_raw('circuit-status')