Closed liorzam closed 4 months ago
Latest commit: 83da0141469794fb8decbb1e35789b9ecc6cbde1
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
[!WARNING]
Rate Limit Exceeded
@liorzam has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 10 minutes and 31 seconds before requesting another review.
How to resolve this issue?
After the wait time has elapsed, a review can be triggered using the `@coderabbitai review` command as a PR comment. Alternatively, push new commits to this PR. We recommend that you space out your commits to avoid hitting the rate limit.How do rate limits work?
CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our [FAQ](https://coderabbit.ai/docs/faq) for further information.Commits
Files that changed from the base of the PR and between 9b198a8f849cc57840e14699c408608765be65fc and 83da0141469794fb8decbb1e35789b9ecc6cbde1.
The recent changes introduce enhanced test reporting, new alert definitions, and data migration updates across various files. Key modifications include configuring Jest HTML reporter options, adding new alert definitions, updating Prisma schema and migrations, and refining transaction-related utility functions. These updates aim to improve test visibility, alert accuracy, and data migration logging.
File Path | Change Summary |
---|---|
.github/actions/test-action/action.yml | Added environment variables, Jest HTML reporter options, and a step to store test results using actions/upload-artifact . |
.gitignore | Added services/workflows-service/test-report.html to the .gitignore file. |
services/workflows-service/jest.config.cjs | Added a custom reporter configuration for Jest tests. |
services/workflows-service/package.json | Added "jest-html-reporter": "^3.10.2" to the dependencies. |
services/workflows-service/prisma/data-migrations | Updated subproject commit hash. |
services/.../migrations/20240527134049_add_file_name_for_data_migration/migration.sql | Added a new column fileName to the DataMigrationVersion table. |
services/workflows-service/prisma/schema.prisma | Added fileName field to the DataMigrationVersion model. |
services/.../scripts/alerts/generate-alerts.ts | Added new alert definitions MGAV_CC and MGAV_APM , and modified generateAlertDefinitions function call. |
services/.../src/alert/alert.service.intg.test.ts | Introduced createCounterparty function and logic to create transactions and trigger alerts. |
services/.../data-analytics/data-analytics.service.ts | Added evaluateMerchantGroupAverage method to DataAnalyticsService class. |
services/.../data-analytics/types.ts | Added InlineRule variant and TMerchantGroupAverage type. |
services/.../data-migration/scripts/migrate.ts | Refactored logging of data migration process filenames. |
services/.../transaction/test-utils/transaction-factory.ts | Modified createBusinessCounterparty function and added businessTypeFn parameter. |
In the code's vast and winding maze,
New alerts and tests now blaze.
With Jest reports and Prisma's might,
Data flows clear, day and night.
Transactions tallied, alerts ring true,
A rabbit's touch, just for you.
🐇✨
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?
Summary by CodeRabbit
New Features
evaluateMerchantGroupAverage
for advanced data analytics.Bug Fixes
Chores
jest-html-reporter
dependency for enhanced test reporting..gitignore
to exclude test report files.Refactor
Tests