Skip to content

Commit

Permalink
chore: fix nav import and tests (#1200)
Browse files Browse the repository at this point in the history
Signed-off-by: Jason C. Leach <[email protected]>
  • Loading branch information
jleach authored Jul 19, 2024
1 parent f65e147 commit e1daf27
Show file tree
Hide file tree
Showing 71 changed files with 211 additions and 425 deletions.
2 changes: 1 addition & 1 deletion packages/legacy/core/App/components/buttons/HeaderHome.tsx
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
import { useNavigation } from '@react-navigation/core'
import { useNavigation } from '@react-navigation/native'
import React from 'react'
import { useTranslation } from 'react-i18next'

Expand Down
2 changes: 1 addition & 1 deletion packages/legacy/core/App/components/buttons/InfoIcon.tsx
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
import { useNavigation } from '@react-navigation/core'
import { useNavigation } from '@react-navigation/native'
import { StackNavigationProp } from '@react-navigation/stack'
import React from 'react'
import { useTranslation } from 'react-i18next'
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
import { useNavigation } from '@react-navigation/core'
import { useNavigation } from '@react-navigation/native'
import { StackNavigationProp } from '@react-navigation/stack'
import React from 'react'
import { useTranslation } from 'react-i18next'
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ import {
} from '@credo-ts/core'
import { useAgent, useConnectionById } from '@credo-ts/react-hooks'
import { markProofAsViewed } from '@hyperledger/aries-bifold-verifier'
import { useNavigation } from '@react-navigation/core'
import { useNavigation } from '@react-navigation/native'
import { StackNavigationProp } from '@react-navigation/stack'
import React, { useEffect, useState } from 'react'
import { useTranslation } from 'react-i18next'
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
import { useNavigation } from '@react-navigation/core'
import { useNavigation } from '@react-navigation/native'
import { StackNavigationProp } from '@react-navigation/stack'
import React, { useState } from 'react'
import { useTranslation } from 'react-i18next'
Expand Down
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
import { CredentialExchangeRecord } from '@credo-ts/core'
import { BrandingOverlay } from '@hyperledger/aries-oca'
import { Attribute, CredentialOverlay, Predicate } from '@hyperledger/aries-oca/build/legacy'
import { useNavigation } from '@react-navigation/core'
import { useNavigation } from '@react-navigation/native'
import startCase from 'lodash.startcase'
import React, { useEffect, useState } from 'react'
import { useTranslation } from 'react-i18next'
Expand Down
2 changes: 1 addition & 1 deletion packages/legacy/core/App/components/misc/QRScanner.tsx
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
import { useNavigation } from '@react-navigation/core'
import { useNavigation } from '@react-navigation/native'
import React, { useState } from 'react'
import { useTranslation } from 'react-i18next'
import { View, Modal, Pressable, StyleSheet, Text } from 'react-native'
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
import { useNavigation } from '@react-navigation/core'
import { useNavigation } from '@react-navigation/native'
import { StackNavigationProp } from '@react-navigation/stack'
import React, { useState } from 'react'
import { useTranslation } from 'react-i18next'
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
import { useNavigation } from '@react-navigation/core'
import { useNavigation } from '@react-navigation/native'
import { StackNavigationProp } from '@react-navigation/stack'
import React, { useEffect, useState } from 'react'
import { useTranslation } from 'react-i18next'
Expand Down
6 changes: 3 additions & 3 deletions packages/legacy/core/App/hooks/chat-messages.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ import {
} from '@credo-ts/core'
import { useBasicMessagesByConnectionId } from '@credo-ts/react-hooks'
import { isPresentationReceived } from '@hyperledger/aries-bifold-verifier'
import { useNavigation } from '@react-navigation/core'
import { useNavigation } from '@react-navigation/native'
import { StackNavigationProp } from '@react-navigation/stack'
import React, { Fragment, useEffect, useState } from 'react'
import { useTranslation } from 'react-i18next'
Expand Down Expand Up @@ -90,9 +90,9 @@ export const useChatMessagesByConnection = (connection: ConnectionRecord): Exten
const transformedMessages: Array<ExtendedChatMessage> = basicMessages.map((record: BasicMessageRecord) => {
const role = getMessageEventRole(record)
// eslint-disable-next-line
const linkRegex = /(?:https?\:\/\/\w+(?:\.\w+)+\S*)|(?:[\w\d\.\_\-]+@\w+(?:\.\w+)+)/gmi
const linkRegex = /(?:https?\:\/\/\w+(?:\.\w+)+\S*)|(?:[\w\d\.\_\-]+@\w+(?:\.\w+)+)/gim
// eslint-disable-next-line
const mailRegex = /^[\w\d\.\_\-]+@\w+(?:\.\w+)+$/gmi
const mailRegex = /^[\w\d\.\_\-]+@\w+(?:\.\w+)+$/gim
const links = record.content.match(linkRegex) ?? []
const handleLinkPress = (link: string) => {
if (link.match(mailRegex)) {
Expand Down
2 changes: 1 addition & 1 deletion packages/legacy/core/App/navigators/OnboardingStack.tsx
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
/* eslint-disable @typescript-eslint/no-non-null-assertion */
import { ParamListBase, RouteConfig, StackNavigationState, useNavigation } from '@react-navigation/core'
import { ParamListBase, RouteConfig, StackNavigationState, useNavigation } from '@react-navigation/native'
import { StackNavigationOptions, StackNavigationProp, createStackNavigator } from '@react-navigation/stack'
import { StackNavigationEventMap } from '@react-navigation/stack/lib/typescript/src/types'
import React from 'react'
Expand Down
2 changes: 1 addition & 1 deletion packages/legacy/core/App/navigators/RootStack.tsx
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
import { ProofState } from '@credo-ts/core'
import { useAgent, useProofByState } from '@credo-ts/react-hooks'
import { ProofCustomMetadata, ProofMetadata } from '@hyperledger/aries-bifold-verifier'
import { useNavigation } from '@react-navigation/core'
import { useNavigation } from '@react-navigation/native'
import { StackCardStyleInterpolator, StackNavigationProp, createStackNavigator } from '@react-navigation/stack'
import React, { useEffect, useRef, useState } from 'react'
import { useTranslation } from 'react-i18next'
Expand Down
3 changes: 1 addition & 2 deletions packages/legacy/core/App/screens/AttemptLockout.tsx
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
import { useNavigation } from '@react-navigation/core'
import { CommonActions } from '@react-navigation/native'
import { useNavigation, CommonActions } from '@react-navigation/native'
import React, { useEffect, useState } from 'react'
import { useTranslation } from 'react-i18next'
import { StyleSheet, Text, View } from 'react-native'
Expand Down
2 changes: 1 addition & 1 deletion packages/legacy/core/App/screens/Chat.tsx
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import { BasicMessageRepository, ConnectionRecord } from '@credo-ts/core'
import { useAgent, useBasicMessagesByConnectionId, useConnectionById } from '@credo-ts/react-hooks'
import { useIsFocused, useNavigation } from '@react-navigation/core'
import { useIsFocused, useNavigation } from '@react-navigation/native'
import { StackNavigationProp, StackScreenProps } from '@react-navigation/stack'
import React, { useCallback, useEffect, useMemo, useState } from 'react'
import { useTranslation } from 'react-i18next'
Expand Down
2 changes: 1 addition & 1 deletion packages/legacy/core/App/screens/ContactDetails.tsx
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import { CredentialState } from '@credo-ts/core'
import { useAgent, useConnectionById, useCredentialByState } from '@credo-ts/react-hooks'
import { useNavigation } from '@react-navigation/core'
import { useNavigation } from '@react-navigation/native'
import { StackNavigationProp, StackScreenProps } from '@react-navigation/stack'
import React, { useCallback, useMemo, useState } from 'react'
import { useTranslation } from 'react-i18next'
Expand Down
2 changes: 1 addition & 1 deletion packages/legacy/core/App/screens/CredentialOffer.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ import { CredentialPreviewAttribute } from '@credo-ts/core'
import { useCredentialById } from '@credo-ts/react-hooks'
import { BrandingOverlay } from '@hyperledger/aries-oca'
import { Attribute, CredentialOverlay } from '@hyperledger/aries-oca/build/legacy'
import { useIsFocused } from '@react-navigation/core'
import { useIsFocused } from '@react-navigation/native'
import { StackScreenProps } from '@react-navigation/stack'
import React, { useEffect, useState } from 'react'
import { useTranslation } from 'react-i18next'
Expand Down
2 changes: 1 addition & 1 deletion packages/legacy/core/App/screens/CredentialOfferAccept.tsx
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import { CredentialState } from '@credo-ts/core'
import { useCredentialById } from '@credo-ts/react-hooks'
import { useNavigation } from '@react-navigation/core'
import { useNavigation } from '@react-navigation/native'
import React, { useEffect, useState } from 'react'
import { useTranslation } from 'react-i18next'
import { AccessibilityInfo, Modal, ScrollView, StyleSheet, Text, View } from 'react-native'
Expand Down
3 changes: 1 addition & 2 deletions packages/legacy/core/App/screens/ListCredentials.tsx
Original file line number Diff line number Diff line change
@@ -1,8 +1,7 @@
import { AnonCredsCredentialMetadataKey } from '@credo-ts/anoncreds'
import { CredentialState } from '@credo-ts/core'
import { useCredentialByState } from '@credo-ts/react-hooks'
import { useNavigation } from '@react-navigation/core'
import { useIsFocused } from '@react-navigation/native'
import { useNavigation, useIsFocused } from '@react-navigation/native'
import { StackNavigationProp } from '@react-navigation/stack'
import React, { useEffect } from 'react'
import { useTranslation } from 'react-i18next'
Expand Down
2 changes: 1 addition & 1 deletion packages/legacy/core/App/screens/NameWallet.tsx
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
import { useAgent } from '@credo-ts/react-hooks'
import { useNavigation } from '@react-navigation/core'
import { useNavigation } from '@react-navigation/native'
import React, { useState } from 'react'
import { useTranslation } from 'react-i18next'
import { StyleSheet, Text, View } from 'react-native'
Expand Down
3 changes: 1 addition & 2 deletions packages/legacy/core/App/screens/Onboarding.tsx
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
import { useNavigation } from '@react-navigation/core'
import { useFocusEffect } from '@react-navigation/native'
import { useNavigation, useFocusEffect } from '@react-navigation/native'
import { StackNavigationProp } from '@react-navigation/stack'
import React, { Ref, useCallback, useEffect, useRef, useState } from 'react'
import { useTranslation } from 'react-i18next'
Expand Down
3 changes: 1 addition & 2 deletions packages/legacy/core/App/screens/PINCreate.tsx
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
import { ParamListBase, useNavigation } from '@react-navigation/core'
import { CommonActions } from '@react-navigation/native'
import { CommonActions, ParamListBase, useNavigation } from '@react-navigation/native'
import { StackNavigationProp, StackScreenProps } from '@react-navigation/stack'
import React, { useState, useRef, useEffect } from 'react'
import { useTranslation } from 'react-i18next'
Expand Down
3 changes: 1 addition & 2 deletions packages/legacy/core/App/screens/PINEnter.tsx
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
import { useNavigation } from '@react-navigation/core'
import { CommonActions } from '@react-navigation/native'
import { useNavigation, CommonActions } from '@react-navigation/native'
import React, { useEffect, useState } from 'react'
import { useTranslation } from 'react-i18next'
import { Keyboard, StyleSheet, Text, Image, View, DeviceEventEmitter } from 'react-native'
Expand Down
2 changes: 1 addition & 1 deletion packages/legacy/core/App/screens/Preface.tsx
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
import { useNavigation } from '@react-navigation/core'
import { useNavigation } from '@react-navigation/native'
import { StackNavigationProp } from '@react-navigation/stack'
import React, { useState } from 'react'
import { useTranslation } from 'react-i18next'
Expand Down
2 changes: 1 addition & 1 deletion packages/legacy/core/App/screens/ProofRequest.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ import {
import { CredentialExchangeRecord, DifPexInputDescriptorToCredentials, ProofState } from '@credo-ts/core'
import { useConnectionById, useProofById } from '@credo-ts/react-hooks'
import { Attribute, Predicate } from '@hyperledger/aries-oca/build/legacy'
import { useIsFocused } from '@react-navigation/core'
import { useIsFocused } from '@react-navigation/native'
import moment from 'moment'
import React, { useEffect, useMemo, useState } from 'react'
import { useTranslation } from 'react-i18next'
Expand Down
2 changes: 1 addition & 1 deletion packages/legacy/core/App/screens/ProofRequestAccept.tsx
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import { ProofState } from '@credo-ts/core'
import { useProofById } from '@credo-ts/react-hooks'
import { useNavigation } from '@react-navigation/core'
import { useNavigation } from '@react-navigation/native'
import React, { useEffect, useState } from 'react'
import { useTranslation } from 'react-i18next'
import { Modal, ScrollView, StyleSheet, Text, View } from 'react-native'
Expand Down
3 changes: 1 addition & 2 deletions packages/legacy/core/App/screens/ProofRequesting.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -10,8 +10,7 @@ import {
linkProofWithTemplate,
sendProofRequest,
} from '@hyperledger/aries-bifold-verifier'
import { useIsFocused } from '@react-navigation/core'
import { useFocusEffect } from '@react-navigation/native'
import { useIsFocused, useFocusEffect } from '@react-navigation/native'
import React, { useCallback, useEffect, useState } from 'react'
import { useTranslation } from 'react-i18next'
import {
Expand Down
2 changes: 1 addition & 1 deletion packages/legacy/core/App/screens/PushNotification.tsx
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
import { useAgent } from '@credo-ts/react-hooks'
import { CommonActions, ParamListBase, useNavigation } from '@react-navigation/core'
import { CommonActions, ParamListBase, useNavigation } from '@react-navigation/native'
import { StackNavigationProp, StackScreenProps } from '@react-navigation/stack'
import React, { useMemo, useState } from 'react'
import { useTranslation } from 'react-i18next'
Expand Down
2 changes: 1 addition & 1 deletion packages/legacy/core/App/screens/RenameContact.tsx
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
import { useConnectionById } from '@credo-ts/react-hooks'
import { useNavigation } from '@react-navigation/core'
import { useNavigation } from '@react-navigation/native'
import { StackScreenProps } from '@react-navigation/stack'
import React, { useState } from 'react'
import { useTranslation } from 'react-i18next'
Expand Down
3 changes: 1 addition & 2 deletions packages/legacy/core/App/screens/Splash.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -4,8 +4,7 @@ import { useAgent } from '@credo-ts/react-hooks'
import { agentDependencies } from '@credo-ts/react-native'
import { RemoteOCABundleResolver } from '@hyperledger/aries-oca/build/legacy'
import { GetCredentialDefinitionRequest, GetSchemaRequest } from '@hyperledger/indy-vdr-shared'
import { useNavigation } from '@react-navigation/core'
import { CommonActions } from '@react-navigation/native'
import { useNavigation, CommonActions } from '@react-navigation/native'
import React, { useEffect, useState } from 'react'
import { useTranslation } from 'react-i18next'
import { DeviceEventEmitter, StyleSheet } from 'react-native'
Expand Down
2 changes: 1 addition & 1 deletion packages/legacy/core/App/screens/Terms.tsx
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
import { useNavigation } from '@react-navigation/core'
import { useNavigation } from '@react-navigation/native'
import { StackNavigationProp } from '@react-navigation/stack'
import React, { useCallback, useState } from 'react'
import { useTranslation } from 'react-i18next'
Expand Down
2 changes: 1 addition & 1 deletion packages/legacy/core/App/screens/UseBiometry.tsx
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
import { CommonActions, useNavigation } from '@react-navigation/core'
import { CommonActions, useNavigation } from '@react-navigation/native'
import { StackNavigationProp } from '@react-navigation/stack'
import React, { useState, useEffect } from 'react'
import { useTranslation } from 'react-i18next'
Expand Down
2 changes: 1 addition & 1 deletion packages/legacy/core/App/types/navigators.ts
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
import { CredentialExchangeRecord } from '@credo-ts/core'
import { NavigatorScreenParams } from '@react-navigation/core'
import { NavigatorScreenParams } from '@react-navigation/native'
import { StackNavigationOptions } from '@react-navigation/stack'

export enum Screens {
Expand Down
45 changes: 45 additions & 0 deletions packages/legacy/core/__mocks__/@react-navigation/native.ts
Original file line number Diff line number Diff line change
@@ -0,0 +1,45 @@
// import { useFocusEffect } from '@react-navigation/native'

const navigate = jest.fn()
const dispatch = jest.fn()
const replace = jest.fn()
const navigation = {
__timestamp: process.hrtime(),
navigate,
replace,
setOptions: jest.fn(),
getParent: () => {
return {
navigate,
dispatch,
replace,
}
},
getState: jest.fn(() => ({
index: jest.fn(),
})),
goBack: jest.fn(),
pop: jest.fn(),
reset: jest.fn(),
isFocused: () => true,
dispatch,
}

const useNavigation = () => {
return navigation
}

const useIsFocused = () => {
return true
}

const CommonActions = {
navigate: jest.fn(),
reset: jest.fn(),
goBack: jest.fn(),
}

const useFocusEffect = jest.fn()
const createNavigatorFactory = jest.fn()

export { useNavigation, useIsFocused, useFocusEffect, createNavigatorFactory, CommonActions }
14 changes: 0 additions & 14 deletions packages/legacy/core/__mocks__/custom/@react-navigation/native.ts

This file was deleted.

Original file line number Diff line number Diff line change
@@ -1,16 +1,10 @@
import { useNavigation } from '@react-navigation/core'
import { useNavigation } from '@react-navigation/native'
import { render, fireEvent, act } from '@testing-library/react-native'
import React from 'react'
import CameraDisclosureModal from '../../App/components/modals/CameraDisclosureModal'
import { testIdWithKey } from '../../App/utils/testable'

let requestCameraUse = jest.fn(() => Promise.resolve(true))
jest.mock('@react-navigation/core', () => {
return require('../../__mocks__/custom/@react-navigation/core')
})
jest.mock('@react-navigation/native', () => {
return require('../../__mocks__/custom/@react-navigation/native')
})

describe('CameraDisclosureModal Component', () => {
beforeAll(() => {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -16,13 +16,6 @@ jest.mock('../../App/container-api')
jest.mock('react-native/Libraries/EventEmitter/NativeEventEmitter')
jest.mock('@react-native-community/netinfo', () => mockRNCNetInfo)
jest.mock('react-native/Libraries/Animated/NativeAnimatedHelper')
jest.mock('@react-navigation/core', () => {
return require('../../__mocks__/custom/@react-navigation/core')
})
jest.mock('@react-navigation/native', () => {
return require('../../__mocks__/custom/@react-navigation/native')
})

jest.mock('@hyperledger/anoncreds-react-native', () => ({}))
jest.mock('@hyperledger/aries-askar-react-native', () => ({}))
jest.mock('@hyperledger/indy-vdr-react-native', () => ({}))
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,13 +3,6 @@ import React from 'react'

import HeaderRightHome from '../../App/components/buttons/HeaderHome'

jest.mock('@react-navigation/core', () => {
return require('../../__mocks__/custom/@react-navigation/core')
})
jest.mock('@react-navigation/native', () => {
return require('../../__mocks__/custom/@react-navigation/native')
})

describe('Header Right Home Component', () => {
test('Renders correctly', () => {
const tree = render(<HeaderRightHome />)
Expand Down
8 changes: 1 addition & 7 deletions packages/legacy/core/__tests__/components/NewQRView.test.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -6,15 +6,9 @@ import NewQRView from '../../App/components/misc/NewQRView'
import { ConfigurationContext } from '../../App/contexts/configuration'
import { StoreProvider, defaultState } from '../../App/contexts/store'
import { testIdWithKey } from '../../App/utils/testable'
import { useNavigation } from '../../__mocks__/custom/@react-navigation/core'
import { useNavigation } from '@react-navigation/native'
import configurationContext from '../contexts/configuration'

jest.mock('@react-navigation/core', () => {
return require('../../__mocks__/custom/@react-navigation/core')
})
jest.mock('@react-navigation/native', () => {
return require('../../__mocks__/custom/@react-navigation/native')
})
jest.mock('react-native-orientation-locker', () => {
return require('../../__mocks__/custom/react-native-orientation-locker')
})
Expand Down
7 changes: 0 additions & 7 deletions packages/legacy/core/__tests__/screens/Chat.test.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -16,13 +16,6 @@ import { BasicMessageMetadata } from '../../App/types/metadata'
import configurationContext from '../contexts/configuration'
import navigationContext from '../contexts/navigation'

jest.mock('@react-navigation/core', () => {
return require('../../__mocks__/custom/@react-navigation/core')
})
jest.mock('@react-navigation/native', () => {
return require('../../__mocks__/custom/@react-navigation/native')
})

const props = { params: { connectionId: '1' } }

const connection = new ConnectionRecord({
Expand Down
Loading

0 comments on commit e1daf27

Please sign in to comment.