This needs to be eager now, as we won't verify (and thus get the tested warning) otherwise.

pull/1451/head
jMyles 2019-12-23 14:38:26 -08:00
parent b1345c24db
commit 332d072c40
2 changed files with 2 additions and 2 deletions

View File

@ -30,7 +30,7 @@ def test_learner_learns_about_domains_separately(ursula_federated_test_config, c
assert len(big_learner.known_nodes) == 5
new_first_domain_learner = lonely_ursula_maker(domains={"nucypher1.test_suite"}).pop()
new_second_domain_learner = lonely_ursula_maker(domains={"nucypher2.test_suite"})
_new_second_domain_learner = lonely_ursula_maker(domains={"nucypher2.test_suite"})
new_first_domain_learner._current_teacher_node = big_learner
new_first_domain_learner.learn_from_teacher_node()

View File

@ -40,7 +40,7 @@ def test_blockchain_ursula_stamp_verification_tolerance(blockchain_ursulas):
lonely_blockchain_learner.remember_node(blockchain_teacher)
globalLogPublisher.addObserver(warning_trapper)
lonely_blockchain_learner.learn_from_teacher_node()
lonely_blockchain_learner.learn_from_teacher_node(eager=True)
globalLogPublisher.removeObserver(warning_trapper)
# We received one warning during learning, and it was about this very matter.