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

#11654 Repair and re-enable Expense Agent permission and posted-history tests

Expense Agent tests were repaired and re-enabled. The fixes cover permission checks, posted-history fixtures and foreign-currency fixtures, and 14 exclusions were removed from the disabled-test manifest. The Travel Requests API page was also adjusted, and HTTP API test exclusions stay in place.

Open in Claude11654.mdPull request #11654

Pull request · merged into main (BC30) on 2026-10-01 by t-prda · 6 files (+211 -210), 1 AL · fix · tier official · reviewed (checked by Opus)

What changed

  • Fixed three permission-related tests by organizing permission observations under the correct restricted role before assertions
  • Replaced invalid posted-history fixture with a properly posted zero-amount line to preserve genuine history
  • Re-enabled Approve/Submit/Reject Travel Request page-action tests after compilation fix
  • Classified codeunit 148339 as UnitTest and repaired foreign-currency fixtures in six API tests
  • Removed 14 role/history/action test exclusions while preserving HTTP API exclusions requiring separate authentication

AL objects touched

Context

  • Work items: AB#625894, AB#650245, AB#650246, AB#650247, AB#650277, AB#650370
  • Labels: AL: Apps (W1), ExpenseManagement, Team: Finance
  • Systems: development
  • From the description: “Fix the three Expense Mgmt. Read/Edit/Admin tests by collecting all nine permission observations under the exact restricted role.”

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).