Commit ef234bf7 authored by Sean McGivern's avatar Sean McGivern

Merge branch 'jejacks0n/finalize-published-experiment-spec' into 'master'

Finish up pending spec for published experiments

See merge request gitlab-org/gitlab!73894
parents f922bc59 eea81d74
...@@ -160,18 +160,17 @@ RSpec.describe API::Experiments do ...@@ -160,18 +160,17 @@ RSpec.describe API::Experiments do
end end
it 'publishes into a collection of experiments that have been run in the request' do it 'publishes into a collection of experiments that have been run in the request' do
pending 'requires gitlab-experiment >= 0.5.4 -- resolved in a follow up MR' expect(RequestStore).to receive(:clear!).and_wrap_original do |clear|
expect(ApplicationExperiment.published_experiments['null_hypothesis']).to include(
get api('/experiments', user)
expect(ApplicationExperiment.published_experiments).to eq(
'null_hypothesis' => {
excluded: false, excluded: false,
experiment: 'null_hypothesis', experiment: 'null_hypothesis',
key: 'abc123',
variant: 'candidate' variant: 'candidate'
} )
)
clear.call
end
get api('/experiments', user)
end end
end end
end end
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment