fix(b2-workflow): broadcast module-target to every module instead of positional mapping

fix #26

Author: alandefreitas

b2-workflow/dist/index.js

@@ -156,7 +156,7 @@ test('passes through explicit module paths untouched', async () => {
     expect(buildArgs).toContain('libs/math/example');
 });
 
-// Multiple targets should map positionally to the module list with the last reused as needed.
+// Multiple targets should be broadcast to every module.
 test('applies per-module targets when multiple values are provided', async () => {
     const inputs = createInputs({
         modules: ['filesystem', 'chrono'],
@@ -165,9 +165,23 @@ test('applies per-module targets when multiple values are provided', async () =>
     await main(inputs);
     const buildArgs = exec.getExecOutput.mock.calls[2][1];
     expect(buildArgs).toContain('libs/filesystem/test');
+    expect(buildArgs).toContain('libs/filesystem/example');
+    expect(buildArgs).toContain('libs/chrono/test');
     expect(buildArgs).toContain('libs/chrono/example');
 });
 
+// Multiple targets for a single module should all appear in the B2 arguments.
+test('broadcasts all targets to a single module', async () => {
+    const inputs = createInputs({
+        modules: ['beast2'],
+        module_target: ['test', 'example']
+    });
+    await main(inputs);
+    const buildArgs = exec.getExecOutput.mock.calls[2][1];
+    expect(buildArgs).toContain('libs/beast2/test');
+    expect(buildArgs).toContain('libs/beast2/example');
+});
+
 test('derives address model and architecture from arch input when unspecified', async () => {
     const inputs = createInputs({
         arch: 'arm64'

b2-workflow/dist/index.js.map

@@ -287,8 +287,7 @@ export async function main(inputs: Inputs): Promise<void> {
     if (moduleTargets.length === 0) {
         moduleTargets = ['test'];
     }
-    for (let index = 0; index < inputs.modules.length; ++index) {
-        const moduleEntry = inputs.modules[index];
+    for (const moduleEntry of inputs.modules) {
         const module = moduleEntry && moduleEntry.trim ? moduleEntry.trim() : moduleEntry;
         if (!module) {
             continue;
@@ -297,8 +296,9 @@ export async function main(inputs: Inputs): Promise<void> {
         if (hasExplicitTarget) {
             b2_args.push(module);
         } else {
-            const target = moduleTargets[Math.min(index, moduleTargets.length - 1)];
-            b2_args.push(`libs/${module}/${target}`);
+            for (const target of moduleTargets) {
+                b2_args.push(`libs/${module}/${target}`);
+            }
         }
     }
 

b2-workflow/src/index.test.ts

@@ -174,7 +174,7 @@ When this input is left as the default (\`test\`), every module listed above con
 \`libs/filesystem/example\` or \`libs/filesystem//unit_tests\`), the action forwards it verbatim without prefixing it.
 
 This input accepts a single value or a list separated by spaces, commas, or new lines. When multiple values are
-provided they are applied positionally to the module list, and the final value is reused for any remaining modules.`
+provided, every target is applied to every module.`
     },
 
     extra_args: {