Skip to content

test(server): handle string groups claim (#22973)#23208

Merged
crenshaw-dev merged 1 commit intoargoproj:masterfrom
crenshaw-dev:invalid-claims-test
May 29, 2025
Merged

test(server): handle string groups claim (#22973)#23208
crenshaw-dev merged 1 commit intoargoproj:masterfrom
crenshaw-dev:invalid-claims-test

Conversation

@crenshaw-dev
Copy link
Copy Markdown
Member

Should've added a test in #23202 for #22973

This should prevent regressions

Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
@crenshaw-dev crenshaw-dev requested a review from a team as a code owner May 29, 2025 19:15
@bunnyshell
Copy link
Copy Markdown

bunnyshell bot commented May 29, 2025

❌ Preview Environment deleted from Bunnyshell

Available commands (reply to this comment):

  • 🚀 /bns:deploy to deploy the environment

@crenshaw-dev crenshaw-dev enabled auto-merge (squash) May 29, 2025 19:52
@crenshaw-dev crenshaw-dev merged commit 5509dd8 into argoproj:master May 29, 2025
27 checks passed
@codecov
Copy link
Copy Markdown

codecov bot commented May 29, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
⚠️ Please upload report for BASE (master@7fda067). Learn more about missing BASE report.
⚠️ Report is 412 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff            @@
##             master   #23208   +/-   ##
=========================================
  Coverage          ?   60.05%           
=========================================
  Files             ?      342           
  Lines             ?    57845           
  Branches          ?        0           
=========================================
  Hits              ?    34736           
  Misses            ?    20338           
  Partials          ?     2771           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@crenshaw-dev crenshaw-dev deleted the invalid-claims-test branch May 29, 2025 21:08
chansuke pushed a commit to chansuke/argo-cd that referenced this pull request Jun 4, 2025
…3208)

Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
dsuhinin pushed a commit to dsuhinin/argo-cd that referenced this pull request Jun 16, 2025
…3208)

Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
Signed-off-by: dsuhinin <suhinin.dmitriy@gmail.com>
dsuhinin pushed a commit to dsuhinin/argo-cd that referenced this pull request Jun 16, 2025
…3208)

Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
Signed-off-by: dsuhinin <suhinin.dmitriy@gmail.com>
enneitex pushed a commit to enneitex/argo-cd that referenced this pull request Aug 24, 2025
…3208)

Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
Signed-off-by: enneitex <etienne.divet@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants