Fixed BacktraceLine::IN_APP_PATH regexp issue

Sergey Kuznetsov
1 parent 5a59cc4f
Showing 2 changed files with 13 additions and 1 deletions Show diff stats
app/models/backtrace_line.rb
spec/models/backtrace_line.rb
 class BacktraceLine
   include Mongoid::Document
-  IN_APP_PATH = %r{^\[PROJECT_ROOT\]\/(?!(vendor))}
+  IN_APP_PATH = %r{^\[PROJECT_ROOT\](?!(\/vendor))}
   GEMS_PATH   = %r{\[GEM_ROOT\]\/gems\/([^\/]+)}
   field :number, :type => Integer
@@ -0,0 +1,12 @@
+require 'spec_helper'
+
+describe BacktraceLine do
+  subject { described_class.new(raw_line) }
+
+  describe "root at the start of decorated filename" do
+    let(:raw_line) { { 'number' => rand(999), 'file' => '[PROJECT_ROOT]/app/controllers/pages_controller.rb', 'method' => ActiveSupport.methods.shuffle.first.to_s } }
+    it "should leave leading root symbol in filepath" do
+      subject.decorated_path.should == '/app/controllers/'
+    end
+  end
+end
1	class BacktraceLine	1	class BacktraceLine
2	include Mongoid::Document	2	include Mongoid::Document
3	- IN_APP_PATH = %r{^\[PROJECT_ROOT\]\/(?!(vendor))}	3	+ IN_APP_PATH = %r{^\[PROJECT_ROOT\](?!(\/vendor))}
4	GEMS_PATH = %r{\[GEM_ROOT\]\/gems\/([^\/]+)}	4	GEMS_PATH = %r{\[GEM_ROOT\]\/gems\/([^\/]+)}
5		5
6	field :number, :type => Integer	6	field :number, :type => Integer
@@ -0,0 +1,12 @@		@@ -0,0 +1,12 @@
	1	+require 'spec_helper'
	2	+
	3	+describe BacktraceLine do
	4	+ subject { described_class.new(raw_line) }
	5	+
	6	+ describe "root at the start of decorated filename" do
	7	+ let(:raw_line) { { 'number' => rand(999), 'file' => '[PROJECT_ROOT]/app/controllers/pages_controller.rb', 'method' => ActiveSupport.methods.shuffle.first.to_s } }
	8	+ it "should leave leading root symbol in filepath" do
	9	+ subject.decorated_path.should == '/app/controllers/'
	10	+ end
	11	+ end
	12	+end