Merge pull request fbeline#25 from fbeline/factory-static-method

fbeline · web-flow · commit 98f83b5064fd · 2019-03-05T10:56:33.000-03:00
Factory static method
diff --git a/src/creational/factory/factory_es6.js b/src/creational/factory/factory_es6.js
@@ -1,6 +1,6 @@
 class BmwFactory {
 
-    create(type) {
+    static create(type) {
         if (type === 'X5')
             return new Bmw(type, 108000, 300);
         if (type === 'X6')
diff --git a/test/factory_es6-test.js b/test/factory_es6-test.js
@@ -1,15 +1,14 @@
 const expect = require('chai').expect;
 import BmwFactory from '../src/creational/factory/factory_es6';
 
-describe('factory es6 test', () => {
-    it('sanity', () => {
-        const bmwFactory = new BmwFactory();
-        const x5 = bmwFactory.create('X5');
-        const x6 = bmwFactory.create('X6');
+describe('Factory es6 test', () => {
+    it('We can create a X5 instance', () => {
+        const x5 = BmwFactory.create('X5');
+        expect(x5.model).to.equal('X5');
+    });
 
+    it('The X5 price is properly set', () => {
+        const x5 = BmwFactory.create('X5');
         expect(x5.price).to.equal(108000);
-        expect(x6.price).to.equal(111000);
-        expect(x5.maxSpeed).to.equal(300);
-        expect(x6.maxSpeed).to.equal(320);
     });
 });
