Migrate to ES6

Author: Dzianis Khviadzinich

extensions/it-optional.js

@@ -1,19 +1,13 @@
-'use strict';
-
-exports = module.exports = testOptional;
-
-function testOptional(title, fn) {
-    
-    it(title, function() {
-        try {
-           fn.call(this);
-        } catch (err) {
-            if (err.message=="Not implemented") {
-                this.test.skip();
-            } else {
-                throw err;
-            }
-        }
-    });
-
-}
+module.exports = function testOptional(title, fn) {
+  it(title, function () {
+    try {
+      fn.call(this);
+    } catch (err) {
+      if (err.message === 'Not implemented') {
+        this.test.skip();
+      } else {
+        throw err;
+      }
+    }
+  });
+};

task/01-first-tasks.js

@@ -1,12 +1,12 @@
 function concatenateStrings(value1, value2) {
-    return value1 + value2
+  return value1 + value2;
 }
 
-function testingPassFunction(value1, value2) {
-    throw new Error('Not implemented');
+function testingPassFunction() {
+  throw new Error('Not implemented');
 }
 
 module.exports = {
-    concatenateStrings: concatenateStrings,
-    testingPassFunction: testingPassFunction,
-}
+  concatenateStrings,
+  testingPassFunction,
+};

test/01-first-tests.js

@@ -1,12 +1,12 @@
-var assert = require('assert');
-var tasks = require('../task/01-first-tasks');
+const assert = require('assert');
+const tasks = require('../task/01-first-tasks');
 it.optional = require('../extensions/it-optional');
 
-describe('01-first-tests', function() {
-    it('concatenateStrings should return concatenation of two strings', function() {
-      assert.equal(tasks.concatenateStrings('a', 'b'), 'ab');
-    });
-    it.optional('testingPassFunction should ignore a test', function() {
-        assert.equal(tasks.testingPassFunction(), 'ab');
-      });
-});
+describe('01-first-tests', () => {
+  it('concatenateStrings should return concatenation of two strings', () => {
+    assert.equal(tasks.concatenateStrings('a', 'b'), 'ab');
+  });
+  it.optional('testingPassFunction should ignore a test', () => {
+    assert.equal(tasks.testingPassFunction(), 'test');
+  });
+});