Skip to content

[API tests] Initialize cancellation reason-code fixtures - #11225

Draft
Prangshuman Das (t-prda) wants to merge 13 commits into
prdas/646383-split-vatfrom
prdas/646383-split-cancellation-reasons
Draft

[API tests] Initialize cancellation reason-code fixtures#11225
Prangshuman Das (t-prda) wants to merge 13 commits into
prdas/646383-split-vatfrom
prdas/646383-split-cancellation-reasons

Conversation

@t-prda

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

Copy link
Copy Markdown
Contributor

Scope

Ensure a reason code exists before corrective-credit-memo and invoice-cancellation scenarios. Keep changes in four API test codeunits; no production cancellation logic is changed.

AB#646383 — follow-up split from PR #10085; link only, not an instruction to resolve the umbrella work item.

Evidence and limits

Four corrective-credit-memo methods in CUs139728/139828 failed in AU in saved result sets 2 and 4. Six invoice methods additionally reach the changed fixture; those are dependency coverage, not observed failures.

Native GitHub stack #11232

This draft is stacked above #11224. #10085 is the bottom API-authentication fix. The review diff contains only this layer's fix plus removal of its 10 temporary method exclusions. Earlier layers are inherited, not repeated in this diff.

Existing PR/branch history is preserved; no force-push or merge into main was performed. NAV selection/company/native exclusions and existing out-of-scope Expense exclusions remain untouched.

Validation

  • Parent-tip ancestry and exact layer file scope verified; git diff whitespace checks and exclusion JSON validation passed.
  • Only this layer's listed temporary exclusions are removed; existing whole-codeunit exclusions are retained.
  • Full runtime validation of this stacked head is pending. Historical failures are issue evidence, not proof this head passes.
  • Repository CI currently filters PR base branches to main, releases/* and features/*; retargeted personal-branch layers do not automatically trigger that workflow on subsequent pushes. No workflow-policy expansion was made.

Methods re-enabled by this layer

  • 139728.TestActionCancelCorrectiveCreditMemo
  • 139728.TestActionCancelAndSendCorrectiveCreditMemo
  • 139828.TestActionCancelCorrectiveCreditMemo
  • 139828.TestActionCancelAndSendCorrectiveCreditMemo
  • 139709.TestActionCancelInvoice
  • 139709.TestActionCancelAndSendInvoice
  • 139709.TestActionSendCancelledInvoice
  • 139809.TestActionCancelInvoice
  • 139809.TestActionCancelAndSendInvoice
  • 139809.TestActionSendCancelledInvoice

Split from the preserved PR10085 snapshot. 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 8, 2026
@github-actions github-actions Bot added this to the Version 30.0 milestone Sep 8, 2026
@github-actions github-actions Bot added the Team: Integrations GitHub request for Integrations area label Sep 8, 2026
Preserve published branch history and re-enable only the 10 methods covered by this layer.

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
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
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
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