Skip to content

Add isMultiDay extension property and improve test naming conventions#13

Merged
kozw merged 2 commits intomainfrom
enhancements
Nov 6, 2025
Merged

Add isMultiDay extension property and improve test naming conventions#13
kozw merged 2 commits intomainfrom
enhancements

Conversation

@kozw
Copy link
Owner

@kozw kozw commented Nov 6, 2025

No description provided.

Copilot AI review requested due to automatic review settings November 6, 2025 23:55
@codecov
Copy link

codecov bot commented Nov 6, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.30%. Comparing base (88ed1e5) to head (fd04c3c).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #13      +/-   ##
==========================================
+ Coverage   97.28%   97.30%   +0.02%     
==========================================
  Files          15       15              
  Lines        2137     2153      +16     
==========================================
+ Hits         2079     2095      +16     
  Misses         58       58              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Contributor

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 adds multi-day detection functionality to calendar components, enabling determination of whether events and todos span multiple calendar days. The implementation includes a shared helper function with comprehensive test coverage for various scenarios.

  • Adds isMultiDay getter to EventComponent and TodoComponent extensions
  • Implements _isMultiDay helper function with logic for both all-day and timed components
  • Adds 14 new test cases covering edge cases for events and todos (duration-based, time-based, all-day scenarios)

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
lib/src/semantic/extensions/components.dart Adds _isMultiDay helper function and isMultiDay getters to EventComponent and TodoComponent extensions with comprehensive documentation
test/semantic/extensions/components_test.dart Adds comprehensive test coverage for isMultiDay functionality across different event and todo scenarios; modifies test description capitalization for timezone tests

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@kozw kozw merged commit ed24088 into main Nov 6, 2025
9 checks passed
@kozw kozw deleted the enhancements branch November 6, 2025 23:59
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