[armanist]

Closes #606. Part of #595. The changelog entry is written in a final pull request, after the sub-issues.

What changed

• install:openapi creates modules/<Name>/resources before resources/openapi when the module has no resources folder, for example one generated from the DefaultApi template.
• copyResources() creates the public/assets/OpenApiUi folder before copying the Swagger UI files into it.
• testGenerateOpenapiSpecificationCreatesSpecFile used an @OA docblock fixture and only checked that spec.json was not empty, so it passed with an empty specification now that doctrine/annotations is gone. Its fixture is an attribute and it checks the title.

Why

Both folders were assumed to exist. Without them the command failed with mkdir(): No such file or directory and copy(...): Failed to open stream.

LocalFileSystemAdapter::makeDirectory() is not made recursive, because other callers depend on it.

Breaking changes

• None.

Testing

• New: testCopyResourcesCreatesTheTargetFolderWhenItIsMissing and testExecCreatesTheResourcesFoldersOfAModuleWithoutThem (runs the whole command on a module without resources). Both fail without the fix.
• Changed: testGenerateOpenapiSpecificationCreatesSpecFile now asserts the specification title.
• Known gap: the fix was not re-run end to end in a skeleton project; the two new tests cover each missing folder.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!