feat: SDKE-534 Implement iOS Integration Test App #451
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
denischilik
requested review from
BrandonStalnaker,
jamesnrokt and
nickolas-dimitrakas
nickolas-dimitrakas
requested changes
Nov 13, 2025
- Add Tuist project configuration for IntegrationTests - Add WireMock recorder script for API mocking - Add initial test app implementation - Add proxy mappings for events and identify endpoints - Add README and generation scripts Add NO_OPEN parameter to generate.sh to control Xcode opening - Remove unused configuration variables - Add NO_OPEN parameter (defaults to false) - Conditionally call tuist generate with --no-open flag when NO_OPEN=true - Allows generate.sh to be used both standalone (opens Xcode) and from other scripts (skips Xcode) Refactor run_wiremock_recorder.sh to use detached mode with auto-cleanup - Add generate.sh call at the beginning to build project - Run WireMock in detached mode (-d flag) with named container - Extract logic into three functions: start_wiremock, wait_for_wiremock, stop_wiremock - Add health check waiting for WireMock to be ready (checks admin API) - Implement trap for automatic cleanup on EXIT/INT/TERM signals - Script now stays running until interrupted, ensuring proper container cleanup Add build_application function to run_wiremock_recorder.sh Add reset_simulators function Add find_device, start_simulator and install_application functions Add launch_application and wait_for_app_completion functions Add automatic device selection and test execution flow - Add find_available_device() function to automatically select available iPhone simulator - Add find_app_path() function to locate built application - Add complete test execution flow after WireMock starts - Update device priority list to include iPhone 17 - Move DEVICE_NAME to global variables for dynamic assignment Fix build command to use generic iOS Simulator platform and add quiet mode - Change destination to 'generic/platform=iOS Simulator' to avoid dependency on specific device name - Add -quiet flag to reduce build output verbosity - change order Update README: clarify purpose as API recording tool for testing - Update description to focus on recording API requests for later use in tests - Simplify prerequisites and overview sections - Remove detailed technical sections (Build Process, WireMock Recording, Device Selection, Benefits) - Consolidate troubleshooting into single section for port conflicts - Remove CI/CD integration examples - Update development workflow to emphasize recording purpose Use local SDK sources directly instead of building xcframework - Update Project.swift to use .local(path: '../') package reference - Remove Tuist/Package.swift to allow Xcode to resolve local package - Simplify generate.sh to only generate project (no xcframework build) - Update README to reflect new simpler approach - Remove temp_artifacts directory (no longer needed) Benefits: - No need to rebuild xcframework for every change - Xcode automatically picks up latest source code changes - Much faster development workflow - Simpler and more maintainable approach Remove generate.sh script and update documentation - Delete generate.sh (no longer needed - tuist generate is called directly) - Update README to remove generate.sh section - Simplify available scripts documentation to only show run_wiremock_recorder.sh - add documentation how to use tuist - cleanup .gitignore Remove comment
denischilik
force-pushed
the
feat/SDKE-534-Implement-iOS-Test-App
branch
from
b205baa to
f8c9b58
Compare
BrandonStalnaker
approved these changes
Nov 13, 2025
Collaborator
BrandonStalnaker
left a comment
BrandonStalnaker
left a comment
There was a problem hiding this comment.
LGTM thanks fro addressing that one thing earlier
jamesnrokt
approved these changes
Nov 13, 2025
|
|
||
| Tools for recording mParticle Apple SDK API requests using WireMock for later use in integration testing. | ||
|
|
||
| ## Prerequisites |
Collaborator
There was a problem hiding this comment.
We should probably add the installation of any tools just to CONTRIBUTING.md
Also there's mention of docker commands below so I'm assuming this should be a prerequisite too?
Contributor
Author
There was a problem hiding this comment.
Agree, will update in then PR
| // main.swift | ||
| // IntegrationTests | ||
| // | ||
| // Created by Denis Chilik on 11/4/25. |
Collaborator
There was a problem hiding this comment.
Nit: Should be either removed or changed to Rokt. Ideally this should be in the project config
Contributor
Author
There was a problem hiding this comment.
Agree, but let discuss with after the daily first. To keep it consistent.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Background
This change introduces an iOS Integration Test App for the mParticle Apple SDK.
Previously, there was no automated way to record and replay SDK API interactions for integration testing.
The new Integration Test App, combined with WireMock recording, provides a standardized framework to simulate and verify SDK network behavior across SDK updates.
What Has Changed
Checklist
Reference Issue (For employees only. Ignore if you are an outside contributor)