Skip to content

test(server-nestjs): stub every required env of the plugin module spec - #2883

Open
shikanime wants to merge 1 commit into
mainfrom
fix/plugin-spec-env
Open

shikanime wants to merge 1 commit into
mainfrom
fix/plugin-spec-env

Conversation

@shikanime

Copy link
Copy Markdown
Member

Pourquoi

plugin.module.spec.ts n'est pas hermétique : il stubbe 19 variables d'environnement mais pas DB_URL, ni les variables requises par les modules Keycloak et observabilité, activés inconditionnellement. En l'absence d'un .env local fournissant ces valeurs, le test échoue avec ZodError: DB_URL Required. Refs #2882

Quoi

  • Complète le stubbing : DB_URL, les 7 variables Keycloak requises, GRAFANA_URL et DSO_OBSERVABILITY_CHART_VERSION.
  • Vérification : le test passe dans un workspace sans .env.

Références

The spec stubbed 19 env vars but not DB_URL, nor the keycloak and
observability variables required by their unconditionally enabled
modules, so it only passed where a local .env provided them.

Stub the full required set so the test passes without a .env.

Refs #2882

Co-authored-by: Automata <automata@shikanime.studio>
Signed-off-by: William Phetsinorath <william.phetsinorath-open@interieur.gouv.fr>
Change-Id: I829cedfe41997cb5297d5e651efc2eb96a6a6964
@shikanime
shikanime requested a review from a team as a code owner October 20, 2026 15:23
@github-actions github-actions Bot added the built label Oct 20, 2026
@cloud-pi-native-sonarqube

Copy link
Copy Markdown

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant