Skip to content

Commit aca26cb

Browse files
committed
Do not setup mocks when useMocks option is false
1 parent 62d3963 commit aca26cb

File tree

4 files changed

+15
-18
lines changed

4 files changed

+15
-18
lines changed

eslint.config.mjs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ export default [
1313
'@nx/enforce-module-boundaries': [
1414
'error',
1515
{
16-
enforceBuildableLibDependency: true,
16+
enforceBuildableLibDependency: false,
1717
allow: ['^.*/eslint(\\.base)?\\.config\\.[cm]?js$'],
1818
depConstraints: [
1919
{

libs/transactions-journey/e2e-tests/fixture/transaction.ts

Lines changed: 12 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -14,14 +14,18 @@ import { MakeTransferPage } from '../page-objects/pages/make-transfer';
1414

1515
export const test = baseTest.extend<TransactionFixture>({
1616
// mocks data setup, can be overridden or bypassed via "useMocks"
17-
transactionMockSetup: async ({ page }, use) =>
18-
await use((transactions: Partial<TransactionDetailsDataType>[]) =>
19-
setupPageMocks(page, addTransactionsToMock(transactions))
20-
),
21-
transactionsMockSetup: async ({ page }, use) =>
22-
use((transactions: TransactionsListDataType) =>
23-
setupPageMocks(page, defaultTransactionsMock)
24-
),
17+
transactionMockSetup: async ({ page, useMocks }, use) =>
18+
await use((transactions: Partial<TransactionDetailsDataType>[]) => {
19+
if (useMocks) {
20+
setupPageMocks(page, addTransactionsToMock(transactions));
21+
}
22+
}),
23+
transactionsMockSetup: async ({ page, useMocks }, use) =>
24+
await use(() => {
25+
if (useMocks) {
26+
setupPageMocks(page, defaultTransactionsMock);
27+
}
28+
}),
2529
makeTransferPage: async ({ page, baseURL }, use) => {
2630
await use(new MakeTransferPage(page, { baseURL, url: '/make-transfer' }));
2731
},

libs/transactions-journey/e2e-tests/model/transaction.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -44,6 +44,7 @@ export interface TransactionFixture {
4444
makeTransferPage: MakeTransferPage;
4545
transactionDetailsPage: TransactionDetailsPage;
4646
transactionsPage: TransactionsPage;
47+
useMocks: boolean;
4748
}
4849

4950
export const getAmountValue = (

libs/transactions-journey/e2e-tests/specs/transaction-details.spec.ts

Lines changed: 1 addition & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -12,15 +12,7 @@ export function testTransactionDetails(
1212
) {
1313
test.describe(
1414
'Transactions details',
15-
{
16-
tag: [
17-
'@e2e',
18-
'@transactions',
19-
'@transactions-details',
20-
'@mocks',
21-
'@ephemeral',
22-
],
23-
},
15+
{ tag: ['@e2e', '@transactions', '@transactions-details', '@mocks'] },
2416
() => {
2517
test.beforeEach(async ({ transactionMockSetup }) => {
2618
await transactionMockSetup(testData.transactions);

0 commit comments

Comments
 (0)