feat(server): added the ability to obtain instance http.createServer () for socket.io (#165)

* feat(server): added the ability to obtain instance http.createServer () for socket.io

* fix(server): stupid semicolon

* fix(server): stupid "Return statement should not contain assignment."

* fix(server): stupid name variables & methods, add unit test on check httpInstance

Author: ntvsx193

src/Server/index.js

@@ -34,6 +34,7 @@ class Server {
     this.config = Config
     this.event = Event
     this.log = new CatLog('adonis:framework')
+    this.httpInstance = null
   }
 
   /**
@@ -264,6 +265,19 @@ class Server {
     })
   }
 
+  /**
+   *
+   * @returns {*}
+   * @public
+   */
+  getInstance () {
+    if (!this.httpInstance) {
+      this.httpInstance = http.createServer(this.handle.bind(this))
+    }
+
+    return this.httpInstance
+  }
+
   /**
    * starting a server on a given port and host
    *
@@ -277,7 +291,7 @@ class Server {
    */
   listen (host, port) {
     this.log.info('serving app on %s:%s', host, port)
-    http.createServer(this.handle.bind(this)).listen(port, host)
+    this.getInstance().listen(port, host)
   }
 
 }

test/unit/server.spec.js

@@ -278,6 +278,16 @@ describe("Server", function () {
     expect(res.error.text).to.equal('Forbidden')
   })
 
+  it("should server instance is null", function * () {
+    expect(this.server.httpInstance).to.be.null
+  })
+
+  it("should server instance is http.Server", function * () {
+    const httpServer = this.server.getInstance()
+    expect(httpServer).to.be.instanceOf(http.Server)
+    expect(this.server.httpInstance).to.be.instanceOf(http.Server)
+  })
+
   it('should listen to server on a given port and host using listen method', function * () {
     Route.get('/','HomeController.index')
     this.server.listen('0.0.0.0', 8000)