[bugfix] #1 prevent thrown error while unregister unregistered component

TerrificBridge.js

@@ -250,6 +250,9 @@ export class TerrificBridge {
             }
 
             const id = node.getAttribute("data-t-id");
+            if (!id || id === null) {
+                return void 0;
+            }
             const tModule = this._app.getModuleById(id);
 
             if (bridge._config.debug) {
@@ -321,7 +324,7 @@ export class TerrificBridge {
 }
 
 /**
- * The TerrificBridgeInstance is a singleton and 
+ * The TerrificBridgeInstance is a singleton and
  * will be instanciated here.
  */
 TerrificBridgeInstance = new TerrificBridge(!(NODE_ENV === "production"));

TerrificBridge.spec.js

@@ -51,6 +51,39 @@ describe("TerrificBridge", () => {
         });
     });
 
+    it("should unregister unregistered terrific component", () => {
+        TerrificBridge.reset();
+        TerrificBridge.configure({ debug: true });
+
+        const uiStopStub = sinon.spy();
+
+        T.Module.CanRegister = T.createModule({
+            start(resolve) {
+                resolve();
+            },
+            stop() {
+                uiStopStub();
+                this._events.disconnect();
+            }
+        });
+
+        class CanRegister extends Component {
+            componentWillUnmount() {
+                TerrificBridge.unregisterComponent(this);
+            }
+
+            render() {
+                return <div id="component" data-t-name="CanRegister" />;
+            }
+        }
+
+        const tree = mount(<CanRegister />);
+        TerrificBridge.load();
+        tree.unmount();
+
+        expect(uiStopStub.callCount).toEqual(0);
+    });
+
     describe("components", () => {
         describe("registration", () => {
             it("should mount components successfully", () => {
@@ -245,6 +278,7 @@ describe("TerrificBridge", () => {
 
                 expect(uiStopStub.callCount).toEqual(1);
             });
+
             it("should log unregistration errors from terrific components", () => {
                 TerrificBridge.reset();
                 TerrificBridge.configure({ debug: true });

dist/index.js

@@ -291,6 +291,9 @@ var TerrificBridge = function () {
                 }
 
                 var id = node.getAttribute("data-t-id");
+                if (!id || id === null) {
+                    return void 0;
+                }
                 var tModule = _this2._app.getModuleById(id);
 
                 if (bridge._config.debug) {
@@ -395,14 +398,14 @@ var TerrificBridge = function () {
     }, {
         key: "terrific",
         get: function get() {
-            return this._t;
+            return this._t || window.T || void 0;
         }
     }]);
     return TerrificBridge;
 }();
 
 /**
- * The TerrificBridgeInstance is a singleton and 
+ * The TerrificBridgeInstance is a singleton and
  * will be instanciated here.
  */
 

package.json

@@ -1,6 +1,6 @@
 {
     "name": "react-terrific-bridge",
-    "version": "1.2.6",
+    "version": "1.2.7",
     "main": "dist/index.js",
     "author": "Jan Biasi <biasijan@gmail.com>",
     "contributors": [