Adapt to PR feedback

assetgraph · Sep 22, 2018 · 71a1879 · 71a1879
1 parent 93ccbcf
commit 71a1879
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 6 deletions.
diff --git a/lib/assets/Asset.js b/lib/assets/Asset.js
@@ -388,18 +388,22 @@ class Asset extends EventEmitter {
           const { headers = {}, ...requestOptions } =
             this.assetGraph.requestOptions || {};
           const firstIncomingRelation = this.incomingRelations[0];
+          let Referer;
 
           if (
             firstIncomingRelation &&
             firstIncomingRelation.from.protocol &&
             firstIncomingRelation.from.protocol.startsWith('http')
           ) {
-            headers.Referer = firstIncomingRelation.from.url;
+            Referer = firstIncomingRelation.from.url;
           }
 
           const response = await this.assetGraph.teepee.request({
             ...requestOptions,
-            headers,
+            headers: {
+              ...headers,
+              Referer
+            },
             method: metadataOnly ? 'HEAD' : 'GET',
             url,
             json: false

diff --git a/test/assets/Asset.js b/test/assets/Asset.js
@@ -49,8 +49,6 @@ describe('assets/Asset', function() {
 
       await assetGraph.loadAssets('/');
       await assetGraph.populate();
-
-      expect(assetGraph, 'to contain asset', 'Css');
     });
     it('should not add an HTTP referer header to an http request when following a relation from a non-HTTP asset', async function() {
       httpception([
@@ -77,8 +75,6 @@ describe('assets/Asset', function() {
         text: '<link rel="stylesheet" href="https://www.example.com/style.css">'
       });
       await assetGraph.populate();
-
-      expect(assetGraph, 'to contain asset', 'Css');
     });
 
     it('should error when there is no file handle and the asset is not in a graph', function() {