1berry/docker-cli
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #3792 from thaJeztah/compose_tests

Browse Source 
cli/compose: remove redundant reflection from tests
This commit is contained in:
Sebastiaan van Stijn 2022-09-29 16:15:21 +02:00 committed by GitHub
commit ef80de39ab
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 9 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
cli/compose/loader/loader_test.go
View File

@ -3,7 +3,6 @@ package loader
import (
"bytes"
"os"
"reflect"
"sort"
"testing"
"time"
@ -792,8 +791,10 @@ services:
// Default behavior keeps the `env_file` entries
configWithEnvFiles, err := Load(configDetails)
assert.NilError(t, err)
assert.DeepEqual(t, configWithEnvFiles.Services[0].EnvFile, types.StringList{"example1.env",
"example2.env"})
assert.DeepEqual(t, configWithEnvFiles.Services[0].EnvFile, types.StringList{
"example1.env",
"example2.env",
})
assert.DeepEqual(t, configWithEnvFiles.Services[0].Environment, expectedEnvironmentMap)
// Custom behavior removes the `env_file` entries
@ -862,7 +863,7 @@ services:
service: foo
`)
assert.ErrorType(t, err, reflect.TypeOf(&ForbiddenPropertiesError{}))
assert.ErrorType(t, err, &ForbiddenPropertiesError{})
props := err.(*ForbiddenPropertiesError).Properties
assert.Check(t, is.Len(props, 2))
@ -1343,7 +1344,6 @@ func TestLoadVolumesWarnOnDeprecatedExternalNameVersion34(t *testing.T) {
}
assert.Check(t, is.DeepEqual(expected, volumes))
assert.Check(t, is.Contains(buf.String(), "volume.external.name is deprecated"))
}
func patchLogrus() (*bytes.Buffer, func()) {
@ -1483,7 +1483,6 @@ func TestLoadNetworksWarnOnDeprecatedExternalNameVersion35(t *testing.T) {
}
assert.Check(t, is.DeepEqual(expected, networks))
assert.Check(t, is.Contains(buf.String(), "network.external.name is deprecated"))
}
func TestLoadNetworksWarnOnDeprecatedExternalNameVersion34(t *testing.T) {
@ -1564,7 +1563,7 @@ func TestLoadInit(t *testing.T) {
booleanTrue := true
booleanFalse := false
var testcases = []struct {
testcases := []struct {
doc string
yaml string
init *bool
@ -1651,7 +1650,7 @@ services:
}
func TestTransform(t *testing.T) {
var source = []interface{}{
source := []interface{}{
"80-82:8080-8082",
"90-92:8090-8092/udp",
"85:8500",

5
cli/compose/template/template_test.go
View File

@ -2,7 +2,6 @@ package template
import (
"fmt"
"reflect"
"testing"
"gotest.tools/v3/assert"
@ -125,8 +124,8 @@ func TestMandatoryVariableErrors(t *testing.T) {
for _, tc := range testCases {
_, err := Substitute(tc.template, defaultMapping)
assert.ErrorContains(t, err, tc.expectedError)
assert.ErrorType(t, err, reflect.TypeOf(&InvalidTemplateError{}))
assert.Check(t, is.ErrorContains(err, tc.expectedError))
assert.Check(t, is.ErrorType(err, &InvalidTemplateError{}))
}
}