Commit e46e63a
[SDK-115] Re-enable embedded test: send membershipLevel, allow tester:// URLs
Two missing pieces vs the iOS BCIT embedded test:
1. The BCIT campaign's audience predicate is on `membershipLevel == "premium"`,
not on `isPremium == true`. Android was sending the wrong field, so the
audience never matched and /api/embedded-messaging/messages returned
`placements: []`. iOS sends `membershipLevel: "premium" / "standard"`
(see EmbeddedMessageTestViewModel.swift:90); this commit does the same
from both the test and the on-screen toggle.
2. The deeplink button on placement 2157 fires `openUrl` with
`tester://testview`. The Iterable SDK drops URLs whose scheme isn't in
IterableConfig.allowedProtocols. iOS configures
`["tester", "https", "http"]`; Android did not, so the SDK silently
dropped the URL and the URL handler never fired. Add the same allowed
protocols in BaseIntegrationTest.
Removed the @ignore on testEmbeddedMessageMVP. Renamed the
isPremium-themed toggle / labels in EmbeddedMessageTestActivity to
membership-level wording for parity.
Local verification (CI mode, dated user, full non-push suite):
Tests 7/7 completed. (0 skipped) (0 failed)
BUILD SUCCESSFUL
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>1 parent f2b2afd commit e46e63a
3 files changed
Lines changed: 28 additions & 36 deletions
File tree
- integration-tests/src
- androidTest/java/com/iterable/integration/tests
- main
- java/com/iterable/integration/tests/activities
- res/layout
Lines changed: 8 additions & 15 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
21 | 21 | | |
22 | 22 | | |
23 | 23 | | |
24 | | - | |
25 | 24 | | |
26 | 25 | | |
27 | 26 | | |
| |||
110 | 109 | | |
111 | 110 | | |
112 | 111 | | |
113 | | - | |
114 | | - | |
115 | | - | |
116 | | - | |
117 | | - | |
118 | | - | |
119 | 112 | | |
120 | 113 | | |
121 | 114 | | |
| |||
141 | 134 | | |
142 | 135 | | |
143 | 136 | | |
144 | | - | |
145 | | - | |
146 | | - | |
147 | | - | |
| 137 | + | |
| 138 | + | |
| 139 | + | |
| 140 | + | |
148 | 141 | | |
149 | 142 | | |
150 | | - | |
| 143 | + | |
151 | 144 | | |
152 | 145 | | |
153 | 146 | | |
154 | | - | |
| 147 | + | |
155 | 148 | | |
156 | 149 | | |
157 | 150 | | |
| |||
260 | 253 | | |
261 | 254 | | |
262 | 255 | | |
263 | | - | |
264 | | - | |
| 256 | + | |
| 257 | + | |
265 | 258 | | |
266 | 259 | | |
267 | 260 | | |
| |||
Lines changed: 18 additions & 19 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
38 | 38 | | |
39 | 39 | | |
40 | 40 | | |
41 | | - | |
| 41 | + | |
42 | 42 | | |
43 | | - | |
| 43 | + | |
44 | 44 | | |
45 | | - | |
| 45 | + | |
46 | 46 | | |
47 | | - | |
| 47 | + | |
48 | 48 | | |
49 | 49 | | |
50 | 50 | | |
51 | 51 | | |
52 | | - | |
53 | | - | |
| 52 | + | |
| 53 | + | |
54 | 54 | | |
55 | | - | |
| 55 | + | |
56 | 56 | | |
57 | | - | |
| 57 | + | |
58 | 58 | | |
59 | 59 | | |
60 | 60 | | |
61 | 61 | | |
62 | 62 | | |
63 | 63 | | |
64 | | - | |
65 | | - | |
66 | | - | |
67 | | - | |
68 | | - | |
| 64 | + | |
| 65 | + | |
| 66 | + | |
| 67 | + | |
69 | 68 | | |
70 | | - | |
| 69 | + | |
71 | 70 | | |
72 | | - | |
| 71 | + | |
73 | 72 | | |
74 | 73 | | |
75 | | - | |
76 | | - | |
77 | | - | |
| 74 | + | |
| 75 | + | |
| 76 | + | |
78 | 77 | | |
79 | 78 | | |
80 | | - | |
| 79 | + | |
81 | 80 | | |
82 | 81 | | |
83 | 82 | | |
| |||
Lines changed: 2 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
24 | 24 | | |
25 | 25 | | |
26 | 26 | | |
27 | | - | |
| 27 | + | |
28 | 28 | | |
29 | 29 | | |
30 | 30 | | |
| |||
38 | 38 | | |
39 | 39 | | |
40 | 40 | | |
41 | | - | |
| 41 | + | |
42 | 42 | | |
43 | 43 | | |
44 | 44 | | |
| |||
0 commit comments