Skip to content

Commit d170ef0

Browse files
committed
use git repo in tests
1 parent 70940f0 commit d170ef0

File tree

2 files changed

+13
-9
lines changed

2 files changed

+13
-9
lines changed

internal/provider/cached_image_data_source_test.go

+4-4
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ func TestAccCachedImageDataSource(t *testing.T) {
3535
}
3636
cache_repo = %q
3737
verbose = true
38-
}`, deps.BuilderImage, deps.RepoDir, deps.RepoDir, deps.CacheRepo)
38+
}`, deps.BuilderImage, "/workspace", deps.RepoURL, deps.CacheRepo)
3939
resource.Test(t, resource.TestCase{
4040
PreCheck: func() { testAccPreCheck(t) },
4141
ProtoV6ProviderFactories: testAccProtoV6ProviderFactories,
@@ -46,7 +46,7 @@ func TestAccCachedImageDataSource(t *testing.T) {
4646
// Inputs should still be present.
4747
resource.TestCheckResourceAttr("data.envbuilder_cached_image.test", "cache_repo", deps.CacheRepo),
4848
resource.TestCheckResourceAttr("data.envbuilder_cached_image.test", "extra_env.FOO", "bar"),
49-
resource.TestCheckResourceAttr("data.envbuilder_cached_image.test", "git_url", deps.RepoDir),
49+
resource.TestCheckResourceAttr("data.envbuilder_cached_image.test", "git_url", deps.RepoURL),
5050
// Should be empty
5151
resource.TestCheckNoResourceAttr("data.envbuilder_cached_image.test", "git_username"),
5252
resource.TestCheckNoResourceAttr("data.envbuilder_cached_image.test", "git_password"),
@@ -94,7 +94,7 @@ func TestAccCachedImageDataSource(t *testing.T) {
9494
}
9595
cache_repo = %q
9696
verbose = true
97-
}`, deps.BuilderImage, deps.RepoDir, deps.RepoDir, deps.CacheRepo)
97+
}`, deps.BuilderImage, "/workspace", deps.RepoURL, deps.CacheRepo)
9898
resource.Test(t, resource.TestCase{
9999
PreCheck: func() { testAccPreCheck(t) },
100100
ProtoV6ProviderFactories: testAccProtoV6ProviderFactories,
@@ -105,7 +105,7 @@ func TestAccCachedImageDataSource(t *testing.T) {
105105
// Inputs should still be present.
106106
resource.TestCheckResourceAttr("data.envbuilder_cached_image.test", "cache_repo", deps.CacheRepo),
107107
resource.TestCheckResourceAttr("data.envbuilder_cached_image.test", "extra_env.FOO", "bar"),
108-
resource.TestCheckResourceAttr("data.envbuilder_cached_image.test", "git_url", deps.RepoDir),
108+
resource.TestCheckResourceAttr("data.envbuilder_cached_image.test", "git_url", deps.RepoURL),
109109
resource.TestCheckResourceAttr("data.envbuilder_cached_image.test", "exists", "false"),
110110
resource.TestCheckResourceAttr("data.envbuilder_cached_image.test", "image", deps.BuilderImage),
111111
// Should be empty

internal/provider/provider_test.go

+9-5
Original file line numberDiff line numberDiff line change
@@ -71,9 +71,11 @@ func testAccPreCheck(t *testing.T) {
7171
type testDependencies struct {
7272
BuilderImage string
7373
RepoDir string
74+
RepoURL string
7475
CacheRepo string
7576
GitImage string
7677
SSHDir string
78+
SSHPort string
7779
}
7880

7981
func setup(t testing.TB, files map[string]string) testDependencies {
@@ -98,12 +100,17 @@ func setup(t testing.TB, files map[string]string) testDependencies {
98100
"authorized_keys": testSSHUserPubKey,
99101
}, sshDir)
100102

103+
// TODO(mafredri): Use a dynamic port?
104+
sshPort := "2222"
105+
101106
return testDependencies{
102107
BuilderImage: envbuilderImageRef,
103108
CacheRepo: reg + "/test",
104109
RepoDir: repoDir,
110+
RepoURL: fmt.Sprintf("ssh://git@localhost:%s/srv/git/repo.git", sshPort),
105111
GitImage: gitImageRef,
106112
SSHDir: sshDir,
113+
SSHPort: sshPort,
107114
}
108115
}
109116

@@ -120,9 +127,6 @@ func seedCache(ctx context.Context, t testing.TB, deps testDependencies) {
120127
ensureImage(ctx, t, cli, deps.GitImage)
121128
ensureImage(ctx, t, cli, deps.BuilderImage)
122129

123-
// TODO(mafredri): Use a dynamic port?
124-
sshPort := "2222"
125-
126130
gitCtr, err := cli.ContainerCreate(ctx, &container.Config{
127131
Image: deps.GitImage,
128132
Env: []string{
@@ -133,7 +137,7 @@ func seedCache(ctx context.Context, t testing.TB, deps testDependencies) {
133137
},
134138
}, &container.HostConfig{
135139
PortBindings: nat.PortMap{
136-
"22/tcp": []nat.PortBinding{{HostIP: "localhost", HostPort: sshPort}},
140+
"22/tcp": []nat.PortBinding{{HostIP: "localhost", HostPort: deps.SSHPort}},
137141
},
138142
Binds: []string{
139143
deps.RepoDir + ":/srv/git/repo.git",
@@ -159,7 +163,7 @@ func seedCache(ctx context.Context, t testing.TB, deps testDependencies) {
159163
"ENVBUILDER_INIT_SCRIPT=exit",
160164
"ENVBUILDER_PUSH_IMAGE=true",
161165
"ENVBUILDER_VERBOSE=true",
162-
fmt.Sprintf("ENVBUILDER_GIT_URL=ssh://git@localhost:%s/srv/git/repo.git", sshPort),
166+
"ENVBUILDER_GIT_URL=" + deps.RepoURL,
163167
"ENVBUILDER_GIT_SSH_PRIVATE_KEY_PATH=/tmp/ssh/id_ed25519",
164168
},
165169
Labels: map[string]string{

0 commit comments

Comments
 (0)