Skip to content

Commit

Permalink
Fix integration test with invalid configuration (#1182)
Browse files Browse the repository at this point in the history
## Changes

The indentation mistake on the `path` field under `notebook` meant the
pipeline had a single entry with a `nil` notebook field. This was
allowed but incorrect.

While working on the `dyn.Value` approach, this yielded a non-nil but
zeroed `notebook` field and a failure to translate an empty path.

## Tests

Correcting the indentation made the test fail because the file is not a
notebook. I changed it to a `file` reference and the test now passes.
  • Loading branch information
pietern authored Feb 7, 2024
1 parent de363fa commit b64e113
Showing 1 changed file with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -3,5 +3,5 @@ resources:
bar:
name: test-bundle-pipeline-{{.unique_id}}
libraries:
- notebook:
path: "./foo.py"
- file:
path: "./foo.py"

0 comments on commit b64e113

Please sign in to comment.