wp-media / wp-rocket-e2e

Playwright E2E testing repo
GNU General Public License v3.0
0 stars 0 forks source link

Discard error messages related to table already existing #132

Closed MathieuLamiot closed 2 months ago

MathieuLamiot commented 2 months ago

Context We regularly have errors reported on install or upgrade that a table such as RUCSS or ATF already exists (for instance #54, see also on Slack, we have many examples). Those errors can occur because multiple requests try to create/update the table after an install/update; this is not an issue.

Expected behavior

Acceptance Criteria Run smoke E2E tests multiple times until this error occurs in the logs. The related test must not fail because of this.

Additional information The other possibility I can think of is to prevent the error from happening in the first place. This would require a lock mechanism, which I think is risky and probably overkill for this.

MathieuLamiot commented 2 months ago

Integrated to #67