test: re-enable undefined prop test

Rather than checking for failure, we verify that the computable part of
the configuration gets outputted.

Signed-off-by: Andrea Luzzardi <aluzzardi@gmail.com>
This commit is contained in:
Andrea Luzzardi 2021-01-20 18:54:44 -08:00
parent 72c0851e01
commit 69cd9dd323
2 changed files with 4 additions and 3 deletions

View File

@ -1,5 +1,7 @@
package testing package testing
hello: "world"
bar: string bar: string
#dagger: { #dagger: {

View File

@ -29,9 +29,8 @@ test::compute(){
"$dagger" "${DAGGER_BINARY_ARGS[@]}" compute "$d"/compute/noop "$dagger" "${DAGGER_BINARY_ARGS[@]}" compute "$d"/compute/noop
test::one "Compute: simple should succeed" --exit=0 --stdout="{}" \ test::one "Compute: simple should succeed" --exit=0 --stdout="{}" \
"$dagger" "${DAGGER_BINARY_ARGS[@]}" compute "$d"/compute/simple "$dagger" "${DAGGER_BINARY_ARGS[@]}" compute "$d"/compute/simple
test::one "Compute: unresolved should be ignored" --exit=0 --stdout='{"hello":"world"}' \
disable test::one "Compute: unresolved should fail (FIXME https://github.com/blocklayerhq/dagger/issues/28)" --exit=1 --stdout= \ "$dagger" "${DAGGER_BINARY_ARGS[@]}" compute "$d"/compute/undefined_prop
"$dagger" "${DAGGER_BINARY_ARGS[@]}" compute "$d"/compute/invalid/undefined_prop
} }
test::fetchcontainer(){ test::fetchcontainer(){