[BUGFIX beta] Test nested addon import

lib/models/addon.js

@@ -376,6 +376,31 @@ var Addon = CoreObject.extend({
     return tree;
   },
 
+  /**
+    This method climbs up the hierarchy of addons
+    up to the host application.
+
+    This prevents previous addons (prior to `this.import`, ca 2.7.0)
+    to break at importing assets when they are used nested in other addons.
+
+    @private
+    @method _findHost
+    ```
+  */
+  _findHost: function() {
+    var app = this.app;
+    var parent = this.parent;
+    while (parent.parent) {
+      if (parent.app) {
+        app = parent.app;
+        break;
+      }
+
+      parent = parent.parent;
+    }
+    return app;
+  },
+
   /**
     This method is called when the addon is included in a build. You
     would typically use this hook to perform additional imports
@@ -420,10 +445,7 @@ var Addon = CoreObject.extend({
      @param {String} [options.destDir] Destination directory, defaults to the name of the directory the asset is in
    */
   import: function(asset, options) {
-    var app = this.app;
-    while (app.app) {
-      app = app.app;
-    }
+    var app = this._findHost();
     app.import(asset, options);
   },
 

tests/acceptance/nested-addons-smoke-test-slow.js

@@ -0,0 +1,59 @@
+'use strict';
+
+var path       = require('path');
+var fs         = require('fs-extra');
+
+var runCommand          = require('../helpers/run-command');
+var acceptance          = require('../helpers/acceptance');
+var copyFixtureFiles    = require('../helpers/copy-fixture-files');
+var createTestTargets   = acceptance.createTestTargets;
+var teardownTestTargets = acceptance.teardownTestTargets;
+var linkDependencies    = acceptance.linkDependencies;
+var cleanupRun          = acceptance.cleanupRun;
+
+var chai = require('../chai');
+var expect = chai.expect;
+var file = chai.file;
+var dir = chai.dir;
+
+var appName  = 'some-cool-app';
+
+describe('Acceptance: nested-addons-smoke-test', function() {
+  this.timeout(360000);
+
+  before(function() {
+    return createTestTargets(appName);
+  });
+
+  after(function() {
+    return teardownTestTargets();
+  });
+
+  beforeEach(function() {
+    return linkDependencies(appName);
+  });
+
+  afterEach(function() {
+    return cleanupRun(appName).then(function() {
+      expect(dir('tmp/' + appName)).to.not.exist;
+    });
+  });
+
+  it('addons with nested addons compile correctly', function() {
+    return copyFixtureFiles('addon/with-nested-addons')
+      .then(function() {
+        var packageJsonPath = path.join(__dirname, '..', '..', 'tmp', appName, 'package.json');
+        var packageJson = fs.readJsonSync(packageJsonPath);
+        packageJson.devDependencies['ember-top-addon'] = 'latest';
+
+        return fs.writeJsonSync(packageJsonPath, packageJson);
+      })
+      .then(function() {
+        return runCommand(path.join('.', 'node_modules', 'ember-cli', 'bin', 'ember'), 'build');
+      })
+      .then(function() {
+        expect(file('dist/assets/vendor.js')).to.contain('INNER_ADDON_IMPORT_WITH_APP_IMPORT');
+        expect(file('dist/assets/vendor.js')).to.contain('INNER_ADDON_IMPORT_WITH_THIS_IMPORT');
+      });
+  });
+});