Skip to content
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

Add Slack Notifications to Cypress Test Results #752

Merged
merged 1 commit into from
Dec 12, 2024

Conversation

FahadDarw
Copy link
Contributor

Summary:
This PR enhances the Cypress test configuration by integrating Slack notifications to provide real-time updates on test results. Additionally, it ensures the reporting functionality is consistent and well-structured.


Changes:

  1. Added Slack Notifications:

    • Integrated Slack notifications using a webhook.
    • Configured the after:run event to send a summary of Cypress test results to a specified Slack channel.
    • Notifications include total tests, passed tests, and failed tests with dynamic color-coded messaging.
    • Other information has been added.
  2. Updated Cypress Reporter Configuration:

    • Fixed a typo in the mochawesomeReporterOptions (quitequiet).
    • Verified reporting outputs JSON files to cypress/reports/mocha.
  3. Improved CI Feedback Loop:

    • Notifications enhance visibility of test results for the team, reducing reliance on manual checks.

DrizzlyOwl
DrizzlyOwl previously approved these changes Dec 5, 2024
@FahadDarw FahadDarw dismissed DrizzlyOwl’s stale review December 5, 2024 10:22

The merge-base changed after approval.

DrizzlyOwl
DrizzlyOwl previously approved these changes Dec 5, 2024
@FahadDarw FahadDarw dismissed DrizzlyOwl’s stale review December 5, 2024 11:08

The merge-base changed after approval.

@DrizzlyOwl DrizzlyOwl disabled auto-merge December 5, 2024 11:10
DrizzlyOwl
DrizzlyOwl previously approved these changes Dec 5, 2024
@FahadDarw FahadDarw dismissed DrizzlyOwl’s stale review December 5, 2024 11:16

The merge-base changed after approval.

DrizzlyOwl
DrizzlyOwl previously approved these changes Dec 11, 2024
@FahadDarw FahadDarw dismissed DrizzlyOwl’s stale review December 11, 2024 10:44

The merge-base changed after approval.

@DrizzlyOwl DrizzlyOwl force-pushed the added_Slack_Notification branch from 5909b16 to 283bfa5 Compare December 12, 2024 16:44
@DrizzlyOwl DrizzlyOwl force-pushed the added_Slack_Notification branch from 283bfa5 to d181d6e Compare December 12, 2024 16:45
@DrizzlyOwl DrizzlyOwl merged commit ac77a2e into master Dec 12, 2024
3 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.

4 participants