diff options
Diffstat (limited to 'spec/experiments/members/invite_email_experiment_spec.rb')
-rw-r--r-- | spec/experiments/members/invite_email_experiment_spec.rb | 41 |
1 files changed, 26 insertions, 15 deletions
diff --git a/spec/experiments/members/invite_email_experiment_spec.rb b/spec/experiments/members/invite_email_experiment_spec.rb index 4376c021385..539230e39b9 100644 --- a/spec/experiments/members/invite_email_experiment_spec.rb +++ b/spec/experiments/members/invite_email_experiment_spec.rb @@ -3,26 +3,14 @@ require 'spec_helper' RSpec.describe Members::InviteEmailExperiment do - subject :invite_email do - experiment('members/invite_email', actor: double('Member', created_by: double('User', avatar_url: '_avatar_url_'))) - end + subject(:invite_email) { experiment('members/invite_email', **context) } + + let(:context) { { actor: double('Member', created_by: double('User', avatar_url: '_avatar_url_')) } } before do allow(invite_email).to receive(:enabled?).and_return(true) end - describe "#rollout_strategy" do - it "resolves to round_robin" do - expect(invite_email.rollout_strategy).to eq(:round_robin) - end - end - - describe "#variants" do - it "has all the expected variants" do - expect(invite_email.variants).to match(%i[avatar permission_info control]) - end - end - describe "exclusions", :experiment do it "excludes when created by is nil" do expect(experiment('members/invite_email')).to exclude(actor: double(created_by: nil)) @@ -34,4 +22,27 @@ RSpec.describe Members::InviteEmailExperiment do expect(experiment('members/invite_email')).to exclude(actor: member_without_avatar_url) end end + + describe "variant resolution", :clean_gitlab_redis_shared_state do + it "proves out round robin in variant selection", :aggregate_failures do + instance_1 = described_class.new('members/invite_email', **context) + allow(instance_1).to receive(:enabled?).and_return(true) + instance_2 = described_class.new('members/invite_email', **context) + allow(instance_2).to receive(:enabled?).and_return(true) + instance_3 = described_class.new('members/invite_email', **context) + allow(instance_3).to receive(:enabled?).and_return(true) + + instance_1.try { } + + expect(instance_1.variant.name).to eq('permission_info') + + instance_2.try { } + + expect(instance_2.variant.name).to eq('control') + + instance_3.try { } + + expect(instance_3.variant.name).to eq('avatar') + end + end end |