Add specs for printing backtrace to STDERR in case of unhandled excep…

…tion or crash (#4171)
crystal-lang · Mar 21, 2017 · e4d1f92 · e4d1f92
1 parent ae678b2
commit e4d1f92
Show file tree

Hide file tree

Showing 3 changed files with 67 additions and 0 deletions.
diff --git a/spec/std/callstack_spec.cr b/spec/std/callstack_spec.cr
@@ -26,4 +26,36 @@ describe "Backtrace" do
     output.should_not match(/src\/exception\.cr/)
     output.should_not match(/src\/raise\.cr/)
   end
+
+  it "prints exception backtrace to stderr" do
+    tempfile = Tempfile.new("compiler_spec_output")
+    tempfile.close
+    sample = "#{__DIR__}/data/exception_backtrace_sample"
+
+    `bin/crystal build --debug #{sample.inspect} -o #{tempfile.path.inspect}`
+    File.exists?(tempfile.path).should be_true
+
+    output, error = {IO::Memory.new, IO::Memory.new}.tap do |outio, errio|
+      Process.run tempfile.path, output: outio, error: errio
+    end
+
+    output.to_s.empty?.should be_true
+    error.to_s.should contain("IndexError")
+  end
+
+  it "prints crash backtrace to stderr" do
+    tempfile = Tempfile.new("compiler_spec_output")
+    tempfile.close
+    sample = "#{__DIR__}/data/crash_backtrace_sample"
+
+    `bin/crystal build --debug #{sample.inspect} -o #{tempfile.path.inspect}`
+    File.exists?(tempfile.path).should be_true
+
+    output, error = {IO::Memory.new, IO::Memory.new}.tap do |outio, errio|
+      Process.run tempfile.path, output: outio, error: errio
+    end
+
+    output.to_s.empty?.should be_true
+    error.to_s.should contain("Invalid memory access")
+  end
 end
diff --git a/spec/std/data/crash_backtrace_sample b/spec/std/data/crash_backtrace_sample
@@ -0,0 +1,18 @@
+class Kls
+  def callee1
+    foo = "foo".to_unsafe
+    foo.realloc(100)
+  end
+end
+
+def callee2
+  yield
+end
+
+def callee3
+  callee2 do
+    Kls.new.callee1
+  end
+end
+
+callee3
diff --git a/spec/std/data/exception_backtrace_sample b/spec/std/data/exception_backtrace_sample
@@ -0,0 +1,17 @@
+class Kls
+  def callee1
+    (0..10).to_a[123]
+  end
+end
+
+def callee2
+  yield
+end
+
+def callee3
+  callee2 do
+    Kls.new.callee1
+  end
+end
+
+callee3