Skip to content

Commit 019c7fb

Browse files
reduce diff of test_closing_connection_closes_commands
Signed-off-by: varun-edachali-dbx <varun.edachali@databricks.com>
1 parent fa15730 commit 019c7fb

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

tests/unit/test_client.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -102,7 +102,7 @@ def test_closing_connection_closes_commands(self, mock_thrift_client_class):
102102

103103
for closed in (True, False):
104104
with self.subTest(closed=closed):
105-
# set initial state based on whether the command is already closed
105+
# Set initial state based on whether the command is already closed
106106
initial_state = (
107107
CommandState.CLOSED if closed else CommandState.SUCCEEDED
108108
)
@@ -114,7 +114,7 @@ def test_closing_connection_closes_commands(self, mock_thrift_client_class):
114114
mock_execute_response.is_staging_operation = False
115115
mock_execute_response.command_id = Mock(spec=CommandId)
116116

117-
# Mock the backend that will be used by the real ThriftResultSet
117+
# Mock the backend that will be used
118118
mock_backend = Mock(spec=ThriftDatabricksClient)
119119
mock_backend.staging_allowed_local_path = None
120120
mock_thrift_client_class.return_value = mock_backend
@@ -132,7 +132,7 @@ def test_closing_connection_closes_commands(self, mock_thrift_client_class):
132132
# Mock execute_command to return our real result set
133133
cursor.backend.execute_command = Mock(return_value=real_result_set)
134134

135-
# Execute a command - this should set cursor.active_result_set to our real result set
135+
# Execute a command
136136
cursor.execute("SELECT 1")
137137

138138
# Close the connection

0 commit comments

Comments
 (0)