The asset-test and prerelease-asset-test workflows have a hard coded set of pnpm test commands. Elasticsearch-assets doesn't use these reusable workflows because it has one more test suite it needs to run. And now we are adding integration tests to standard-assets that will force us to use a custom workflow.
It should be possible to pass a list of test commands to the workflow, separated by the type of setup the test needs (e2e, integration, unit, encrypted, etc). Then the list will be used as a matrix for each job type. This would allow us to use the same reusable workflow for all of the assets again.
We should probably do the same type of thing for the prerelease tests, since only a few lines of the workflow are different.
The
asset-testandprerelease-asset-testworkflows have a hard coded set ofpnpm testcommands. Elasticsearch-assets doesn't use these reusable workflows because it has one more test suite it needs to run. And now we are adding integration tests to standard-assets that will force us to use a custom workflow.It should be possible to pass a list of test commands to the workflow, separated by the type of setup the test needs (e2e, integration, unit, encrypted, etc). Then the list will be used as a matrix for each job type. This would allow us to use the same reusable workflow for all of the assets again.
We should probably do the same type of thing for the prerelease tests, since only a few lines of the workflow are different.