Added basics for unit testing

Author: michbeck100

Gruntfile.coffee

@@ -0,0 +1,73 @@
+module.exports = (grunt) ->
+
+  # Project configuration.
+  grunt.initConfig
+    pkg: grunt.file.readJSON "package.json"
+    coffeelint:
+      app: [
+        "*.coffee"
+        "accessories/**/*.coffee"
+        "test/**/*.coffee"
+      ]
+      options:
+        no_trailing_whitespace:
+          level: "ignore"
+        max_line_length:
+          value: 100
+        indentation:
+          value: 2
+          level: "error"
+        no_unnecessary_fat_arrows:
+          level: 'ignore'
+
+    mochaTest:
+      test:
+        options:
+          reporter: "spec"
+          require: ['coffee-errors'] #needed for right line numbers in errors
+        src: ["test/*"]
+      # blanket is used to record coverage
+      testBlanket:
+        options:
+          reporter: "dot"
+        src: ["test/*"]
+      coverage:
+        options:
+          reporter: "html-cov"
+          quiet: true
+          captureFile: "coverage.html"
+        src: ["test/*"]
+
+  grunt.loadNpmTasks "grunt-coffeelint"
+  grunt.loadNpmTasks "grunt-mocha-test"
+
+  grunt.registerTask "blanket", =>
+    blanket = require "blanket"
+
+    blanket(
+      pattern: (file) ->
+        if file.match "pimatic-hap/accessories" then return true
+        #if file.match "pimatic/node_modules" then return false
+        withoutPrefix = file.replace(/.*\/node_modules\/pimatic/, "")
+        return (not withoutPrefix.match 'node_modules') and (not withoutPrefix.match "/test/")
+      loader: "./node-loaders/coffee-script"
+    )
+
+  grunt.registerTask "clean-coverage", =>
+    fs = require "fs"
+    path = require "path"
+
+    replaceAll = (find, replace, str) =>
+      escapeRegExp = (str) => str.replace(/[\-\[\]\/\{\}\(\)\*\+\?\.\\\^\$\|]/g, "\\$&")
+      str.replace(new RegExp(escapeRegExp(find), 'g'), replace)
+
+    file = "#{__dirname}/coverage.html"
+    html = fs.readFileSync(file).toString()
+    html = replaceAll path.dirname(__dirname), "", html
+    fs.writeFileSync file, html
+
+  # Default task(s).
+  grunt.registerTask "default", ["coffeelint", "mochaTest:test"]
+  grunt.registerTask "test", ["coffeelint", "mochaTest:test"]
+  grunt.registerTask "coverage",
+    ["blanket", "mochaTest:testBlanket", "mochaTest:coverage", "clean-coverage"]

accessories/base.coffee

@@ -23,7 +23,7 @@ module.exports = (env) ->
       @getService(Service.AccessoryInformation)
         .setCharacteristic(Characteristic.Manufacturer, "Pimatic")
         .setCharacteristic(Characteristic.Model, "Rev-1")
-        .setCharacteristic(Characteristic.SerialNumber, serialNumber);
+        .setCharacteristic(Characteristic.SerialNumber, serialNumber)
 
       @addService(Service.BridgingState)
         .getCharacteristic(Characteristic.Reachable)
@@ -69,6 +69,8 @@ module.exports = (env) ->
       return false
 
     getServiceOverride: (service) =>
-      if @hapConfig != null && @hapConfig != undefined && @hapConfig.service != null && @hapConfig.service != undefined && @hapConfig.service of @supportedServiceOverrides
+      if @hapConfig != null && @hapConfig != undefined &&
+      @hapConfig.service != null && @hapConfig.service != undefined &&
+      @hapConfig.service of @supportedServiceOverrides
         return @supportedServiceOverrides[@hapConfig.service]
       return service

accessories/shutter.coffee

@@ -43,10 +43,10 @@ module.exports = (env) ->
           .setCharacteristic(Characteristic.LockCurrentState, @getLockCurrentState(position))
 
     getLockCurrentState: (position) =>
-            if position == 'up'
-              return Characteristic.LockCurrentState.UNSECURED
-            else if position == "down"
-              return Characteristic.LockCurrentState.SECURED
-            else
-              # stopped somewhere in between
-              return Characteristic.LockCurrentState.UNKNOWN
+      if position == 'up'
+        return Characteristic.LockCurrentState.UNSECURED
+      else if position == "down"
+        return Characteristic.LockCurrentState.SECURED
+      else
+        # stopped somewhere in between
+        return Characteristic.LockCurrentState.UNKNOWN

accessories/thermostat.coffee

@@ -26,9 +26,9 @@ module.exports = (env) ->
         .on 'get', (callback) =>
           if @_temperature == null
             device.getTemperatureSetpoint().then( (temp) =>
-                @_temperature = temp
-                callback(null, @_temperature)
-              )
+              @_temperature = temp
+              callback(null, @_temperature)
+            )
           else
             callback(null, @_temperature)
 
@@ -79,7 +79,8 @@ module.exports = (env) ->
       device.on 'mode', (mode) =>
         if mode == "auto"
           @getService(Service.Thermostat)
-            .setCharacteristic(Characteristic.TargetHeatingCoolingState, Characteristic.TargetHeatingCoolingState.AUTO)
+            .setCharacteristic(Characteristic.TargetHeatingCoolingState,
+            Characteristic.TargetHeatingCoolingState.AUTO)
 
     setTemperatureTo: (temp) =>
       if @_temperature is temp then return

hap-config-schema.coffee

@@ -15,4 +15,4 @@ module.exports = {
       type: "integer"
       default: 51826
   }
-};
+}

package.json

@@ -5,6 +5,7 @@
   "main": "hap.coffee",
   "files": [
     "accessories",
+    "test",
     "hap.coffee",
     "README.md",
     "hap-config-schema.coffee",
@@ -18,10 +19,23 @@
     "url": "git://github.com/michbeck100/pimatic-hap.git"
   },
   "configSchema": "hap-config-schema.coffee",
+  "scripts": {
+    "test": "grunt test"
+  },
   "dependencies": {
     "hap-nodejs": "0.2.5",
     "color-convert": "1.0.0"
   },
+  "devDependencies": {
+    "cassert": "0.1.2",
+    "grunt-cli": "0.1.11",
+    "grunt": "0.4.2",
+    "grunt-coffeelint": "0.0.8",
+    "grunt-mocha-test": "0.8.1",
+    "blanket": "1.1.5",
+    "coffee-errors": "0.8.4",
+    "mocha": "1.17.0"
+  },
   "peerDependencies": {
     "pimatic": "0.8.*"
   },