Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[No QA] Enable group chat name tests by removing skip from last message preview #56461

Merged
merged 9 commits into from
Feb 17, 2025
Merged

[No QA] Enable group chat name tests by removing skip from last message preview #56461

Show file tree
Hide file tree
Changes from 7 commits
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
3e3d87a
Enable group chat name tests by removing skip from last message previ…
pasyukevich Feb 6, 2025
676a7c7
Downgrade react-native-onyx version to 2.0.91 in package.json and pac…
pasyukevich Feb 6, 2025
da2875c
Revert Onyx downgrade
pasyukevich Feb 6, 2025
003b702
Fix typo
pasyukevich Feb 6, 2025
0ed885c
Merge branch 'main' into bugfix/tests-failing-debug
pasyukevich Feb 13, 2025
2b56209
fix: update Jest command in GitHub Actions workflow to use npm
pasyukevich Feb 13, 2025
e054d4f
fix: update Jest command in GitHub Actions workflow to use npm run test
pasyukevich Feb 13, 2025
b7ac351
Merge branch 'main' into bugfix/tests-failing-debug
pasyukevich Feb 17, 2025
97ac148
fix: eslint
pasyukevich Feb 17, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion .github/workflows/test.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,7 @@ jobs:
key: ${{ runner.os }}-jest

- name: Jest tests
run: NODE_OPTIONS="$NODE_OPTIONS --experimental-vm-modules" npx jest --silent --shard=${{ fromJSON(matrix.chunk) }}/${{ strategy.job-total }} --max-workers ${{ steps.cpu-cores.outputs.count }}
run: NODE_OPTIONS="$NODE_OPTIONS --experimental-vm-modules" npm test -- --silent --shard=${{ fromJSON(matrix.chunk) }}/${{ strategy.job-total }} --max-workers ${{ steps.cpu-cores.outputs.count }}

storybookTests:
if: ${{ github.actor != 'OSBotify' && github.actor != 'imgbot[bot]' || github.event_name == 'workflow_call' }}
Expand Down
4 changes: 2 additions & 2 deletions tests/ui/GroupChatNameTests.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,9 +6,9 @@
import {act, render, screen, waitFor} from '@testing-library/react-native';
import React from 'react';
import Onyx from 'react-native-onyx';
import * as Localize from '@libs/Localize';

Check failure on line 9 in tests/ui/GroupChatNameTests.tsx

View workflow job for this annotation

GitHub Actions / Changed files ESLint check 

Namespace imports from @libs are not allowed. Use named imports instead. Example: import { method } from "@libs/module"
import * as AppActions from '@userActions/App';

Check failure on line 10 in tests/ui/GroupChatNameTests.tsx

View workflow job for this annotation

GitHub Actions / Changed files ESLint check 

Namespace imports from @userActions are not allowed. Use named imports instead. Example: import { action } from "@userActions/module"
import * as User from '@userActions/User';

Check failure on line 11 in tests/ui/GroupChatNameTests.tsx

View workflow job for this annotation

GitHub Actions / Changed files ESLint check 

Namespace imports from @userActions are not allowed. Use named imports instead. Example: import { action } from "@userActions/module"
import App from '@src/App';
import CONST from '@src/CONST';
import ONYXKEYS from '@src/ONYXKEYS';
Expand Down Expand Up @@ -306,7 +306,7 @@
return waitFor(() => expect(displayNameTexts).toHaveLength(1));
}));

it.skip('Should show last message preview in LHN', () =>
it('Should show last message preview in LHN', () =>
signInAndGetApp('A, B, C, D', participantAccountIDs4).then(() => {
// Verify the sidebar links are rendered
const sidebarLinksHintText = Localize.translateLocal('sidebarScreen.listOfChats');
Expand All @@ -324,7 +324,7 @@
return waitFor(() => expect(lastChatText?.props?.children).toBe('B: Test'));
}));

it.skip('Should sort the names before displaying', () =>
it('Should sort the names before displaying', () =>
signInAndGetApp('', [USER_E_ACCOUNT_ID, ...participantAccountIDs4]).then(() => {
// Verify the sidebar links are rendered
const sidebarLinksHintText = Localize.translateLocal('sidebarScreen.listOfChats');
Expand Down
Loading