Skip to content

test (frontend): add unit tests#21

Merged
krisstern merged 21 commits intojenkinsci:mainfrom
giovanni-vaccarino:frontend-testing
Jul 12, 2025
Merged

test (frontend): add unit tests#21
krisstern merged 21 commits intojenkinsci:mainfrom
giovanni-vaccarino:frontend-testing

Conversation

@giovanni-vaccarino
Copy link
Copy Markdown
Contributor

Description

This PR introduces the unit tests for the frontend project. It includes both tests for API-related modules and React components. Moreover a new workflow have been added to integrate the frontend tests in the CI.

Submitter checklist

  • Make sure you are opening from a topic/feature/bugfix branch (right side) and not your main branch!
  • Ensure that the pull request title represents the desired changelog entry
  • Please describe what you did

@giovanni-vaccarino giovanni-vaccarino marked this pull request as ready for review July 6, 2025 14:00
@giovanni-vaccarino giovanni-vaccarino requested a review from a team as a code owner July 6, 2025 14:00
Comment thread frontend/src/tests/Sidebar.test.tsx
Comment thread frontend/src/tests/api_chatbot.test.ts
Comment thread frontend/src/tests/api_chatbot.test.ts
Comment thread frontend/src/tests/chatbotStorage.test.ts
Comment thread frontend/src/tests/chatbotStorage.test.ts
@krisstern
Copy link
Copy Markdown
Member

There are currently also some merge conflicts waiting to be resolved as well

Copy link
Copy Markdown
Member

@krisstern krisstern left a comment

Choose a reason for hiding this comment

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

LGTM

@krisstern krisstern merged commit 51caa52 into jenkinsci:main Jul 12, 2025
19 checks passed
@giovanni-vaccarino giovanni-vaccarino deleted the frontend-testing branch August 3, 2025 13:03
@berviantoleo berviantoleo added the tests This PR adds/removes/updates test cases label Aug 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

tests This PR adds/removes/updates test cases

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants