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.

Sign up for GitHub

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

A couple extra tests would be ideal to add, to ensure existing behavior #2

Closed

clintval opened this issue Nov 9, 2023 · 0 comments · Fixed by #6

Labels

good first issue

Member

clintval commented Nov 9, 2023

nf-dotenv/plugins/nf-dotenv/src/test/nextflow/dotenv/DotenvTest.groovy

Lines 199 to 200 in 66dec8f

 // TODO: Need a test that tests for default value return from config 

 // TODO: Need a test for environment variable override

clintval added the good first issue label

clintval mentioned this issue

Round out all features and ensure tests work too #6

Merged

clintval closed this as completed in #6

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment