Skip to content

test: use real calendar provider where possible #74

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

Merged
merged 24 commits into from
Apr 23, 2025

Conversation

williscool
Copy link
Owner

@williscool williscool commented Apr 14, 2025

I thought it was an android api its a concept from this app so want to
mock as little of it as possible

TODones:

  • there is like a storage clearing bug with the tests in concert now
  • figure out where to document that since we delegate to real calendar provider in the mock it essentially serves as a test for it

Copy link

Build artifacts for this PR are available:

You can download these artifacts from the "Artifacts" section of the workflow run.

Copy link

github-actions bot commented Apr 14, 2025

Code Coverage Report

Overall Project 13.26%

There is no coverage information present for the Files changed

Copy link

📊 Code Coverage Summary

Type Coverage
Line 0.73
Branch

View detailed coverage report

Copy link

Build artifacts for this PR are available:

You can download these artifacts from the "Artifacts" section of the workflow run.

Copy link

📊 Code Coverage Summary

Type Coverage
Line 13.16
Branch

View detailed coverage report

Copy link

Build artifacts for this PR are available:

You can download these artifacts from the "Artifacts" section of the workflow run.

Copy link

📊 Code Coverage Summary

Type Coverage
Line 13.36
Branch

View detailed coverage report

@williscool williscool enabled auto-merge (squash) April 15, 2025 05:10
Copy link

Build artifacts for this PR are available:

You can download these artifacts from the "Artifacts" section of the workflow run.

Copy link

📊 Code Coverage Summary

Type Coverage
Line 12.9
Branch

View detailed coverage report

@williscool williscool force-pushed the test/use_real_calendar_provider_in_tests branch from cee4913 to 52f1825 Compare April 22, 2025 03:07
Copy link

Build artifacts for this PR are available:

You can download these artifacts from the "Artifacts" section of the workflow run.

Copy link

📊 Code Coverage Summary

Type Coverage
Line
Branch

View detailed coverage report

@williscool williscool force-pushed the test/use_real_calendar_provider_in_tests branch from 52f1825 to 7d97b33 Compare April 22, 2025 22:22
Copy link

Build artifacts for this PR are available:

You can download these artifacts from the "Artifacts" section of the workflow run.

Copy link

📊 Code Coverage Summary

Type Coverage
Line 24.84
Branch

View detailed coverage report

Copy link

Build artifacts for this PR are available:

You can download these artifacts from the "Artifacts" section of the workflow run.

@williscool williscool force-pushed the test/use_real_calendar_provider_in_tests branch from 41508c9 to 97a02b8 Compare April 23, 2025 05:38
Copy link

📊 Code Coverage Summary

Type Coverage
Line 25.17
Branch

View detailed coverage report

Copy link

Build artifacts for this PR are available:

You can download these artifacts from the "Artifacts" section of the workflow run.

Copy link

Build artifacts for this PR are available:

You can download these artifacts from the "Artifacts" section of the workflow run.

Copy link

📊 Code Coverage Summary

Coverage Type Coverage
Overall 13.26
Changed Files 100

View detailed coverage report

Copy link

Build artifacts for this PR are available:

You can download these artifacts from the "Artifacts" section of the workflow run.

Copy link

📊 Code Coverage Summary

Coverage Type Coverage
Overall 13.26
Changed Files 100

View detailed coverage report

Copy link

📊 Code Coverage Summary

Coverage Type Coverage
Overall 13.25
Changed Files 28.94

View detailed coverage report

Copy link

Build artifacts for this PR are available:

You can download these artifacts from the "Artifacts" section of the workflow run.

@williscool williscool merged commit 5f76d5e into master Apr 23, 2025
10 checks passed
Copy link

📊 Code Coverage Summary

Coverage Type Coverage
Overall 13.26
Changed Files 100

View detailed coverage report

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