Add test case and fix for glGetDebugInfo regression under Node. (#111)

* Add test case and fix for glGetDebugInfo regression under Node.

* Add "experimental.js" to exported "files"  in package.json.

* Add comments to some top-level test index.js files

package.json

@@ -29,7 +29,8 @@
     "shaderlib",
     "src",
     "headless.js",
-    "node-io.js"
+    "node-io.js",
+    "experimental.js"
   ],
   "dependencies": {
     "autobind-decorator": "^1.3.3",

src/webgl/context.js

@@ -205,9 +205,9 @@ export function glGetDebugInfo(gl) {
   // specs by returning null for unsupported extension. Instead,
   // it returns an object without GL_UNMASKED_VENDOR_WEBGL and GL_UNMASKED_RENDERER_WEBGL.
   return {
-    vendor: info.GL_UNMASKED_VENDOR_WEBGL ?
+    vendor: (info && info.GL_UNMASKED_VENDOR_WEBGL) ?
       gl.getParameter(GL_UNMASKED_VENDOR_WEBGL) : 'unknown',
-    renderer: info.GL_UNMASKED_RENDERER_WEBGL ?
+    renderer: (info && info.GL_UNMASKED_RENDERER_WEBGL) ?
       gl.getParameter(GL_UNMASKED_RENDERER_WEBGL) : 'unknown'
   };
 }

test/headless-nowebgl.js

@@ -1,6 +1,9 @@
+// Note that we do two test runs on luma.gl, with and without headless-gl
+// This file imports tests that should run *without* headless-gl included
+
 require('babel-core/register');
 require('babel-polyfill');
 
 // Quick test that webgl independent code works
-require('./webgl/core-spec');
+require('./webgl/context-no-headless.spec');
 require('./webgl-independent');

test/webgl/context.spec.js

@@ -0,0 +1,18 @@
+const {createGLContext, glGetDebugInfo} = require('../../src/headless');
+const {isWebGLContext} = require('../../src/webgl/webgl-checks');
+
+const test = require('tape-catch');
+
+test('WebGL#headless context creation', t => {
+  const gl = createGLContext();
+  t.ok(isWebGLContext(gl), 'Context creation ok');
+  t.end();
+});
+
+test('WebGL#glGetDebugInfo', t => {
+  const gl = createGLContext();
+  const info = glGetDebugInfo(gl);
+  t.ok(typeof info.vendor === 'string', 'info.vendor ok');
+  t.ok(typeof info.renderer === 'string', 'info.renderer ok');
+  t.end();
+});

test/webgl/index.js

@@ -1,24 +1,27 @@
+// Note that we do two test runs on luma.gl, with and without headless-gl
+// This file imports tests that should run *with* headless-gl included
+
+// helpers & utils
 import './helpers/query-manager.spec';
 
 // webgl
-import './webgl-spec';
-import './core-spec.js';
-import './vertex-attributes-spec';
-import './buffer-spec';
-import './program-spec';
-import './texture-spec';
-import './framebuffer-spec';
-import './renderbuffer-spec';
-import './draw-spec';
-import './uniforms-spec';
+import './context.spec.js';
+import './vertex-attributes.spec';
+import './buffer.spec';
+import './program.spec';
+import './texture.spec';
+import './framebuffer.spec';
+import './renderbuffer.spec';
+import './draw.spec';
+import './uniforms.spec';
 
 // Extensions
 import './timer-query.spec';
 
 // import './fbo-spec';
 
 // webgl2
-import './vertex-array-object-spec';
+import './vertex-array-object.spec';
 
 // misc
 import './io-spec';