Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Jan 21, 2026

Two syntax errors in internal/api/auth_test.go were causing linter failures:

  • Line 51: Stray closing parenthesis after mockTokenCredential.GetToken() method
  • Line 732: Missing closing brace for TestUnit_NewAuthBase_InitializesCliTokenCache() function

Both errors prevented the file from compiling and blocked all typecheck validation.


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: mawasile <50197777+mawasile@users.noreply.github.com>
@mawasile mawasile marked this pull request as ready for review January 21, 2026 08:04
@mawasile mawasile requested a review from a team as a code owner January 21, 2026 08:04
Copy link
Contributor Author

Copilot AI commented Jan 21, 2026

Just as a heads up, I was blocked by some firewall rules while working on your feedback. Expand below for details.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • login.microsoftonline.com
    • Triggering command: /tmp/go-build3879096377/b001/api.test /tmp/go-build3879096377/b001/api.test -test.testlogfile=/tmp/go-build3879096377/b001/testlog.txt -test.paniconexit0 -test.timeout=10m0s -test.run=TestUnit -test.v=true ider-power-platf-ifaceassert ux-amd64/pkg/tool/linux_amd64/compile (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

Copilot AI changed the title [WIP] Add credential caching and retrieval mechanism in auth fix: correct syntax errors in auth_test.go Jan 21, 2026
Copilot AI requested a review from mawasile January 21, 2026 08:04
@mawasile mawasile merged commit 1a2c8b0 into polatengin/711-lack-of-credential-reuse-in-gettokenforscopes Jan 21, 2026
9 of 10 checks passed
@mawasile mawasile deleted the copilot/sub-pr-1017 branch January 21, 2026 08:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants