Skip to content

Commit b86fb42

Browse files
committed
Make test more cohesive with the spec
1 parent 918c989 commit b86fb42

File tree

1 file changed

+20
-9
lines changed

1 file changed

+20
-9
lines changed

tests/stub/authorization.py

Lines changed: 20 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -730,7 +730,7 @@ def tearDown(self):
730730
self._server.reset()
731731
super().tearDown()
732732

733-
def read_return_1_failure_return_2_succeed_script(self):
733+
def read_return_1_failure_return_2_and_3_succeed_script(self):
734734
return """
735735
!: BOLT 4
736736
!: AUTO HELLO
@@ -747,10 +747,20 @@ def read_return_1_failure_return_2_succeed_script(self):
747747
S: <EXIT>
748748
----
749749
C: RUN "RETURN 2 as n" {} {"mode": "r"}
750-
C: PULL {"n": 1000}
751750
S: SUCCESS {"fields": ["n"]}
752-
RECORD [1]
753-
SUCCESS {"type": "r"}
751+
{+
752+
C: PULL {"n": 1}
753+
S: RECORD [1]
754+
SUCCESS {"type": "r", "has_more": true}
755+
+}
756+
C: DISCARD {"n": -1}
757+
S: SUCCESS {}
758+
----
759+
C: RUN "RETURN 3 as n" {} {"mode": "r"}
760+
C: PULL {"n": "*"}
761+
S: SUCCESS {"fields": ["n"]}
762+
RECORD [1]
763+
SUCCESS {"type": "r"}
754764
}}
755765
756766
+}
@@ -759,12 +769,12 @@ def read_return_1_failure_return_2_succeed_script(self):
759769
@driver_feature(types.Feature.AUTHORIZATION_EXPIRED_TREATMENT)
760770
def test_should_drop_connection_after_AuthorizationExpired(self):
761771
self._server.start(
762-
script=self.read_return_1_failure_return_2_succeed_script()
772+
script=self.read_return_1_failure_return_2_and_3_succeed_script()
763773
)
764774
driver = Driver(self._backend, self._uri, self._auth,
765775
userAgent=self._userAgent)
766776

767-
session1 = driver.session('r')
777+
session1 = driver.session('r', fetchSize=1)
768778
session2 = driver.session('r')
769779

770780
session1.run('RETURN 2 as n').next()
@@ -782,25 +792,26 @@ def test_should_drop_connection_after_AuthorizationExpired(self):
782792
# fetching another connection and run a query to force
783793
# drivers which lazy close the connection do it
784794
session3 = driver.session('r')
785-
session3.run('RETURN 2 as n').next()
795+
session3.run('RETURN 3 as n').next()
786796
session3.close()
787797

788798
hangup_count = self._server.count_responses("<HANGUP>")
789799

790800
self.assertEqual(accept_count, hangup_count)
801+
self.assertGreaterEqual(accept_count, 2)
791802

792803
driver.close()
793804

794805
@driver_feature(types.Feature.AUTHORIZATION_EXPIRED_TREATMENT)
795806
def test_should_be_able_to_use_current_sessions_after_AuthorizationExpired(self):
796807
self._server.start(
797-
script=self.read_return_1_failure_return_2_succeed_script()
808+
script=self.read_return_1_failure_return_2_and_3_succeed_script()
798809
)
799810

800811
driver = Driver(self._backend, self._uri, self._auth,
801812
userAgent=self._userAgent)
802813

803-
session1 = driver.session('r')
814+
session1 = driver.session('r', fetchSize=1)
804815
session2 = driver.session('r')
805816

806817
session1.run('RETURN 2 as n').next()

0 commit comments

Comments
 (0)