grape-slack-bot 1.2.3 → 1.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 3472ece85beebd447e192f73013849e3e37a366fdc22b4f4864ef0908f58f502
-  data.tar.gz: 7f25928036698338c354238d9615ccd980d136a0c88056be1a1989304c95eae9
+  metadata.gz: 669dbf492365849f4628ed362f0a047e70d15cb80a62a236260be26d76fbaacc
+  data.tar.gz: e17819f3afa4096b5623c4a8d19f6443522c2010d528ac6a14fefc17fb3c3309
 SHA512:
-  metadata.gz: a2902bfc733cd9aec81df154e0c43ac2bfe9eca7fd6105cb7b6d2f1ea1631daae093e2a96698388accaf35b46c4da826218333bab3c78bed7781c17583e01312
-  data.tar.gz: 04a70f1a3a5251d44d2b4bb21cdaa113296d50c51b0fae010f27f5c1a7caeb5b41eb635c6d349672701a5aef13c58f39249f8e740a2310ab5024dd822e7341ce
+  metadata.gz: 05de3c891473d52722c368bd587e7a90b7d8e9eaa96c4f6bd59bd4bbb5f867c265e2fab1b2e95076abd8b75c0dd9460a51ea460909afd538e9c0c0962fe8250c
+  data.tar.gz: c80d3a362202a526df44c32dad70270109ef344180d7d83d27cca9a24c9f308d422c7f8f91215a0a31b357ef2e86d14771b048f344051a7bc48ca27f80fa73d1

data/CHANGELOG.md

@@ -1,3 +1,7 @@
+# 1.3.0
+
+* Clean up callback arguments, remove unused `method_name`
+
 # 1.2.3
 
 * Minor fix for Events API

data/lib/slack_bot/callback.rb

@@ -14,19 +14,18 @@ module SlackBot
       nil
     end
 
-    def self.create(class_name:, method_name:, user:, channel_id: nil, config: nil)
+    def self.create(class_name:, user:, channel_id: nil, config: nil)
       callback =
-        new(class_name: class_name, method_name: method_name, user: user, channel_id: channel_id, config: config)
+        new(class_name: class_name, user: user, channel_id: channel_id, config: config)
       callback.save
       callback
     end
 
     attr_reader :id, :data, :args, :config
-    def initialize(id: nil, class_name: nil, method_name: nil, user: nil, channel_id: nil, extra: nil, config: nil)
+    def initialize(id: nil, class_name: nil, user: nil, channel_id: nil, extra: nil, config: nil)
       @id = id
       @data = {
         class_name: class_name,
-        method_name: method_name,
         user_id: user&.id,
         channel_id: channel_id,
         extra: extra

data/lib/slack_bot/command.rb

@@ -48,7 +48,7 @@ module SlackBot
 
     private
 
-    def open_modal(view_name, method_name: nil, context: nil)
+    def open_modal(view_name, context: nil)
       view = self.class.view_klass.new(
         args: args,
         current_user: @current_user,
@@ -61,7 +61,6 @@ module SlackBot
         trigger_id: params[:trigger_id],
         channel_id: params[:channel_id],
         class_name: self.class.name,
-        method_name: method_name,
         user: @current_user,
         payload: payload,
         config: config

data/lib/slack_bot/interaction.rb

@@ -12,7 +12,6 @@ module SlackBot
       trigger_id:,
       payload:,
       class_name:,
-      method_name:,
       user:,
       channel_id:,
       config: nil
@@ -20,7 +19,6 @@ module SlackBot
       callback =
         Callback.create(
           class_name: class_name,
-          method_name: method_name,
           user: user,
           channel_id: channel_id,
           config: config
@@ -45,7 +43,6 @@ module SlackBot
       view_id:,
       payload:,
       class_name: nil,
-      method_name: nil,
       user: nil,
       channel_id: nil,
       config: nil
@@ -54,7 +51,6 @@ module SlackBot
       callback ||=
         Callback.create(
           class_name: class_name,
-          method_name: method_name,
           user: user,
           channel_id: channel_id,
           config: config

data/lib/slack_bot.rb

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

data/spec/slack_bot/callback_spec.rb

@@ -4,7 +4,6 @@ describe SlackBot::Callback do
   subject(:callback) {
     described_class.new(
       class_name: "Test",
-      method_name: "test",
       user: user,
       channel_id: "test_channel_id",
       extra: { test: "test" },
@@ -34,7 +33,7 @@ describe SlackBot::Callback do
     end
 
     context "when callback is found" do
-      let(:data) { { class_name: "Test", method_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", extra: { test: "test" }, args: "" } }
 
       it "returns callback" do
         expect(find).to be_a(described_class)
@@ -43,7 +42,6 @@ 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.method_name).to eq("test")
         expect(find.extra).to eq({ test: "test" })
       end
     end
@@ -58,21 +56,20 @@ describe SlackBot::Callback do
   end
 
   describe ".create" do
-    subject(:create) { described_class.create(class_name: "Test", method_name: "test", user: user, channel_id: "test_channel_id", config: config) }
+    subject(:create) { described_class.create(class_name: "Test", user: user, channel_id: "test_channel_id", config: config) }
 
     before do
       allow_any_instance_of(described_class).to receive(:generate_id).and_return("test_callback_id")
       allow(callback_storage_instance).to receive(:write).with("slack-bot-callback:test_callback_id", {
         args: "",
         class_name: "Test",
-        method_name: "test",
         user_id: 1,
         channel_id: "test_channel_id",
         extra: nil
       }, expires_in: 1.hour)
     end
 
-    let(:data) { { class_name: "Test", method_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", extra: nil } }
 
     it "returns callback" do
       expect(create).to be_a(described_class)
@@ -81,7 +78,6 @@ 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.method_name).to eq("test")
       expect(create.extra).to eq(nil)
     end
   end
@@ -94,7 +90,6 @@ describe SlackBot::Callback do
     let(:data) {
       {
         class_name: "Test",
-        method_name: "test",
         user_id: 1,
         channel_id: "test_channel_id",
         extra: { test: "test" },
@@ -113,7 +108,6 @@ 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.method_name).to eq("test")
       expect(reload.extra).to eq({ test: "test" })
     end
 
@@ -131,7 +125,6 @@ describe SlackBot::Callback do
     let(:callback) {
       described_class.new(
         class_name: "Test",
-        method_name: "test",
         user: user,
         channel_id: "test_channel_id",
         extra: { test: "test" },
@@ -144,7 +137,6 @@ describe SlackBot::Callback do
       allow(callback_storage_instance).to receive(:write).with("slack-bot-callback:test_callback_id", {
         args: "",
         class_name: "Test",
-        method_name: "test",
         user_id: 1,
         channel_id: "test_channel_id",
         extra: { test: "test" }
@@ -165,7 +157,6 @@ describe SlackBot::Callback do
       described_class.new(
         id: "test_callback_id",
         class_name: "Test",
-        method_name: "test",
         user: user,
         channel_id: "test_channel_id",
         extra: { test: "test" },
@@ -177,7 +168,6 @@ describe SlackBot::Callback do
       allow(callback_storage_instance).to receive(:write).with("slack-bot-callback:test_callback_id", {
         args: "",
         class_name: "Test",
-        method_name: "test",
         user_id: 1,
         channel_id: "test_channel_id",
         extra: { test: "test" },

metadata

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