Skip to content

test: tighten module run source resolution coverage#7062

Draft
edmundmiller wants to merge 1 commit intopr-7057-stack-basefrom
stacked-module-run-test-review
Draft

test: tighten module run source resolution coverage#7062
edmundmiller wants to merge 1 commit intopr-7057-stack-basefrom
stacked-module-run-test-review

Conversation

@edmundmiller
Copy link
Copy Markdown
Member

Summary

  • refactor CmdModuleRun to expose a small resolveModuleSource() seam for focused tests
  • replace brittle full-cmd.run() specs with source-resolution specs
  • add data-driven and interaction-based Spock coverage for local-vs-registry dispatch

Why

This PR is stacked on top of #7057 and focuses only on the tests we want to see for local nextflow module run behavior.

In particular, it proves:

  • local module directories resolve to main.nf
  • local module files resolve directly
  • local sources dispatch to the local resolver and bypass remote resolution
  • registry-style sources dispatch to the remote resolver and bypass local resolution
  • error cases still raise AbortOperationException

Test Plan

  • ./gradlew :nextflow:test --tests 'nextflow.cli.module.CmdModuleRunTest'

Depends on:

@edmundmiller edmundmiller marked this pull request as draft April 23, 2026 17:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant