Skip to content

Commit 6422558

Browse files
committed
tests(perps): restore # in PR/issue references and add targeted color-no-hex disables\n\n- PerpsMarketDetailsView.view.test.tsx: 'Bug #25315' label restored with eslint-disable-next-line\n- PerpsMarketListView.view.test.tsx: 'Bug regression: #25571' label restored with eslint-disable-next-line\n- PerpsConnectionManager.test.ts: '(PR #25334)' and comment 'PR #26780' restored with eslint-disable-next-line\n\nKeeps @metamask/design-tokens/color-no-hex enforcement while preserving conventional GitHub cross-references.
1 parent bdc53b5 commit 6422558

File tree

3 files changed

+8
-4
lines changed

3 files changed

+8
-4
lines changed

app/components/UI/Perps/Views/PerpsMarketDetailsView/PerpsMarketDetailsView.view.test.tsx

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -136,7 +136,8 @@ describe('PerpsMarketDetailsView', () => {
136136
).not.toBeOnTheScreen();
137137
});
138138

139-
describe('Bug 25315: Geo-restriction for Close and Modify actions', () => {
139+
// eslint-disable-next-line @metamask/design-tokens/color-no-hex
140+
describe('Bug #25315: Geo-restriction for Close and Modify actions', () => {
140141
it('shows geo block bottom sheet when Close is pressed (geo-restricted user)', async () => {
141142
renderPerpsMarketDetailsView();
142143

app/components/UI/Perps/Views/PerpsMarketListView/PerpsMarketListView.view.test.tsx

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,8 @@ const commodityMarket: PerpsMarketData = {
3838
const marketDataWithCategories = [cryptoMarket, commodityMarket];
3939

4040
describe('PerpsMarketListView', () => {
41-
describe('Bug regression: 25571', () => {
41+
// eslint-disable-next-line @metamask/design-tokens/color-no-hex
42+
describe('Bug regression: #25571', () => {
4243
it('renders market list header and list with default state (no category filtering)', async () => {
4344
renderPerpsMarketListView();
4445

app/components/UI/Perps/services/PerpsConnectionManager.test.ts

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -867,7 +867,8 @@ describe('PerpsConnectionManager', () => {
867867
});
868868
});
869869

870-
describe('DEX Abstraction Cache Clearing (PR 25334)', () => {
870+
// eslint-disable-next-line @metamask/design-tokens/color-no-hex
871+
describe('DEX Abstraction Cache Clearing (PR #25334)', () => {
871872
beforeEach(() => {
872873
jest.clearAllMocks();
873874
});
@@ -972,7 +973,8 @@ describe('PerpsConnectionManager', () => {
972973

973974
describe('foreground reconnection — single reconnection flow', () => {
974975
it('PerpsConnectionManager has no AppState listener — only the hook triggers foreground reconnect', () => {
975-
// This test documents the fix for the race condition introduced by PR 26780.
976+
// eslint-disable-next-line @metamask/design-tokens/color-no-hex
977+
// This test documents the fix for the race condition introduced by PR #26780.
976978
// Previously, PerpsConnectionManager registered its own AppState listener in
977979
// setupStateMonitoring(), which competed with usePerpsConnectionLifecycle hook.
978980
//

0 commit comments

Comments
 (0)