Skip to content

Commit 4894cbc

Browse files
committed
Refactor: Clean up import statements and improve error message formatting
1 parent 32fda64 commit 4894cbc

7 files changed

Lines changed: 13 additions & 14 deletions

File tree

datalab_kernel/backends/webapi.py

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -167,8 +167,7 @@ def _raise_for_status(self, response: httpx.Response) -> None:
167167
raise
168168
except Exception:
169169
raise RuntimeError(
170-
f"DataLab server error (HTTP 500). "
171-
f"URL: {response.request.url}"
170+
f"DataLab server error (HTTP 500). URL: {response.request.url}"
172171
) from None
173172

174173
response.raise_for_status()

datalab_kernel/kernel.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,11 +14,11 @@
1414
from __future__ import annotations
1515

1616
from ipykernel.ipkernel import IPythonKernel
17+
from sigima import create_image, create_signal
1718

1819
from datalab_kernel import __version__
1920
from datalab_kernel.plotter import Plotter
2021
from datalab_kernel.workspace import Workspace
21-
from sigima import create_image, create_signal
2222

2323

2424
class DataLabKernel(IPythonKernel):

datalab_kernel/tests/conftest.py

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,9 @@ def pytest_configure(config):
2020
config.addinivalue_line("markers", "standalone: tests for standalone mode only")
2121
config.addinivalue_line("markers", "live: tests requiring live DataLab instance")
2222
config.addinivalue_line("markers", "contract: contract tests for both modes")
23-
config.addinivalue_line("markers", "integration: integration tests with external services")
23+
config.addinivalue_line(
24+
"markers", "integration: integration tests with external services"
25+
)
2426
config.addinivalue_line("markers", "webapi: tests requiring WebAPI backend")
2527

2628

datalab_kernel/tests/contract/test_api.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,8 +16,8 @@
1616

1717
import numpy as np
1818
import pytest
19-
2019
from sigima import create_image, create_signal
20+
2121
from datalab_kernel.plotter import Plotter
2222
from datalab_kernel.workspace import StandaloneBackend, Workspace
2323

datalab_kernel/tests/data.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,6 @@
1111
from __future__ import annotations
1212

1313
import numpy as np
14-
1514
from sigima import ImageObj, SignalObj, create_image, create_signal
1615

1716

datalab_kernel/tests/test_webapi_backend.py

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -196,13 +196,13 @@ def test_short_id_pattern_matching(self):
196196

197197
# Should not match
198198
invalid_cases = [
199-
"s01", # Not enough digits
200-
"s0001", # Too many digits
201-
"x001", # Wrong prefix
202-
"signal1", # Not a short ID
203-
"", # Empty
204-
"s", # Just letter
205-
"001", # Just numbers
199+
"s01", # Not enough digits
200+
"s0001", # Too many digits
201+
"x001", # Wrong prefix
202+
"signal1", # Not a short ID
203+
"", # Empty
204+
"s", # Just letter
205+
"001", # Just numbers
206206
]
207207
for case in invalid_cases:
208208
match = re.match(pattern, case.lower())

datalab_kernel/tests/unit/test_objects.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,6 @@
1111
from __future__ import annotations
1212

1313
import numpy as np
14-
1514
from sigima import (
1615
ImageObj,
1716
SignalObj,

0 commit comments

Comments
 (0)