Skip to content

Commit 7886e21

Browse files
Gayathri Srividya Rajavarapuclaude
authored andcommitted
fix: move imports to top of test_rest.py to resolve ruff E402 errors
Remove duplicate `import json` and `import pytest` added mid-file for issue #3422 tests; move `import json` to the top-level import block. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
1 parent ed898ba commit 7886e21

1 file changed

Lines changed: 1 addition & 7 deletions

File tree

tests/catalog/test_rest.py

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,7 @@
1818
from __future__ import annotations
1919

2020
import base64
21+
import json
2122
import os
2223
from collections.abc import Callable
2324
from typing import Any, cast
@@ -3167,10 +3168,6 @@ def test_load_table_without_storage_credentials(
31673168
# Tests for issue #3422: REST catalog auth cannot be configured via environment
31683169
# variables unless auth JSON strings are decoded.
31693170

3170-
import json
3171-
3172-
import pytest
3173-
31743171

31753172
@pytest.mark.parametrize(
31763173
"auth_dict, expected_header, mocker_type",
@@ -3290,9 +3287,6 @@ def test_rest_catalog_with_basic_auth_flat_properties(rest_mock: Mocker) -> None
32903287
assert rest_mock.last_request.headers["Authorization"] == expected_auth_header
32913288

32923289

3293-
import pytest
3294-
3295-
32963290
@pytest.mark.parametrize(
32973291
"catalog_properties, token_response, expected_scopes",
32983292
[

0 commit comments

Comments
 (0)