Skip to content

[API tests] Ensure vendor-payment journal number series - #11228

Draft
Prangshuman Das (t-prda) wants to merge 9 commits into
prdas/646383-split-quote-datesfrom
prdas/646383-split-journal-numbering
Draft

[API tests] Ensure vendor-payment journal number series#11228
Prangshuman Das (t-prda) wants to merge 9 commits into
prdas/646383-split-quote-datesfrom
prdas/646383-split-journal-numbering

Conversation

@t-prda

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

Copy link
Copy Markdown
Contributor

Scope

Assign a generated number series only when the test-created vendor-payment journal batch lacks one. Keep production numbering behavior unchanged.

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

Evidence and limits

Two observed CZ failures in saved result set 2: CU139843.TestCreateVendorPaymentWithoutDocNo and CU139867.TestApplyVendorEntryAppliesPayment. There are 18 direct test callers of the shared helper, but only these two failures are evidenced by those saved results.

Native GitHub stack #11232

This draft is stacked above #11227. #10085 is the bottom API-authentication fix. The review diff contains only this layer's fix plus removal of its 2 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

  • 139843.TestCreateVendorPaymentWithoutDocNo
  • 139867.TestApplyVendorEntryAppliesPayment

Split from the preserved PR10085 snapshot. AB#646383

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Preserve published branch history and re-enable only the 2 methods covered by this layer.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 3952f078-a881-4da8-ad96-13b727e48a91
@t-prda
Prangshuman Das (t-prda) changed the base branch from main to prdas/646383-split-quote-dates September 8, 2026 17:06
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

Team: Integrations GitHub request for Integrations area

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant