Skip to content

Commit a77e99e

Browse files
committed
fix test for CODER_AGENT_SUBSYSTEM
1 parent 01aaeef commit a77e99e

File tree

1 file changed

+22
-3
lines changed

1 file changed

+22
-3
lines changed

integration/integration_test.go

+22-3
Original file line numberDiff line numberDiff line change
@@ -127,6 +127,7 @@ func TestLogs(t *testing.T) {
127127
envbuilderEnv("GIT_URL", srv.URL),
128128
"CODER_AGENT_URL=" + logSrv.URL,
129129
"CODER_AGENT_TOKEN=" + token,
130+
"ENVBUILDER_INIT_SCRIPT=env",
130131
}})
131132
require.NoError(t, err)
132133
ctx, cancel := context.WithTimeout(context.Background(), time.Minute)
@@ -137,9 +138,27 @@ func TestLogs(t *testing.T) {
137138
case <-logsDone:
138139
}
139140

140-
// Ensure that CODER_AGENT_SUBSYSTEM is set correctly inside the container.
141-
output := execContainer(t, ctrID, "echo $CODER_AGENT_SUBSYSTEM")
142-
require.Equal(t, "envbuilder", strings.TrimSpace(output))
141+
// Wait for the container to exit
142+
client, err := client.NewClientWithOpts(client.FromEnv, client.WithAPIVersionNegotiation())
143+
require.NoError(t, err)
144+
require.Eventually(t, func() bool {
145+
status, err := client.ContainerInspect(ctx, ctrID)
146+
if !assert.NoError(t, err) {
147+
return false
148+
}
149+
return !status.State.Running
150+
}, 10*time.Second, time.Second, "container never exited")
151+
152+
// Check the expected log output
153+
logReader, err := client.ContainerLogs(ctx, ctrID, container.LogsOptions{
154+
ShowStdout: true,
155+
ShowStderr: true,
156+
})
157+
require.NoError(t, err)
158+
logBytes, err := io.ReadAll(logReader)
159+
require.NoError(t, err)
160+
logs := string(logBytes)
161+
require.Contains(t, logs, "CODER_AGENT_SUBSYSTEM=envbuilder")
143162
}
144163

145164
func TestInitScriptInitCommand(t *testing.T) {

0 commit comments

Comments
 (0)