Display samples for workflow execution (ENHC0010030)

[joshsadam]

What does this PR do and why?

Describe in detail what your merge request does and why.

Screenshots or screen recordings

Screenshots are required for UI changes, and strongly recommended for all other pull requests.

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

1. Create a new project
2. Add a sample to the project and upload sequence files to the sample.
3. Go back to your new project samples page and select the new sample and launch a pipeline for it.
4. Go to workflow executions and select the new one.
5. Select the tab for Samplesheet
6. Files should be displayed as above.

PR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

• I have evaluated the PR acceptance checklist for this PR.

[deepsidhu85]

This is looking good. Other than the update required to _samplesheet.html.erb to only output the puid and name of the attachment if it was provided in the samplesheet when the pipeline was ran, just one comment below

[JeffreyThiessen]

Overall it looks good. Just one edge case issue.

After launching a workflow execution with 0 samples (not sure if this is intended in the first place), when clicking on the samplesheet tab, the application throws a error to the user because of an undefined method.

[github-actions]

Simplecov Report

Covered	Threshold
92.4%	90%

[deepsidhu85]

Looks good to me!

[JeffreyThiessen]

👍