Skip to content

Conversation

@josecelano
Copy link
Member

@josecelano josecelano commented Nov 13, 2023

Reestablish E2E Tests.

@josecelano josecelano requested a review from a team as a code owner November 13, 2023 13:03
@josecelano josecelano linked an issue Nov 13, 2023 that may be closed by this pull request
@josecelano
Copy link
Member Author

ACK f35a067

Hi @da2ce7 these are the minimal changes needed to reestablish the E2E tests. I'm planning to make other improvements and close other issues related to docker with new PRs.

@josecelano josecelano requested a review from da2ce7 November 13, 2023 13:28
@josecelano josecelano merged commit 64479b7 into torrust:develop Nov 13, 2023
josecelano added a commit that referenced this pull request Nov 14, 2023
d92443d fix: [#342] E2E test execution with MySQL (Jose Celano)
31351fa fix: [#342] disable clippy warning (Jose Celano)
f8aa238 fix: [#342] broken E2E tests after renaming Category::category_id field (Jose Celano)
e78607d fix: [#342] SQLite data file path inside the container for E2E tests (Jose Celano)
1cce823 fix: [#342] index container not running for E2E tests (Jose Celano)
03943ef fix: [#342] fix en vars to run E2E tests (Jose Celano)

Pull request description:

  The E2E test suite execution was not reestablished correctly [here](#378).

  Some env vars were renamed (like `TORRUST_INDEX_E2E_SHARED`) so we were not running E2E tests (although they seemed to pass, we were only executing the ones that do not require a shared env).

  This PR only reestablishes the E2E tests. There are a lot of pending refactors that will be done in new PRs like [renaming env vars](#361).

Top commit has no ACKs.

Tree-SHA512: fc8f9f89dab83b8b2cb263befc614c9c8ebec884a060e4e5c6a958827d537b4c1198e0c8cdb4b709902a9869366088dc91ac33cca15a1fdf357a215a8328589e
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

Reestablish E2E Tests

1 participant