@flighthq/render-gl 0.3.1-next.887.9f59a84 → 0.3.1-next.970.9691e7b

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@flighthq/render-gl",
-  "version": "0.3.1-next.887.9f59a84",
+  "version": "0.3.1-next.970.9691e7b",
   "author": "Joshua Granick and other contributors",
   "license": "MIT",
   "repository": {
@@ -38,16 +38,16 @@
     "clean:dist": "tsx ../../scripts/clean-package-dist.ts"
   },
   "dependencies": {
-    "@flighthq/color": "0.3.1-next.887.9f59a84",
-    "@flighthq/entity": "0.3.1-next.887.9f59a84",
-    "@flighthq/log": "0.3.1-next.887.9f59a84",
-    "@flighthq/scene2d": "0.3.1-next.887.9f59a84",
-    "@flighthq/geometry": "0.3.1-next.887.9f59a84",
-    "@flighthq/render": "0.3.1-next.887.9f59a84",
-    "@flighthq/registry": "0.3.1-next.887.9f59a84",
-    "@flighthq/signals": "0.3.1-next.887.9f59a84",
-    "@flighthq/texture": "0.3.1-next.887.9f59a84",
-    "@flighthq/types": "0.3.1-next.887.9f59a84"
+    "@flighthq/color": "0.3.1-next.970.9691e7b",
+    "@flighthq/entity": "0.3.1-next.970.9691e7b",
+    "@flighthq/log": "0.3.1-next.970.9691e7b",
+    "@flighthq/scene2d": "0.3.1-next.970.9691e7b",
+    "@flighthq/geometry": "0.3.1-next.970.9691e7b",
+    "@flighthq/render": "0.3.1-next.970.9691e7b",
+    "@flighthq/registry": "0.3.1-next.970.9691e7b",
+    "@flighthq/signals": "0.3.1-next.970.9691e7b",
+    "@flighthq/texture": "0.3.1-next.970.9691e7b",
+    "@flighthq/types": "0.3.1-next.970.9691e7b"
   },
   "devDependencies": {
     "typescript": "^5.3.0"

package/src/glTestHelper.test.ts

@@ -0,0 +1,51 @@
+import { makeGL } from './glTestHelper';
+
+describe('makeGL', () => {
+  it('keeps draw validation outside a test-installed mock implementation', () => {
+    const gl = makeGL();
+    const draw = vi.fn();
+    (gl.drawElements as ReturnType<typeof vi.fn>).mockImplementation(draw);
+
+    gl.drawElements(gl.TRIANGLES, 1, gl.UNSIGNED_SHORT, 1);
+
+    expect(draw).not.toHaveBeenCalled();
+    expect(gl.getError()).toBe(gl.INVALID_OPERATION);
+
+    gl.drawElements(gl.TRIANGLES, 1, gl.UNSIGNED_SHORT, 0);
+    expect(draw).toHaveBeenCalledOnce();
+  });
+
+  it('sets and consumes the first GL draw error without throwing', () => {
+    const gl = makeGL();
+
+    expect(() => gl.drawElements(gl.TRIANGLES, -1, gl.UNSIGNED_SHORT, 0)).not.toThrow();
+    gl.drawElements(0xffff, 1, gl.UNSIGNED_SHORT, 0);
+
+    expect(gl.drawElements).not.toHaveBeenCalled();
+    expect(gl.getError()).toBe(gl.INVALID_VALUE);
+    expect(gl.getError()).toBe(gl.NO_ERROR);
+  });
+
+  it('reports a type-misaligned index offset as INVALID_OPERATION', () => {
+    const gl = makeGL();
+
+    gl.drawElements(gl.TRIANGLES, 1, gl.UNSIGNED_SHORT, 1);
+
+    expect(gl.getError()).toBe(gl.INVALID_OPERATION);
+    expect(gl.getError()).toBe(gl.NO_ERROR);
+  });
+
+  it('validates non-indexed and instanced draw entry points', () => {
+    const gl = makeGL();
+
+    gl.drawArrays(0xffff, 0, 1);
+    expect(gl.getError()).toBe(gl.INVALID_ENUM);
+
+    gl.drawArraysInstanced(gl.TRIANGLES, 0, 1, -1);
+    expect(gl.getError()).toBe(gl.INVALID_VALUE);
+
+    gl.drawElementsInstanced(gl.TRIANGLES, 1, gl.FLOAT, 0, 1);
+    expect(gl.getError()).toBe(gl.INVALID_ENUM);
+    expect(gl.getError()).toBe(gl.NO_ERROR);
+  });
+});