e-Learning-by-SSE / nm-self-learning

https://staging.sse.uni-hildesheim.de/selflearn
4 stars 4 forks source link

[Quiz] Fehlender "Zur nächsten Lerneinheit" Button bei Quiz-Fehlversuch #226

Closed palfner-sse closed 2 weeks ago

palfner-sse commented 1 month ago

closes #199 closes #189

spark-sse commented 3 weeks ago

für die Funktionalität wann der Dialog gezeigt wird, hätte ich gerne einen test.

Für die Konzeption davon können wir uns zusammen setzen. Es sind noch nicht alle Beispiele für UI-Tests hochgeladen

ansonsten sieht es erstmal gut aus

palfner-sse commented 3 weeks ago

Alles klar. Lass uns um 13 Uhr ein Termin finden.

spark-sse commented 2 weeks ago

Bei den Tests kommt folgende Meldung

console.error
    Warning: An update to TransitionRootFn inside a test was not wrapped in act(...).

    When testing, code that causes React state updates should be wrapped into act(...):

    act(() => {
      /* fire events that update state */
    });
    /* assert on the output */

    This ensures that you're testing the behavior the user would see in the browser. Learn more at [https://reactjs.org/link/wrap-tests-with-act](https://reactjs.org/link/wrap-tests-with-act%1B[39m)
        at fn (/home/jenkins/workspace/Teaching_nm-self-learn_PR-226@2/node_modules/@headlessui/react/dist/headlessui.dev.cjs:8598:9)
        at fn (/home/jenkins/workspace/Teaching_nm-self-learn_PR-226@2/node_modules/@headlessui/react/dist/headlessui.dev.cjs:7969:3)
        at fn (/home/jenkins/workspace/Teaching_nm-self-learn_PR-226@2/node_modules/@headlessui/react/dist/headlessui.dev.cjs:8859:21)
        at fn (/home/jenkins/workspace/Teaching_nm-self-learn_PR-226@2/libs/ui/common/src/lib/dialog/dialog.tsx:7:2)
        at fn (/home/jenkins/workspace/Teaching_nm-self-learn_PR-226@2/apps/site/pages/courses/[courseSlug]/[lessonSlug]/quiz.tsx:377:2)
        at div
        at fn (/home/jenkins/workspace/Teaching_nm-self-learn_PR-226@2/apps/site/pages/courses/[courseSlug]/[lessonSlug]/quiz.tsx:170:2)
spark-sse commented 2 weeks ago

hab die änderungen fix selber gemacht. @palfner-sse bitte meine letzten beiden commits mal anschauen.