fix(inspectedApp): read posted data structure correctly

inspectedApp processes message objects posted from ngHint.
It was looking for properties on the root of the object that exist in a
nested object.
This caused the Batarang scope tree to appear blank.

Author: gary-b

panel/components/inspected-app/inspected-app.js

@@ -88,21 +88,21 @@ function inspectedAppService($rootScope, $q) {
         });
       } else if (hint.event === 'scope:new') {
         addNewScope(hint);
-      } else if (hint.id && scopes[hint.id]) {
-        var scope = scopes[hint.id];
+      } else if (hint.data.id && scopes[hint.data.id]) {
+        var scope = scopes[hint.data.id];
         if (hint.event === 'scope:destroy') {
           if (scope.parent) {
             scope.parent.children.splice(scope.parent.children.indexOf(child), 1);
           }
-          delete scopes[hint.id];
+          delete scopes[hint.data.id];
         } else if (hint.event === 'model:change') {
-          scope.models[hint.path] = (typeof hint.value === 'undefined') ?
-                                                undefined : JSON.parse(hint.value);
+          scope.models[hint.data.path] = (typeof hint.data.value === 'undefined') ?
+                                                undefined : JSON.parse(hint.data.value);
         } else if (hint.event === 'scope:link') {
-          scope.descriptor = hint.descriptor;
+          scope.descriptor = hint.data.descriptor;
         }
       }
-      $rootScope.$broadcast(hint.event, hint);
+      $rootScope.$broadcast(hint.event, hint.data);
     }
   }
 
@@ -112,13 +112,13 @@ function inspectedAppService($rootScope, $q) {
   }
 
   function addNewScope (hint) {
-    scopes[hint.child] = {
-      parent: hint.parent,
+    scopes[hint.data.child] = {
+      parent: hint.data.parent,
       children: [],
       models: {}
     };
-    if (scopes[hint.parent]) {
-      scopes[hint.parent].children.push(hint.child);
+    if (scopes[hint.data.parent]) {
+      scopes[hint.data.parent].children.push(hint.data.child);
     }
   }
 

panel/components/inspected-app/inspected-app.spec.js

@@ -1,13 +1,15 @@
 'use strict';
 
 describe('inspectedApp', function() {
-  var inspectedApp, port;
+  var inspectedApp, rootScope, port;
 
   beforeEach(function() {
     module('batarang.inspected-app')
     window.chrome = createMockChrome();
-    inject(function(_inspectedApp_) {
+    inject(function(_inspectedApp_, _$rootScope_) {
       inspectedApp = _inspectedApp_;
+      rootScope = _$rootScope_;
+      spyOn(rootScope, '$broadcast');
     });
   });
 
@@ -26,20 +28,27 @@ describe('inspectedApp', function() {
     }));
 
     it('should track new scopes', inject(function ($browser) {
-      port.onMessage.trigger(JSON.stringify({ event: 'scope:new', child: 1 }));
+      port.onMessage.trigger(JSON.stringify({ event: 'scope:new', data: { child: 1 } }));
       $browser.defer.flush();
 
       expect(inspectedApp.scopes).toEqual({ 1: { parent: undefined, children: [], models: {} } });
     }));
 
     it('should track updates to scope descriptors', inject(function ($browser) {
-      port.onMessage.trigger(JSON.stringify({ event: 'scope:new', child: 1 }));
-      port.onMessage.trigger(JSON.stringify({ event: 'scope:link', id: 1, descriptor: 'pasta' }));
+      port.onMessage.trigger(JSON.stringify({ event: 'scope:new', data: { child: 1 } }));
+      port.onMessage.trigger(JSON.stringify({ event: 'scope:link', data: { id: 1, descriptor: 'pasta' } }));
       $browser.defer.flush();
 
       expect(inspectedApp.scopes[1].descriptor).toBe('pasta');
     }));
-  })
+    it('should broadcast message from $rootScope', inject(function ($browser) {
+      var message = { event: 'scope:new', data: { child: 1 } };
+      port.onMessage.trigger(JSON.stringify(message));
+      $browser.defer.flush();
+
+      expect(rootScope.$broadcast).toHaveBeenCalledWith(message.event, message.data);
+    }));
+  });
 
   describe('watch', function () {
     it('should call chrome devtools APIs', function() {