Skip to content

Commit 03e5506

Browse files
committed
Fix failing specs
1 parent 93a2a42 commit 03e5506

File tree

1 file changed

+9
-6
lines changed

1 file changed

+9
-6
lines changed

spec/datadog/appsec/api_security/route_extractor_spec.rb

Lines changed: 9 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -96,19 +96,20 @@
9696
before do
9797
allow(request).to receive(:env).and_return({
9898
'action_dispatch.routes' => route_set,
99-
'action_dispatch.request.path_parameters' => {}
99+
'action_dispatch.request.path_parameters' => {},
100+
'PATH_INFO' => '/users/1'
100101
})
101102
end
102103

103104
let(:router) { double('ActionDispatch::Routing::RouteSet::Router') }
104105
let(:route_set) { double('ActionDispatch::Routing::RouteSet', router: router) }
105-
let(:request) { double('Rack::Request', env: {}, script_name: '', path: '/users/1') }
106+
let(:request) { double('Rack::Request', script_name: '', path: '/users/1') }
106107

107108
it { expect(described_class.route_pattern(request)).to eq('/users/{param:int}') }
108109

109110
it 'persists inferred route in the request env' do
110111
expect { described_class.route_pattern(request) }
111-
.to change { request.env[Datadog::Tracing::Contrib::Rack::Ext::DATADOG_INFERRED_ROUTE] }
112+
.to change { request.env[Datadog::Tracing::Contrib::Rack::RouteInference::DATADOG_INFERRED_ROUTE_ENV_KEY] }
112113
.from(nil).to('/users/{param:int}')
113114
end
114115
end
@@ -157,8 +158,10 @@
157158

158159
context 'when Rails router cannot recognize request' do
159160
before do
160-
allow(request).to receive(:env).and_return({'action_dispatch.routes' => route_set})
161-
allow(request).to receive(:path).and_return('/unmatched/route')
161+
allow(request).to receive(:env).and_return({
162+
'action_dispatch.routes' => route_set,
163+
'PATH_INFO' => '/unmatched/route'
164+
})
162165
allow(router).to receive(:recognize).with(request).and_return([])
163166
end
164167

@@ -175,7 +178,7 @@
175178
end
176179

177180
context 'when route has nested path' do
178-
before { allow(request).to receive(:path).and_return('/some/other/path') }
181+
before { allow(request).to receive(:env).and_return({'PATH_INFO' => '/some/other/path'}) }
179182

180183
it { expect(described_class.route_pattern(request)).to eq('/some/other/path') }
181184
end

0 commit comments

Comments
 (0)