Bluesky app fork with some witchin' additions 💫 witchsky.app
bluesky fork client

Reduce some Sentry logs (#2549)

* Reduce some Sentry logs

* Update tests

authored by

Eric Bailey and committed by
GitHub
9cbd3c09 64f7800b

+8 -7
+3 -3
src/state/persisted/__tests__/migrate.test.ts
··· 26 27 expect(AsyncStorage.getItem).toHaveBeenCalledWith('root') 28 expect(read).toHaveBeenCalledTimes(1) 29 - expect(logger.log).toHaveBeenCalledWith( 30 'persisted state: no migration needed', 31 ) 32 }) ··· 38 39 expect(AsyncStorage.getItem).toHaveBeenCalledWith('root') 40 expect(read).toHaveBeenCalledTimes(1) 41 - expect(logger.log).toHaveBeenCalledWith( 42 'persisted state: no migration needed', 43 ) 44 }) ··· 68 await migrate() 69 70 expect(write).toHaveBeenCalledWith(transform(fixtures.LEGACY_DATA_DUMP)) 71 - expect(logger.log).toHaveBeenCalledWith( 72 'persisted state: migrated legacy storage', 73 ) 74 })
··· 26 27 expect(AsyncStorage.getItem).toHaveBeenCalledWith('root') 28 expect(read).toHaveBeenCalledTimes(1) 29 + expect(logger.info).toHaveBeenCalledWith( 30 'persisted state: no migration needed', 31 ) 32 }) ··· 38 39 expect(AsyncStorage.getItem).toHaveBeenCalledWith('root') 40 expect(read).toHaveBeenCalledTimes(1) 41 + expect(logger.info).toHaveBeenCalledWith( 42 'persisted state: no migration needed', 43 ) 44 }) ··· 68 await migrate() 69 70 expect(write).toHaveBeenCalledWith(transform(fixtures.LEGACY_DATA_DUMP)) 71 + expect(logger.info).toHaveBeenCalledWith( 72 'persisted state: migrated legacy storage', 73 ) 74 })
+2 -2
src/state/persisted/legacy.ts
··· 164 165 if (validate.success) { 166 await write(newData) 167 - logger.log('persisted state: migrated legacy storage') 168 } else { 169 logger.error('persisted state: legacy data failed validation', { 170 error: validate.error, 171 }) 172 } 173 } else { 174 - logger.log('persisted state: no migration needed') 175 } 176 } catch (e: any) { 177 logger.error(e, {
··· 164 165 if (validate.success) { 166 await write(newData) 167 + logger.info('persisted state: migrated legacy storage') 168 } else { 169 logger.error('persisted state: legacy data failed validation', { 170 error: validate.error, 171 }) 172 } 173 } else { 174 + logger.info('persisted state: no migration needed') 175 } 176 } catch (e: any) { 177 logger.error(e, {
+2 -1
src/state/queries/feed.ts
··· 272 }, 273 }) 274 } catch (e) { 275 - logger.warn(`usePinnedFeedsInfos: failed to fetch ${uri}`, { 276 error: e, 277 }) 278 }
··· 272 }, 273 }) 274 } catch (e) { 275 + // expected failure 276 + logger.info(`usePinnedFeedsInfos: failed to fetch ${uri}`, { 277 error: e, 278 }) 279 }
+1 -1
src/state/queries/notifications/unread.tsx
··· 167 } 168 broadcast.postMessage({event: unreadCountStr}) 169 } catch (e) { 170 - logger.error('Failed to check unread notifications', {error: e}) 171 } 172 }, 173
··· 167 } 168 broadcast.postMessage({event: unreadCountStr}) 169 } catch (e) { 170 + logger.warn('Failed to check unread notifications', {error: e}) 171 } 172 }, 173