fixing interpreter test

Program-AR · Sep 27, 2024 · 141cebf · 141cebf
1 parent c54b0c7
commit 141cebf
Show file tree

Hide file tree

Showing 4 changed files with 22 additions and 16 deletions.
diff --git a/src/components/challengeView/SceneButtons/Execute.tsx b/src/components/challengeView/SceneButtons/Execute.tsx
@@ -59,14 +59,14 @@ export const ExecuteButton = ({ challenge }: ExecuteButtonProps) => {
   return <>
     <Tooltip title={t('run.tooltip')}>
       {isSmallScreen ?
-        <IconButton className={styles['icon-button']} onClick={handleExcecute} data-testid='execute-button' data-finishedExecution={finishedExecution}>
+        <IconButton className={styles['icon-button']} onClick={handleExcecute} data-testid='execute-button' data-finishedexecution={finishedExecution}>
           <Stack>
             <Circle color='success' className={styles['circle-icon']} />
             <PlayArrow className={styles['icon']} />
           </Stack>
         </IconButton >
         :
-        <Button variant="contained" color="success" onClick={handleExcecute} data-testid='execute-button' data-finishedExecution={finishedExecution}>{t("run.label")} </Button>
+        <Button variant="contained" color="success" onClick={handleExcecute} data-testid='execute-button' data-finishedexecution={finishedExecution}>{t("run.label")} </Button>
       }
     </Tooltip>
     <EndDialog showModal={showModal} setShowModal={setShowModal} />

diff --git a/src/test/integration/ChallengeView.cy.tsx b/src/test/integration/ChallengeView.cy.tsx
@@ -34,7 +34,7 @@ describe('<ChallengeView />', () => {
   const executeChallengeWithEval = (expression: string, expected: any) => {
     cy.get('[data-testid="scene-iframe"]').should('have.attr', 'data-loaded', 'true').then($iframe => {
       const iframe = $iframe[0] as HTMLIFrameElement;
-      cy.get('[data-testid="execute-button"]').click().should('have.attr', 'data-finishedExecution', 'true').then(() => {
+      cy.get('[data-testid="execute-button"]').click().should('have.attr', 'data-finishedexecution', 'true').then(() => {
         expect((iframe.contentWindow as any).eval(`pilas.escena_actual().${expression}`)).to.equal(expected)
       })
     })

diff --git a/src/test/unit/components/ChallengeView/SceneView.test.tsx b/src/test/unit/components/ChallengeView/SceneView.test.tsx
@@ -7,6 +7,7 @@
 import { SceneView } from "../../../../components/challengeView/SceneView"
 import { screen } from '@testing-library/react'
 import { renderComponent } from "../../../testUtils"
+import React from "react"
 describe('SceneView renders all challenges', () => {
 
     test.skip(`Scene LimpiandoElHumedal loads ok`, () => {

diff --git a/src/test/unit/components/interpreter.test.tsx b/src/test/unit/components/interpreter.test.tsx
@@ -1,7 +1,7 @@
 import { ExecuteButton } from "../../../components/challengeView/SceneButtons/Execute"
 import { Challenge } from "../../../staticData/challenges"
 import { renderComponent } from "../../testUtils"
-import { screen, fireEvent, waitFor } from '@testing-library/react'
+import { screen, fireEvent, waitFor, act } from '@testing-library/react'
 import { interpreterFactory } from "../../../components/challengeView/SceneButtons/interpreterFactory"
 import { scene } from "../../../components/challengeView/scene"
 
@@ -10,7 +10,7 @@ describe("Interpreter", () => {
 
     beforeEach(() => {
 
-        scene['restartScene'] = (_descriptor: string) => {}
+        scene['restartScene'] = (_descriptor: string) => { }
 
         interpreterMock = {
             run: jest.fn(() => false),
@@ -19,23 +19,28 @@ describe("Interpreter", () => {
 
         interpreterFactory['createInterpreter'] = () => interpreterMock
 
+        scene['isTheProblemSolved'] = () => true
+
     })
 
-    const challenge = (): Challenge => {
-        return {
-            id: 1,
-            sceneDescriptor: `new EscenaLita(["[[A,L,T],[-,-,E],[-,-,-]]","[[A,T,L],[-,-,E],[-,-,-]]"])`,
-            toolboxBlockIds: ["AgarrarTomate"],
-            imageURL: () => ''
-        }
+    const challenge: Challenge = {
+        id: 1,
+        sceneDescriptor: `new EscenaLita(["[[A,L,T],[-,-,E],[-,-,-]]","[[A,T,L],[-,-,E],[-,-,-]]"])`,
+        toolboxBlockIds: ["AgarrarTomate"],
+        imageURL: () => ''
     }
 
     test('Interpreter runs on execution', async () => {
-        renderComponent(<ExecuteButton challenge={challenge()} />)
+        renderComponent(<ExecuteButton challenge={challenge} />)
         const button = await screen.findByTestId('execute-button')
-        fireEvent.click(button)
-        await new Promise(r => setTimeout(r, 2000));
-        //await waitFor()
+        act(() => {
+            fireEvent.click(button);
+        });
+
+        await waitFor(() => {
+            expect(button).toHaveAttribute('data-finishedexecution', 'true');
+        });
+
         expect(interpreterMock.run).toHaveBeenCalled();
     })
 })