Skip to content

Kay Miao and Karen Lau: DAP20#31

Open
kmiao369 wants to merge 1 commit intoFullStackWithLawrence:mainfrom
kmiao369:main
Open

Kay Miao and Karen Lau: DAP20#31
kmiao369 wants to merge 1 commit intoFullStackWithLawrence:mainfrom
kmiao369:main

Conversation

@kmiao369
Copy link
Copy Markdown

Add success and failure notifications to Python unit test workflow and ensure failure notifications are not skipped.

DAP20 Capstone – Kay Miao and Karen Lau

Type of Change

  • Enhancement

Changes

  • Added notification logic for both success and failure cases in the Python unit test workflow
  • Ensured the notification job always runs by using if: always(), preventing it from being skipped when tests fail

Testing

Success scenario:

  • Committed valid changes and verified that the workflow completes successfully
  • Confirmed success notification is displayed

Failure scenario:

  • Introduced a temporary failure (e.g., forced test failure)
  • Verified that the notification job still runs and displays failure details

Notes

The merge-main-to-dev-branches workflow may fail due to branch synchronization logic unrelated to this change. The Python unit test notification workflow has been updated and tested independently and functions as expected.

…flow, and ensure failure notification not skipped
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.

1 participant