Skip to content

Add API tests for Paywalls and CustomerCenter presenters#735

Merged
facumenzella merged 6 commits into
mainfrom
other/integration-tests2
Mar 10, 2026
Merged

Add API tests for Paywalls and CustomerCenter presenters#735
facumenzella merged 6 commits into
mainfrom
other/integration-tests2

Conversation

@facumenzella

@facumenzella facumenzella commented Oct 30, 2025

Copy link
Copy Markdown
Member

Summary

  • Add PaywallsPresenterAPITests to validate the public static paywall entry points (Present, PresentIfNeeded, PaywallResult, PaywallResultType)
  • Add CustomerCenterPresenterAPITests that wires every callback on CustomerCenterPresenter to ensure signatures stay callable from integrations

Note: These live under IntegrationTests/ due to the existing project naming convention, but they are API surface tests, not integration tests.

@facumenzella facumenzella added the pr:other Changes to our CI configuration files and scripts label Oct 30, 2025
@facumenzella facumenzella marked this pull request as ready for review October 30, 2025 10:33
@facumenzella facumenzella requested a review from a team as a code owner October 30, 2025 10:33
Comment thread IntegrationTests/Assets/APITests/CustomerCenterPresenterAPITests.cs
Comment thread IntegrationTests/Assets/APITests/CustomerCenterPresenterAPITests.cs
Comment thread IntegrationTests/Assets/APITests/PaywallsPresenterAPITests.cs
@facumenzella facumenzella requested a review from vegaro November 3, 2025 17:47

@ajpallares ajpallares left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Is the title of this PR accurate? It says "Integration tests" but it actually adds API tests

@tonidero

tonidero commented Mar 2, 2026

Copy link
Copy Markdown
Contributor

It says "Integration tests" but it actually adds API tests

Yeah... in our Unity SDK, the naming could be improved, and the IntegrationTest project actually just does API tests 😅

@facumenzella facumenzella changed the title Add static integation tests for Paywalls and CustomerCenter Add API tests for Paywalls and CustomerCenter presenters Mar 10, 2026
@facumenzella facumenzella requested a review from ajpallares March 10, 2026 13:59
@facumenzella facumenzella enabled auto-merge (squash) March 10, 2026 14:00

@vegaro vegaro left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I think everything is covered. I added tests for PaywallsBehaviour in cesar/integration-tests2-missing if you want to take a look

facumenzella added a commit that referenced this pull request Mar 10, 2026
This belongs in the other/integration-tests2 branch (PR #735), not here.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
vegaro and others added 2 commits March 10, 2026 16:52
@facumenzella facumenzella merged commit fe63b8a into main Mar 10, 2026
8 checks passed
@facumenzella facumenzella deleted the other/integration-tests2 branch March 10, 2026 16:08
facumenzella added a commit that referenced this pull request Mar 10, 2026
This belongs in the other/integration-tests2 branch (PR #735), not here.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
ajpallares pushed a commit that referenced this pull request Mar 11, 2026
**This is an automatic release.**

## RevenueCat SDK
### ✨ New Features
* Add PurchaseLogic support for paywall presentation when
`PurchasesAreCompletedBy.MyApp` (#842) via Toni Rico (@tonidero)
### 📦 Dependency Updates
* [AUTOMATIC BUMP] Updates purchases-hybrid-common to 17.46.1 (#854) via
RevenueCat Git Bot (@RCGitBot)
* [Android
9.23.1](https://github.com/RevenueCat/purchases-android/releases/tag/9.23.1)
* [Android
9.23.0](https://github.com/RevenueCat/purchases-android/releases/tag/9.23.0)
* [iOS
5.61.0](https://github.com/RevenueCat/purchases-ios/releases/tag/5.61.0)
* [AUTOMATIC BUMP] Updates purchases-hybrid-common to 17.46.0 (#853) via
RevenueCat Git Bot (@RCGitBot)
* [Android
9.23.1](https://github.com/RevenueCat/purchases-android/releases/tag/9.23.1)
* [Android
9.23.0](https://github.com/RevenueCat/purchases-android/releases/tag/9.23.0)
* [iOS
5.61.0](https://github.com/RevenueCat/purchases-ios/releases/tag/5.61.0)
* [AUTOMATIC BUMP] Updates purchases-hybrid-common to 17.44.0 (#850) via
RevenueCat Git Bot (@RCGitBot)
* [Android
9.23.1](https://github.com/RevenueCat/purchases-android/releases/tag/9.23.1)
* [Android
9.23.0](https://github.com/RevenueCat/purchases-android/releases/tag/9.23.0)
* [iOS
5.61.0](https://github.com/RevenueCat/purchases-ios/releases/tag/5.61.0)

## RevenueCatUI SDK
### ✨ New Features
* Add full screen paywall presentation support (#839) via Cesar de la
Vega (@vegaro)

### 🔄 Other Changes
* Redo Subtester app and fix Android emulators support (#855) via Cesar
de la Vega (@vegaro)
* Support PaywallActivity screenOrientation manifest override (#844) via
Toni Rico (@tonidero)
* refactor: Use PaywallView instead of PaywallActivity on Android (#841)
via Toni Rico (@tonidero)
* Add API tests for Paywalls and CustomerCenter presenters (#735) via
Facundo Menzella (@facumenzella)
* Bump fastlane-plugin-revenuecat_internal from `f5c099b` to `e146447`
(#852) via dependabot[bot] (@dependabot[bot])
* Bump fastlane-plugin-revenuecat_internal from `8cd957f` to `f5c099b`
(#848) via dependabot[bot] (@dependabot[bot])
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr:other Changes to our CI configuration files and scripts

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants