Add configuration option for user access scopes strategy

CHANGELOG.md

@@ -1,5 +1,6 @@
 Unreleased
 ----------
+* Add configuration option for user access strategy [#1599](https://github.com/Shopify/shopify_app/pull/1599)
 * Fixes a bug with `EnsureAuthenticatedLinks` causing deep links to not work [#1549](https://github.com/Shopify/shopify_app/pull/1549)
 * Ensure online token is properly used when using `current_shopify_session` [#1566](https://github.com/Shopify/shopify_app/pull/1566)
 * Added debug logs, you can read more about logging [here](./docs/logging.md). [#1545](https://github.com/Shopify/shopify_app/pull/1545)

lib/shopify_app/configuration.rb

@@ -83,7 +83,17 @@ def shop_access_scopes_strategy
       ShopifyApp::AccessScopes::ShopStrategy
     end
 
+    def user_access_scopes_strategy=(class_name)
+      unless class_name.is_a?(String)
+        raise ConfigurationError, "Invalid user access scopes strategy - expected a string"
+      end
+
+      @user_access_scopes_strategy = class_name.safe_constantize
+    end
+
     def user_access_scopes_strategy
+      return @user_access_scopes_strategy if @user_access_scopes_strategy
+
       return ShopifyApp::AccessScopes::NoopStrategy unless reauth_on_access_scope_changes
 
       ShopifyApp::AccessScopes::UserStrategy

test/shopify_app/configuration_test.rb

@@ -209,4 +209,23 @@ class ConfigurationTest < ActiveSupport::TestCase
     assert_equal ShopifyApp::AccessScopes::ShopStrategy, ShopifyApp.configuration.shop_access_scopes_strategy
     assert_equal ShopifyApp::AccessScopes::UserStrategy, ShopifyApp.configuration.user_access_scopes_strategy
   end
+
+  test "user access scopes strategy is configurable with a string" do
+    my_strategy = "Object"
+    ShopifyApp.configure do |config|
+      config.user_access_scopes_strategy = my_strategy
+    end
+
+    assert_equal ShopifyApp::AccessScopes::NoopStrategy, ShopifyApp.configuration.shop_access_scopes_strategy
+    assert_equal Object, ShopifyApp.configuration.user_access_scopes_strategy
+  end
+
+  test "user access scopes strategy is not configurable with a constant" do
+    error = assert_raises ShopifyApp::ConfigurationError do
+      ShopifyApp.configure do |config|
+        config.user_access_scopes_strategy = Object
+      end
+    end
+    assert_equal "Invalid user access scopes strategy - expected a string", error.message
+  end
 end