Closed liorzam closed 1 month ago
Latest commit: 9b5bec8d46a57f033c2fef3f1ae0dbfd1f4f35d5
Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.
Click here to learn what changesets are, and how to add one.
Click here if you're a maintainer who wants to add a changeset to this PR
The recent updates enhance the CI workflow and testing infrastructure. Key changes include adding Jest HTML reporter configurations, storing test results as artifacts, and updating the test-action
version in the CI workflow. Additionally, database migration logging and schema were improved, and a new dependency on jest-html-reporter
was introduced. The .gitignore
file was updated to exclude specific test reports, and minor script refactoring was performed.
File Path | Change Summary |
---|---|
.github/actions/test-action/action.yml |
Added environment variables for Jest HTML reporter and a step to store test results using actions/upload-artifact . |
.github/workflows/ci.yml |
Updated Test job to use specific version bal-2106 of test-action . |
.gitignore |
Excluded services/workflows-service/test-report.html from git tracking. |
services/workflows-service/package.json |
Added jest-html-reporter dependency version ^3.10.2 . |
services/workflows-service/prisma/data-migrations |
Updated subproject commit hash. |
services/.../20240527134049_add_file_name_for_data_migration/migration.sql |
Added fileName column to DataMigrationVersion table. |
services/workflows-service/prisma/schema.prisma |
Added nullable fileName field to DataMigrationVersion model. |
services/.../scripts/alerts/generate-alerts.ts |
Refactored generateAlertDefinitions function call for conciseness. |
services/.../src/data-migration/scripts/migrate.ts |
Updated logging to extract and log fileName , and included fileName in dataMigrationRepository.create method call. |
services/workflows-service/jest.config.cjs |
Added custom reporter configuration for Jest HTML reporter. |
services/.../src/test/db-setup.ts |
Set LOG_LEVEL environment variable to 'error' before initializing database setup. |
In the code's dance, a change so bright,
Jest reports now shine in the night.
Migrations log with names so clear,
CI flows with less to fear.
Git ignores the test report's flight,
A rabbit's joy in code's delight.
🐇✨
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Link to an report example: https://github.com/ballerine-io/ballerine/actions/runs/9287773639/artifacts/1548657493
Report example:
Summary by CodeRabbit
New Features
fileName
column in theDataMigrationVersion
table for better tracking of data migrations.Improvements
.gitignore
to exclude test report files.Refactor
test-action
.Chores
jest-html-reporter
as a new dependency inpackage.json
.LOG_LEVEL
to 'error' in database setup scripts.