nebius/SWE-rebench-V2-PRs huggingface dataset:
Repository: CTFd/CTFd
Pull request #2583 (2024-08-17 12:14:02)

Fix typo in unlocks API
Small typo fix in unlocks API endpoint

codecov[bot]: ## [Codecov](https://app.codecov.io/gh/CTFd/CTFd/pull/2583?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=CTFd) Report
All modified and coverable lines are covered by tests :white_check_mark:
> Project coverage is 85.66%. Comparing base [(`162bca6`)](https://app.codecov.io/gh/CTFd/CTFd/commit/162bca6649d1cd17d6c4fb930010e53b183cd46d?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=CTFd) to head [(`1c4229d`)](https://app.codecov.io/gh/CTFd/CTFd/commit/1c4229dbc22f18647bb597f7c14611a85ac86127?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=CTFd).
> Report is 31 commits behind head on master.

<details><summary>Additional details and impacted files</summary>


```diff
@@            Coverage Diff             @@
##           master    #2583      +/-   ##
==========================================
- Coverage   87.57%   85.66%   -1.92%     
==========================================
  Files         155      154       -1     
  Lines        9650     9681      +31     
==========================================
- Hits         8451     8293     -158     
- Misses       1199     1388     +189     
```

</details>

[:umbrella: View full report in Codecov by Sentry](https://app.codecov.io/gh/CTFd/CTFd/pull/2583?dropdown=coverage&src=pr&el=continue&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=CTFd).   
:loudspeaker: Have feedback on the report? [Share it here](https://about.codecov.io/codecov-pr-comment-feedback/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=CTFd).


Patch:
diff --git a/CTFd/api/v1/unlocks.py b/CTFd/api/v1/unlocks.py
index 7927691e..05666675 100644
--- a/CTFd/api/v1/unlocks.py
+++ b/CTFd/api/v1/unlocks.py
@@ -138,7 +138,7 @@ class UnlockList(Resource):
             return (
                 {
                     "success": False,
-                    "errors": {"target": "You've already unlocked this this target"},
+                    "errors": {"target": "You've already unlocked this target"},
                 },
                 400,
             )


Test patch:
diff --git a/tests/teams/test_hints.py b/tests/teams/test_hints.py
index ac68a315..a9d1b2b9 100644
--- a/tests/teams/test_hints.py
+++ b/tests/teams/test_hints.py
@@ -54,7 +54,7 @@ def test_hint_team_unlock():
             assert r.status_code == 400
             assert (
                 r.get_json()["errors"]["target"]
-                == "You've already unlocked this this target"
+                == "You've already unlocked this target"
             )
 
             # Assert that we see a hint
Repository: argoproj-labs/argocd-operator
Pull request #740 (2022-07-25 09:48:56)

fix: imagePullPolicy in dex deployment
**What type of PR is this?**
 /kind bug

**What does this PR do / why we need it**:

Fixes https://github.com/redhat-developer/gitops-operator/issues/132
Removing the `ImagePullPolicy` in dex deployment
The default policy of `IfNotPresent` property will be used in the deployment.

**Have you updated the necessary documentation?**

* [ ] Documentation update is required by this PR.
* [ ] Documentation has been updated.

**Which issue(s) this PR fixes**:

Fixes #?

**How to test changes / Special notes to the reviewer**:


iam-veeramalla: The following test failed.
```
    logger.go:42: 18:36:56 | 1-021_validate_notification_controller/3-disable_notification | test step failed 3-disable_notification
    case.go:254: failed in step 3-disable_notification
    case.go:256: resource matched of kind: apps/v1, Kind=Deployment
```
iam-veeramalla: @saumeya unfortunately the log captured by the e2e tests are not significant enough to identify the root cause for the failure. 
Can you run the below test on you setup to understand the issue ?
`kubectl kuttl test ./tests/k8s --config ./tests/kuttl-tests.yaml --test 1-021_validate_notification_controller`

Patch:
diff --git a/controllers/argocd/dex.go b/controllers/argocd/dex.go
index 74347ef6..a93a9388 100644
--- a/controllers/argocd/dex.go
+++ b/controllers/argocd/dex.go
@@ -229,10 +229,9 @@ func (r *ReconcileArgoCD) reconcileDexDeployment(cr *argoprojv1a1.ArgoCD) error
 			"/shared/argocd-dex",
 			"rundex",
 		},
-		Image:           getDexContainerImage(cr),
-		ImagePullPolicy: corev1.PullAlways,
-		Name:            "dex",
-		Env:             proxyEnvVars(),
+		Image: getDexContainerImage(cr),
+		Name:  "dex",
+		Env:   proxyEnvVars(),
 		LivenessProbe: &corev1.Probe{
 			ProbeHandler: corev1.ProbeHandler{
 				HTTPGet: &corev1.HTTPGetAction{


Test patch:
diff --git a/controllers/argocd/dex_test.go b/controllers/argocd/dex_test.go
index 9f0fdd86..c143ed51 100644
--- a/controllers/argocd/dex_test.go
+++ b/controllers/argocd/dex_test.go
@@ -421,7 +421,6 @@ func TestReconcileArgoCD_reconcileDexDeployment(t *testing.T) {
 				},
 				VolumeMounts: []corev1.VolumeMount{
 					{Name: "static-files", MountPath: "/shared"}},
-				ImagePullPolicy: corev1.PullAlways,
 			},
 		},
 		ServiceAccountName: "argocd-argocd-dex-server",
@@ -512,7 +511,6 @@ func TestReconcileArgoCD_reconcileDexDeployment_withUpdate(t *testing.T) {
 				},
 				VolumeMounts: []corev1.VolumeMount{
 					{Name: "static-files", MountPath: "/shared"}},
-				ImagePullPolicy: corev1.PullAlways,
 			},
 		},
 		ServiceAccountName: "argocd-argocd-dex-server",