Updating dependencies and fixing failing Ping Bell test

Updating dependencies and fixing failing Ping Bell test

Fixing Package info

Fighting with Travis

Author: markpete

.travis.yml

@@ -1,10 +1,10 @@
 language: node_js
 
 node_js:
-  - '0.10'
-  - '0.12'
-  - '4'
   - '5'
+  - '7'
+  - 'lts/*'
+  - 'node'
 
 before_install:
   - npm install -g gulp

exercises/ping_bell/exercise.js

@@ -40,7 +40,7 @@ exercise.addVerifyProcessor(verifyProcessor(exercise, function (test, done) {
   test.equals(piezo.pin, pins.piezo, 'connect_speaker_to_pin', {pin: pins.piezo})
 
   var initial = {
-    tone: {callCount: piezo.tone.callCount}
+    play: {callCount: piezo.play.callCount}
   }
 
   var buffer = new Buffer('HAI!?')
@@ -50,7 +50,7 @@ exercise.addVerifyProcessor(verifyProcessor(exercise, function (test, done) {
   // Allow some time for the udp packet to reach server and sound to be played
   setTimeout(function () {
     try {
-      test.truthy(piezo.tone.callCount > initial.tone.callCount, 'speaker_played_tune_on_udp_message')
+      test.truthy(piezo.play.callCount > initial.play.callCount, 'speaker_played_tune_on_udp_message')
 
       done()
     } catch (error) {

exercises/ping_bell/solution/solution.js

@@ -7,8 +7,12 @@ board.on('ready', function () {
   var server = dgram.createSocket('udp4')
 
   server.on('message', function () {
-    piezo.tone(five.Piezo.Notes.c4, 1000)
-  })
+    piezo.play({
+      song: 'C D F D A',
+      beats: 1 / 4,
+      tempo: 100
+    });
+  });
 
   server.bind(1337)
-})
+});

nodebot-workshop.js

@@ -3,7 +3,7 @@
 const workshop = require('workshopper-adventure')({
     appDir    : __dirname
   , languages : ['en', 'ja', 'fr']
-  , menu      : {fg: 'black', bg: /^win/.test(process.platform) ? 'yellow' : 220}
+  , menu      : {fg: 'black', bg: 220}
   , header    : require('workshopper-adventure/default/header')
   , footer    : require('workshopper-adventure/default/footer')
 })

package.json

@@ -36,25 +36,25 @@
     "url": "https://github.com/tableflip/nodebot-workshop/issues"
   },
   "dependencies": {
-    "async": "^2.0.1",
-    "dnode": "^1.2.0",
-    "ioboard": "^3.0",
-    "johnny-five": "^0.10.0",
-    "proxyquire": "^1.0.0",
-    "sinon": "^1.9",
-    "workshopper-adventure": "4.6.1",
-    "workshopper-exercise": "^2.5.3"
+    "async": "^2.6.0",
+    "dnode": "^1.2.2",
+    "ioboard": "^3.0.0",
+    "johnny-five": "^0.14.1",
+    "proxyquire": "^1.8.0",
+    "sinon": "^4.4.2",
+    "workshopper-adventure": "^6.0.3",
+    "workshopper-exercise": "^3.0.1"
   },
   "optionalDependencies": {
-    "node-notifier": "^4.3.1"
+    "node-notifier": "^5.2.1"
   },
   "devDependencies": {
     "gulp": "^3.9.1",
-    "gulp-jshint": "^2.0.0",
-    "gulp-shell": "^0.5.1",
-    "jshint": "^2.8.0",
-    "jshint-stylish": "^2.1.0",
-    "pre-commit": "^1.1.2"
+    "gulp-jshint": "^2.1.0",
+    "gulp-shell": "^0.6.5",
+    "jshint": "^2.9.5",
+    "jshint-stylish": "^2.2.1",
+    "pre-commit": "^1.2.2"
   },
   "pre-commit": [
     "lint",

tests/verify-solutions.js

@@ -79,7 +79,7 @@ if (process.argv.length > 2){
 
 // foo/exercises/ping_bell/solution/solution.js -> Ping Bell
 function nameFromPath(path){
-  var regex = /exercises\/(\w+)\/solution\/solution\.js/
+  var regex = /exercises(?:\/|\\)(\w+)(?:\/|\\)solution(?:\/|\\)solution\.js/
   var res = path.match(regex)
   if(!res) throw new Error('Cannot establish exercise from path: ' + path)
   return res[1]