Skip to content

Commit

Permalink
update test logic
Browse files Browse the repository at this point in the history
  • Loading branch information
russellwheatley committed Dec 20, 2024
1 parent 6b9ff91 commit 4cde55c
Showing 1 changed file with 5 additions and 5 deletions.
10 changes: 5 additions & 5 deletions packages/app/lib/common/unitTestUtils.ts
Original file line number Diff line number Diff line change
Expand Up @@ -40,11 +40,11 @@ export const createCheckV9Deprecation = (moduleNames: string[]): CheckV9Deprecat
const firebaseAppDeprecationMessage = warnMessage.includes('Please use `getApp()` instead.');
if (checkFirebaseAppDeprecationWarning) {
throw new Error(`Console warn was called unexpectedly with: ${warnMessage}`);

Check warning on line 42 in packages/app/lib/common/unitTestUtils.ts

View check run for this annotation

Codecov / codecov/patch

packages/app/lib/common/unitTestUtils.ts#L42

Added line #L42 was not covered by tests
}

if (!firebaseAppDeprecationMessage) {
// we want to ignore all firebase app deprecation warnings (e.g. "Please use `getApp()` instead.") unless actually testing for it which we do above
throw new Error(`Console warn was called unexpectedly with: ${warnMessage}`);
} else {
if (!firebaseAppDeprecationMessage) {
// we want to ignore all firebase app deprecation warnings (e.g. "Please use `getApp()` instead.") unless actually testing for it which we do above
throw new Error(`Console warn was called unexpectedly with: ${warnMessage}`);

Check warning on line 46 in packages/app/lib/common/unitTestUtils.ts

View check run for this annotation

Codecov / codecov/patch

packages/app/lib/common/unitTestUtils.ts#L46

Added line #L46 was not covered by tests
}
}
});
// Do not call `mockRestore()` unless removing the spy
Expand Down

0 comments on commit 4cde55c

Please sign in to comment.