Skip to content
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

[SDK-2480] Created initial MobileBoost test apps and Github Actions #1461

Open
wants to merge 18 commits into
base: master
Choose a base branch
from

Conversation

nsingh-branch
Copy link
Contributor

Reference

SDK-2480 -- MobileBoost/GPT Driver Automated Testing

Summary

This change contains the first 3 linking testbed apps (DelayedInit, SwiftUI, Scenes-based) for iOS as well as a Github Action for triggering the tests on new PRs and for receiving the results. There were also a series of changes to the regular testbed to have it work better with MobileBoost tests.

Each of the testbeds requires the Branch.xcframework to be added for it to work properly, which is automatically done in the Github Action.

Motivation

To enhance our automated testing and QA processes.

Type Of Change

  • New feature (non-breaking change which adds functionality)

Testing Instructions

Try out the new sample apps by integrating them with the Branch.xcframework.
The github action be run by opening a PR or manually from the actions tab, but this will require it being merged in. It can also be tested locally with Act.

cc @BranchMetrics/saas-sdk-devs for visibility.

@NidhiDixit09
Copy link
Collaborator

NidhiDixit09 commented Jan 8, 2025

Workflow - Run MobileBoost Tests on PR / build-and-test (pull_request) is failing. Its failing I guess beacuse of some linking error.
Workflow - Unit tests / verify (pull_request) is also failing because of some version related changes I guess in Branch Testbed app.
cc: @nsingh-branch @gdeluna-branch

@@ -19,16 +19,20 @@
<key>CFBundlePackageType</key>
<string>APPL</string>
<key>CFBundleShortVersionString</key>
<string>1.1</string>
<string>3.7.0</string>
Copy link
Contributor

Choose a reason for hiding this comment

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

Tests are not happy with this change
XCTAssertEqualObjects(application.displayVersionString, @"1.1");

Changed integration method to SPM (local)
Changed deployment target
Updated enableLogging method temp.
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.

3 participants