Fix aframevr#1817 - Add basic test for inspector

src/components/scene/inspector.js

@@ -1,6 +1,7 @@
 /* global AFRAME */
-var registerComponent = require('../../core/component').registerComponent;
+var AFRAME_INJECTED = require('../../constants').AFRAME_INJECTED;
 var pkg = require('../../../package');
+var registerComponent = require('../../core/component').registerComponent;
 
 var INSPECTOR_URL = pkg.homepage + 'releases/' + pkg.version + '/aframe-inspector.min.js';
 
@@ -30,6 +31,8 @@ module.exports.Component = registerComponent('inspector', {
     // Inject.
     script = document.createElement('script');
     script.src = this.data.url;
+    script.setAttribute('data-name', 'aframe-inspector');
+    script.setAttribute(AFRAME_INJECTED, '');
     script.onload = function () {
       AFRAME.INSPECTOR.open();
       window.removeEventListener('keydown', self.onKeydown);

tests/components/scene/inspector.test.js

@@ -0,0 +1,29 @@
+/* global assert, setup, suite, teardown, test */
+
+suite('inspector', function () {
+  setup(function (done) {
+    var el = this.sceneEl = document.createElement('a-scene');
+    document.body.appendChild(el);
+
+    el.addEventListener('loaded', function () { done(); });
+  });
+
+  teardown(function () {
+    var el = this.sceneEl;
+    el.parentNode.removeChild(el);
+  });
+
+  test('adds inspector SCRIPT element to page', function () {
+    var el = this.sceneEl;
+
+    // Have to call method directly because Chrome doesn't provide
+    // a reliable method for KeyboardEvent
+    el.components.inspector.onKeydown({
+      keyCode: 73,
+      ctrlKey: true,
+      altKey: true
+    });
+
+    assert.equal(document.querySelectorAll('script[data-name=aframe-inspector]').length, 1);
+  });
+});