refactored test_operations.coffee

test_operations: using `eq` instead of `ok` wherever possible, as per
satyr's suggestion
satyr's suggestion
jashkenas · michaelficarra · michaelficarra · michaelficarra · michaelficarra · michaelficarra
diff --git a/.npmignore b/.npmignore
@@ -0,0 +1,11 @@
+*.coffee
+*.html
+.DS_Store
+.git*
+Cakefile
+documentation/
+examples/
+extras/
+raw/
+src/
+test/
diff --git a/Cakefile b/Cakefile
@@ -73,7 +73,7 @@ task 'build:full', 'rebuild the source twice, and run the tests', ->
 
 
 task 'build:parser', 'rebuild the Jison parser (run build first)', ->
-  extend global, require('utils')
+  extend global, require('util')
   require 'jison'
   parser = require('./lib/grammar').parser
   fs.writeFile 'lib/parser.js', parser.generate()
@@ -143,36 +143,78 @@ task 'loc', 'count the lines of source code in the CoffeeScript compiler', ->
     console.log stdout.trim()
 
 
+# Run the CoffeeScript test suite.
 runTests = (CoffeeScript) ->
-  startTime = Date.now()
-  passedTests = failedTests = 0
+  startTime   = Date.now()
+  currentFile = null
+  passedTests = 0
+  failures    = []
 
-  for name, func of require 'assert'
+  # Mix in the assert module globally, to make it available for tests.
+  addGlobal = (name, func) ->
     global[name] = ->
       passedTests += 1
       func arguments...
 
+  addGlobal name, func for name, func of require 'assert'
+
+  # Convenience aliases.
   global.eq = global.strictEqual
   global.CoffeeScript = CoffeeScript
 
+  # Our test helper function for delimiting different test cases.
+  global.test = (description, fn) ->
+    try
+      fn()
+    catch e
+      e.description = description if description?
+      e.source      = fn.toString() if fn.toString?
+      failures.push file: currentFile, error: e
+
+  # A recursive functional equivalence helper; uses egal for testing equivalence.
+  # See http://wiki.ecmascript.org/doku.php?id=harmony:egal
+  arrayEqual = (a, b) ->
+    if a is b
+      # 0 isnt -0
+      a isnt 0 or 1/a is 1/b
+    else if a instanceof Array and b instanceof Array
+      return no unless a.length is b.length
+      return no for el, idx in a when not arrayEqual el, b[idx]
+      yes
+    else
+      # NaN is NaN
+      a isnt a and b isnt b
+
+  global.arrayEq = (a, b, msg) -> ok arrayEqual(a,b), msg
+
+  # When all the tests have run, collect and print errors.
+  # If a stacktrace is available, output the compiled function source.
   process.on 'exit', ->
     time = ((Date.now() - startTime) / 1000).toFixed(2)
     message = "passed #{passedTests} tests in #{time} seconds#{reset}"
-    if failedTests
-      log "failed #{failedTests} and #{message}", red
-    else
-      log message, green
-
+    return log(message, green) unless failures.length
+    log "failed #{failures.length} and #{message}", red
+    for fail in failures
+      {error, file}      = fail
+      jsFile             = file.replace(/\.coffee$/,'.js')
+      match              = error.stack?.match(new RegExp(fail.file+":(\\d+):(\\d+)"))
+      [match, line, col] = match if match
+      log "\n  #{error.toString()}", red
+      log "  #{error.description}", red if error.description
+      log "  #{jsFile}: line #{line or 'unknown'}, column #{col or 'unknown'}", red
+      console.log "  #{error.source}" if error.source
+
+  # Run every test in the `test` folder, recording failures.
   fs.readdir 'test', (err, files) ->
     files.forEach (file) ->
       return unless file.match(/\.coffee$/i)
       fileName = path.join 'test', file
       fs.readFile fileName, (err, code) ->
+        currentFile = fileName
         try
           CoffeeScript.run code.toString(), {fileName}
-        catch err
-          failedTests += 1
-          log "failed #{fileName}", red, '\n' + err.stack?.toString()
+        catch e
+          failures.push file: currentFile, error: e
 
 
 task 'test', 'run the CoffeeScript language test suite', ->
@@ -182,5 +224,6 @@ task 'test', 'run the CoffeeScript language test suite', ->
 task 'test:browser', 'run the test suite against the merged browser script', ->
   source = fs.readFileSync 'extras/coffee-script.js', 'utf-8'
   result = {}
+  global.testingBrowser = yes
   (-> eval source).call result
   runTests result.CoffeeScript
diff --git a/documentation/coffee/block_comment.coffee b/documentation/coffee/block_comment.coffee
@@ -1,4 +1,6 @@
 ###
-CoffeeScript Compiler v0.9.6
+CoffeeScript Compiler v1.0.0
 Released under the MIT License
-###
+###
+
+
diff --git a/documentation/coffee/conditionals.coffee b/documentation/coffee/conditionals.coffee
@@ -8,4 +8,6 @@ else
 
 date = if friday then sue else jill
 
-options or= defaults
+options or= defaults
+
+
diff --git a/documentation/coffee/do.coffee b/documentation/coffee/do.coffee
@@ -0,0 +1,4 @@
+for fileName in list
+  do (fileName) ->
+    fs.readFile fileName, (err, contents) ->
+      compile fileName, contents.toString()
diff --git a/documentation/coffee/existence.coffee b/documentation/coffee/existence.coffee
@@ -1,6 +1,8 @@
 solipsism = true if mind? and not world?
 
-speed ?= 140
+speed ?= 75
+
+footprints = yeti ? "bear"
 
 
 
diff --git a/documentation/coffee/expressions_assignment.coffee b/documentation/coffee/expressions_assignment.coffee
@@ -1 +1,3 @@
-six = (one = 1) + (two = 2) + (three = 3)
+six = (one = 1) + (two = 2) + (three = 3)
+
+
diff --git a/documentation/coffee/expressions_try.coffee b/documentation/coffee/expressions_try.coffee
@@ -3,4 +3,5 @@ alert(
     nonexistent / undefined
   catch error
     "And the error is ... " + error
-)
+)
+
diff --git a/documentation/coffee/heredocs.coffee b/documentation/coffee/heredocs.coffee
@@ -2,4 +2,6 @@ html = '''
        <strong>
          cup of coffeescript
        </strong>
-       '''
+       '''
+
+
diff --git a/documentation/coffee/heregexes.coffee b/documentation/coffee/heregexes.coffee
@@ -6,4 +6,6 @@ OPERATOR = /// ^ (
    | ([&|<>])\2=?      # logic / shift
    | \?\.              # soak access
    | \.{2,3}           # range or splat
-) ///
+) ///
+
+
diff --git a/documentation/coffee/objects_and_arrays.coffee b/documentation/coffee/objects_and_arrays.coffee
@@ -2,7 +2,7 @@ song = ["do", "re", "mi", "fa", "so"]
 
 singers = {Jagger: "Rock", Elvis: "Roll"}
 
-matrix = [
+bitlist = [
   1, 0, 1
   0, 0, 1
   1, 1, 0
@@ -14,4 +14,6 @@ kids =
     age:  11
   sister:
     name: "Ida"
-    age:  9
+    age:  9
+
+
diff --git a/documentation/coffee/objects_reserved.coffee b/documentation/coffee/objects_reserved.coffee
@@ -1,3 +1,5 @@
 $('.account').attr class: 'active'
 
-log object.class
+log object.class
+
+
diff --git a/documentation/coffee/parallel_assignment.coffee b/documentation/coffee/parallel_assignment.coffee
@@ -1,4 +1,6 @@
 theBait   = 1000
 theSwitch = 0
 
-[theBait, theSwitch] = [theSwitch, theBait]
+[theBait, theSwitch] = [theSwitch, theBait]
+
+
diff --git a/documentation/coffee/prototypes.coffee b/documentation/coffee/prototypes.coffee
@@ -1,2 +1,3 @@
 String::dasherize = ->
-  this.replace /_/g, "-"
+  this.replace /_/g, "-"
+
diff --git a/documentation/coffee/slices.coffee b/documentation/coffee/slices.coffee
@@ -4,3 +4,4 @@ copy    = numbers[0...numbers.length]
 
 middle  = copy[3..6]
 
+
diff --git a/documentation/coffee/splats.coffee b/documentation/coffee/splats.coffee
@@ -22,4 +22,6 @@ awardMedals contenders...
 
 alert "Gold: " + gold
 alert "Silver: " + silver
-alert "The Field: " + rest
+alert "The Field: " + rest
+
+
diff --git a/documentation/coffee/try.coffee b/documentation/coffee/try.coffee
@@ -4,4 +4,5 @@ try
 catch error
   print error
 finally
-  cleanUp()
+  cleanUp()
+