Skip to content

Conversation

@LogicalGuy77
Copy link
Contributor

This PR sets up Cypress Testing environment for e2e tests. Includes a basic test to see app loads correctly.

* feat: added Unit testing using Jest for utils file (#15)

* fix: update import path for Kubeflow styles

Signed-off-by: LogicalGuy77 <harshitacademia@gmail.com>

* fix: update build scripts to ensure CSS assets are copied correctly

Signed-off-by: LogicalGuy77 <harshitacademia@gmail.com>

* fix: enhance frontend dependency installation and style setup

Signed-off-by: LogicalGuy77 <harshitacademia@gmail.com>

* fix: enhance frontend dependency installation and style setup

Signed-off-by: LogicalGuy77 <harshitacademia@gmail.com>

* fix: enhance frontend dependency installation and style setup

Signed-off-by: LogicalGuy77 <harshitacademia@gmail.com>

* feat: setup Jest and __mocks__ for unit testing

* feat: enhance type definitions and add tests for predictor extension specifications

* fix: format code for consistency in utils.jest.spec.ts

* fix: update unit test command to use Jest instead of production

---------

Signed-off-by: LogicalGuy77 <harshitacademia@gmail.com>

* feat: add Cypress support for end-to-end testing and new serve scripts (#16)

Signed-off-by: LogicalGuy77 <harshitacademia@gmail.com>

---------

Signed-off-by: LogicalGuy77 <harshitacademia@gmail.com>
Signed-off-by: Harshit Nayan <138717001+LogicalGuy77@users.noreply.github.com>
Signed-off-by: LogicalGuy77 <harshitacademia@gmail.com>
@juliusvonkohout
Copy link
Contributor

@Griffin-Sullivan for review.

Copy link
Contributor

@Griffin-Sullivan Griffin-Sullivan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overall looks good! Haven't ran the tests to see if everything is set up properly, but they are dummy tests anyway. Either this PR or the next we will want to add the test execution to Git Hub Actions

@juliusvonkohout juliusvonkohout merged commit ccccf41 into kserve:master Jun 23, 2025
6 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