Skip to content

Commit

Permalink
Use a dummy port for integration test.
Browse files Browse the repository at this point in the history
An empty port uses :80 by default, which requires admin privileges or something.
  • Loading branch information
mitchell-as committed Oct 2, 2024
1 parent 851072a commit 65fd3fd
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion internal/scriptrun/test/integration/scriptrun_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -120,7 +120,8 @@ func (suite *ScriptRunSuite) TestEnvIsSet() {
cfg.Set(constants.AsyncRuntimeConfig, true)

out := capturer.CaptureOutput(func() {
scriptRun := scriptrun.New(primer.New(auth, outputhelper.NewCatcher(), subshell.New(cfg), proj, cfg, blackhole.New(), model.NewSvcModel("")))
port := ":12345" // pick a high number that doesn't require admin privileges
scriptRun := scriptrun.New(primer.New(auth, outputhelper.NewCatcher(), subshell.New(cfg), proj, cfg, blackhole.New(), model.NewSvcModel(port)))
script, err := proj.ScriptByName("run")
require.NoError(t, err, "Error: "+errs.JoinMessage(err))
err = scriptRun.Run(script, nil)
Expand Down

0 comments on commit 65fd3fd

Please sign in to comment.