Skip to content

Commit d64727e

Browse files
committed
test(remote-config): avoid dereference on potentially undefined var
not sure how this could be undefined but something caused it to crash in a CI run, avoiding the crash now. The test would fail in practice if it did not crash, and then potentially retry
1 parent 93100c7 commit d64727e

File tree

1 file changed

+5
-1
lines changed

1 file changed

+5
-1
lines changed

packages/remote-config/e2e/config.e2e.js

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -748,7 +748,11 @@ describe('remoteConfig()', function () {
748748
if (!spy.called) return false;
749749
for (let i = 0; i < spy.callCount; i++) {
750750
const callbackEvent = spy.getCall(i).args[0];
751-
if (callbackEvent.updatedKeys && callbackEvent.updatedKeys.includes(paramName)) {
751+
if (
752+
callbackEvent &&
753+
callbackEvent.updatedKeys &&
754+
callbackEvent.updatedKeys.includes(paramName)
755+
) {
752756
return true;
753757
}
754758
}

0 commit comments

Comments
 (0)