fix at_mobiles generated hash

lib/dingtalk_sdk/robot.rb

 # frozen_string_literal: true
 
-require 'json'
 require 'base64'
 require 'openssl'
 require 'dingtalk_sdk'
@@ -117,9 +116,9 @@ module DingtalkSdk
         }
       end
 
-      def at_mobile(mobile)
+      def at_mobiles(mobiles)
         @is_at_all = false
-        @at_mobile_list = [*mobile].map(&:to_s).uniq
+        @at_mobile_list = [*mobiles].map(&:to_s).uniq
       end
 
       def at_all
@@ -132,16 +131,12 @@ module DingtalkSdk
           h.merge! @mesg
 
           if @is_at_all
-            h[:isAtAll] = true
+            h[:at] = { isAtAll: true }
           elsif @at_mobile_list.try(:size).try(:positive?)
-            h[:atMobiles] = @at_mobile_list
+            h[:at] = { isAtAll: false, atMobiles: @at_mobile_list }
           end
         end
       end
-
-      def to_json(*_args)
-        JSON.generate to_h
-      end
     end
   end
 end

spec/dingtalk_sdk/robot_spec.rb

@@ -38,18 +38,17 @@ RSpec.describe DingtalkSdk::Robot do
 
       expect(mesg_h[:msgtype]).to eq('text')
       expect(mesg_h[:text][:content]).to eq('hello world')
-      expect(mesg_h[:isAtAll]).to be_nil
-      expect(mesg_h[:atMobiles]).to be_nil
+      expect(mesg_h[:at]).to be_nil
     end
 
     it 'should compose message with at mobiles' do
       builder = DingtalkSdk::Robot::MessageBuilder.new
       builder.text text: 'hello world'
-      builder.at_mobile [123, 456]
+      builder.at_mobiles [123, 456]
       mesg_h = builder.to_h
 
-      expect(mesg_h[:isAtAll]).to be_nil
-      expect(mesg_h[:atMobiles]).to eq(%w[123 456])
+      expect(mesg_h[:at][:isAtAll]).to be_nil
+      expect(mesg_h[:at][:atMobiles]).to eq(%w[123 456])
     end
   end
 end