Skip to content

feat: adding github action to run all tests #4804

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

nqodube
Copy link

@nqodube nqodube commented Jul 6, 2025

This pull request introduces a new GitHub Actions workflow for manual test execution, configures Jest to generate JUnit-style test reports, and adds a new dependency to support this functionality. Below is a summary of the most important changes:

GitHub Actions Workflow:

  • .github/workflows/run-all-tests.yml: Added a new workflow named "Manual Flowise Test Run" to enable manual execution of tests via workflow_dispatch. The workflow includes steps for checking out the repository, setting up Node.js, installing dependencies, building modules, running tests, and uploading test results as artifacts. It also integrates with a unit test result publishing action.

Jest Configuration:

  • packages/server/jest.config.js: Updated Jest configuration to include the jest-junit reporter. This generates JUnit-style XML test reports and stores them in the test-results directory with the filename results.xml.

Dependencies:

@HenryHengZJ
Copy link
Contributor

is this meant to be your forked changes?

@nqodube
Copy link
Author

nqodube commented Jul 6, 2025

Yes they are, tested the action in our forked repo.

@nqodube
Copy link
Author

nqodube commented Jul 7, 2025

@HenryHengZJ are you asking because you may not want to run due to Github action minutes?

They are only triggered on demand and not as part of every PR.

Also, It may be beneficial for anyone that forks to also run these on their forked version too.

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.

2 participants