Add TEST_DATABASE_URL to the default .env

[jvanbaarsen]

Currently it is a bit implicit that you can use the TEST_DATABASE_URL
env variable. I was able to figure this out by looking through the
source code, and since someone posted this on the issue tracker it means
more people find it a bit implicit.

Fixes: redwoodjs/redwood#1620

[dthyresson]

Should TEST_DATABASE_URL be included in the .example and .defaults as well?

Since

# schema.prisma defaults
DATABASE_URL=file:./dev.db

is in defaults? Would TEST_DATABASE_URL be set to the same?

Or a different database?

Perhaps a comment describes its purpose?

[jvanbaarsen]

@dthyresson I was in doubt about this. The reason for me not to include it was that the DATABASE_URL in .env is pretty much specific for Postgresql. This is also the only provider where you need to set the TEST_DATABASE_URL. For SQLite this works out of the box (Because it will just create a .db file in the temp directory).

What I want to achieve with this is mostly awareness that this option is available for you to set, since this caused me a bunch of confusion.

[dthyresson]

I was in doubt about this.

Thanks @jvanbaarsen -- perhaps @thedavidprice @cannikin or @peterp can determine if it should or not. Agree that the goals is not to be confused if you need to test against a postgres database

[cannikin]

My first thought is to add to .env.defaults with the current default path of .redwood/test.db but commented out with a note about usage:

# These environment variables will be used by default if you do not create any
# yourself in .env. This file should be safe to check into your version control
# system. Any custom values should go in .env and .env should *not* be checked
# into version control.

# schema.prisma defaults
DATABASE_URL=file:./dev.db

# location of the test database for api service scenarios (defaults to ./.redwood/test.db it not set)
# TEST_DATABASE_URL=file:./.redwood/test.db

# disables Prisma CLI update notifier
PRISMA_HIDE_UPDATE_MESSAGE=true

[thedavidprice]

I +1 Rob's suggestion above.

Also, we most definitely need to finish the Test Documentation! I've added a note about this on that PR as well.

[thedavidprice]

Instead of adding to .env, add to .env.defaults with the current default path of .redwood/test.db but commented out with a note about usage:

# These environment variables will be used by default if you do not create any
# yourself in .env. This file should be safe to check into your version control
# system. Any custom values should go in .env and .env should *not* be checked
# into version control.

# schema.prisma defaults
DATABASE_URL=file:./dev.db

# location of the test database for api service scenarios (defaults to ./.redwood/test.db it not set)
# TEST_DATABASE_URL=file:./.redwood/test.db

# disables Prisma CLI update notifier
PRISMA_HIDE_UPDATE_MESSAGE=true

[cannikin]

Yes, this should go in .env.defaults please!

[jvanbaarsen]

Allright! I'll update this PR 👍