Skip to content

[API tests] Separate APIV1 fixture and verification reliability fixes - #11322

Draft
Prangshuman Das (t-prda) wants to merge 4 commits into
prdas/646383-split-return-shipment-pdffrom
prdas/646383-split-apiv1-reliability
Draft

[API tests] Separate APIV1 fixture and verification reliability fixes#11322
Prangshuman Das (t-prda) wants to merge 4 commits into
prdas/646383-split-return-shipment-pdffrom
prdas/646383-split-apiv1-reliability

Conversation

@t-prda

@t-prda Prangshuman Das (t-prda) commented Sep 10, 2026

Copy link
Copy Markdown
Contributor

Scope

AB#646383

Draft follow-up to #11230, based on prdas/646383-split-return-shipment-pdf.

  • Identify five posted blank API lines by GUID description and exact record lookup.
  • Regenerate RapidStart fixtures per invocation; refresh all three polling loops, correct expected/actual assertion order, and fix the apply-status qualifier.
  • Keep G/L journal-template handler state in a codeunit global.
  • Enable Foundation setup for the invoice discount-percent scenario.
  • Re-enable exactly ten APIV1 methods temporarily quarantined while these business fixes were separated. These were conservative/deferred exclusions, not newly proven failures. RS create/upload were not additionally quarantined.

Lifetime and preservation

Automation initialization keeps workdate first, followed by idempotent authentication-provider selection, per-test cleanup, and sample generation. Removing the guard makes provider selection per invocation; its existing same-provider early return remains intact.

Preserves all seven earlier layers, APIV2 wildcards, and all 22 Expense exclusion entries. Only eight AL files and the APIV1 exclusion manifest change.

Validation

  • Static context-patch and reviewed-method equivalence checks passed.
  • Exact nine-file scope, protected-manifest byte equality, and git diff --check passed.
  • This stacked head remains runtime untested. Prerequisite [Expense Agent tests] Fix action-result assertions for runtime compilation #11340 has resolved the CU 148339 compilation blocker; that success does not validate this layer. The RapidStart polling tests require background task execution and a compatible validation profile. No local NST was used.

Linked to native GitHub stack #11232: #10085 is the bottom auth PR, followed by #11224 through #11230, then this draft as the top layer. All eight follow-ups are drafts. No branch was force-pushed and no PR was merged. The first seven layers restore 37 deferred methods; this layer restores the remaining ten, preserving pre-existing exclusions.

AB#646383

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@github-actions github-actions Bot added the AL: Apps (W1) Add-on apps for W1 label Sep 10, 2026
@t-prda
Prangshuman Das (t-prda) added this pull request to stack #11232 September 10, 2026 11:43
@github-actions github-actions Bot added the Team: Integrations GitHub request for Integrations area label Sep 10, 2026
@github-actions github-actions Bot added this to the Version 30.0 milestone Sep 10, 2026
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 3952f078-a881-4da8-ad96-13b727e48a91
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 3952f078-a881-4da8-ad96-13b727e48a91
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 3952f078-a881-4da8-ad96-13b727e48a91
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

AL: Apps (W1) Add-on apps for W1 Team: Integrations GitHub request for Integrations area

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant