rename LD variation

rubygems · Mar 16, 2023 · 3293a22 · 3293a22
1 parent bf31f8b
commit 3293a22
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 7 deletions.
diff --git a/app/controllers/concerns/privacy_pass_supportable.rb b/app/controllers/concerns/privacy_pass_supportable.rb
@@ -27,6 +27,6 @@ def delete_privacy_pass_token_redemption
 
   def privacy_pass_enabled?
     ld_context = LaunchDarkly::LDContext.with_key(self.class.name)
-    Rails.configuration.launch_darkly_client.variation("privacy_pass.enabled", ld_context, false)
+    Rails.configuration.launch_darkly_client.variation("gemcutter.privacy_pass.enabled", ld_context, false)
   end
 end
diff --git a/test/integration/sign_in_test.rb b/test/integration/sign_in_test.rb
@@ -336,7 +336,7 @@ class SignInTest < SystemTest
   end
 
   test "signing in when privacy pass is verified" do
-    Rails.configuration.launch_darkly_client.expects(:variation).with("privacy_pass.enabled", anything, anything).returns(true).at_least_once
+    Rails.configuration.launch_darkly_client.expects(:variation).with("gemcutter.privacy_pass.enabled", anything, anything).returns(true).at_least_once
     # privacy pass is meant to _not_ be machine automatable, so we're stubbing in this case
     PrivacyPassRedeemer.expects(:call).with(anything, anything).returns(true)
 
@@ -349,7 +349,7 @@ class SignInTest < SystemTest
   end
 
   test "signing in when privacy pass is not verified and captcha is not triggered" do
-    Rails.configuration.launch_darkly_client.expects(:variation).with("privacy_pass.enabled", anything, anything).returns(true).at_least_once
+    Rails.configuration.launch_darkly_client.expects(:variation).with("gemcutter.privacy_pass.enabled", anything, anything).returns(true).at_least_once
     # privacy pass is meant to _not_ be machine automatable, so we're stubbing in this case
     PrivacyPassRedeemer.expects(:call).with(anything, anything).returns(false)
 
@@ -362,7 +362,7 @@ class SignInTest < SystemTest
   end
 
   test "signing in when privacy pass is not verified and captcha is triggered" do
-    Rails.configuration.launch_darkly_client.expects(:variation).with("privacy_pass.enabled", anything, anything).returns(true).at_least_once
+    Rails.configuration.launch_darkly_client.expects(:variation).with("gemcutter.privacy_pass.enabled", anything, anything).returns(true).at_least_once
     @scope = Rack::Attack::LOGIN_THROTTLE_PER_USER_KEY
     update_limit_for("#{@scope}:#{@user.email}", 4, Rack::Attack::LOGIN_LIMIT_PERIOD)
     # captcha is meant to _not_ be machine automatable, so we're stubbing in this case

diff --git a/test/integration/sign_up_test.rb b/test/integration/sign_up_test.rb
@@ -112,7 +112,8 @@ class SignUpTest < SystemTest
   end
 
   test "sign up when privacy pass is verified" do
-    Rails.configuration.launch_darkly_client.expects(:variation).with("privacy_pass.enabled", anything, anything).returns(true).at_least_once
+    Rails.configuration.launch_darkly_client.expects(:variation).with("gemcutter.privacy_pass.enabled", anything,
+anything).returns(true).at_least_once
     # privacy pass is meant to _not_ be machine automatable, so we're stubbing in this case
     PrivacyPassRedeemer.expects(:call).with(anything, anything).returns(true)
 
@@ -127,7 +128,8 @@ class SignUpTest < SystemTest
   end
 
   test "sign up when privacy pass is not verified and captcha is not triggered" do
-    Rails.configuration.launch_darkly_client.expects(:variation).with("privacy_pass.enabled", anything, anything).returns(true).at_least_once
+    Rails.configuration.launch_darkly_client.expects(:variation).with("gemcutter.privacy_pass.enabled", anything,
+anything).returns(true).at_least_once
     # privacy pass is meant to _not_ be machine automatable, so we're stubbing in this case
     PrivacyPassRedeemer.expects(:call).with(anything, anything).returns(false)
 
@@ -142,7 +144,8 @@ class SignUpTest < SystemTest
   end
 
   test "sign up when privacy pass is not verified and captcha verification is triggered" do
-    Rails.configuration.launch_darkly_client.expects(:variation).with("privacy_pass.enabled", anything, anything).returns(true).at_least_once
+    Rails.configuration.launch_darkly_client.expects(:variation).with("gemcutter.privacy_pass.enabled", anything,
+anything).returns(true).at_least_once
     # privacy pass is meant to _not_ be machine automatable, so we're stubbing in this case
     PrivacyPassRedeemer.expects(:call).with(anything, anything).returns(false)