feat: Add recursiveFetchLogicalFiles

[GordonSmith]

Fixes #4392

Checklist:

• The commit message is properly formatted and free of typos.
  • The commit message title makes sense in a changelog, by itself.
  • The commit message includes a "fixes" reference if appropriate.
  • The commit is signed.
• The change has been fully tested:
  • I have viewed all related gallery items
  • I have viewed all related dermatology items
• There are no similar instances of the same problem that should be addressed
  • I have addressed them here
  • I have raised new issues to address them separately

Testing:

[Copilot]

Pull Request Overview

This PR introduces a new recursiveFetchLogicalFiles method to enhance the DFUService by recursively retrieving logical file information. Key changes include:

• Adding the recursiveFetchLogicalFiles function in DFUService for fetching logical files recursively.
• Extending the LogicalFile class with a fetchAllLogicalFiles method that utilizes the new recursive function.
• Including a new (currently skipped) test case to validate the recursive fetching behavior.

Reviewed Changes

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

File	Description
packages/comms/tests/wsDFU.spec.ts	Added a skipped test for recursive fetching of logical files.
packages/comms/src/services/wsDFU.ts	Introduced recursiveFetchLogicalFiles to recursively fetch logical files.
packages/comms/src/ecl/logicalFile.ts	Added a convenience method that wraps the new recursive fetching logic.

Comments suppressed due to low confidence (3)

packages/comms/src/services/wsDFU.ts:18

• The recursiveFetchLogicalFiles method expects objects with 'NodeGroup' and 'Name' properties, but the recursive call uses 'childSuperFiles', whose type may not fully align. Consider unifying the data types or mapping the LogicalFile instance properly before recursion.

    async recursiveFetchLogicalFiles(superFiles: { NodeGroup: string, Name: string }[]): Promise<string[]> {

packages/comms/src/ecl/logicalFile.ts:180

• Passing 'this' (a LogicalFile instance) into recursiveFetchLogicalFiles may lead to a type mismatch if LogicalFile doesn't exactly match { NodeGroup: string, Name: string }. Consider mapping 'this' to the expected shape.

        return this.connection.recursiveFetchLogicalFiles([this]);

packages/comms/tests/wsDFU.spec.ts:24

• [nitpick] The test for recursiveFetch is currently skipped; please re-enable it once the functionality is stable to ensure comprehensive test coverage for the new recursive fetching behavior.

    it.skip("recursiveFetch", async function () {

[GordonSmith]

@jeclrsg can you review

[jeclrsg]

@GordonSmith asked about the new test being marked as to be skipped