Skip to content

test(W-23836447): add non-DPoP pool server login test to ECALoginTests - #3002

Merged
wmathurin merged 2 commits into
forcedotcom:devfrom
wmathurin:dpop-jkt-pool-servers
Aug 21, 2026
Merged

test(W-23836447): add non-DPoP pool server login test to ECALoginTests#3002
wmathurin merged 2 commits into
forcedotcom:devfrom
wmathurin:dpop-jkt-pool-servers

Conversation

@wmathurin

Copy link
Copy Markdown
Contributor

Summary

  • Adds testECAJwt_ViaLoginPoolServer to ECALoginTests — verifies that a plain ECA JWT login through the pool server produces a valid session (non-DPoP path)
  • Updates AuthFlowTester/README.md: adds the new test to the ECALoginTests table, adds a Notes column, and updates the section description to mention pool server coverage

Note: This commit should have been included in the main dpop-jkt-pool-servers PR. Opening as a separate PR to avoid rebasing the already-reviewed branch.

Test plan

  • AuthFlowTester: testECAJwt_ViaLoginPoolServer passes against login.test1.pc-rnd.salesforce.com

… README

Pool server non-DPoP test was missing from the ECALoginTests table. Also adds
a Notes column and updates the section description to mention pool server coverage.
…ost is provisioned

The @ignore reason was stale — loginPoolHost is already set in ui_test_config.json.
Also cleans up the stale comment block.

@sfdctaka sfdctaka left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@wmathurin
wmathurin merged commit 0e88896 into forcedotcom:dev Aug 21, 2026
8 of 10 checks passed

@brandonpage brandonpage left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

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.

3 participants