mirror of
https://github.com/moby/moby.git
synced 2022-11-09 12:21:53 -05:00
19f3b0715c
Extract a common function for builder.createContainer Extract imageCache for doing cache probes Removes the cacheBuested field from Builder Create a new containerManager class which reduces the interface between the builder and managing containers to 3 functions (from 6) Signed-off-by: Daniel Nephin <dnephin@docker.com>
100 lines
3.5 KiB
Go
100 lines
3.5 KiB
Go
package dockerfile
|
|
|
|
import (
|
|
"bytes"
|
|
"testing"
|
|
|
|
"github.com/stretchr/testify/assert"
|
|
)
|
|
|
|
func strPtr(source string) *string {
|
|
return &source
|
|
}
|
|
|
|
func TestGetAllAllowed(t *testing.T) {
|
|
buildArgs := newBuildArgs(map[string]*string{
|
|
"ArgNotUsedInDockerfile": strPtr("fromopt1"),
|
|
"ArgOverriddenByOptions": strPtr("fromopt2"),
|
|
"ArgNoDefaultInDockerfileFromOptions": strPtr("fromopt3"),
|
|
"HTTP_PROXY": strPtr("theproxy"),
|
|
})
|
|
|
|
buildArgs.AddMetaArg("ArgFromMeta", strPtr("frommeta1"))
|
|
buildArgs.AddMetaArg("ArgFromMetaOverriden", strPtr("frommeta2"))
|
|
buildArgs.AddMetaArg("ArgFromMetaNotUsed", strPtr("frommeta3"))
|
|
|
|
buildArgs.AddArg("ArgOverriddenByOptions", strPtr("fromdockerfile2"))
|
|
buildArgs.AddArg("ArgWithDefaultInDockerfile", strPtr("fromdockerfile1"))
|
|
buildArgs.AddArg("ArgNoDefaultInDockerfile", nil)
|
|
buildArgs.AddArg("ArgNoDefaultInDockerfileFromOptions", nil)
|
|
buildArgs.AddArg("ArgFromMeta", nil)
|
|
buildArgs.AddArg("ArgFromMetaOverriden", strPtr("fromdockerfile3"))
|
|
|
|
all := buildArgs.GetAllAllowed()
|
|
expected := map[string]string{
|
|
"HTTP_PROXY": "theproxy",
|
|
"ArgOverriddenByOptions": "fromopt2",
|
|
"ArgWithDefaultInDockerfile": "fromdockerfile1",
|
|
"ArgNoDefaultInDockerfileFromOptions": "fromopt3",
|
|
"ArgFromMeta": "frommeta1",
|
|
"ArgFromMetaOverriden": "fromdockerfile3",
|
|
}
|
|
assert.Equal(t, expected, all)
|
|
}
|
|
|
|
func TestGetAllMeta(t *testing.T) {
|
|
buildArgs := newBuildArgs(map[string]*string{
|
|
"ArgNotUsedInDockerfile": strPtr("fromopt1"),
|
|
"ArgOverriddenByOptions": strPtr("fromopt2"),
|
|
"ArgNoDefaultInMetaFromOptions": strPtr("fromopt3"),
|
|
"HTTP_PROXY": strPtr("theproxy"),
|
|
})
|
|
|
|
buildArgs.AddMetaArg("ArgFromMeta", strPtr("frommeta1"))
|
|
buildArgs.AddMetaArg("ArgOverriddenByOptions", strPtr("frommeta2"))
|
|
buildArgs.AddMetaArg("ArgNoDefaultInMetaFromOptions", nil)
|
|
|
|
all := buildArgs.GetAllMeta()
|
|
expected := map[string]string{
|
|
"HTTP_PROXY": "theproxy",
|
|
"ArgFromMeta": "frommeta1",
|
|
"ArgOverriddenByOptions": "fromopt2",
|
|
"ArgNoDefaultInMetaFromOptions": "fromopt3",
|
|
}
|
|
assert.Equal(t, expected, all)
|
|
}
|
|
|
|
func TestWarnOnUnusedBuildArgs(t *testing.T) {
|
|
buildArgs := newBuildArgs(map[string]*string{
|
|
"ThisArgIsUsed": strPtr("fromopt1"),
|
|
"ThisArgIsNotUsed": strPtr("fromopt2"),
|
|
"HTTPS_PROXY": strPtr("referenced builtin"),
|
|
"HTTP_PROXY": strPtr("unreferenced builtin"),
|
|
})
|
|
buildArgs.AddArg("ThisArgIsUsed", nil)
|
|
buildArgs.AddArg("HTTPS_PROXY", nil)
|
|
|
|
buffer := new(bytes.Buffer)
|
|
buildArgs.WarnOnUnusedBuildArgs(buffer)
|
|
out := buffer.String()
|
|
assert.NotContains(t, out, "ThisArgIsUsed")
|
|
assert.NotContains(t, out, "HTTPS_PROXY")
|
|
assert.NotContains(t, out, "HTTP_PROXY")
|
|
assert.Contains(t, out, "ThisArgIsNotUsed")
|
|
}
|
|
|
|
func TestIsUnreferencedBuiltin(t *testing.T) {
|
|
buildArgs := newBuildArgs(map[string]*string{
|
|
"ThisArgIsUsed": strPtr("fromopt1"),
|
|
"ThisArgIsNotUsed": strPtr("fromopt2"),
|
|
"HTTPS_PROXY": strPtr("referenced builtin"),
|
|
"HTTP_PROXY": strPtr("unreferenced builtin"),
|
|
})
|
|
buildArgs.AddArg("ThisArgIsUsed", nil)
|
|
buildArgs.AddArg("HTTPS_PROXY", nil)
|
|
|
|
assert.True(t, buildArgs.IsReferencedOrNotBuiltin("ThisArgIsUsed"))
|
|
assert.True(t, buildArgs.IsReferencedOrNotBuiltin("ThisArgIsNotUsed"))
|
|
assert.True(t, buildArgs.IsReferencedOrNotBuiltin("HTTPS_PROXY"))
|
|
assert.False(t, buildArgs.IsReferencedOrNotBuiltin("HTTP_PROXY"))
|
|
}
|