Skip to content

[API tests] Align sales quote document and posting dates - #11227

Draft
Prangshuman Das (t-prda) wants to merge 13 commits into
prdas/646383-split-italian-discountfrom
prdas/646383-split-quote-dates
Draft

[API tests] Align sales quote document and posting dates#11227
Prangshuman Das (t-prda) wants to merge 13 commits into
prdas/646383-split-italian-discountfrom
prdas/646383-split-quote-dates

Conversation

@t-prda

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

Copy link
Copy Markdown
Contributor

Scope

Validate Posting Date and Document Date against WorkDate in the shared V1/V2 sales-quote fixtures. This is separate from the license-safe WorkDate helper retained by the authentication PR.

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

Evidence and limits

Two observed IT failures in saved result set 2: CU139723/139823.TestActionMakeOrderFromQuote, with Document Date greater than Posting Date. The helper has five test callers per codeunit; do not equate those ten callers with ten observed failures.

Native GitHub stack #11232

This draft is stacked above #11226. #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

  • 139723.TestActionMakeOrderFromQuote
  • 139823.TestActionMakeOrderFromQuote

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