Skip to content

Commit d6a6112

Browse files
[pre-commit.ci] auto fixes from pre-commit.com hooks
for more information, see https://pre-commit.ci
1 parent 5caaa4d commit d6a6112

File tree

4 files changed

+23
-14
lines changed

4 files changed

+23
-14
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["Flagsmith-Version"] = self.get_version()
2222
return response
23-
23+
2424
def get_version(self) -> str:
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: 14 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,8 @@
11
import pytest
22
from pytest_mock import MockerFixture
33

4-
from common.core import middleware as middleware_module, utils
4+
from common.core import middleware as middleware_module
5+
from common.core import utils
56

67

78
@pytest.fixture(autouse=True)
@@ -19,7 +20,8 @@ def test_APIResponseVersionHeaderMiddleware__valid_version_info___adds_version_h
1920
get_response = mocker.Mock(return_value=response)
2021
middleware = middleware_module.APIResponseVersionHeaderMiddleware(get_response)
2122
get_versions_from_manifest = mocker.patch.object(
22-
middleware_module, "get_versions_from_manifest",
23+
middleware_module,
24+
"get_versions_from_manifest",
2325
return_value={".": "v1.2.3"},
2426
)
2527

@@ -33,10 +35,13 @@ def test_APIResponseVersionHeaderMiddleware__valid_version_info___adds_version_h
3335
assert get_versions_from_manifest.call_args_list == [mocker.call()]
3436

3537

36-
@pytest.mark.parametrize("version_info", [
37-
{"foo": "bar"},
38-
{},
39-
])
38+
@pytest.mark.parametrize(
39+
"version_info",
40+
[
41+
{"foo": "bar"},
42+
{},
43+
],
44+
)
4045
def test_APIResponseVersionHeaderMiddleware__invalid_version_info___adds_unknown_header(
4146
mocker: MockerFixture,
4247
version_info: dict[str, str] | None,
@@ -47,7 +52,8 @@ def test_APIResponseVersionHeaderMiddleware__invalid_version_info___adds_unknown
4752
get_response = mocker.Mock(return_value=response)
4853
middleware = middleware_module.APIResponseVersionHeaderMiddleware(get_response)
4954
get_versions_from_manifest = mocker.patch.object(
50-
middleware_module, "get_versions_from_manifest",
55+
middleware_module,
56+
"get_versions_from_manifest",
5157
return_value=version_info,
5258
)
5359

@@ -58,4 +64,4 @@ def test_APIResponseVersionHeaderMiddleware__invalid_version_info___adds_unknown
5864
assert result == response
5965
assert response.headers["Flagsmith-Version"] == "unknown"
6066
assert get_response.call_args_list == [mocker.call(request)]
61-
assert get_versions_from_manifest.call_args_list == [mocker.call()]
67+
assert get_versions_from_manifest.call_args_list == [mocker.call()]

tests/unit/common/core/test_utils.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -91,7 +91,7 @@ def test_get_version_info_with_missing_files(fs: FakeFilesystem) -> None:
9191
"has_email_provider": False,
9292
"is_enterprise": True,
9393
"is_saas": False,
94-
'package_versions': {'.': 'unknown'},
94+
"package_versions": {".": "unknown"},
9595
"self_hosted_data": {
9696
"has_logins": False,
9797
"has_users": False,

0 commit comments

Comments
 (0)