Enable RU tests

[spetersenms]

Enables RU tests.

AB#642528

How I validated this

• I read the full diff and it contains only changes I intended.
• I built the affected app(s) locally with no new analyzer warnings.
• I ran the change in Business Central and confirmed it behaves as expected.
• I added or updated tests for the new behavior, or explained below why none are needed.

What I tested and the outcome (required — be specific: scenarios, commands, screenshots for UI changes)

Risk & compatibility

[github-actions]

Copilot PR Review

Iteration 1 · Outcome: completed

Knowledge source: https://github.com/microsoft/BCQuality@186d8a131465475c79244d994acb872cd5c0d4bf

Findings by domain

Findings split into Knowledge-backed (cite a BCQuality article) and Agent (the agent's own judgement, no matching BCQuality rule).

Domain	Findings	Knowledge-backed	Agent	Inline	Fallback
Agent	1	0	1	0	0
Events	1	1	0	1	0

Totals: 1 knowledge-backed · 1 agent findings.

Orchestrator pre-filter (2 file(s) excluded)

• layer-disabled (knowledge) : 2 file(s)

_{Findings produced by the AL review agent v1.7.3. Reply 👎 on any inline comment to flag false positives.}

[github-actions]

$\textbf{🟠\ High\ Severity\ —\ Events}$

RUReportDownloadHandler (codeunit 147210) is a static [EventSubscriber] on File Management.OnBeforeDownloadHandler that is always bound and fires for every download in every session that loads this test app, not just the RU report-validation tests it was written for. The knowledge article's anti-pattern is exactly this shape: an always-on side effect (here, silently copying a server file to the target path and forcing IsHandled := true) implemented as a static subscriber when the behavior is inherently scoped to specific tests. Any other test in the same app that triggers a client-side download will silently hit this handler too, which can mask genuine download failures or change behavior the test did not expect. The article recommends EventSubscriberInstance = Manual with BindSubscription/UnbindSubscription scoped to the tests that need it instead of a session-wide static subscriber.

Knowledge:

• microsoft/knowledge/events/choose-static-vs-manual-subscribers-deliberately.md

_{👍 useful · ❤️ especially valuable · 👎 wrong - reply with why}