fix(clerk-js): address PR review feedback on state tests

Author: jacekradko

packages/clerk-js/src/core/__tests__/state.test.ts

@@ -123,7 +123,6 @@ describe('State', () => {
         // The previous id 'signup_123' should be gone
         expect(signUpResourceSignal().resource).toBe(newSignUpFromReset);
         expect(signUpResourceSignal().resource?.id).toBeUndefined();
-        expect(signUpResourceSignal().resource).not.toBe(existingSignUp);
       });
 
       it('should allow resource update when new resource has an id (not a null update)', () => {
@@ -170,35 +169,6 @@ describe('State', () => {
   });
 
   describe('Simulated client refresh scenarios', () => {
-    it('should protect against null update during email verification (before finalize)', () => {
-      // This test simulates the scenario described in USER-4372:
-      // 1. User is in the middle of sign up with verified email
-      // 2. Client refresh happens (e.g., router.refresh in Next.js)
-      // 3. Server returns sign_up: null because sign up is complete on server side
-      // 4. The null update should be IGNORED to prevent flash of default form
-
-      // Arrange: User has completed email verification but hasn't called finalize yet
-      const signUp = new SignUp({
-        id: 'signup_123',
-        status: 'complete',
-        created_session_id: 'session_123',
-        verifications: {
-          email_address: { status: 'verified' },
-        },
-      } as any);
-
-      expect(signUpResourceSignal().resource?.id).toBe('signup_123');
-      expect(signUp.__internal_future.canBeDiscarded).toBe(false);
-
-      // Act: Simulate client refresh returning null sign_up
-      // This is what happens in Client.fromJSON when server returns sign_up: null
-      const nullSignUp = new SignUp(null);
-
-      // Assert: The null update should be IGNORED
-      expect(signUpResourceSignal().resource?.id).toBe('signup_123');
-      expect(signUpResourceSignal().resource).toBe(signUp);
-    });
-
     it('should allow null update after finalize sets canBeDiscarded to true', async () => {
       // This test verifies that after finalize() is called,
       // the null update IS allowed (because canBeDiscarded is true)