Skip to content

Conversation

tcoch
Copy link
Contributor

@tcoch tcoch commented Sep 15, 2025

Related to #836

cc @7-zete-7

Copy link
Contributor

@7-zete-7 7-zete-7 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would suggest adding a link to the documentation on installing PHPUnit into a project (this document assumes it is installed).

Also this repository already has a document on using Makefile in a project. Maybe add a link to it and rely on its extension rather than writing a separate Makefile?

## I want to run tests on my server

Simple: you only need to set up and deploy you project the same way as for the dev environment,
but you will need to set APP_ENV to test.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
but you will need to set APP_ENV to test.
but you will need to set `APP_ENV` to test.

You can set this value in `compose.override.yaml` directly, or provide your environment values
with a specific `env-file`:

```bash
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
```bash
```console

See #544

(check out [Automating your Workflow with a Makefile](https://symfony.com/doc/current/the-fast-track/en/17-tests.html#automating-your-workflow-with-a-makefile)).
You can start from this template:

```makefile
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
```makefile
```Makefile

See

```Makefile

`dev` environment. That's not ideal for unit tests, and clearly bad if you run
tests against your database.

The solution is to use a Makefile
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Perhaps it would be better to write "one of the possible solutions"?

@tcoch
Copy link
Contributor Author

tcoch commented Sep 21, 2025

Closing: refer to https://symfony.com/doc/current/testing.html if you need some documentation about testing, and check your configuration if you have issues.

@tcoch tcoch closed this Sep 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants