refactor: split test in longPress in two

Author: pierrezimmermann

src/user-event/press/__tests__/longPress.real-timers.test.tsx

@@ -27,30 +27,46 @@ describe('userEvent.longPress with real timers', () => {
     expect(mockOnLongPress).toHaveBeenCalled();
   });
 
-  test('calls onLongPress when duration is greater than specified onLongPressDelay', async () => {
+  test('calls onLongPress when duration is greater than specified longPressDelay', async () => {
     const mockOnLongPress = jest.fn();
     const mockOnPress = jest.fn();
     const user = userEvent.setup();
 
     render(
       <Pressable
-        delayLongPress={1000}
+        delayLongPress={800}
         onLongPress={mockOnLongPress}
         onPress={mockOnPress}
       >
         <Text>press me</Text>
       </Pressable>
     );
-    await user.longPress(screen.getByText('press me'));
-
-    expect(mockOnLongPress).not.toHaveBeenCalled();
-    expect(mockOnPress).toHaveBeenCalledTimes(1);
 
     await user.longPress(screen.getByText('press me'), {
       duration: 1000,
     });
 
     expect(mockOnLongPress).toHaveBeenCalled();
+    expect(mockOnPress).not.toHaveBeenCalled();
+  });
+
+  test('does not calls onLongPress when duration is lesser than specified longPressDelay', async () => {
+    const mockOnLongPress = jest.fn();
+    const mockOnPress = jest.fn();
+    const user = userEvent.setup();
+
+    render(
+      <Pressable
+        delayLongPress={1000}
+        onLongPress={mockOnLongPress}
+        onPress={mockOnPress}
+      >
+        <Text>press me</Text>
+      </Pressable>
+    );
+    await user.longPress(screen.getByText('press me'));
+
+    expect(mockOnLongPress).not.toHaveBeenCalled();
     expect(mockOnPress).toHaveBeenCalledTimes(1);
   });
 

src/user-event/press/__tests__/longPress.test.tsx

@@ -29,23 +29,39 @@ describe('userEvent.longPress with fake timers', () => {
 
     render(
       <Pressable
-        delayLongPress={1000}
+        delayLongPress={800}
         onLongPress={mockOnLongPress}
         onPress={mockOnPress}
       >
         <Text>press me</Text>
       </Pressable>
     );
-    await user.longPress(screen.getByText('press me'));
-
-    expect(mockOnLongPress).not.toHaveBeenCalled();
-    expect(mockOnPress).toHaveBeenCalledTimes(1);
 
     await user.longPress(screen.getByText('press me'), {
       duration: 1000,
     });
 
     expect(mockOnLongPress).toHaveBeenCalled();
+    expect(mockOnPress).not.toHaveBeenCalled();
+  });
+
+  test('does not calls onLongPress when duration is lesser than specified onLongPressDelay', async () => {
+    const mockOnLongPress = jest.fn();
+    const mockOnPress = jest.fn();
+    const user = userEvent.setup();
+
+    render(
+      <Pressable
+        delayLongPress={1000}
+        onLongPress={mockOnLongPress}
+        onPress={mockOnPress}
+      >
+        <Text>press me</Text>
+      </Pressable>
+    );
+    await user.longPress(screen.getByText('press me'));
+
+    expect(mockOnLongPress).not.toHaveBeenCalled();
     expect(mockOnPress).toHaveBeenCalledTimes(1);
   });