Update mapping tests to be in line with the spec (#4575)

Author: teoxoy

src/webgpu/api/validation/buffer/mapping.spec.ts

@@ -201,7 +201,7 @@ g.test('mapAsync,state,mappedAtCreation')
       mappedAtCreation: true,
     });
     await t.testMapAsyncCall(
-      { validationError: true, earlyRejection: false, rejectName: 'OperationError' },
+      { validationError: true, earlyRejection: true, rejectName: 'OperationError' },
       buffer,
       mapMode
     );
@@ -222,7 +222,7 @@ g.test('mapAsync,state,mapped')
     const buffer = t.createMappableBuffer(mapMode, 16);
     await t.testMapAsyncCall('success', buffer, mapMode);
     await t.testMapAsyncCall(
-      { validationError: true, earlyRejection: false, rejectName: 'OperationError' },
+      { validationError: true, earlyRejection: true, rejectName: 'OperationError' },
       buffer,
       mapMode
     );
@@ -254,7 +254,7 @@ g.test('mapAsync,state,mappingPending')
     // that we can unmap the previous mapping in the same stack frame and testing this one doesn't
     // get canceled, but instead is rejected.
     const pending1 = t.testMapAsyncCall(
-      { validationError: false, earlyRejection: true, rejectName: 'OperationError' },
+      { validationError: true, earlyRejection: true, rejectName: 'OperationError' },
       buffer,
       mapMode
     );
@@ -392,7 +392,7 @@ g.test('mapAsync,earlyRejection')
     const p1 = buffer.mapAsync(mapMode, offset1, mapSize); // succeeds
     await t.testMapAsyncCall(
       {
-        validationError: false,
+        validationError: true,
         earlyRejection: true,
         rejectName: 'OperationError',
       },
@@ -405,20 +405,17 @@ g.test('mapAsync,earlyRejection')
   });
 
 g.test('mapAsync,abort_over_invalid_error')
-  .desc(
-    `Test that unmap abort error should have precedence over validation error
-TODO
-  - Add other validation error test (eg. offset is not a multiple of 8)
-  `
-  )
+  .desc('Test that unmap abort error should have precedence over validation error')
   .paramsSubcasesOnly(u =>
     u.combine('mapMode', kMapModeOptions).combine('unmapBeforeResolve', [true, false])
   )
   .fn(async t => {
     const { mapMode, unmapBeforeResolve } = t.params;
     const bufferSize = 8;
-    const buffer = t.createMappableBuffer(mapMode, bufferSize);
-    await buffer.mapAsync(mapMode);
+    const buffer = t.createBufferTracked({
+      usage: 0,
+      size: bufferSize,
+    });
 
     if (unmapBeforeResolve) {
       // unmap abort error should have precedence over validation error
@@ -430,7 +427,7 @@ TODO
       buffer.unmap();
       await pending;
     } else {
-      // map on already mapped buffer should cause validation error
+      // invalid map call should cause validation error
       await t.testMapAsyncCall(
         { validationError: true, earlyRejection: false, rejectName: 'OperationError' },
         buffer,
@@ -455,7 +452,7 @@ g.test('getMappedRange,state,mapped')
 
     // map on already mapped buffer should be rejected
     const pending = t.testMapAsyncCall(
-      { validationError: true, earlyRejection: false, rejectName: 'OperationError' },
+      { validationError: true, earlyRejection: true, rejectName: 'OperationError' },
       buffer,
       mapMode
     );
@@ -490,7 +487,7 @@ g.test('getMappedRange,state,mappedAtCreation')
 
     // map on already mapped buffer should be rejected
     const pending = t.testMapAsyncCall(
-      { validationError: true, earlyRejection: false, rejectName: 'OperationError' },
+      { validationError: true, earlyRejection: true, rejectName: 'OperationError' },
       buffer,
       mapMode
     );
@@ -533,7 +530,7 @@ g.test('getMappedRange,state,mappedAgain')
 
     // call mapAsync again on already mapped buffer should fail
     await t.testMapAsyncCall(
-      { validationError: true, earlyRejection: false, rejectName: 'OperationError' },
+      { validationError: true, earlyRejection: true, rejectName: 'OperationError' },
       buffer,
       mapMode
     );
@@ -674,7 +671,7 @@ g.test('getMappedRange,state,mappingPending')
     /* noawait */ const mapping0 = buffer.mapAsync(mapMode);
     // seconding mapping should be rejected
     const mapping1 = t.testMapAsyncCall(
-      { validationError: false, earlyRejection: true, rejectName: 'OperationError' },
+      { validationError: true, earlyRejection: true, rejectName: 'OperationError' },
       buffer,
       mapMode
     );