run rubocop

.rubocop.yml

+Style/AsciiComments:
+  Enabled: false
\ No newline at end of file

Gemfile

-source "https://rubygems.org"
+# frozen_string_literal: true
 
-git_source(:github) {|repo_name| "https://github.com/#{repo_name}" }
+source 'https://rubygems.org'
+
+git_source(:github) { |repo_name| "https://github.com/#{repo_name}" }
 
 # Specify your gem's dependencies in dingtalk.gemspec
 gemspec

Rakefile

-require "bundler/gem_tasks"
-require "rspec/core/rake_task"
+# frozen_string_literal: true
+
+require 'bundler/gem_tasks'
+require 'rspec/core/rake_task'
+require 'rubocop/rake_task'
 
 RSpec::Core::RakeTask.new(:spec)
 
-task :default => :spec
+task default: :spec
+
+RuboCop::RakeTask.new(:rubocop) do |task|
+  task.options = ['-A']
+end

bin/console

 #!/usr/bin/env ruby
+# frozen_string_literal: true
 
-require "bundler/setup"
-require "dingtalk"
+require 'bundler/setup'
+require 'dingtalk'
 
 # You can add fixtures and/or initialization code here to make experimenting
 # with your gem easier. You can also use a different console, if you like.
@@ -10,5 +11,5 @@ require "dingtalk"
 # require "pry"
 # Pry.start
 
-require "irb"
+require 'irb'
 IRB.start(__FILE__)

dingtalk_sdk.gemspec

+# frozen_string_literal: true
 
-lib = File.expand_path("../lib", __FILE__)
+lib = File.expand_path('lib', __dir__)
 $LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
-require "dingtalk_sdk/version"
+require 'dingtalk_sdk/version'
 
 Gem::Specification.new do |spec|
-  spec.name          = "dingtalk_sdk"
+  spec.name          = 'dingtalk_sdk'
   spec.version       = DingtalkSdk::VERSION
-  spec.authors       = ["francis"]
-  spec.email         = ["francis.tm@gmail.com"]
+  spec.authors       = ['francis']
+  spec.email         = ['francis.tm@gmail.com']
 
-  spec.summary       = %q{Dingtalk integration}
-  spec.description   = %q{Dingtalk ruby SKD}
-  spec.homepage      = "https://github.com/francistm/dingtalk-sdk"
-  spec.license       = "MIT"
+  spec.summary       = 'Dingtalk integration'
+  spec.description   = 'Dingtalk ruby SKD'
+  spec.homepage      = 'https://github.com/francistm/dingtalk-sdk'
+  spec.license       = 'MIT'
 
   # Prevent pushing this gem to RubyGems.org. To allow pushes either set the 'allowed_push_host'
   # to allow pushing to a single host or delete this section to allow pushing to any host.
   if spec.respond_to?(:metadata)
-    spec.metadata["allowed_push_host"] = "https://rubygems.pkg.github.com"
+    spec.metadata['allowed_push_host'] = 'https://rubygems.pkg.github.com'
 
-    spec.metadata["homepage_uri"] = spec.homepage
-    spec.metadata["source_code_uri"] = "https://github.com/francistm/dingtalk-sdk"
+    spec.metadata['homepage_uri'] = spec.homepage
+    spec.metadata['source_code_uri'] = 'https://github.com/francistm/dingtalk-sdk'
     # spec.metadata["changelog_uri"] = "TODO: Put your gem's CHANGELOG.md URL here."
   else
-    raise "RubyGems 2.0 or newer is required to protect against " \
-      "public gem pushes."
+    raise 'RubyGems 2.0 or newer is required to protect against ' \
+      'public gem pushes.'
   end
 
   # Specify which files should be added to the gem when it is released.
   # The `git ls-files -z` loads the files in the RubyGem that have been added into git.
-  spec.files         = Dir.chdir(File.expand_path('..', __FILE__)) do
+  spec.files = Dir.chdir(File.expand_path(__dir__)) do
     `git ls-files -z`.split("\x0").reject { |f| f.match(%r{^(test|spec|features)/}) }
   end
-  spec.bindir        = "exe"
+  spec.bindir        = 'exe'
   spec.executables   = spec.files.grep(%r{^exe/}) { |f| File.basename(f) }
-  spec.require_paths = ["lib"]
-
-  spec.add_runtime_dependency     "httparty", "~> 0.18"
-  spec.add_runtime_dependency     "activesupport", "~> 6.0"
-
-  spec.add_development_dependency "pry", "~> 0.13"
-  spec.add_development_dependency "webmock", "~> 3.8"
-  spec.add_development_dependency "bundler", "~> 2.1.0"
-  spec.add_development_dependency "rake", "~> 10.0"
-  spec.add_development_dependency "rspec", "~> 3.0"
+  spec.require_paths = ['lib']
+  spec.required_ruby_version = '> 2.4'
+
+  spec.add_runtime_dependency     'activesupport', '~> 6.0'
+  spec.add_runtime_dependency     'httparty', '~> 0.18'
+
+  spec.add_development_dependency 'bundler', '~> 2.1.0'
+  spec.add_development_dependency 'pry', '~> 0.13'
+  spec.add_development_dependency 'rake', '~> 10.0'
+  spec.add_development_dependency 'rspec', '~> 3.0'
+  spec.add_development_dependency 'rubocop', '~> 0.80'
+  spec.add_development_dependency 'webmock', '~> 3.8'
 end

lib/dingtalk_sdk.rb

+# frozen_string_literal: true
+
 require 'uri'
 require 'base64'
 require 'openssl'
 
-require "dingtalk_sdk/version"
-require "dingtalk_sdk/auth"
-require "dingtalk_sdk/user"
-require "dingtalk_sdk/department"
-require "dingtalk_sdk/access_token"
-require "dingtalk_sdk/corp_conversation"
+require 'dingtalk_sdk/version'
+require 'dingtalk_sdk/auth'
+require 'dingtalk_sdk/user'
+require 'dingtalk_sdk/department'
+require 'dingtalk_sdk/access_token'
+require 'dingtalk_sdk/corp_conversation'
 
 module DingtalkSdk
   class Error < StandardError; end
@@ -16,7 +18,8 @@ module DingtalkSdk
     attr_reader :code, :message
 
     def initialize(code:, message:)
-      @code, @message, = code, message
+      @code = code
+      @message = message
     end
   end
 
@@ -44,7 +47,9 @@ module DingtalkSdk
     attr_reader :agent_id, :app_key, :app_secret
 
     def initialize(agent_id:, app_key:, app_secret:)
-      @agent_id, @app_key, @app_secret = agent_id, app_key, app_secret
+      @agent_id = agent_id
+      @app_key = app_key
+      @app_secret = app_secret
     end
   end
 
@@ -56,18 +61,18 @@ module DingtalkSdk
     def login_free_signature(app_secret, options = {})
       default_options = {
         timestamp: nil,
-        url_encode: true,
+        url_encode: true
       }
 
       options = default_options.merge(options)
 
-      timestamp = unless options[:timestamp].nil?
-        options[:timestamp]
-      else
-        (Time.now.localtime("+08:00").to_f * 1000).to_i
-      end
+      timestamp = if options[:timestamp].nil?
+                    (Time.now.localtime('+08:00').to_f * 1000).to_i
+                  else
+                    options[:timestamp]
+                  end
 
-      signature_str = OpenSSL::HMAC.digest("SHA256", app_secret, timestamp.to_s)
+      signature_str = OpenSSL::HMAC.digest('SHA256', app_secret, timestamp.to_s)
       signature_str_base64 = Base64.encode64(signature_str).strip
 
       Signature.new(signature_str_base64)

lib/dingtalk_sdk/access_token.rb

-require "dingtalk_sdk/core"
-require "dingtalk_sdk/request_url"
+# frozen_string_literal: true
+
+require 'dingtalk_sdk/core'
+require 'dingtalk_sdk/request_url'
 
 module DingtalkSdk
   module AccessToken
@@ -14,4 +16,4 @@ module DingtalkSdk
       request.add_const :appsecret, ->(r) { r.app_secret }, in: :query
     end
   end
-end
\ No newline at end of file
+end

lib/dingtalk_sdk/auth.rb

-require "dingtalk_sdk/core"
-require "dingtalk_sdk/request_url"
-require "active_support/core_ext/object/to_query"
+# frozen_string_literal: true
+
+require 'dingtalk_sdk/core'
+require 'dingtalk_sdk/request_url'
+require 'active_support/core_ext/object/to_query'
 
 module DingtalkSdk
   module Auth
@@ -17,11 +19,11 @@ module DingtalkSdk
 
     # 生成扫码登录跳转地址
     # {https://ding-doc.dingtalk.com/doc#/serverapi2/kymkv6}
-    def get_qr_connect_uri(redirect_uri, state = "")
+    def get_qr_connect_uri(redirect_uri, state = '')
       qs = {}.tap do |h|
         h[:appid] = @app_key
-        h[:response_type] = "code"
-        h[:scope] = "snsapi_login"
+        h[:response_type] = 'code'
+        h[:scope] = 'snsapi_login'
         h[:state] = state
         h[:redirect_uri] = redirect_uri
       end
@@ -43,4 +45,4 @@ module DingtalkSdk
       request.add_arg :tmp_auth_code, required: true, in: :body
     end
   end
-end
\ No newline at end of file
+end

lib/dingtalk_sdk/core.rb

-require "json"
-require "httparty"
-require "active_support"
-require "active_support/core_ext/hash/keys"
-require "active_support/core_ext/hash/deep_merge"
+# frozen_string_literal: true
 
-require "dingtalk_sdk"
+require 'json'
+require 'httparty'
+require 'active_support'
+require 'active_support/core_ext/hash/keys'
+require 'active_support/core_ext/hash/deep_merge'
+
+require 'dingtalk_sdk'
 
 module DingtalkSdk
   module Core
@@ -22,41 +24,39 @@ module DingtalkSdk
         @query_const = {}
       end
 
-      def is_json?
+      def json?
         @format == :json
       end
 
-      def is_x_form?
+      def x_form?
         @format == :x_form
       end
 
-      def is_form_data?
+      def form_data?
         @format == :form_data
       end
 
-      def is_arg_required?(arg_name)
+      def arg_required?(arg_name)
         @required_args.include? arg_name
       end
 
-      def has_body?
+      def body?
         !@body_args.empty? || !@body_const.empty?
       end
 
-      def has_query?
+      def query?
         !@query_args.empty? || !@query_const.empty?
       end
 
-      def has_body_const?
+      def body_const?
         !@body_const.empty?
       end
 
-      def has_query_const?
+      def query_const?
         !@query_const.empty?
       end
 
-      def format=(format)
-        @format = format
-      end
+      attr_writer :format
 
       def add_arg(arg_name, option)
         case option[:in]
@@ -65,14 +65,14 @@ module DingtalkSdk
         when :query
           @query_args << arg_name
         else
-          raise DingtalkSdk::Error.new("unknown argument '#{arg_name}' position")
+          raise DingtalkSdk::Error, "unknown argument '#{arg_name}' position"
         end
 
         @required_args << arg_name if option[:required]
       end
 
       def add_const(arg_name, value, option)
-        const_hash = {arg_name => value}
+        const_hash = { arg_name => value }
 
         case option[:in]
         when :body
@@ -80,7 +80,7 @@ module DingtalkSdk
         when :query
           @query_const.merge! const_hash
         else
-          raise DingtalkSdk::Error.new("unknown argument '#{arg_name}' position")
+          raise DingtalkSdk::Error, "unknown argument '#{arg_name}' position"
         end
       end
     end
@@ -92,33 +92,34 @@ module DingtalkSdk
       define_method request_name do |method_args = {}|
         request_options = {}.tap do |h|
           builder.required_args.each do |arg|
-            raise DingtalkSdk::Error.new("missing required argument '#{arg}' when invoke request #{request_name}") \
+            raise DingtalkSdk::Error, "missing required argument '#{arg}' when invoke request #{request_name}" \
               if method_args[arg].nil?
           end
 
-          [:body, :query].each do |arg_pos|
-            h[arg_pos] = {} if builder.send(:"has_#{arg_pos}?")
+          %i[body query].each do |arg_pos|
+            h[arg_pos] = {} if builder.send(:"#{arg_pos}?")
 
             builder.send(:"#{arg_pos}_args").each do |arg_name|
               arg_value = method_args[arg_name]
               next if arg_pos == :query && arg_value.nil?
+
               h[arg_pos][arg_name] = arg_value
             end
 
             builder.send(:"#{arg_pos}_const").each do |arg_name, arg_value|
-              if arg_value.respond_to?(:call)
-                h[arg_pos][arg_name] = arg_value.call(self)
-              else
-                h[arg_pos][arg_name] = arg_value
-              end
+              h[arg_pos][arg_name] = if arg_value.respond_to?(:call)
+                                       arg_value.call(self)
+                                     else
+                                       arg_value
+                                     end
             end
           end
 
-          if builder.is_form_data?
+          if builder.form_data?
             h[:multipart] = true
-          elsif builder.is_json?
-            h[:body] = h[:body].to_json if Hash === h[:body]
-            h[:headers] = {:"Content-Type" => "application/json"}
+          elsif builder.json?
+            h[:body] = h[:body].to_json if h[:body].is_a?(Hash)
+            h[:headers] = { "Content-Type": 'application/json' }
           end
         end
 
@@ -128,7 +129,7 @@ module DingtalkSdk
         if symbolized_response[:errcode] != 0
           raise DingtalkSdk::RequestFailedError.new(
             code: symbolized_response[:errcode],
-            message: symbolized_response[:errmsg],
+            message: symbolized_response[:errmsg]
           )
         end
 
@@ -136,4 +137,4 @@ module DingtalkSdk
       end
     end
   end
-end
\ No newline at end of file
+end

lib/dingtalk_sdk/corp_conversation.rb

+# frozen_string_literal: true
+
 require 'dingtalk_sdk/core'
 require 'dingtalk_sdk/request_url'
 
@@ -56,4 +58,4 @@ module DingtalkSdk
       request.add_const :agent_id, ->(r) { r.agent_id }, in: :body
     end
   end
-end
\ No newline at end of file
+end

lib/dingtalk_sdk/department.rb

-require "dingtalk_sdk/core"
-require "dingtalk_sdk/request_url"
+# frozen_string_literal: true
+
+require 'dingtalk_sdk/core'
+require 'dingtalk_sdk/request_url'
 
 module DingtalkSdk
   module Department
@@ -24,4 +26,4 @@ module DingtalkSdk
       request.add_arg :access_token, in: :query, required: true
     end
   end
-end
\ No newline at end of file
+end

lib/dingtalk_sdk/request_url.rb

@@ -3,58 +3,58 @@
 module DingtalkSdk
   module RequestUrl
     # 获取 AccessToken
-    ACCESS_TOKEN = "https://oapi.dingtalk.com/gettoken"
+    ACCESS_TOKEN = 'https://oapi.dingtalk.com/gettoken'
 
     # 扫码登录第三方网站 OAuth 跳转地址
     # {https://ding-doc.dingtalk.com/doc#/serverapi2/kymkv6}
-    CONNECT_QR_REDIRECT = "https://oapi.dingtalk.com/connect/qrconnect"
+    CONNECT_QR_REDIRECT = 'https://oapi.dingtalk.com/connect/qrconnect'
 
     # 企业内应用免登录获取用户ID
     # {https://ding-doc.dingtalk.com/doc#/serverapi2/clotub}
-    INT_LOGIN_FREE_GET_USER_INFO = "https://oapi.dingtalk.com/user/getuserinfo"
+    INT_LOGIN_FREE_GET_USER_INFO = 'https://oapi.dingtalk.com/user/getuserinfo'
 
     # 服务端通过临时授权码获取授权用户的个人信息
     # {https://ding-doc.dingtalk.com/doc#/serverapi2/etaarr}
-    GET_USER_INFO_SNS = "https://oapi.dingtalk.com/sns/getuserinfo_bycode"
+    GET_USER_INFO_SNS = 'https://oapi.dingtalk.com/sns/getuserinfo_bycode'
 
     # 获取用户详情
     # {https://ding-doc.dingtalk.com/doc#/serverapi2/ege851/AaRQe}
-    GET_USER_PROFILE = "https://oapi.dingtalk.com/user/get"
+    GET_USER_PROFILE = 'https://oapi.dingtalk.com/user/get'
 
     # 通过 unionId 获取 userId
     # {https://ding-doc.dingtalk.com/doc#/serverapi2/ege851/602f4b15}
-    GET_USERID_FROM_UNIONID = "https://oapi.dingtalk.com/user/getUseridByUnionid"
+    GET_USERID_FROM_UNIONID = 'https://oapi.dingtalk.com/user/getUseridByUnionid'
 
     # 根据手机号获取 userId
     # {https://ding-doc.dingtalk.com/doc#/serverapi2/ege851/soV11}
-    GET_USERID_FROM_MOBILE = "https://oapi.dingtalk.com/user/get_by_mobile"
+    GET_USERID_FROM_MOBILE = 'https://oapi.dingtalk.com/user/get_by_mobile'
 
     # 获取部门列表
     # {https://ding-doc.dingtalk.com/doc#/serverapi2/dubakq/e6e1604e}
-    GET_DEPARTMENT_LIST = "https://oapi.dingtalk.com/department/list"
+    GET_DEPARTMENT_LIST = 'https://oapi.dingtalk.com/department/list'
 
     # 获取部门详情
     # {https://ding-doc.dingtalk.com/doc#/serverapi2/dubakq/5bf960de}
-    GET_DEPARTMENT_PROFILE = "https://oapi.dingtalk.com/department/get"
+    GET_DEPARTMENT_PROFILE = 'https://oapi.dingtalk.com/department/get'
 
     # 发送工作通知消息
     # {https://ding-doc.dingtalk.com/doc#/serverapi2/pgoxpy}
-    SEND_CORP_CONVERSATION = "https://oapi.dingtalk.com/topapi/message/corpconversation/asyncsend_v2"
+    SEND_CORP_CONVERSATION = 'https://oapi.dingtalk.com/topapi/message/corpconversation/asyncsend_v2'
 
     # 查询工作通知消息的发送进度
     # {https://ding-doc.dingtalk.com/doc#/serverapi2/pgoxpy/e2262dad}
-    CORP_CONVERSATION_SEND_PROGRESS = "https://oapi.dingtalk.com/topapi/message/corpconversation/getsendprogress"
+    CORP_CONVERSATION_SEND_PROGRESS = 'https://oapi.dingtalk.com/topapi/message/corpconversation/getsendprogress'
 
     # 查询工作通知消息的发送结果
     # {https://ding-doc.dingtalk.com/doc#/serverapi2/pgoxpy/a5920210}
-    CORP_CONVERSATION_SEND_RESULT = "https://oapi.dingtalk.com/topapi/message/corpconversation/getsendresult"
+    CORP_CONVERSATION_SEND_RESULT = 'https://oapi.dingtalk.com/topapi/message/corpconversation/getsendresult'
 
     # 工作通知消息撤回
     # {https://ding-doc.dingtalk.com/doc#/serverapi2/pgoxpy/hYyV8}
-    RECALL_CORP_CONVERSATION = "https://oapi.dingtalk.com/topapi/message/corpconversation/recall"
+    RECALL_CORP_CONVERSATION = 'https://oapi.dingtalk.com/topapi/message/corpconversation/recall'
 
     # 上传媒体文件
     # {https://ding-doc.dingtalk.com/doc#/serverapi2/bcmg0i/08d5a73b}
-    UPLOAD_MEDIA_FILE = "https://oapi.dingtalk.com/media/upload"
+    UPLOAD_MEDIA_FILE = 'https://oapi.dingtalk.com/media/upload'
   end
-end
\ No newline at end of file
+end

lib/dingtalk_sdk/storage.rb

+# frozen_string_literal: true
+
 require 'dingtalk_sdk/core'
 require 'dingtalk_sdk/request_url'
 
@@ -15,4 +17,4 @@ module DingtalkSdk
       request.add_arg :media, in: :body, required: true
     end
   end
-end
\ No newline at end of file
+end

lib/dingtalk_sdk/user.rb

-require "dingtalk_sdk/core"
-require "dingtalk_sdk/request_url"
+# frozen_string_literal: true
+
+require 'dingtalk_sdk/core'
+require 'dingtalk_sdk/request_url'
 
 module DingtalkSdk
   module User
@@ -18,7 +20,7 @@ module DingtalkSdk
     # {https://ding-doc.dingtalk.com/doc#/serverapi2/ege851/602f4b15}
     # @!method get_userid_by_unionid(unionid:, access_token:)
     # @return [Hash]
-    add_request :get_userid_by_unionid, :get, DingtalkSdk::RequestUrl::GET_USERID_FROM_UNIONID  do |request|
+    add_request :get_userid_by_unionid, :get, DingtalkSdk::RequestUrl::GET_USERID_FROM_UNIONID do |request|
       request.add_arg :unionid, required: true, in: :query
       request.add_arg :access_token, required: true, in: :query
     end
@@ -32,4 +34,4 @@ module DingtalkSdk
       request.add_arg :access_token, required: true, in: :query
     end
   end
-end
\ No newline at end of file
+end

lib/dingtalk_sdk/version.rb

+# frozen_string_literal: true
+
 module DingtalkSdk
-  VERSION = "0.1.0"
+  VERSION = '0.1.0'
 end

spec/dingtalk_sdk/access_token_spec.rb

-require "json"
-require "dingtalk_sdk"
-require "dingtalk_sdk/access_token"
+# frozen_string_literal: true
+
+require 'json'
+require 'dingtalk_sdk'
+require 'dingtalk_sdk/access_token'
 
 RSpec.describe DingtalkSdk::AccessToken do
   before :each do
-    @agent_id = "mocked_agent_id"
-    @app_key = "mocked_app_key"
-    @app_secret = "mocked_app_secret"
-    @mocked_access_token = "mocked_access_token"
+    @agent_id = 'mocked_agent_id'
+    @app_key = 'mocked_app_key'
+    @app_secret = 'mocked_app_secret'
+    @mocked_access_token = 'mocked_access_token'
 
     @dingtalk_request = DingtalkSdk::Request.new(
       agent_id: @agent_id,
       app_key: @app_key,
-      app_secret: @app_secret,
+      app_secret: @app_secret
     )
 
     response_body = {}.tap do |h|
@@ -23,12 +25,12 @@ RSpec.describe DingtalkSdk::AccessToken do
 
     stub_request(:get, /oapi.dingtalk.com/)
       .with(query: { appkey: @app_key, appsecret: @app_secret })
-      .to_return(status: 200, body: response_body.to_json )
+      .to_return(status: 200, body: response_body.to_json)
   end
 
-  it "should return correct access_token from request" do
+  it 'should return correct access_token from request' do
     response = @dingtalk_request.get_access_token
 
     expect(response[:access_token]).to eq(@mocked_access_token)
   end
-end
\ No newline at end of file
+end

spec/dingtalk_sdk/auth_spec.rb

-require "cgi"
-require "uri"
-require "json"
-require "securerandom"
+# frozen_string_literal: true
 
-require "dingtalk_sdk"
-require "dingtalk_sdk/auth"
+require 'cgi'
+require 'uri'
+require 'json'
+require 'securerandom'
+
+require 'dingtalk_sdk'
+require 'dingtalk_sdk/auth'
 
 RSpec.describe DingtalkSdk::Auth do
   before :each do
-    @agent_id = "mocked_agent_id"
-    @app_key = "mocked_app_key"
-    @app_secret = "mocked_app_secret"
+    @agent_id = 'mocked_agent_id'
+    @app_key = 'mocked_app_key'
+    @app_secret = 'mocked_app_secret'
 
     @int_user_id = 100
-    @access_token = "mocked_access_token"
-    @int_login_code = "mocked_int_login_code"
-    @timestamp = (Time.now.localtime("+08:00").to_f * 1000).to_i
+    @access_token = 'mocked_access_token'
+    @int_login_code = 'mocked_int_login_code'
+    @timestamp = (Time.now.localtime('+08:00').to_f * 1000).to_i
 
     @signature = DingtalkSdk.login_free_signature(@app_secret, timestamp: @timestamp)
 
     @dingtalk_request = DingtalkSdk::Request.new(
       agent_id: @agent_id,
       app_key: @app_key,
-      app_secret: @app_secret,
+      app_secret: @app_secret
     )
 
     # 企业内部免登 响应结果
     get_user_info_response_body = {}.tap do |h|
       h[:errcode] = 0
-      h[:errmsg] = "ok"
+      h[:errmsg] = 'ok'
       h[:userid] = @int_user_id
       h[:is_sys] = false
       h[:sys_level] = 0
@@ -37,31 +39,32 @@ RSpec.describe DingtalkSdk::Auth do
     # 钉钉内免登第三方网站 响应结果
     get_user_info_by_code_response_body = {}.tap do |h|
       h[:errcode] = 0
-      h[:errmsg] = "ok"
+      h[:errmsg] = 'ok'
       h[:user_info] = {
-        nick: "张三",
-        openid: "liSii8KCxxxxx",
-        unionid: "7Huu46kk"
+        nick: '张三',
+        openid: 'liSii8KCxxxxx',
+        unionid: '7Huu46kk'
       }
     end
 
-    stub_request(:get, /oapi.dingtalk.com\/user\/getuserinfo/)
+    stub_request(:get, %r{oapi.dingtalk.com/user/getuserinfo})
       .with(
         query: {
           code: @int_login_code,
-          access_token: @access_token,
-        })
+          access_token: @access_token
+        }
+      )
       .to_return(status: 200, body: get_user_info_response_body.to_json)
 
-    stub_request(:post, /oapi.dingtalk.com\/sns\/getuserinfo_bycode/)
+    stub_request(:post, %r{oapi.dingtalk.com/sns/getuserinfo_bycode})
       .with(
         query: {
           accessKey: @app_key,
           timestamp: @timestamp.to_s,
-          signature: @signature.to_s,
+          signature: @signature.to_s
         },
-        body: { tmp_auth_code: "23152698ea18304da4d0ce1xxxxx" }.to_json,
-        headers: { :"Content-Type" => "application/json" }
+        body: { tmp_auth_code: '23152698ea18304da4d0ce1xxxxx' }.to_json,
+        headers: { "Content-Type": 'application/json' }
       )
       .to_return(status: 200, body: get_user_info_by_code_response_body.to_json)
   end
@@ -69,7 +72,7 @@ RSpec.describe DingtalkSdk::Auth do
   it 'should get user id in internal app login-free scenario' do
     response = @dingtalk_request.get_int_login_free_user_id(
       code: @int_login_code,
-      access_token: @access_token,
+      access_token: @access_token
     )
 
     expect(response[:userid]).to eq(@int_user_id)
@@ -83,25 +86,25 @@ RSpec.describe DingtalkSdk::Auth do
       accessKey: @app_key,
       timestamp: @timestamp,
       signature: @signature.to_s,
-      tmp_auth_code: "23152698ea18304da4d0ce1xxxxx"
+      tmp_auth_code: '23152698ea18304da4d0ce1xxxxx'
     )
 
     expect(response[:user_info]).to be_a(Hash)
-    expect(response[:user_info][:nick]).to eq("张三")
-    expect(response[:user_info][:openid]).to eq("liSii8KCxxxxx")
-    expect(response[:user_info][:unionid]).to eq("7Huu46kk")
+    expect(response[:user_info][:nick]).to eq('张三')
+    expect(response[:user_info][:openid]).to eq('liSii8KCxxxxx')
+    expect(response[:user_info][:unionid]).to eq('7Huu46kk')
   end
 
   it 'should create connect qr scan page url in dingtalk.com' do
     state = SecureRandom.hex(6)
-    redirect_uri = "http://example.org/callback"
+    redirect_uri = 'http://example.org/callback'
     qr_connect_uri = @dingtalk_request.get_qr_connect_uri(redirect_uri, state)
 
     uri = URI(qr_connect_uri)
-    query = CGI::parse(uri.query)
+    query = CGI.parse(uri.query)
 
-    expect(query["appid"].first).to eq(@app_key)
-    expect(query["state"].first).to eq(state)
-    expect(query["redirect_uri"].first).to eq(redirect_uri)
+    expect(query['appid'].first).to eq(@app_key)
+    expect(query['state'].first).to eq(state)
+    expect(query['redirect_uri'].first).to eq(redirect_uri)
   end
-end
\ No newline at end of file
+end

spec/dingtalk_sdk_spec.rb

+# frozen_string_literal: true
+
 RSpec.describe DingtalkSdk do
-  it "has a version number" do
+  it 'has a version number' do
     expect(DingtalkSdk::VERSION).not_to be nil
   end
 
   # 测试样本见 https://ding-doc.dingtalk.com/doc#/faquestions/hxs5v9 底部
-  it "sign correct timestamp with app_secret in personal login-free scenario" do
-    timestamp = 1546084445901
-    app_secret = "testappSecret"
+  it 'sign correct timestamp with app_secret in personal login-free scenario' do
+    timestamp = 1_546_084_445_901
+    app_secret = 'testappSecret'
 
     signature = DingtalkSdk.login_free_signature(app_secret, timestamp: timestamp)
 
     expect(signature).to be_a(DingtalkSdk::Signature)
-    expect(signature.url_encoded).to eq("HCbG3xNE3vzhO%2Bu7qCUL1jS5hsu2n5r2cFhnTrtyDAE%3D")
+    expect(signature.url_encoded).to eq('HCbG3xNE3vzhO%2Bu7qCUL1jS5hsu2n5r2cFhnTrtyDAE%3D')
   end
 end

spec/spec_helper.rb

-require "pry"
-require "bundler/setup"
-require "webmock/rspec"
-require "dingtalk_sdk"
+# frozen_string_literal: true
+
+require 'pry'
+require 'bundler/setup'
+require 'webmock/rspec'
+require 'dingtalk_sdk'
 
 RSpec.configure do |config|
   # Enable flags like --only-failures and --next-failure
-  config.example_status_persistence_file_path = ".rspec_status"
+  config.example_status_persistence_file_path = '.rspec_status'
 
   # Disable RSpec exposing methods globally on `Module` and `main`
   config.disable_monkey_patching!