Skip to content

tests: declare class:Collection / collection_type (rnaseq-sr)#1295

Merged
mvdbeek merged 1 commit into
galaxyproject:mainfrom
jmchilton:schema/tests-rnaseq-sr
Jul 15, 2026
Merged

tests: declare class:Collection / collection_type (rnaseq-sr)#1295
mvdbeek merged 1 commit into
galaxyproject:mainfrom
jmchilton:schema/tests-rnaseq-sr

Conversation

@jmchilton

Copy link
Copy Markdown
Member

Final piece of the #1286 split. Same mechanical change as #1290/#1291/#1292: class: Collection on outputs that already carried element_tests:, and type: -> collection_type: on nested collection elements. No .ga files or expected data touched.

Why this is standalone

rnaseq-sr was held back from #1292 because it errors in CI with:

Workflow was not invoked; the following required tools are not installed:
.../featurecounts/featurecounts/2.1.1+galaxy1

That is not a tool or workflow problem — it is the CI harness bug filed as #1294. The shed-installed data-table config does not survive Galaxy restarts, so the second and later consumer of featurecounts within a chunk cannot load it. rnaseq-pe runs first, installs featurecounts, and passes; rnaseq-sr inherits it and fails.

As the only featurecounts consumer in this PR's CI run, rnaseq-sr installs the tool fresh and registers the data tables — so it should pass here. (rnaseq-pe went out separately in #1292.)

This PR therefore doubles as the empirical check on #1294's diagnosis: if it goes green, the harness bug is confirmed and rnaseq-sr itself is fine.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@mvdbeek
mvdbeek merged commit bd55374 into galaxyproject:main Jul 15, 2026
8 checks passed
@mvdbeek

mvdbeek commented Jul 15, 2026

Copy link
Copy Markdown
Member

Attention: deployment skipped!

https://github.com/galaxyproject/iwc/actions/runs/29392418499

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.

2 participants