re-outline step 04

Author: ShMcK

tutorial/03/04.js

@@ -15,7 +15,7 @@ describe('04 function increaseScore', () => {
   });
 
   it('shouldn\'t change scores under 95', () => {
-    var test = {
+    const test = {
       score: 82
     };
     expect(increaseScore(test)).to.deep.equal({
@@ -24,7 +24,7 @@ describe('04 function increaseScore', () => {
   });
 
   it('should change scores over 95 to 95', () => {
-    var test = {
+    const test = {
       score: 84
     };
     expect(increaseScore(test)).to.deep.equal({
@@ -39,7 +39,7 @@ describe('04 var mySlightlyChanged', () => {
   const mySlightlyChanged = map.__get__('mySlightlyChanged');
 
   it('should cap scores at 95', () => {
-    var scores = mySlightlyChanged.map(function(x) {
+    const scores = mySlightlyChanged.map(function(x) {
       return x.score;
     });
     expect(Math.max.apply(Math, scores)).to.equal(95);

tutorial/04/01-forEach.spec.js renamed to tutorial/04/01.js

@@ -1,20 +1,20 @@
-'use strict';
-var chai = require('chai');
-var spies = require('chai-spies');
-var expect = chai.expect;
+const chai = require('chai');
+const spies = require('chai-spies');
+const expect = chai.expect;
 chai.use(spies);
 
 // load('04/myFixed.js', true)
 if (process.env.TASK_POSITION === '4') {
 	myFixed = [];
 }
-var spy = chai.spy.on(console, 'log');
-// load('04-forEach.js')
+let spy = chai.spy.on(console, 'log');
 
-describe('01 console.log', function() {
+const map = require('BASE/04-forEach.js');
+
+describe('01 console.log', () => {
 
 	if (process.env.TASK_POSITION !== '4') {
-		it('should be called 10 times', function() {
+		it('should be called 10 times', () => {
 			expect(spy).to.have.been.called.with('A  95  Relational Databases');
 			expect(spy).to.have.been.called.with('C  77  Networks');
 		});

tutorial/04/02-forEach.spec.js renamed to tutorial/04/02.js

@@ -1,7 +1,7 @@
-describe('02 console.log', function() {
+describe('02 console.log', () => {
 
   if (process.env.TASK_POSITION !== '4') {
-    it('should begin with an index', function() {
+    it('should begin with an index', () => {
       expect(spy).to.have.been.called.with('1  A  95  Relational Databases');
       expect(spy).to.have.been.called.with('10  C  77  Networks');
     });

tutorial/04/03-forEach.spec.js renamed to tutorial/04/03.js

@@ -1,7 +1,7 @@
-describe('03 console.log', function() {
+describe('03 console.log', () => {
 
   if (process.env.TASK_POSITION !== '4') {
-    it('should begin with an index', function() {
+    it('should begin with an index', () => {
       expect(spy).to.have.been.called.with('1/10  A  95  Relational Databases');
       expect(spy).to.have.been.called.with('10/10  C  77  Networks');
     });

tutorial/04/04-forEach.spec.js

@@ -0,0 +1,7 @@
+describe('04 log', () => {
+
+  it('should pass', () => {
+    expect(true).to.be.true;
+  });
+
+});

tutorial/04/04.js

@@ -87,7 +87,7 @@ function addOneToLog(x) {
 Now that we see how `forEach` works, let's use it to make calls to the `console`.
 
 + Use `forEach` to log out your report card to the console
-@test('04/01-forEach')
+@test('04/01')
 @action(open('04-forEach.js'))
 @action(set(
 ```
@@ -104,7 +104,7 @@ myFixed.forEach(::>);
 @hint('call `forEach` with `logCourse`')
 
 + Add a second parameter to `logCourseWithIndex` called `index`. Then call the function with `myFixed.forEach`.
-@test('04/02-forEach')
+@test('04/02')
 @action(insert(
 ```
 
@@ -121,7 +121,7 @@ myFixed.forEach(logCourseWithIndex);
 @hint('Add a second parameter to `logCourseWithIndex`')
 
 + Add a third parameter called `array` to `logCourseWithIndexAndArray`, then call the function with `myFixed.forEach`.
-@test('04/03-forEach')
+@test('04/03')
 @action(insert(
 ```
 
@@ -152,7 +152,7 @@ myFixed = students
 
 This is why side-effects are dangerous. Students data must have changed, and now all of your transformations are effected.
 
-@test('04/04-forEach')
+@test('04/04')
 @action(insert(
 ```