Skip to content

Paywalls: tests for PurchaseButton layout logic#3468

Merged
NachoSoto merged 2 commits into
mainfrom
paywalls-template-4-no-offer-details
Nov 23, 2023
Merged

Paywalls: tests for PurchaseButton layout logic#3468
NachoSoto merged 2 commits into
mainfrom
paywalls-template-4-no-offer-details

Conversation

@NachoSoto

Copy link
Copy Markdown
Contributor

This adds test coverage for the logic added in #3361, now extracted from PurchaseButton into a TemplateViewConfiguration extension.

This adds test coverage for the logic added in #3361, now extracted from `PurchaseButton` into a `TemplateViewConfiguration` extension.
@NachoSoto NachoSoto added the test label Nov 21, 2023
@NachoSoto NachoSoto requested review from a team November 21, 2023 23:15
@codecov

codecov Bot commented Nov 21, 2023

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (e31a531) 86.07% compared to head (d115aea) 86.07%.
Report is 3 commits behind head on main.

❗ Current head d115aea differs from pull request most recent head 2033ad1. Consider uploading reports for the commit 2033ad1 to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #3468   +/-   ##
=======================================
  Coverage   86.07%   86.07%           
=======================================
  Files         237      237           
  Lines       17204    17204           
=======================================
  Hits        14809    14809           
  Misses       2395     2395           

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

Comment thread Tests/RevenueCatUITests/Data/TemplateViewConfigurationTests.swift Outdated
@NachoSoto NachoSoto enabled auto-merge (squash) November 23, 2023 01:18
@NachoSoto NachoSoto merged commit b62650a into main Nov 23, 2023
@NachoSoto NachoSoto deleted the paywalls-template-4-no-offer-details branch November 23, 2023 01:37
tonidero pushed a commit that referenced this pull request Dec 1, 2023
**This is an automatic release.**

### RevenueCatUI
* `Paywalls`: remove empty space when template 4 has no offer details
(#3469) via NachoSoto (@NachoSoto)
### Other Changes
* `Concurrency`: address strict concurrency issues on `SystemInfo`
(#3462) via NachoSoto (@NachoSoto)
* `CircleCI`: upgrade to Xcode 15.1 (#3403) via NachoSoto (@NachoSoto)
* `Paywalls`: tests for `PurchaseButton` layout logic (#3468) via
NachoSoto (@NachoSoto)
* `Paywalls`: simplified `PaywallViewMode` logic (#3470) via NachoSoto
(@NachoSoto)
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