Skip to content

Conversation

hellyzh
Copy link
Collaborator

@hellyzh hellyzh commented Aug 11, 2025

No description provided.

@hellyzh hellyzh requested a review from Copilot August 11, 2025 05:27
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR improves error handling in the UI test workflow when calculating duration for email reports. It adds validation and error suppression to prevent failures when processing timestamp data.

  • Enhanced validation to check for malformed timestamp data before processing
  • Added error suppression and fallback handling for the dateutils.ddiff command
  • Improved robustness of the duration calculation logic

Copy link

codecov bot commented Aug 13, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 90.04%. Comparing base (2df6478) to head (e8c878a).
⚠️ Report is 3 commits behind head on dev.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff            @@
##              dev   #14410    +/-   ##
========================================
  Coverage   90.03%   90.04%            
========================================
  Files         650      650            
  Lines       38924    38924            
  Branches     7675     7807   +132     
========================================
+ Hits        35046    35048     +2     
+ Misses       1759     1758     -1     
+ Partials     2119     2118     -1     

see 2 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@hellyzh hellyzh requested a review from qinqingxu August 13, 2025 09:03
@hellyzh hellyzh merged commit 0e7f46d into dev Aug 14, 2025
20 of 21 checks passed
@hellyzh hellyzh deleted the hellyzh/mail branch August 14, 2025 02:04
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