Skip to content

Commit f8d80db

Browse files
author
Georgios Hadjiharalambous
committed
make header name const
1 parent 7160976 commit f8d80db

4 files changed

Lines changed: 12 additions & 6 deletions

File tree

sdk/batch/speechmatics/batch/__init__.py

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -32,9 +32,11 @@
3232
from ._models import TranscriptFilteringConfig
3333
from ._models import TranscriptionConfig
3434
from ._models import TranslationConfig
35+
from ._transport import PROCESSING_DATA_HEADER
3536

3637
__all__ = [
3738
"AsyncClient",
39+
"PROCESSING_DATA_HEADER",
3840
"AuthBase",
3941
"AuthenticationError",
4042
"AutoChaptersConfig",

sdk/batch/speechmatics/batch/_async_client.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,6 +31,7 @@
3131
from ._models import JobType
3232
from ._models import Transcript
3333
from ._models import TranscriptionConfig
34+
from ._transport import PROCESSING_DATA_HEADER
3435
from ._transport import Transport
3536

3637

@@ -537,7 +538,7 @@ async def _submit_and_create_job_details(
537538
"""Submit job and create JobDetails response."""
538539
extra_headers: Optional[dict[str, dict[str, int]]] = None
539540
if requested_parallel is not None:
540-
extra_headers = {"X-SM-Processing-Data": {"requested_parallel": requested_parallel}}
541+
extra_headers = {PROCESSING_DATA_HEADER: {"requested_parallel": requested_parallel}}
541542
response = await self._transport.post("/jobs", multipart_data=multipart_data, extra_headers=extra_headers)
542543
job_id = response.get("id")
543544
if not job_id:

sdk/batch/speechmatics/batch/_transport.py

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,8 @@
2525
from ._logging import get_logger
2626
from ._models import ConnectionConfig
2727

28+
PROCESSING_DATA_HEADER = "X-SM-Processing-Data"
29+
2830

2931
class Transport:
3032
"""

tests/batch/test_submit_job.py

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,7 @@
1313
from speechmatics.batch import JobStatus
1414
from speechmatics.batch import JobType
1515
from speechmatics.batch import TranscriptionConfig
16+
from speechmatics.batch import PROCESSING_DATA_HEADER
1617

1718

1819
def _make_client(api_key: str = "test-key") -> AsyncClient:
@@ -53,8 +54,8 @@ async def test_header_sent_when_requested_parallel_provided(self):
5354

5455
extra_headers = _captured_extra_headers(mock_post)
5556
assert extra_headers is not None
56-
assert "X-SM-Processing-Data" in extra_headers
57-
payload = extra_headers["X-SM-Processing-Data"]
57+
assert PROCESSING_DATA_HEADER in extra_headers
58+
payload = extra_headers[PROCESSING_DATA_HEADER]
5859
assert payload == {"requested_parallel": 4}
5960

6061
@pytest.mark.asyncio
@@ -81,7 +82,7 @@ async def test_header_value_is_valid_json(self):
8182
extra_headers = _captured_extra_headers(mock_post)
8283
# Must be parseable JSON
8384
assert extra_headers is not None
84-
parsed = extra_headers["X-SM-Processing-Data"]
85+
parsed = extra_headers[PROCESSING_DATA_HEADER]
8586
assert parsed["requested_parallel"] == 8
8687

8788
@pytest.mark.asyncio
@@ -95,7 +96,7 @@ async def test_requested_parallel_one(self):
9596

9697
extra_headers = _captured_extra_headers(mock_post)
9798
assert extra_headers is not None
98-
payload = extra_headers["X-SM-Processing-Data"]
99+
payload = extra_headers[PROCESSING_DATA_HEADER]
99100
assert payload["requested_parallel"] == 1
100101

101102
@pytest.mark.asyncio
@@ -120,7 +121,7 @@ async def test_header_sent_with_fetch_data_config(self):
120121

121122
extra_headers = _captured_extra_headers(mock_post)
122123
assert extra_headers is not None
123-
payload = extra_headers["X-SM-Processing-Data"]
124+
payload = extra_headers[PROCESSING_DATA_HEADER]
124125
assert payload == {"requested_parallel": 2}
125126

126127

0 commit comments

Comments
 (0)