Fix rubocop

Author: Unai Abrisketa

bin/console

@@ -1,5 +1,5 @@
-# frozen_string_literal: true
 #!/usr/bin/env ruby
+# frozen_string_literal: true
 
 require "bundler/setup"
 require "dotenv/load"

lib/beyond_api/connection.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require 'faraday'
+require "faraday"
 
 module BeyondApi
   class Connection
@@ -13,8 +13,8 @@ def self.default
         faraday.options[:timeout] = BeyondApi.configuration.timeout.to_i
         faraday.response :logger, LOGGER, { headers: BeyondApi.configuration.log_headers,
                                             bodies: BeyondApi.configuration.log_bodies }
-        faraday.headers['Accept'] = 'application/json'
-        faraday.headers['Content-Type'] = 'application/json'
+        faraday.headers["Accept"] = "application/json"
+        faraday.headers["Content-Type"] = "application/json"
         faraday.request(:multipart)
         faraday.request(:url_encoded)
         faraday.adapter(:net_http)
@@ -27,7 +27,7 @@ def self.token
         faraday.options[:timeout] = BeyondApi.configuration.timeout.to_i
         faraday.response :logger, LOGGER, { headers: BeyondApi.configuration.log_headers,
                                             bodies: BeyondApi.configuration.log_bodies }
-        faraday.headers['Accept'] = 'application/json'
+        faraday.headers["Accept"] = "application/json"
         faraday.adapter(:net_http)
         faraday.basic_auth(BeyondApi.configuration.client_id,
                            BeyondApi.configuration.client_secret)

lib/beyond_api/ext.rb

@@ -53,8 +53,8 @@ class Array
   def camelize_keys
     map do |elem|
       case elem
-        when Hash, Array; elem.camelize_keys
-        else; elem
+      when Hash, Array then elem.camelize_keys
+      else; elem
       end
     end
   end

lib/beyond_api/request.rb

@@ -9,7 +9,7 @@ class << self
         define_method(method) do |session, path, params = {}|
           response = BeyondApi::Connection.default.send(method) do |request|
             request.url(session.api_url + path)
-            request.headers['Authorization'] = "Bearer #{ session.access_token }" unless session.access_token.nil?
+            request.headers["Authorization"] = "Bearer #{session.access_token}" unless session.access_token.nil?
             request.params = params.to_h.camelize_keys
           end
 
@@ -21,7 +21,7 @@ class << self
         define_method(method) do |session, path, body = {}, params = {}|
           response = BeyondApi::Connection.default.send(method) do |request|
             request.url(session.api_url + path)
-            request.headers['Authorization'] = "Bearer #{ session.access_token }" unless session.access_token.nil?
+            request.headers["Authorization"] = "Bearer #{session.access_token}" unless session.access_token.nil?
             request.params = params.to_h.camelize_keys
             request.body = body.camelize_keys.to_json
           end
@@ -34,8 +34,8 @@ class << self
     def self.upload(session, path, file_binary, content_type, params)
       response = BeyondApi::Connection.default.post do |request|
         request.url(session.api_url + path)
-        request.headers['Authorization'] = "Bearer #{ session.access_token }" unless session.access_token.nil?
-        request.headers['Content-Type'] = content_type
+        request.headers["Authorization"] = "Bearer #{session.access_token}" unless session.access_token.nil?
+        request.headers["Content-Type"] = content_type
         request.params = params.to_h.camelize_keys
         request.body = file_binary
       end

lib/beyond_api/resources/newsletter_target.rb

@@ -28,7 +28,9 @@ class NewsletterTarget < Base
     #   session.newsletter_target.create({ submit_url: "https://example.org/cgi-bin/subscribe.php" })
     #
     def create(submit_url)
-      response, status = BeyondApi::Request.post(@session, "/newsletter-target", { submit_url: submit_url})
+      response, status = BeyondApi::Request.post(@session,
+                                                 "/newsletter-target",
+                                                 { submit_url: submit_url })
 
       handle_response(response, status)
     end
@@ -47,7 +49,8 @@ def create(submit_url)
     #   session.newsletter_target.delete
     #
     def delete
-      response, status = BeyondApi::Request.delete(@session, "/newsletter-target")
+      response, status = BeyondApi::Request.delete(@session,
+                                                   "/newsletter-target")
 
       handle_response(response, status, respond_with_true: true)
     end
@@ -63,7 +66,8 @@ def delete
     #   @newsletter_target = session.newsletter_target.find
     #
     def find
-      response, status = BeyondApi::Request.get(@session, "/newsletter-target")
+      response, status = BeyondApi::Request.get(@session,
+                                                "/newsletter-target")
 
       handle_response(response, status)
     end
@@ -89,7 +93,9 @@ def find
     #   session.newsletter_target.update({ submit_url: "https://example.org/cgi-bin/subscribe.php" })
     #
     def update(submit_url)
-      response, status = BeyondApi::Request.put(@session, "/newsletter-target", { submit_url: submit_url })
+      response, status = BeyondApi::Request.put(@session,
+                                                "/newsletter-target",
+                                                { submit_url: submit_url })
 
       handle_response(response, status)
     end

lib/beyond_api/utils.rb

@@ -7,6 +7,7 @@ module Utils
     def handle_response(response, status, respond_with_true: false)
       if status.between?(200, 299)
         return true if respond_with_true
+
         response = sanitize_response(response)
         BeyondApi.configuration.object_struct_responses ? to_object_struct(response) : response
       else
@@ -34,6 +35,7 @@ def sanitize_response(hash)
       {}.tap do |h|
         hash.each do |key, value|
           next if key == "_links" && BeyondApi.configuration.remove_response_links
+
           key = sanitize_key(key) if BeyondApi.configuration.remove_response_key_underscores
           h[key.underscore.to_sym] = transform(value)
         end
@@ -66,18 +68,18 @@ def handle_all_request(url, resource, params = {})
 
     private
 
-      def transform(thing)
-        case thing
-        when Hash; sanitize_response(thing)
-        when Array; thing.map { |v| transform(v) }
-        else; thing
-        end
+    def transform(thing)
+      case thing
+      when Hash; sanitize_response(thing)
+      when Array; thing.map { |v| transform(v) }
+      else; thing
       end
+    end
 
-      def all_paginated(url, params = {})
-        response, status = BeyondApi::Request.get(@session, url, params)
+    def all_paginated(url, params = {})
+      response, status = BeyondApi::Request.get(@session, url, params)
 
-        handle_response(response, status)
-      end
+      handle_response(response, status)
+    end
   end
 end