Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow setting the name attribute on test collections #16654

Closed
TomHarrop opened this issue Sep 6, 2023 · 2 comments
Closed

Allow setting the name attribute on test collections #16654

TomHarrop opened this issue Sep 6, 2023 · 2 comments

Comments

@TomHarrop
Copy link
Contributor

I'd like to be able to set the element_identifier in tests.

If I do this

<test>
  <param name="paired_input">
    <collection type="paired" name="sample1">
      <element name="forward" ftype="fastqsanger.gz" value="sample1_1.fastq" />
      <element name="reverse" ftype="fastqsanger.gz" value="sample1_2.fastq" />
    </collection>
  </param>
</test>

element_identifier resolves to sample1, so this works as expected.

But linting fails with the error

.. ERROR: Invalid XML found in file: tool.xml. Errors [/tmp/tmp4dtg0hlu:245:0:ERROR:SCHEMASV:SCHEMAV_CVC_COMPLEX_TYPE_3_2_1: Element 'collection', attribute 'name': The attribute 'name' is not allowed.]

Let me know if this issue should be posted somewhere else.

Thanks!

@bernt-matthias
Copy link
Contributor

If I get galaxyproject/tools-iuc#5464 (comment) it's only the xsd that misses the name attribute.

@bernt-matthias
Copy link
Contributor

Should be easy to fix.

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

No branches or pull requests

2 participants