Skip to content

Commit 053e18c

Browse files
[pre-commit.ci] auto fixes from pre-commit.com hooks
for more information, see https://pre-commit.ci
1 parent 50b4ef2 commit 053e18c

File tree

3 files changed

+20
-12
lines changed

3 files changed

+20
-12
lines changed

src/common/core/middleware.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@ def __call__(self, request: HttpRequest) -> HttpResponse:
2020
response = self.get_response(request)
2121
response.headers["X-Flagsmith-Version"] = self.get_version() or "unknown"
2222
return response
23-
23+
2424
def get_version(self) -> str | None:
2525
"""Obtains the version number from the manifest file"""
2626
manifest_versions = get_versions_from_manifest()

src/common/core/utils.py

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -17,9 +17,12 @@ class SelfHostedData(TypedDict):
1717
has_logins: bool
1818

1919

20-
VersionManifest = TypedDict("VersionManifest", {
21-
".": str, # This key is used to store the version of the package itself
22-
})
20+
VersionManifest = TypedDict(
21+
"VersionManifest",
22+
{
23+
".": str, # This key is used to store the version of the package itself
24+
},
25+
)
2326

2427

2528
class VersionInfo(TypedDict):
@@ -92,7 +95,7 @@ def get_versions_from_manifest() -> VersionManifest:
9295
raw_content = get_file_contents(VERSIONS_INFO_FILE_LOCATION)
9396
if not raw_content:
9497
return {".": UNKNOWN}
95-
98+
9699
manifest: VersionManifest = json.loads(raw_content)
97100
return manifest
98101

tests/unit/common/core/test_middleware.py

Lines changed: 12 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,8 @@ def test_APIResponseVersionHeaderMiddleware__valid_version_info___adds_version_h
1313
get_response = mocker.Mock(return_value=response)
1414
middleware = middleware_module.APIResponseVersionHeaderMiddleware(get_response)
1515
get_versions_from_manifest = mocker.patch.object(
16-
middleware_module, "get_versions_from_manifest",
16+
middleware_module,
17+
"get_versions_from_manifest",
1718
return_value={".": "v1.2.3"},
1819
)
1920

@@ -27,10 +28,13 @@ def test_APIResponseVersionHeaderMiddleware__valid_version_info___adds_version_h
2728
assert get_versions_from_manifest.call_args_list == [mocker.call()]
2829

2930

30-
@pytest.mark.parametrize("version_info", [
31-
{},
32-
None,
33-
])
31+
@pytest.mark.parametrize(
32+
"version_info",
33+
[
34+
{},
35+
None,
36+
],
37+
)
3438
def test_APIResponseVersionHeaderMiddleware__invalid_version_info___adds_unknown_header(
3539
mocker: MockerFixture,
3640
version_info: dict[str, str] | None,
@@ -41,7 +45,8 @@ def test_APIResponseVersionHeaderMiddleware__invalid_version_info___adds_unknown
4145
get_response = mocker.Mock(return_value=response)
4246
middleware = middleware_module.APIResponseVersionHeaderMiddleware(get_response)
4347
get_versions_from_manifest = mocker.patch.object(
44-
middleware_module, "get_versions_from_manifest",
48+
middleware_module,
49+
"get_versions_from_manifest",
4550
return_value=version_info,
4651
)
4752

@@ -52,4 +57,4 @@ def test_APIResponseVersionHeaderMiddleware__invalid_version_info___adds_unknown
5257
assert result == response
5358
assert response.headers["X-Flagsmith-Version"] == "unknown"
5459
assert get_response.call_args_list == [mocker.call(request)]
55-
assert get_versions_from_manifest.call_args_list == [mocker.call()]
60+
assert get_versions_from_manifest.call_args_list == [mocker.call()]

0 commit comments

Comments
 (0)