Merge pull request #608 from apiaryio/honzajavorek/fix-dot-reporter

Fixed dot reporter

Author: w-vi

src/dredd.coffee

@@ -47,7 +47,6 @@ class Dredd
     configureReporters @configuration, @stats, @tests, @runner
 
   run: (callback) ->
-
     @configDataIsEmpty = true
 
     @configuration.files ?= []

src/reporters/dot-reporter.coffee

@@ -1,50 +1,66 @@
-logger = require './../logger'
-prettifyResponse = require './../prettify-response'
+
+logger = require('../logger')
+prettifyResponse = require('../prettify-response')
+
 
 class DotReporter
+
   constructor: (emitter, stats, tests) ->
-    @type = "dot"
+    @type = 'dot'
     @stats = stats
     @tests = tests
-    @configureEmitter emitter
+    @configureEmitter(emitter)
     @errors = []
 
-  configureEmitter: (emitter) =>
-    emitter.on 'start', ->
-      logger.info 'Beginning Dredd testing...'
+  configureEmitter: (emitter) ->
+    emitter.on('start', (rawBlueprint, callback) ->
+      logger.info('Beginning Dredd testing...')
+      callback()
+    )
 
-    emitter.on 'end', (callback) =>
+    emitter.on('end', (callback) =>
       if @stats.tests > 0
         if @errors.length > 0
-          @write "\n"
-          logger.info "Displaying failed tests..."
+          @write('\n')
+          logger.info('Displaying failed tests...')
           for test in @errors
-            logger.fail test.title + " duration: #{test.duration}ms"
-            logger.fail test.message
-            logger.request "\n" + prettifyResponse(test.request) + "\n"
-            logger.expected "\n" + prettifyResponse(test.expected) + "\n"
-            logger.actual "\n" + prettifyResponse(test.actual) + "\n\n"
-        @write "\n"
-        logger.complete "#{@stats.passes} passing, #{@stats.failures} failing, #{@stats.errors} errors, #{@stats.skipped} skipped"
-        logger.complete "Tests took #{@stats.duration}ms"
+            logger.fail(test.title + " duration: #{test.duration}ms")
+            logger.fail(test.message)
+            logger.request("\n" + prettifyResponse(test.request) + "\n")
+            logger.expected("\n" + prettifyResponse(test.expected) + "\n")
+            logger.actual("\n" + prettifyResponse(test.actual) + "\n\n")
+        @write('\n')
+
+        logger.complete("""\
+          #{@stats.passes} passing, #{@stats.failures} failing, \
+          #{@stats.errors} errors, #{@stats.skipped} skipped
+        """)
+        logger.complete("Tests took #{@stats.duration}ms")
         callback()
+    )
 
-    emitter.on 'test pass', (test) =>
-      @write "."
+    emitter.on('test pass', (test) =>
+      @write('.')
+    )
 
-    emitter.on 'test skip', (test) =>
-      @write "-"
+    emitter.on('test skip', (test) =>
+      @write('-')
+    )
 
-    emitter.on 'test fail', (test) =>
-      @write "F"
-      @errors.push test
+    emitter.on('test fail', (test) =>
+      @write('F')
+      @errors.push(test)
+    )
 
-    emitter.on 'test error', (error, test) =>
-      @write "E"
+    emitter.on('test error', (error, test) =>
+      @write('E')
       test.message = "\nError: \n"  + error + "\nStacktrace: \n" + error.stack + "\n"
-      @errors.push test
+      @errors.push(test)
+    )
 
   write: (str) ->
-    process.stdout.write str
+    process.stdout.write(str)
+
+
 
 module.exports = DotReporter

test/unit/reporters/dot-reporter-test.coffee

@@ -44,8 +44,8 @@ describe 'DotReporter', () ->
       loggerStub.info.restore()
 
     it 'should log that testing has begun', () ->
-      emitter.emit 'start'
-      assert.ok loggerStub.info.called
+      emitter.emit 'start', '', () ->
+        assert.ok loggerStub.info.called
 
   describe 'when ending', () ->
 
@@ -153,4 +153,3 @@ describe 'DotReporter', () ->
 
     it 'should write an E', () ->
       assert.ok dotReporter.write.calledWith('E')
-