Remove unnecessary space.enter() from some tests

These calls presumably exist because the method that they are testing
requires that the client has entered the space. But calling
`space.enter()` in the context of these tests does nothing useful — the
underlying Realtime object is a mock, and the `presenceMap` objects in
the tests’ contexts have already been configured such that a member is
present for the space’s connection ID.

Author: lawrence-forooghian

src/Locations.test.ts

@@ -49,7 +49,6 @@ describe('Locations', () => {
 
     it<SpaceTestContext>('sends a presence update on location set', async ({ space, presence }) => {
       const spy = vi.spyOn(presence, 'update');
-      await space.enter();
       await space.locations.set('location1');
       expect(spy).toHaveBeenCalledWith(createLocationUpdate({ current: 'location1' }));
     });

src/Locks.test.ts

@@ -49,8 +49,6 @@ describe('Locks', () => {
       space,
       presence,
     }) => {
-      await space.enter();
-
       const presenceUpdate = vi.spyOn(presence, 'update');
 
       const lockID = 'test';
@@ -71,8 +69,6 @@ describe('Locks', () => {
     });
 
     it<SpaceTestContext>('includes attributes in the lock request when provided', async ({ space, presence }) => {
-      await space.enter();
-
       const presenceUpdate = vi.spyOn(presence, 'update');
 
       const lockID = 'test';
@@ -89,8 +85,6 @@ describe('Locks', () => {
     });
 
     it<SpaceTestContext>('errors if a PENDING request already exists', async ({ space }) => {
-      await space.enter();
-
       const lockID = 'test';
       await space.locks.acquire(lockID);
       expect(space.locks.acquire(lockID)).rejects.toThrowError();
@@ -108,7 +102,6 @@ describe('Locks', () => {
       });
 
     it<SpaceTestContext>('sets a PENDING request to LOCKED', async ({ space }) => {
-      await space.enter();
       const member = (await space.members.getSelf())!;
 
       const emitSpy = vi.spyOn(space.locks, 'emit');
@@ -173,8 +166,6 @@ describe('Locks', () => {
       },
     ])('$name', ({ desc, otherConnId, otherTimestamp, expectedSelfStatus, expectedOtherStatus }) => {
       it<SpaceTestContext>(desc, async ({ client, space }) => {
-        await space.enter();
-
         // process a PENDING request for the other member, which should
         // transition to LOCKED
         let msg = Realtime.PresenceMessage.fromValues({
@@ -230,7 +221,6 @@ describe('Locks', () => {
     });
 
     it<SpaceTestContext>('sets a released request to UNLOCKED', async ({ space }) => {
-      await space.enter();
       const member = (await space.members.getSelf())!;
 
       let msg = Realtime.PresenceMessage.fromValues({
@@ -263,7 +253,6 @@ describe('Locks', () => {
     });
 
     it<SpaceTestContext>('sets all locks to UNLOCKED when a member leaves', async ({ space }) => {
-      await space.enter();
       const member = (await space.members.getSelf())!;
 
       let msg = Realtime.PresenceMessage.fromValues({
@@ -311,7 +300,6 @@ describe('Locks', () => {
     });
 
     it<SpaceTestContext>('removes the identified lock request from presence extras', async ({ space, presence }) => {
-      await space.enter();
       const member = (await space.members.getSelf())!;
 
       const lockID = 'test';