Skip to content

Conversation

@omsuneri
Copy link
Member

This creates the edge test cases for every function that lies in mathutils.js in /utils/tests which will be the main test folder for the tests of the utils directory.

@omsuneri
Copy link
Member Author

@walterbender i tried running the test

Screenshot 2024-12-11 at 8 19 43 PM

@walterbender
Copy link
Member

(1) How did you run the test?
(2) Which test is failing?

@omsuneri
Copy link
Member Author

@walterbender i had not passed any input for the jest so the test test case runs without any input

@omsuneri
Copy link
Member Author

@walterbender we’ll create a GitHub Action that will run our Jest tests whenever there are changes pushed to the repo

@omsuneri
Copy link
Member Author

@walterbender i ll be doing the github actions soon for the jest tests

@walterbender
Copy link
Member

I'm still confused as to what failed.

@haroon10725
Copy link
Contributor

@walterbender I think the tests are not properly written. 27 failed.

@omsuneri
Copy link
Member Author

@walterbender I ll try to review it again if I find some error

@omsuneri
Copy link
Member Author

@walterbender i had resolved the test file and refactored it such that now it works absolutely fine

Screen.Recording.2024-12-12.at.3.25.05.AM.mov

@omsuneri
Copy link
Member Author

@walterbender now i think i should move forward for github actions to automate the complete process of testing

@walterbender walterbender merged commit a2c31c9 into sugarlabs:master Dec 11, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants