Skip to content

Commit 31ae8fe

Browse files
authored
Remove temp install (#108)
- The CLI test can now use the latest golang SDK (had to specifically get a version with the new migration scheme before) - Update a log info telling users to use `postgresql` instead of `postgres` as the db driver
1 parent 2f18174 commit 31ae8fe

File tree

2 files changed

+1
-6
lines changed

2 files changed

+1
-6
lines changed

cmd/dbos/cli_integration_test.go

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -159,11 +159,6 @@ func testProjectInitialization(t *testing.T, cliPath string) {
159159
modCmd := exec.Command("go", "mod", "tidy")
160160
modOutput, err := modCmd.CombinedOutput()
161161
require.NoError(t, err, "go mod tidy failed: %s", string(modOutput))
162-
163-
// TEMPORARY: go get github.com/dbos-inc/dbos-transact-golang/cmd/dbos@dbos-migration
164-
tmpCmd := exec.Command("go", "get", "github.com/dbos-inc/dbos-transact-golang/cmd/dbos@dbos-migration")
165-
tmpOutput, err := tmpCmd.CombinedOutput()
166-
require.NoError(t, err, "Failed to get dbos-migration: %s", string(tmpOutput))
167162
}
168163

169164
// testApplicationLifecycle starts the application and triggers workflows

cmd/dbos/postgres.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -174,7 +174,7 @@ func startDockerPostgres() error {
174174
return err
175175
}
176176

177-
logger.Info("Postgres available", "url", fmt.Sprintf("postgresql://postgres:%s@localhost:5432", password))
177+
logger.Info("Postgres available", "url", fmt.Sprintf("postgres://postgres:%s@localhost:5432", password))
178178
return nil
179179
}
180180

0 commit comments

Comments
 (0)