Skip to content

Conversation

sumeruchat
Copy link
Contributor

@sumeruchat sumeruchat commented Aug 27, 2025

🔹 Jira Ticket(s)

This PR adds the BCIT test suite for In-App Messages.
It also fixes the Push Notifications BCIT test suite.

Check the file InAppMessageIntegrationTests.

Demo Push:

BCIT_push_demo

Demo InApp

BCIT_inApp_demo

sumeruchat added 30 commits July 1, 2025 17:28
- Add comprehensive GitHub Actions workflows for parallel test execution
- Implement complete test suites for push notifications, in-app messages, embedded messages, and deep linking
- Add backend integration layer for API validation and metrics verification
- Create production-ready test infrastructure with proper error handling and cleanup
- Support for real Iterable API testing with server keys and certificates

Framework provides end-to-end validation of all business critical SDK workflows.
…ation-tests

* master:
  MOB-11819: Prepare for Release 6.5.13 (#930)
  MOB-11622: Improve multi-window support (#926)
  Revert "Reapply "test commit""
  Reapply "test commit"
  Revert "test commit"
  test commit
  Make Repository more AI agent friendly - add readme and test and build scripts for AI agents (#924)
  Fix compilation errors in multi-window support
  MOB-11622: Fix InApp messages displaying in wrong window with Stage Manager (#917)
  [MOB-8537] updates action runner logic to check allowed protocols (#769)
@sumeruchat sumeruchat marked this pull request as draft August 27, 2025 15:14
Copy link

codecov bot commented Aug 27, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 70.22%. Comparing base (6e2fdec) to head (d9e3b35).
⚠️ Report is 892 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff             @@
##           master     #942       +/-   ##
===========================================
- Coverage   85.19%   70.22%   -14.98%     
===========================================
  Files          91      109       +18     
  Lines        6301    10044     +3743     
===========================================
+ Hits         5368     7053     +1685     
- Misses        933     2991     +2058     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Base automatically changed from feature/MOB-10966-business-critical-integration-tests to master August 29, 2025 14:34
@sumeruchat sumeruchat force-pushed the feature/MOB-11464-in-app-tests branch from 35a68c9 to 6ce6d90 Compare September 2, 2025 16:54
# Conflicts:
#	.github/workflows/bcit-integration-test-push-notification.yml
#	tests/business-critical-integration/README.md
#	tests/business-critical-integration/integration-test-app/IterableSDK-Integration-Tester.xcodeproj/project.pbxproj
#	tests/business-critical-integration/integration-test-app/IterableSDK-Integration-Tester/App/AppDelegate+IntegrationTest.swift
#	tests/business-critical-integration/integration-test-app/IterableSDK-Integration-Tester/App/AppDelegate.swift
#	tests/business-critical-integration/integration-test-app/IterableSDK-Integration-Tester/App/Backend/BackendStatusViewController.swift
#	tests/business-critical-integration/integration-test-app/IterableSDK-Integration-Tester/App/Backend/IterableAPIClient.swift
#	tests/business-critical-integration/integration-test-app/IterableSDK-Integration-Tester/App/Backend/PushNotificationSender.swift
#	tests/business-critical-integration/integration-test-app/IterableSDK-Integration-Tester/Tests/IntegrationTestBase.swift
#	tests/business-critical-integration/integration-test-app/IterableSDK-Integration-Tester/Tests/PushNotificationIntegrationTests.swift
#	tests/business-critical-integration/scripts/build.sh
#	tests/business-critical-integration/scripts/run-tests.sh
#	tests/business-critical-integration/scripts/setup-local-environment.sh
#	tests/business-critical-integration/scripts/validate-setup.sh
@joaodordio joaodordio added BCIT Run Business Critical Integration Tests BCIT-PUSH Run business criticial push notification integration test labels Oct 9, 2025
@joaodordio joaodordio marked this pull request as ready for review October 9, 2025 13:40
@sumeruchat sumeruchat self-assigned this Oct 9, 2025
@sumeruchat
Copy link
Contributor Author

Reviewing functionality of in app messages

Code Looks good! Great job!

Three things

  1. We need to add a test for silent push
  2. We need to verify with Akshay what metrics validation means
  3. Verify with Akshay what it means to test handlers and navigation to different modules

@joaodordio joaodordio self-assigned this Oct 10, 2025
@sumeruchat
Copy link
Contributor Author

Did a review of the changes. Changes look good and they cover all criteria.

  1. The tests are actually failing
  2. The script does not detect the tests failing and reports success.

@joaodordio joaodordio changed the title MOB-11464 WIP: In App Business Critical Integration Tests MOB-11464: In App Business Critical Integration Tests Oct 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

BCIT Run Business Critical Integration Tests BCIT-INAPP BCIT-PUSH Run business criticial push notification integration test

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants