BC Observatoryunofficial, made by waldo
code change · fixAL Developmentofficial - Microsoftreviewed - checked by Opus

#12325 [Expense tests] Re-enable reviewed APIs and consolidate Expense fixes

Expense Agent API tests are re-enabled. Eleven Expense API and helper paths move to shared authentication, 51 Expense API exclusions are removed, and nine non-API exclusions stay. Clean test codeunit execution is turned on, and 193 temporary API exclusions are added to the existing APIV1 and APIV2 manifests.

Open in Claude12325.mdPull request #12325

Pull request · merged into main (BC30) on 2026-10-06 by t-prda · 21 files (+1405 -375), 1 AL · fix · tier official · reviewed (checked by Opus)

What changed

  • Migrated 11 Expense API and helper paths to shared authentication
  • Removed 51 Expense API test exclusions; kept 9 non-API Expense exclusions
  • Enabled the existing enableCleanTestCodeunitExecution setting using existing DisabledTests selectors
  • Added 193 temporary method-specific exclusions to the existing APIV1 and APIV2 manifests, to be removed in a follow-up PR
  • Consolidated test fixes for policy URL composition, response isolation, assigned-user filtering and denied-approval handling

AL objects touched

Context

  • Work items: AB#646383, AB#653119
  • Labels: AL: Apps (W1), Build: Automation, ExpenseManagement, Team: Finance
  • Systems: development
  • From the description: “Migrate the 11 reviewed Expense API/helper paths to shared authentication and remove exactly 51 existing Expense API exclusions.”

Source: metadata of a merged pull request in microsoft/BCApps (bcapps): title, labels, changed paths and counts, with a summary in our words. No source text and no description beyond one short quote (D10, D61).