Use memoize option when using Cloud

Author: bkeepers

lib/flipper/cloud/configuration.rb

@@ -40,6 +40,9 @@ def self.brow_instances
       # Public: net/http write timeout for all http requests (default: 5).
       attr_accessor :write_timeout
 
+      # Public: Memoize adapter operations. Defaults to true.
+      attr_accessor :memoize
+
       # Public: IO stream to send debug output too. Off by default.
       #
       #  # for example, this would send all http request information to STDOUT
@@ -85,6 +88,7 @@ def initialize(options = {})
         @sync_interval = options.fetch(:sync_interval) { ENV.fetch("FLIPPER_CLOUD_SYNC_INTERVAL", 10).to_f }
         @sync_secret = options.fetch(:sync_secret) { ENV["FLIPPER_CLOUD_SYNC_SECRET"] }
         @local_adapter = options.fetch(:local_adapter) { Adapters::Memory.new }
+        @memoize = options.fetch(:memoize, true)
         @debug_output = options[:debug_output]
         @adapter_block = ->(adapter) { adapter }
         self.url = options.fetch(:url) { ENV.fetch("FLIPPER_CLOUD_URL", DEFAULT_URL) }

lib/flipper/cloud/dsl.rb

@@ -7,7 +7,10 @@ class DSL < SimpleDelegator
 
       def initialize(cloud_configuration)
         @cloud_configuration = cloud_configuration
-        super Flipper.new(@cloud_configuration.adapter, instrumenter: @cloud_configuration.instrumenter)
+        super Flipper.new(@cloud_configuration.adapter,
+            instrumenter: @cloud_configuration.instrumenter,
+            memoize: @cloud_configuration.memoize
+        )
       end
 
       def sync

lib/flipper/cloud/engine.rb

@@ -15,7 +15,8 @@ class Engine < Rails::Engine
             if ENV["FLIPPER_CLOUD_TOKEN"]
               Flipper::Cloud.new(
                 local_adapter: config.adapter,
-                instrumenter: app.config.flipper.instrumenter
+                instrumenter: app.config.flipper.instrumenter,
+                memoize: app.config.flipper.memoize
               )
             else
               warn "Missing FLIPPER_CLOUD_TOKEN environment variable. Disabling Flipper::Cloud."

spec/flipper/cloud/dsl_spec.rb

@@ -79,4 +79,27 @@
       expect(enable_stub).to have_been_requested
     end
   end
+
+  context "when memoize = :poll" do
+    let(:local_adapter) do
+      Flipper::Adapters::OperationLogger.new Flipper::Adapters::Memory.new
+    end
+
+    let(:cloud_configuration) do
+      cloud_configuration = Flipper::Cloud::Configuration.new({
+        token: "asdf",
+        sync_secret: "tasty",
+        local_adapter: local_adapter,
+        memoize: :poll
+      })
+    end
+
+    subject do
+      described_class.new(cloud_configuration)
+    end
+
+    it "uses a poll adaptor" do
+      expect(subject.adapter).to be_a(Flipper::Adapters::Poll)
+    end
+  end
 end