Skip to content

[Master]-rack failed BCApps validation (AUTODETECTED)#7582

Open
neeleshsinghal wants to merge 2 commits intomicrosoft:mainfrom
neeleshsinghal:users/v-nesinghal/fix-aging-tests-vat-setup
Open

[Master]-rack failed BCApps validation (AUTODETECTED)#7582
neeleshsinghal wants to merge 2 commits intomicrosoft:mainfrom
neeleshsinghal:users/v-nesinghal/fix-aging-tests-vat-setup

Conversation

@neeleshsinghal
Copy link
Copy Markdown
Contributor

@neeleshsinghal neeleshsinghal commented Apr 8, 2026

Workitem

Bug 630306: [master][NAV] Track failed BCApps validation (AUTODETECTED)

Fixes AB#630306

The tests AgedAccountsPayableExportsDocumentTypeAndNo and
AgedAccountsRecExportsDocumentTypeAndNo failed in IT localization
with error 'The VAT Business Posting Group table is empty'.

Fix: Ensure VAT posting setup exists before creating vendor/customer
in the test, so LibraryPurchase.CreateVendor and LibrarySales.CreateCustomer
can assign valid VAT posting groups.

Fixes AB#630306
@neeleshsinghal neeleshsinghal requested a review from a team as a code owner April 8, 2026 10:04
@github-actions github-actions bot added AL: Apps (W1) Add-on apps for W1 From Fork Pull request is coming from a fork Linked Issue is linked to a Azure Boards work item labels Apr 8, 2026
@github-actions github-actions bot added this to the Version 29.0 milestone Apr 8, 2026
…alizations

The previous fix using LibraryERM.CreateVATPostingSetupWithAccounts still failed in some localizations like FR because LibraryPurchase.CreateVendor and LibrarySales.CreateCustomer have complex VAT setup dependencies. Since these tests only need a vendor/customer record to exist (not full VAT functionality), we now create minimal records directly using Init/Insert, which works in all localizations.
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 From Fork Pull request is coming from a fork Linked Issue is linked to a Azure Boards work item

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant