grape-slack-bot 1.3.0 → 1.5.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 669dbf492365849f4628ed362f0a047e70d15cb80a62a236260be26d76fbaacc
-  data.tar.gz: e17819f3afa4096b5623c4a8d19f6443522c2010d528ac6a14fefc17fb3c3309
+  metadata.gz: 0f8fc51cbef0f57cb132347ce71c02fba8ca5130749cca04ed3ffdbefb482ce1
+  data.tar.gz: a1a96768a308005a8bf8aa709c3c3a29f927c31a624c4ad9bfc6d7e60b9b01ad
 SHA512:
-  metadata.gz: 05de3c891473d52722c368bd587e7a90b7d8e9eaa96c4f6bd59bd4bbb5f867c265e2fab1b2e95076abd8b75c0dd9460a51ea460909afd538e9c0c0962fe8250c
-  data.tar.gz: c80d3a362202a526df44c32dad70270109ef344180d7d83d27cca9a24c9f308d422c7f8f91215a0a31b357ef2e86d14771b048f344051a7bc48ca27f80fa73d1
+  metadata.gz: 2d372b748e6242a8766799b762afbfb205023491cb5c1fb6f453cc2454d389d89eecfb0e3f7257ef92b545b3215e579103c325dcc5ba0a40bb5a48f6ab4451d3
+  data.tar.gz: 2c72725ce9911a95c74ae015e6f24aa77eae793afd97b7c2d5c934598c3c9195c61bd386bdd40661b2ce3d89bd875e426ed926c3ec711f709090ed2e591ffaf8

data/CHANGELOG.md

@@ -1,3 +1,11 @@
+# 1.5.0
+
+* Complete upgrade of callback storage logic
+
+# 1.4.0
+
+* Allow setting callback expiration time on save and update
+
 # 1.3.0
 
 * Clean up callback arguments, remove unused `method_name`

data/lib/slack_bot/callback.rb

@@ -4,6 +4,7 @@ require 'active_support/core_ext/numeric/time'
 module SlackBot
   class Callback
     CALLBACK_CACHE_KEY = "slack-bot-callback".freeze
+    CALLBACK_RECORD_EXPIRES_IN = 15.minutes.freeze
 
     def self.find(id, config: nil)
       return if id.blank?
@@ -14,31 +15,33 @@ module SlackBot
       nil
     end
 
-    def self.create(class_name:, user:, channel_id: nil, config: nil)
+    def self.create(class_name:, user:, channel_id: nil, config: nil, payload: nil, expires_in: nil)
       callback =
-        new(class_name: class_name, user: user, channel_id: channel_id, config: config)
+        new(class_name: class_name, user: user, channel_id: channel_id, payload: payload, config: config, expires_in: expires_in)
       callback.save
       callback
     end
 
-    attr_reader :id, :data, :args, :config
-    def initialize(id: nil, class_name: nil, user: nil, channel_id: nil, extra: nil, config: nil)
+    attr_reader :id, :data, :args, :config, :expires_in
+    def initialize(id: nil, class_name: nil, user: nil, channel_id: nil, payload: nil, config: nil, expires_in: nil)
       @id = id
       @data = {
         class_name: class_name,
         user_id: user&.id,
         channel_id: channel_id,
-        extra: extra
+        payload: payload
       }
       @args = SlackBot::Args.new
       @config = config || SlackBot::Config.current_instance
+      @expires_in = expires_in || CALLBACK_RECORD_EXPIRES_IN
     end
 
     def reload
-      @data = read_data
-      SlackBot::DevConsole.log_check("SlackBot::Callback#read_data: #{id} | #{data}")
-      raise SlackBot::Errors::CallbackNotFound if data.nil?
+      cached_data = read_data
+      SlackBot::DevConsole.log_check("SlackBot::Callback#read_data: #{id} | #{cached_data}")
+      raise SlackBot::Errors::CallbackNotFound if cached_data.nil?
 
+      @data = cached_data
       parse_args
       self
     end
@@ -55,7 +58,12 @@ module SlackBot
       return if id.blank?
       return if data.blank?
 
-      @data = data.merge(payload)
+      if @data[:payload].is_a?(Hash)
+        @data[:payload] = @data[:payload].merge(payload)
+      else
+        @data[:payload] = payload
+      end
+
       save
     end
 
@@ -72,6 +80,23 @@ module SlackBot
       end
     end
 
+    def user=(user)
+      @user = user
+      @data[:user_id] = user&.id
+    end
+
+    def class_name=(class_name)
+      @data[:class_name] = class_name
+    end
+
+    def channel_id=(channel_id)
+      @data[:channel_id] = channel_id
+    end
+
+    def payload=(payload)
+      @data[:payload] = payload
+    end
+
     def handler_class
       return if class_name.blank?
 
@@ -80,6 +105,7 @@ module SlackBot
 
     def method_missing(method_name, *args, &block)
       return data[method_name.to_sym] if data.key?(method_name.to_sym)
+      return data[:payload][method_name.to_s] if data[:payload].is_a?(Hash) && data[:payload].key?(method_name.to_s)
 
       super
     end
@@ -91,7 +117,7 @@ module SlackBot
     end
 
     def serialize_args
-      data[:args] = args.to_s
+      @data[:args] = args.to_s
     end
 
     def generate_id
@@ -102,8 +128,9 @@ module SlackBot
       config.callback_storage_instance.read("#{CALLBACK_CACHE_KEY}:#{id}")
     end
 
-    def write_data(data)
-      config.callback_storage_instance.write("#{CALLBACK_CACHE_KEY}:#{id}", data, expires_in: 1.hour)
+    def write_data(data, expires_in: nil)
+      expires_in ||= CALLBACK_RECORD_EXPIRES_IN
+      config.callback_storage_instance.write("#{CALLBACK_CACHE_KEY}:#{id}", data, expires_in: expires_in)
     end
 
     def delete_data

data/lib/slack_bot/grape_extension.rb

@@ -88,7 +88,7 @@ module SlackBot
           callback = SlackBot::Callback.find(callback_id, config: config)
           raise SlackBot::Errors::CallbackNotFound.new if callback.blank?
 
-          SlackBot::DevConsole.log_check "SlackApi::Interactions##{__method__}: #{callback.id} #{callback.extra} #{callback.user_id} #{user&.id}"
+          SlackBot::DevConsole.log_check "SlackApi::Interactions##{__method__}: #{callback.id} #{callback.payload} #{callback.user_id} #{user&.id}"
 
           if callback.user_id != user.id
             raise "Callback user is not equal to action user"

data/lib/slack_bot.rb

@@ -22,5 +22,5 @@ require 'slack_bot/pager'
 require 'slack_bot/grape_extension'
 
 module SlackBot
-  VERSION = '1.3.0'.freeze
+  VERSION = '1.5.0'.freeze
 end

data/spec/slack_bot/callback_spec.rb

@@ -6,7 +6,7 @@ describe SlackBot::Callback do
       class_name: "Test",
       user: user,
       channel_id: "test_channel_id",
-      extra: { test: "test" },
+      payload: { test: "test" },
       config: config
     )
   }
@@ -33,7 +33,7 @@ describe SlackBot::Callback do
     end
 
     context "when callback is found" do
-      let(:data) { { class_name: "Test", user_id: 1, channel_id: "test_channel_id", extra: { test: "test" }, args: "" } }
+      let(:data) { { class_name: "Test", user_id: 1, channel_id: "test_channel_id", payload: { test: "test" }, args: "" } }
 
       it "returns callback" do
         expect(find).to be_a(described_class)
@@ -42,7 +42,7 @@ describe SlackBot::Callback do
         expect(find.user).to eq(user)
         expect(find.user_id).to eq(1)
         expect(find.channel_id).to eq("test_channel_id")
-        expect(find.extra).to eq({ test: "test" })
+        expect(find.payload).to eq({ test: "test" })
       end
     end
 
@@ -65,11 +65,11 @@ describe SlackBot::Callback do
         class_name: "Test",
         user_id: 1,
         channel_id: "test_channel_id",
-        extra: nil
-      }, expires_in: 1.hour)
+        payload: nil
+      }, expires_in: SlackBot::Callback::CALLBACK_RECORD_EXPIRES_IN)
     end
 
-    let(:data) { { class_name: "Test", user_id: 1, channel_id: "test_channel_id", extra: nil } }
+    let(:data) { { class_name: "Test", user_id: 1, channel_id: "test_channel_id", payload: nil } }
 
     it "returns callback" do
       expect(create).to be_a(described_class)
@@ -78,7 +78,7 @@ describe SlackBot::Callback do
       expect(create.user).to eq(user)
       expect(create.user_id).to eq(1)
       expect(create.channel_id).to eq("test_channel_id")
-      expect(create.extra).to eq(nil)
+      expect(create.payload).to eq(nil)
     end
   end
 
@@ -92,7 +92,7 @@ describe SlackBot::Callback do
         class_name: "Test",
         user_id: 1,
         channel_id: "test_channel_id",
-        extra: { test: "test" },
+        payload: { test: "test payload" },
         args: ""
       }
     }
@@ -108,7 +108,7 @@ describe SlackBot::Callback do
       expect(reload.user).to eq(user)
       expect(reload.user_id).to eq(1)
       expect(reload.channel_id).to eq("test_channel_id")
-      expect(reload.extra).to eq({ test: "test" })
+      expect(reload.payload).to eq({ test: "test payload" })
     end
 
     context "when callback is not found" do
@@ -127,7 +127,7 @@ describe SlackBot::Callback do
         class_name: "Test",
         user: user,
         channel_id: "test_channel_id",
-        extra: { test: "test" },
+        payload: { test: "test payload" },
         config: config
       )
     }
@@ -139,12 +139,12 @@ describe SlackBot::Callback do
         class_name: "Test",
         user_id: 1,
         channel_id: "test_channel_id",
-        extra: { test: "test" }
-      }, expires_in: 1.hour)
+        payload: { test: "test payload" }
+      }, expires_in: SlackBot::Callback::CALLBACK_RECORD_EXPIRES_IN)
     end
 
     it "returns callback" do
-      expect { save }.not_to raise_error
+      save
       expect(callback).to be_a(described_class)
       expect(callback.id).to eq("test_callback_id")
       expect(callback.args).to be_a(SlackBot::Args)
@@ -159,7 +159,7 @@ describe SlackBot::Callback do
         class_name: "Test",
         user: user,
         channel_id: "test_channel_id",
-        extra: { test: "test" },
+        payload: { test: "test payload" },
         config: config
       )
     }
@@ -170,15 +170,14 @@ describe SlackBot::Callback do
         class_name: "Test",
         user_id: 1,
         channel_id: "test_channel_id",
-        extra: { test: "test" },
-        test: "test"
-      }, expires_in: 1.hour)
+        payload: { test: "test payload 2" }
+      }, expires_in: SlackBot::Callback::CALLBACK_RECORD_EXPIRES_IN)
     end
 
-    let(:payload) { { test: "test" } }
+    let(:payload) { { test: "test payload 2" } }
 
     it "returns callback" do
-      expect { update }.not_to raise_error
+      update
       expect(callback).to be_a(described_class)
       expect(callback.id).to eq("test_callback_id")
       expect(callback.args).to be_a(SlackBot::Args)

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: grape-slack-bot
 version: !ruby/object:Gem::Version
-  version: 1.3.0
+  version: 1.5.0
 platform: ruby
 authors:
 - Andrei Makarov