Skip to content

[FEATURE] MDX-32 modification Decouverte 1 NR_Datacenter_NOV - #17355

Open
joffreyboches-modulix wants to merge 3 commits into
devfrom
tmp-datacenter-nov-qcu-img
Open

[FEATURE] MDX-32 modification Decouverte 1 NR_Datacenter_NOV#17355
joffreyboches-modulix wants to merge 3 commits into
devfrom
tmp-datacenter-nov-qcu-img

Conversation

@joffreyboches-modulix

@joffreyboches-modulix joffreyboches-modulix commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

☀️ Problème

⛱️ Proposition

🧴 Remarques

🏊 Pour tester

https://app-pr17355.review.pix.fr/modules/da86e4a7/centre-de-donnee-novice

@pix-bot-github

pix-bot-github commented Sep 1, 2026

Copy link
Copy Markdown

@github-actions

github-actions Bot commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

Les tests Modulix ont échoué. Voici les détails :


> pix-api@5.492.0 modulix:test
> npm run test:api:path -- 'tests/devcomp/unit/infrastructure/datasources/learning-content/module-datasource_test.js' 'tests/devcomp/unit/infrastructure/datasources/learning-content/validation/module-validation_test.js' 'tests/devcomp/unit/infrastructure/repositories/module-repository_test.js' && TEST_SETUP_MODE=unit npm run test:api:path -- 'tests/devcomp/acceptance/module-instantiation_test.js'


> pix-api@5.492.0 test:api:path
> NODE_ENV=test node --env-file-if-exists=.env ./node_modules/mocha/bin/mocha tests/devcomp/unit/infrastructure/datasources/learning-content/module-datasource_test.js tests/devcomp/unit/infrastructure/datasources/learning-content/validation/module-validation_test.js tests/devcomp/unit/infrastructure/repositories/module-repository_test.js

.env not found. Continuing without it.
Running with "unit" setup mode


  ........................................................
  ..........!.............................................
  ....................................

  147 passing (27s)
  1 failing

  1) Unit | Infrastructure | Datasources | Learning Content | ModuleDatasource
       #list
         modules content
           module "centre-de-donnee-novice" should contain a valid structure:
     AssertionError: 
============================================================

Error: "sections[1].grains[0].components[2].element.title" is not allowed to be empty.
Valeur concernée à rechercher : ""

============================================================
: expected ValidationError: "sections[1].grains[0].c… { …(2) } to equal undefined
      at Context.<anonymous> (file:///home/runner/work/pix/pix/api/tests/devcomp/unit/infrastructure/datasources/learning-content/module-datasource_test.js:162:33)



@joffreyboches-modulix
joffreyboches-modulix marked this pull request as ready for review September 1, 2026 16:32
@joffreyboches-modulix
joffreyboches-modulix requested a review from a team as a code owner September 1, 2026 16:32
@joffreyboches-modulix
joffreyboches-modulix marked this pull request as draft September 2, 2026 07:43
@joffreyboches-modulix
joffreyboches-modulix marked this pull request as ready for review September 2, 2026 09:47
@joffreyboches-modulix
joffreyboches-modulix added this pull request to the merge queue Sep 4, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to no response for status checks Sep 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants