From d5457d6d0370d92455e1c70570e0da41b911c8f6 Mon Sep 17 00:00:00 2001 From: Guillaume de Rouville Date: Mon, 28 Jun 2021 19:56:21 +0200 Subject: [PATCH] Skip Docker.#Local in bats tests (not working) + fix stdout redirection issue leading to warning loglevel and tests not failing on missing inputs Signed-off-by: Guillaume de Rouville --- cmd/dagger/cmd/up.go | 2 +- stdlib/universe.bats | 3 ++- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/cmd/dagger/cmd/up.go b/cmd/dagger/cmd/up.go index 8916e515..8acd1210 100644 --- a/cmd/dagger/cmd/up.go +++ b/cmd/dagger/cmd/up.go @@ -54,7 +54,7 @@ var upCmd = &cobra.Command{ func checkInputs(ctx context.Context, st *state.State) { lg := log.Ctx(ctx) - warnOnly := viper.GetBool("force") || !term.IsTerminal(int(os.Stdout.Fd())) + warnOnly := viper.GetBool("force") // FIXME: find a way to merge this with the EnvironmentUp client to avoid // creating the client + solver twice diff --git a/stdlib/universe.bats b/stdlib/universe.bats index 7d51ebb1..cd12a98d 100644 --- a/stdlib/universe.bats +++ b/stdlib/universe.bats @@ -50,7 +50,8 @@ setup() { } @test "docker run: local" { - dagger -e docker-run-local up + skip "Not implemented yet + missing inputs leading to failure" + # dagger -e docker-run-local up } @test "docker build" {