From 3dce56ab0df30e1b9ce69a6f49ca1bcfe3ca68f1 Mon Sep 17 00:00:00 2001 From: Lynn Cyrin Date: Wed, 23 Oct 2019 22:26:34 -0700 Subject: [PATCH] remove v1 regression test --- app_regression_test.go | 59 ------------------------------------------ 1 file changed, 59 deletions(-) delete mode 100644 app_regression_test.go diff --git a/app_regression_test.go b/app_regression_test.go deleted file mode 100644 index 3c8681b..0000000 --- a/app_regression_test.go +++ /dev/null @@ -1,59 +0,0 @@ -package cli - -import ( - "testing" -) - -// TestRegression tests a regression that was merged between versions 1.20.0 and 1.21.0 -// The included app.Run line worked in 1.20.0, and then was broken in 1.21.0. -// Relevant PR: https://github.com/urfave/cli/pull/872 -func TestVersionOneTwoOneRegression(t *testing.T) { - testData := []struct { - testCase string - appRunInput []string - skipArgReorder bool - }{ - { - testCase: "with_dash_dash", - appRunInput: []string{"cli", "command", "--flagone", "flagvalue", "--", "docker", "image", "ls", "--no-trunc"}, - }, - { - testCase: "with_dash_dash_and_skip_reorder", - appRunInput: []string{"cli", "command", "--flagone", "flagvalue", "--", "docker", "image", "ls", "--no-trunc"}, - skipArgReorder: true, - }, - { - testCase: "without_dash_dash", - appRunInput: []string{"cli", "command", "--flagone", "flagvalue", "docker", "image", "ls", "--no-trunc"}, - }, - { - testCase: "without_dash_dash_and_skip_reorder", - appRunInput: []string{"cli", "command", "--flagone", "flagvalue", "docker", "image", "ls", "--no-trunc"}, - skipArgReorder: true, - }, - } - for _, test := range testData { - t.Run(test.testCase, func(t *testing.T) { - // setup - app := NewApp() - app.Commands = []Command{{ - Name: "command", - SkipArgReorder: test.skipArgReorder, - Flags: []Flag{ - StringFlag{ - Name: "flagone", - }, - }, - Action: func(c *Context) error { return nil }, - }} - - // logic under test - err := app.Run(test.appRunInput) - - // assertions - if err != nil { - t.Errorf("did not expected an error, but there was one: %s", err) - } - }) - } -}