Removes call to deprecated OutgoingMessage.prototype._headers

Author: SergeAstapov

packages/fastboot/src/fastboot-response.js

@@ -1,10 +1,11 @@
 'use strict';
+/* eslint-disable no-console */
 
 const FastBootHeaders = require('./fastboot-headers');
 
 class FastbootResponse {
   constructor(response) {
-    this.headers = new FastBootHeaders(response._headers);
+    this.headers = new FastBootHeaders(response.getHeaders());
     this.statusCode = 200;
   }
 }

packages/fastboot/test/fastboot-dependencies-test.js

@@ -6,12 +6,15 @@ const FastBoot = require('./../src/index');
 
 describe('FastBoot with dependencies', function() {
   it('it works with dependencies', function() {
-    var fastboot = new FastBoot({
+    const fastboot = new FastBoot({
       distPath: fixture('app-with-dependencies'),
     });
+    const response = {
+      getHeaders() {},
+    };
 
     return fastboot
-      .visit('/')
+      .visit('/', { response })
       .then(r => r.html())
       .then(html => {
         expect(html).to.match(/https:\/\/emberjs.com/);

packages/fastboot/test/fastboot-info-test.js

@@ -23,7 +23,9 @@ describe('FastBootInfo', function() {
   };
 
   beforeEach(function() {
-    response = {};
+    response = {
+      getHeaders() {},
+    };
     request = {
       cookie: '',
       protocol: 'http',

packages/fastboot/test/fastboot-response-test.js

@@ -7,9 +7,11 @@ describe('FastBootResponse', function() {
 
   beforeEach(function() {
     var mockResponse = {
-      _headers: {
-        'i-am-a': ['mock header', 'me too'],
-        cookie: '',
+      getHeaders() {
+        return {
+          'i-am-a': ['mock header', 'me too'],
+          cookie: '',
+        };
       },
     };
 

packages/fastboot/test/fastboot-shoebox-test.js

@@ -6,12 +6,15 @@ const FastBoot = require('./../src/index');
 
 describe('FastBootShoebox', function() {
   it('can render the escaped shoebox HTML', function() {
-    var fastboot = new FastBoot({
+    const fastboot = new FastBoot({
       distPath: fixture('shoebox'),
     });
+    const response = {
+      getHeaders() {},
+    };
 
     return fastboot
-      .visit('/')
+      .visit('/', { response })
       .then(r => r.html())
       .then(html => {
         expect(html).to.match(
@@ -33,12 +36,16 @@ describe('FastBootShoebox', function() {
   });
 
   it('can render the escaped shoebox HTML with shouldRender set to false', function() {
-    var fastboot = new FastBoot({
+    const fastboot = new FastBoot({
       distPath: fixture('shoebox'),
     });
+    const response = {
+      getHeaders() {},
+    };
 
     return fastboot
       .visit('/', {
+        response,
         shouldRender: false,
       })
       .then(r => r.html())
@@ -62,12 +69,16 @@ describe('FastBootShoebox', function() {
   });
 
   it('cannot render the escaped shoebox HTML when disableShoebox is set to true', function() {
-    var fastboot = new FastBoot({
+    const fastboot = new FastBoot({
       distPath: fixture('shoebox'),
     });
+    const response = {
+      getHeaders() {},
+    };
 
     return fastboot
       .visit('/', {
+        response,
         disableShoebox: true,
       })
       .then(r => r.html())