Skip to content

Commit 6a8714d

Browse files
authored
ref(preprod): Clean up param naming in check-for-updates (#101153)
Things from the client are `provided_foo` things we compute based on those are `foo`. Resolves: EME-392
1 parent fb8c393 commit 6a8714d

File tree

1 file changed

+19
-19
lines changed

1 file changed

+19
-19
lines changed

src/sentry/preprod/api/endpoints/project_preprod_check_for_updates.py

Lines changed: 19 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -58,29 +58,29 @@ def get(self, request: Request, project) -> Response:
5858
"""
5959
Check for updates for a preprod artifact
6060
"""
61-
main_binary_identifier = request.GET.get("main_binary_identifier")
62-
app_id = request.GET.get("app_id")
6361

64-
platform = request.GET.get("platform")
62+
provided_main_binary_identifier = request.GET.get("main_binary_identifier")
63+
provided_app_id = request.GET.get("app_id")
64+
provided_platform = request.GET.get("platform")
6565
provided_build_version = request.GET.get("build_version")
6666
provided_build_number = request.GET.get("build_number")
67-
provided_build_configuration_name = request.GET.get("build_configuration")
67+
provided_build_configuration = request.GET.get("build_configuration")
6868

69-
if not app_id or not platform or not provided_build_version:
69+
if not provided_app_id or not provided_platform or not provided_build_version:
7070
return Response({"error": "Missing required parameters"}, status=400)
7171

72-
if not main_binary_identifier and not provided_build_number:
72+
if not provided_main_binary_identifier and not provided_build_number:
7373
return Response(
7474
{"error": "Either main_binary_identifier or build_number must be provided"},
7575
status=400,
7676
)
7777

78-
provided_build_configuration = None
79-
if provided_build_configuration_name:
78+
build_configuration = None
79+
if provided_build_configuration:
8080
try:
81-
provided_build_configuration = PreprodBuildConfiguration.objects.get(
81+
build_configuration = PreprodBuildConfiguration.objects.get(
8282
project=project,
83-
name=provided_build_configuration_name,
83+
name=provided_build_configuration,
8484
)
8585
except PreprodBuildConfiguration.DoesNotExist:
8686
return Response({"error": "Invalid build configuration"}, status=400)
@@ -93,15 +93,15 @@ def get(self, request: Request, project) -> Response:
9393
def get_base_filters():
9494
filter_kwargs = {
9595
"project": project,
96-
"app_id": app_id,
96+
"app_id": provided_app_id,
9797
}
9898

99-
if platform == "android":
99+
if provided_platform == "android":
100100
filter_kwargs["artifact_type__in"] = [
101101
PreprodArtifact.ArtifactType.AAB,
102102
PreprodArtifact.ArtifactType.APK,
103103
]
104-
elif platform == "ios":
104+
elif provided_platform == "ios":
105105
filter_kwargs["artifact_type"] = PreprodArtifact.ArtifactType.XCARCHIVE
106106

107107
return filter_kwargs
@@ -115,8 +115,8 @@ def get_base_filters():
115115
)
116116

117117
# Add main_binary_identifier filter if provided
118-
if main_binary_identifier:
119-
current_filter_kwargs["main_binary_identifier"] = main_binary_identifier
118+
if provided_main_binary_identifier:
119+
current_filter_kwargs["main_binary_identifier"] = provided_main_binary_identifier
120120

121121
# Add build_number filter if provided
122122
if provided_build_number is not None:
@@ -125,8 +125,8 @@ def get_base_filters():
125125
except ValueError:
126126
return Response({"error": "Invalid build_number format"}, status=400)
127127

128-
if provided_build_configuration:
129-
current_filter_kwargs["build_configuration"] = provided_build_configuration
128+
if build_configuration:
129+
current_filter_kwargs["build_configuration"] = build_configuration
130130

131131
preprod_artifact = PreprodArtifact.objects.filter(**current_filter_kwargs).latest(
132132
"date_added"
@@ -156,8 +156,8 @@ def get_base_filters():
156156
new_build_filter_kwargs = get_base_filters()
157157
if preprod_artifact:
158158
new_build_filter_kwargs["build_configuration"] = preprod_artifact.build_configuration
159-
elif provided_build_configuration:
160-
new_build_filter_kwargs["build_configuration"] = provided_build_configuration
159+
elif build_configuration:
160+
new_build_filter_kwargs["build_configuration"] = build_configuration
161161
all_versions = (
162162
PreprodArtifact.objects.filter(**new_build_filter_kwargs)
163163
.values_list("build_version", flat=True)

0 commit comments

Comments
 (0)