Skip to content

Commit

Permalink
fix tests
Browse files Browse the repository at this point in the history
  • Loading branch information
@frosso
frosso committed Feb 24, 2025
1 parent 471f410 commit 85cdbe7
Show file tree
Hide file tree
Showing 3 changed files with 89 additions and 101 deletions.
0 ...tor/test/add-payment-methods-task.test.js → ..._tests__/add-payment-methods-task.test.js
View file
File renamed without changes.
0 .../currency-information-for-methods.test.js → .../currency-information-for-methods.test.js
View file
File renamed without changes.
190 changes: 89 additions & 101 deletions ...selector/test/setup-complete-task.test.js → ...tor/__tests__/setup-complete-task.test.js
View file
Original file line number Diff line number Diff line change
Expand Up @@ -19,31 +19,29 @@ jest.mock( '../../../data', () => ( {
useEnabledPaymentMethodIds: jest.fn(),
} ) );

const SettingsContextProvider = ( { children } ) => (
<WCPaySettingsContext.Provider
value={ { featureFlags: { multiCurrency: true } } }
>
{ children }
</WCPaySettingsContext.Provider>
);
const renderWithSettingsProvider = ( ui ) =>
render(
<WCPaySettingsContext.Provider value={ global.wcpaySettings }>
{ ui }
</WCPaySettingsContext.Provider>
);

describe( 'SetupComplete', () => {
beforeEach( () => {
useEnabledPaymentMethodIds.mockReturnValue( [
[ 'card', 'bancontact', 'eps', 'ideal', 'p24', 'sepa_debit' ],
() => null,
] );
global.wcpaySettings = { featureFlags: { multiCurrency: true } };
} );

it( 'renders setup complete messaging when context value is undefined', () => {
render(
<SettingsContextProvider>
<WizardContext.Provider value={ { completedTasks: {} } }>
<WizardTaskContext.Provider value={ { isActive: true } }>
<SetupComplete />
</WizardTaskContext.Provider>
</WizardContext.Provider>
</SettingsContextProvider>
renderWithSettingsProvider(
<WizardContext.Provider value={ { completedTasks: {} } }>
<WizardTaskContext.Provider value={ { isActive: true } }>
<SetupComplete />
</WizardTaskContext.Provider>
</WizardContext.Provider>
);

expect(
Expand All @@ -52,18 +50,16 @@ describe( 'SetupComplete', () => {
} );

it( 'renders setup complete messaging when context value is `true`', () => {
render(
<SettingsContextProvider>
<WizardContext.Provider
value={ {
completedTasks: { 'add-payment-methods': true },
} }
>
<WizardTaskContext.Provider value={ { isActive: true } }>
<SetupComplete />
</WizardTaskContext.Provider>
</WizardContext.Provider>
</SettingsContextProvider>
renderWithSettingsProvider(
<WizardContext.Provider
value={ {
completedTasks: { 'add-payment-methods': true },
} }
>
<WizardTaskContext.Provider value={ { isActive: true } }>
<SetupComplete />
</WizardTaskContext.Provider>
</WizardContext.Provider>
);

expect( screen.getByText( /Setup complete/ ) ).toHaveTextContent(
Expand All @@ -72,29 +68,27 @@ describe( 'SetupComplete', () => {
} );

it( 'renders setup complete messaging when context value says that methods have not changed', () => {
render(
<SettingsContextProvider>
<WizardContext.Provider
value={ {
completedTasks: {
'add-payment-methods': {
initialMethods: [
'card',
'bancontact',
'eps',
'ideal',
'p24',
'sepa_debit',
],
},
renderWithSettingsProvider(
<WizardContext.Provider
value={ {
completedTasks: {
'add-payment-methods': {
initialMethods: [
'card',
'bancontact',
'eps',
'ideal',
'p24',
'sepa_debit',
],
},
} }
>
<WizardTaskContext.Provider value={ { isActive: true } }>
<SetupComplete />
</WizardTaskContext.Provider>
</WizardContext.Provider>
</SettingsContextProvider>
},
} }
>
<WizardTaskContext.Provider value={ { isActive: true } }>
<SetupComplete />
</WizardTaskContext.Provider>
</WizardContext.Provider>
);

expect( screen.getByText( /Setup complete/ ) ).toHaveTextContent(
Expand All @@ -107,29 +101,27 @@ describe( 'SetupComplete', () => {
[ 'card', 'ideal' ],
() => null,
] );
render(
<SettingsContextProvider>
<WizardContext.Provider
value={ {
completedTasks: {
'add-payment-methods': {
initialMethods: [
'card',
'bancontact',
'eps',
'ideal',
'p24',
'sepa_debit',
],
},
renderWithSettingsProvider(
<WizardContext.Provider
value={ {
completedTasks: {
'add-payment-methods': {
initialMethods: [
'card',
'bancontact',
'eps',
'ideal',
'p24',
'sepa_debit',
],
},
} }
>
<WizardTaskContext.Provider value={ { isActive: true } }>
<SetupComplete />
</WizardTaskContext.Provider>
</WizardContext.Provider>
</SettingsContextProvider>
},
} }
>
<WizardTaskContext.Provider value={ { isActive: true } }>
<SetupComplete />
</WizardTaskContext.Provider>
</WizardContext.Provider>
);

expect( screen.getByText( /Setup complete/ ) ).toHaveTextContent(
Expand All @@ -142,22 +134,20 @@ describe( 'SetupComplete', () => {
[ 'card', 'ideal' ],
() => null,
] );
render(
<SettingsContextProvider>
<WizardContext.Provider
value={ {
completedTasks: {
'add-payment-methods': {
initialMethods: [ 'card' ],
},
renderWithSettingsProvider(
<WizardContext.Provider
value={ {
completedTasks: {
'add-payment-methods': {
initialMethods: [ 'card' ],
},
} }
>
<WizardTaskContext.Provider value={ { isActive: true } }>
<SetupComplete />
</WizardTaskContext.Provider>
</WizardContext.Provider>
</SettingsContextProvider>
},
} }
>
<WizardTaskContext.Provider value={ { isActive: true } }>
<SetupComplete />
</WizardTaskContext.Provider>
</WizardContext.Provider>
);

expect( screen.getByText( /Setup complete/ ) ).toHaveTextContent(
Expand All @@ -177,22 +167,20 @@ describe( 'SetupComplete', () => {
[ 'card', ...additionalMethods ],
() => null,
] );
render(
<SettingsContextProvider>
<WizardContext.Provider
value={ {
completedTasks: {
'add-payment-methods': {
initialMethods: [ 'card' ],
},
renderWithSettingsProvider(
<WizardContext.Provider
value={ {
completedTasks: {
'add-payment-methods': {
initialMethods: [ 'card' ],
},
} }
>
<WizardTaskContext.Provider value={ { isActive: true } }>
<SetupComplete />
</WizardTaskContext.Provider>
</WizardContext.Provider>
</SettingsContextProvider>
},
} }
>
<WizardTaskContext.Provider value={ { isActive: true } }>
<SetupComplete />
</WizardTaskContext.Provider>
</WizardContext.Provider>
);

expect( screen.getByText( /Setup complete/ ) ).toHaveTextContent(
Expand Down

0 comments on commit 85cdbe7

Please sign in to comment.