Skip to content

Fix workflow e2e Step 6: trigger blur/change so Start button becomes enabled#174

Merged
vharseko merged 3 commits into
masterfrom
copilot/analyze-error-causes
May 13, 2026
Merged

Fix workflow e2e Step 6: trigger blur/change so Start button becomes enabled#174
vharseko merged 3 commits into
masterfrom
copilot/analyze-error-causes

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 12, 2026

  • Diagnose run 25741552234 (myidm/workflow): only one /policy/validateObject POST (empty body) → userName.data-validation-status stuck at errorvalidatorsManager.formValidated($form) returns false → formSubmit() bails silently
  • Replace fill() + native dispatchEvent with pressSequentially + jQuery(el).trigger('change').trigger('blur') so jQuery-bound async validators (unique, no-internal-user-conflict, valid-email-address-format) fire on every keystroke
  • After all fills, call ValidatorsManager.validateAllFields(jQuery('#contractorOnBoardForm')) via page.evaluate to re-issue the policy POST with the populated form state
  • Restore framework-driven expect(startBtn).toBeEnabled() gate (no longer hack disabled away)
  • Keep post-click toHaveCount(0) assertion to catch silent submit failures
  • node --check passes
  • Note: build-docker job failure in same run is unrelated — qemu: uncaught target signal 11 (Segmentation fault) during arm64 apt-get install libc-bin (emulator issue on the runner)

…rmSubmit gate + post-submit assertion

Agent-Logs-Url: https://github.com/OpenIdentityPlatform/OpenIDM/sessions/9e1f6b66-5fbc-476f-8d9a-4560cba1cb3b

Co-authored-by: vharseko <6818498+vharseko@users.noreply.github.com>
… validators actually fire (Step 6)

Agent-Logs-Url: https://github.com/OpenIdentityPlatform/OpenIDM/sessions/e7f2f0e5-a25b-43f4-9797-6802819260dc

Co-authored-by: vharseko <6818498+vharseko@users.noreply.github.com>
@vharseko vharseko requested review from maximthomas and removed request for vharseko May 13, 2026 08:21
@vharseko vharseko marked this pull request as ready for review May 13, 2026 08:45
@vharseko vharseko merged commit 541c8e4 into master May 13, 2026
182 checks passed
@vharseko vharseko deleted the copilot/analyze-error-causes branch May 13, 2026 08:45
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