Commit daf1246
committed
refactor(audience-sample): use EventPropertyKeys for typed-event field lookups in live-fire tests
SampleAppLiveFireTests filled the Resource, Purchase, and
MilestoneReached typed-event forms by looking up TextFields with
inline property-name strings ("currency", "amount", "value", "name").
The SDK already centralises these as EventPropertyKeys.Currency /
.Amount / .Value / .Name, made visible to the sample-app tests
through the existing InternalsVisibleTo grant.
Reference EventPropertyKeys.X from the five lookup sites instead of
re-typing the wire-format strings.
Discovered during the sample-app demo property-key centralisation
pass; kept separate because these keys belong to the SDK's typed-event
surface, not the sample-app demo catalogue.
Per the user's "everything random goes in a constant" stance.
Follow-up to SDK-272 (centralisation of duplicated literals).1 parent f1e936f commit daf1246
1 file changed
Lines changed: 5 additions & 5 deletions
Lines changed: 5 additions & 5 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
152 | 152 | | |
153 | 153 | | |
154 | 154 | | |
155 | | - | |
156 | | - | |
| 155 | + | |
| 156 | + | |
157 | 157 | | |
158 | 158 | | |
159 | 159 | | |
| |||
162 | 162 | | |
163 | 163 | | |
164 | 164 | | |
165 | | - | |
166 | | - | |
| 165 | + | |
| 166 | + | |
167 | 167 | | |
168 | 168 | | |
169 | 169 | | |
| |||
172 | 172 | | |
173 | 173 | | |
174 | 174 | | |
175 | | - | |
| 175 | + | |
176 | 176 | | |
177 | 177 | | |
178 | 178 | | |
| |||
0 commit comments