Fix tests

[clyang82]

We have a question about why the CI passes without this fix, but the local environment does not. The reason is that in the CI environment, PostgreSQL runs in a sidecar container, making it effectively part of the same environment as the test container. In this case, no password is required for the database connection. The following explanation elaborates on this behavior.

Note 1: The PostgreSQL image sets up trust authentication locally so you may notice a password is not required when connecting from localhost (inside the same container). However, a password will be required if connecting from a different host/container.

from https://hub.docker.com/_/postgres

[morvencao]

/lgtm

[machi1990]

In the modified file, I see some things done to the template1 database and I am wondering if these are things the maesto project wants to carry? e.g

maestro/pkg/db/db_session/test.go

Line 99 in 999e92a

query = fmt.Sprintf("CREATE DATABASE %s TEMPLATE template1", config.Name)

maestro/pkg/db/db_session/test.go

Line 87 in 999e92a

if err := dropConnections(dbx, "template1"); err != nil {

Plus some comments that are not actually true e.g

maestro/pkg/db/db_session/test.go

Line 94 in 999e92a

// Rebuild AMS DB

[clyang82]

/assign @qiujian16

[clyang82]

I just realized that I should use ConnectionStringWithName instead of ConnectionString to keep the existing behaviour.
@machi1990 take a look at. Thanks.

[machi1990]

I just realized that I should use ConnectionStringWithName instead of ConnectionString to keep the existing behaviour. @machi1990 take a look at. Thanks.

I see comments referencing AMS, can we update them?