[CZ tests] Make advance-payment posting fixtures self-contained - #11370
Draft
Prangshuman Das (t-prda) wants to merge 1 commit into
Draft
[CZ tests] Make advance-payment posting fixtures self-contained#11370Prangshuman Das (t-prda) wants to merge 1 commit into
Prangshuman Das (t-prda) wants to merge 1 commit into
Conversation
Use the existing find-or-create general posting setup helper in the sales and purchase test libraries instead of requiring data left by another test suite. No production logic or isolation settings change. AB#646383 Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 3952f078-a881-4da8-ad96-13b727e48a91
Prangshuman Das (t-prda)
added a commit
that referenced
this pull request
Sep 11, 2026
Bring in PR #11370 so the advance-payment tests find or create their own posting fixture under Codeunit isolation. Preserve production logic, authentication behavior, exclusions and scheduler policy. AB#646383 Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 3952f078-a881-4da8-ad96-13b727e48a91
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
AB#646383
Make the Czech advance-payment test libraries create their own general posting fixture when a suitable one is absent.
In the purchase and sales libraries'
CreateGLAccount, replaceLibraryERM.FindGeneralPostingSetupwith the existingLibraryERM.FindGeneralPostingSetupInvtBase. That helper reuses a suitable setup or creates posting groups and configured G/L accounts through existing test-library routines.Exactly two test-library lines change. No production posting logic, test-runner isolation, test exclusions, NAV selectors, or company configuration changes.
Evidence and validation
Before this fix, fourteen existing blocked/privacy customer/vendor scenarios in CUs 148108 and 148109 failed before their business assertions because no nonblank General Posting Setup pair existed. The failure persisted in run 34569356813 after discovery was isolated from the primary fixture.
The predecessor CZ test suite creates posting setups. Source inspection shows the baseline's extra Disabled pass can leave runner 130451 selected for subsequent calls; the auth harness uses ordinary Codeunit isolation (130450), which must not depend on another suite's writes. Exact baseline CZ suite-state snapshots were not captured, so the full state transition is not claimed proved.
The fixture fix removes the unconditional dependency on existing data regardless of that runner-history explanation. The chosen find-or-create helper is already used by the inventory test library and is present in the Czech Application Test Library.
Source-equivalence and whitespace checks passed: only the two helper calls changed. All fourteen formerly failing scenarios now pass in combined auth CI, CZ Default, on head
25916aeb85. The lane still fails six separate Expense permission tests, not these advance-payment scenarios.Standalone CI passed on attempt 2 at 2026-09-11T19:28:34Z after the AU timing-test retry. That retry was not initiated by this assistant. Combined auth CI remains failed on the 12 known Expense methods, not these fourteen Czech scenarios. No local NST result is claimed.
Relationship to API authentication work
This independently reviewable test-fixture prerequisite targets main. Its commit
9b196da5f3is included in #10085 atb85fcfbac6to validate under the corrected isolation behavior. It is not folded into the authentication provider or the production application.The separately tracked CU 148339 compilation prerequisite is #11340. The auth branch includes that prerequisite; this two-line PR does not duplicate it. Coverage review recorded no Expense cases in the standalone typed artifacts, so its green workflow is not evidence that the broader auth run's Expense failures are fixed.