Update Firewall manifest to declare implementsPretest

[Steve Lee (SteveL-MSFT)]

PR Summary

• Change Firewall manifest to declare implementsPretest so that when the writeOnly unspecifiedRules property is used, set is still called
• Fix a formatting issue in invoke_synthetic_test() funciton I noticed while exploring different options

PR Context

Fix #1681

[Copilot]

Pull request overview

This pull request updates the Windows Firewall resource manifest so the engine will not run an engine-side pretest (test) gate before invoking set, ensuring set still runs when unspecifiedRules (a writeOnly input) is present and can cause changes (fixes #1681).

Changes:

• Set implementsPretest: true in the Windows Firewall resource manifest so config set isn’t skipped due to the pretest result when writeOnly inputs are involved.
• Reformat invoke_synthetic_test() in the command resource implementation (no functional change observed).

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File	Description
resources/windows_firewall/windows_firewall.dsc.resource.json	Declares implementsPretest: true for the FirewallRuleList resource set method to prevent engine-side pretest from skipping set.
lib/dsc-lib/src/dscresources/command_resource.rs	Formats invoke_synthetic_test() onto multiple lines for readability (behavior unchanged).

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.