Skip to content

Commit a0626e3

Browse files
committed
chore: remove GetSession checks in tests
1 parent 6266b5a commit a0626e3

File tree

1 file changed

+5
-8
lines changed

1 file changed

+5
-8
lines changed

test/mockserver_tests/test_json.py

Lines changed: 5 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,6 @@
2121
BatchCreateSessionsRequest,
2222
ExecuteSqlRequest,
2323
CommitRequest,
24-
GetSessionRequest,
2524
BeginTransactionRequest,
2625
TypeCode,
2726
JsonObject,
@@ -95,14 +94,12 @@ def _test_insert_json(self, description, expected):
9594

9695
# Verify the requests that we got.
9796
requests = self.spanner_service.requests
98-
eq_(5, len(requests))
97+
eq_(4, len(requests))
9998
is_instance_of(requests[0], BatchCreateSessionsRequest)
100-
# We should get rid of this extra round-trip for GetSession....
101-
is_instance_of(requests[1], GetSessionRequest)
102-
is_instance_of(requests[2], BeginTransactionRequest)
103-
is_instance_of(requests[3], ExecuteSqlRequest)
104-
is_instance_of(requests[4], CommitRequest)
105-
request: ExecuteSqlRequest = requests[3]
99+
is_instance_of(requests[1], BeginTransactionRequest)
100+
is_instance_of(requests[2], ExecuteSqlRequest)
101+
is_instance_of(requests[3], CommitRequest)
102+
request: ExecuteSqlRequest = requests[2]
106103
eq_(3, len(request.params))
107104
eq_("1", request.params["a0"])
108105
eq_("Test", request.params["a1"])

0 commit comments

Comments
 (0)