analyzeEmberObject() exits too early not allowing to enumerate `h… (#22)

analyzeEmberObject() exits too early not allowing to enumerate `helpers`

Author: Robert Jackson

lib/gather/analyze-ember-object.js

@@ -2,14 +2,16 @@ module.exports = function analyzeEmberObject(possibleEmberObject) {
   if (typeof possibleEmberObject !== 'object' || possibleEmberObject === null) {
     return undefined;
   }
-  if (possibleEmberObject.default && typeof possibleEmberObject.default.proto !== 'function') {
-    return undefined;
-  }
-
-  if (possibleEmberObject.default.isHelperFactory) {
-    return {
-      type: 'Helper',
-    };
+  let eObjDefault = possibleEmberObject.default;
+
+  if (eObjDefault) {
+    if (eObjDefault.isHelperFactory) {
+      return {
+        type: 'Helper',
+      };
+    } else if (typeof eObjDefault.proto !== 'function') {
+      return undefined;
+    }
   }
 
   let proto = possibleEmberObject.default.proto();

lib/gather/gather-telemetry.test.js

@@ -46,10 +46,10 @@ describe('Provide a personalized `Gathering Function`', () => {
     });
   });
 
-  test('can determine components with a robust function', async () => {
+  test('can determine most Ember types with a robust function', async () => {
     await gatherTelemetry('http://localhost:4200', analyzeEmberObject);
     let telemetry = getTelemetry();
-    expect(Object.values(telemetry).filter(Boolean).length).toEqual(29);
+    expect(Object.values(telemetry).filter(Boolean).length).toEqual(34);
   });
 
   test('can handle external functions', async () => {