Merge pull request #25 from splaplapla/supress-internal-communication

internal apiとの通信をdefault 無効にする

lib/procon_bypass_man/bypass/usb_hid_logger.rb

@@ -28,10 +28,12 @@ def log_procon_to_gadget
         end
       end
 
-      ProconBypassMan.cache.fetch key: 'pressed_buttons_reporter', expires_in: 5 do
-        ProconBypassMan::ReportPressedButtonsJob.perform_async(
-          bypass_value.binary.to_procon_reader.to_hash
-        )
+      if ProconBypassMan.config.enable_reporting_pressed_buttons
+        ProconBypassMan.cache.fetch key: 'pressed_buttons_reporter', expires_in: 5 do
+          ProconBypassMan::ReportPressedButtonsJob.perform_async(
+            bypass_value.binary.to_procon_reader.to_hash
+          )
+        end
       end
     end
   end

lib/procon_bypass_man/configuration.rb

@@ -41,7 +41,7 @@ def device_id
   end
 
   attr_accessor :enable_critical_error_logging
-  attr_writer :verbose_bypass_log, :raw_setting
+  attr_writer :verbose_bypass_log, :raw_setting, :enable_reporting_pressed_buttons
 
   def root=(path)
     @root = path
@@ -154,4 +154,8 @@ def verbose_bypass_log
   def raw_setting
     @raw_setting ||= {}
   end
+
+  def enable_reporting_pressed_buttons
+    @enable_reporting_pressed_buttons ||= false
+  end
 end

spec/lib/procon_bypass_man/configuration_spec.rb

@@ -58,4 +58,16 @@
       end
     end
   end
+
+  describe '#enable_reporting_pressed_buttons' do
+    let(:config) { described_class.new }
+    it 'default false' do
+      expect(config.enable_reporting_pressed_buttons).to eq(false)
+    end
+
+    it do
+      config.enable_reporting_pressed_buttons = true
+      expect(config.enable_reporting_pressed_buttons).to eq(true)
+    end
+  end
 end

spec/lib/procon_bypass_man_spec.rb

@@ -45,6 +45,8 @@
       :digest_path,
       :raw_setting,
       :raw_setting=,
+      :enable_reporting_pressed_buttons,
+      :enable_reporting_pressed_buttons=,
     ].each do |me|
       it "has config.#{me} method" do
         expect(described_class.config.respond_to?(me)).to eq(true)