feat(cli): added support for filtering by group in app get-resource CLI command

[sabre1041]

Resolves #25494

[bunnyshell]

❌ Preview Environment deleted from Bunnyshell

Available commands (reply to this comment):

• 🚀 /bns:deploy to deploy the environment

[nitishfy]

Left a small comment otherwise LGTM!

[codecov]

Codecov Report

❌ Patch coverage is 66.66667% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 62.58%. Comparing base (b5f75f1) to head (6ff9769).
⚠️ Report is 32 commits behind head on master.

Files with missing lines	Patch %	Lines
cmd/argocd/commands/app_resources.go	66.66%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master   #25495      +/-   ##
==========================================
+ Coverage   62.30%   62.58%   +0.28%     
==========================================
  Files         351      352       +1     
  Lines       49396    49746     +350     
==========================================
+ Hits        30775    31135     +360     
+ Misses      15684    15628      -56     
- Partials     2937     2983      +46     

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

[reggie-k]

Thanks! Can you add some tests for this?

[sabre1041]

Thanks! Can you add some tests for this?

There arent any

Thanks! Can you add some tests for this?

Since the new logic that was added is included within the cobra command, there are no existing set of tests that target any of the CLI commands, so I am hesitant to take on establishing such an implementation for a fairly minor enhancement

[reggie-k]

Since the new logic that was added is included within the cobra command, there are no existing set of tests that target any of the CLI commands, so I am hesitant to take on establishing such an implementation for a fairly minor enhancement

Yeah, for unit tests it is true, but we have a lot of existing e2e tests that invoke CLI.

[sabre1041]

Since the new logic that was added is included within the cobra command, there are no existing set of tests that target any of the CLI commands, so I am hesitant to take on establishing such an implementation for a fairly minor enhancement

Yeah, for unit tests it is true, but we have a lot of existing e2e tests that invoke CLI.

@reggie-k e2e test has been added