Merge pull request #416 from kazk/fix/LF/crystal

[Crystal] Fix LF issue

Author: jhoffner

frameworks/crystal/formatter.cr

@@ -6,16 +6,16 @@ module Spec
   end
 
   def self.example(msg, mode = "")
-    puts format_message("<LOG:#{mode.upcase}:Example>#{msg}")
+    puts format_message("\n<LOG:#{mode.upcase}:Example>#{msg}")
   end
 
   def self.log(msg, mode = "", label = "")
-    puts format_message("<LOG:#{mode.upcase}:#{label}>#{msg}")
+    puts format_message("\n<LOG:#{mode.upcase}:#{label}>#{msg}")
   end
 
   class RootContext
     def print_results(elapsed_time)
-      puts "<COMPLETEDIN::>#{Spec.to_human(elapsed_time)}"
+      puts "\n<COMPLETEDIN::>#{Spec.to_human(elapsed_time)}"
     end
   end
 
@@ -25,15 +25,15 @@ module Spec
     end
 
     def push(context)
-      puts "<DESCRIBE::>#{Spec.format_message(context.description)}"
+      puts "\n<DESCRIBE::>#{Spec.format_message(context.description)}"
     end
 
     def pop
-      puts "<COMPLETEDIN::>"
+      puts "\n<COMPLETEDIN::>"
     end
 
     def before_example(description)
-      puts "<IT::>#{Spec.format_message(description)}"
+      puts "\n<IT::>#{Spec.format_message(description)}"
     end
 
     def report(result)
@@ -49,18 +49,18 @@ module Spec
 
       case result.kind
         when :success
-          puts "<PASSED::>Test Passed"
+          puts "\n<PASSED::>Test Passed"
         when :fail
-          puts "<FAILED::>Test Failed: #{msg}"
+          puts "\n<FAILED::>Test Failed: #{msg}"
 
         when :error
-          puts "<ERROR::>#{msg}"
+          puts "\n<ERROR::>#{msg}"
       end
 
       if result.exception
-        puts "<ERROR::>#{Spec.format_message(result.exception.to_s)}"
+        puts "\n<ERROR::>#{Spec.format_message(result.exception.to_s)}"
       end
-      puts "<COMPLETEDIN::>"
+      puts "\n<COMPLETEDIN::>"
     end
 
     def finish

test/runners/crystal_spec.js

@@ -7,7 +7,10 @@ describe('crystal runner', function() {
     runner.assertCodeExamples('crystal');
 
     it('should handle basic code evaluation', function(done) {
-      runner.run({language: 'crystal', code: 'puts 42'}, function(buffer) {
+      runner.run({
+        language: 'crystal',
+        code: 'puts 42'
+      }, function(buffer) {
         expect(buffer.stdout).to.equal('42\n');
         done();
       });
@@ -18,12 +21,14 @@ describe('crystal runner', function() {
         runner.run({
           language: 'crystal',
           code: 'a = 1',
-          fixture: 'describe "test" do\n' +
-                        'it("test2") { 1.should eq(1)}\n' +
-                        'end',
+          fixture: [
+            'describe "test" do',
+            '  it("test2") { 1.should eq(1) }',
+            'end',
+          ].join('\n'),
           testFramework: 'spec'
         }, function(buffer) {
-          expect(buffer.stdout).to.include('<DESCRIBE::>test\n<IT::>test2\n<PASSED::>');
+          expect(buffer.stdout).to.include('<DESCRIBE::>test\n\n<IT::>test2\n\n<PASSED::>');
           done();
         });
       });
@@ -32,32 +37,56 @@ describe('crystal runner', function() {
         runner.run({
           language: 'crystal',
           code: 'a = 1',
-          fixture: 'describe "test" do\n' +
-                        'it("test2") { 1.should eq(2)}\n' +
-                        'end',
+          fixture: [
+            'describe "test" do',
+            '  it("test2") { 1.should eq(2) }',
+            'end',
+          ].join('\n'),
           testFramework: 'spec'
         }, function(buffer) {
-          expect(buffer.stdout).to.contain('<DESCRIBE::>test\n<IT::>test2');
+          expect(buffer.stdout).to.contain('<DESCRIBE::>test\n\n<IT::>test2');
           expect(buffer.stdout).to.contain('<FAILED::>');
           expect(buffer.stdout).to.not.contain('<PASSED::>');
           done();
         });
       });
+
       it('should handle errored code', function(done) {
         runner.run({
           language: 'crystal',
           code: 'A = 1',
-          fixture: 'describe "test" do\n' +
-                        'it("test2") { A.should eq 1 }\n' +
-                        'it("test2") { A.idontexist()}\n' +
-                        'end',
+          fixture: [
+            'describe "test" do',
+            '  it("test2") { A.should eq 1 }',
+            '  it("test2") { A.idontexist() }',
+            'end',
+          ].join('\n'),
           testFramework: 'spec'
         }, function(buffer) {
           expect(buffer.stdout).to.eq('');
           expect(buffer.stderr).to.contain('undefined method');
           done();
         });
       });
+
+      it('should have output format command on independent line', function(done) {
+        runner.run({
+          language: 'crystal',
+          testFramework: 'spec',
+          code: '#',
+          fixture: [
+            'describe "tests" do',
+            '  it "test case" do',
+            '    print "foo"',
+            '    1.should eq 2',
+            '  end',
+            'end',
+          ].join('\n'),
+        }, function(buffer) {
+          expect(buffer.stdout).to.contain('\n<FAILED::>');
+          done();
+        });
+      });
     });
   });
 });