ibandit 0.11.6 → 0.11.7

data/lib/ibandit/iban.rb

@@ -1,26 +1,32 @@
-require 'yaml'
+require "yaml"
 
 module Ibandit
   class IBAN
     attr_reader :errors, :iban, :country_code, :check_digits, :bank_code,
                 :branch_code, :account_number, :swift_bank_code,
-                :swift_branch_code, :swift_account_number
+                :swift_branch_code, :swift_account_number, :source
 
     def initialize(argument)
       if argument.is_a?(String)
-        input = argument.to_s.gsub(/\s+/, '').upcase
-
-        if pseudo_iban?(input) && !PseudoIBANSplitter.new(input).split.nil?
-          local_details = PseudoIBANSplitter.new(input).split
+        input = argument.to_s.gsub(/\s+/, "").upcase
+
+        pseudo_iban_splitter = PseudoIBANSplitter.new(input)
+        is_pseudo_iban_country = Constants::PSEUDO_IBAN_COUNTRY_CODES.
+          include?(pseudo_iban_splitter.country_code)
+        if pseudo_iban?(input) && is_pseudo_iban_country
+          @source = :pseudo_iban
+          local_details = pseudo_iban_splitter.split
           build_iban_from_local_details(local_details)
         else
+          @source = :iban
           @iban = input
           extract_swift_details_from_iban!
         end
       elsif argument.is_a?(Hash)
+        @source = :local_details
         build_iban_from_local_details(argument)
       else
-        raise TypeError, 'Must pass an IBAN string or hash of local details'
+        raise TypeError, "Must pass an IBAN string or hash of local details"
       end
 
       @errors = {}
@@ -51,7 +57,7 @@ module Ibandit
 
       iban.slice(
         structure[:local_check_digit_position] - 1,
-        structure[:local_check_digit_length]
+        structure[:local_check_digit_length],
       )
     end
 
@@ -64,7 +70,7 @@ module Ibandit
         country_code: country_code,
         bank_code: bank_code,
         branch_code: branch_code,
-        account_number: account_number
+        account_number: account_number,
       ).assemble
     end
 
@@ -73,21 +79,27 @@ module Ibandit
     ###############
 
     def valid?
-      [
-        valid_country_code?,
-        valid_characters?,
-        valid_check_digits?,
-        valid_length?,
-        valid_bank_code_length?,
-        valid_branch_code_length?,
-        valid_account_number_length?,
-        valid_format?,
-        valid_bank_code_format?,
-        valid_branch_code_format?,
-        valid_account_number_format?,
-        valid_local_modulus_check?,
-        passes_country_specific_checks?
-      ].all?
+      has_iban = !iban.nil?
+      has_pseudo_iban = !pseudo_iban.nil?
+
+      if has_pseudo_iban && !has_iban
+        valid_pseudo_iban?
+      else
+        valid_iban?
+      end
+    end
+
+    def valid_pseudo_iban_check_digits?
+      return true unless source == :pseudo_iban
+      return true if check_digits == Constants::PSEUDO_IBAN_CHECK_DIGITS
+
+      @errors[:check_digits] =
+        Ibandit.translate(
+          :invalid_check_digits,
+          expected_check_digits: Constants::PSEUDO_IBAN_CHECK_DIGITS,
+          check_digits: check_digits,
+        )
+      false
     end
 
     def valid_country_code?
@@ -131,8 +143,9 @@ module Ibandit
 
     def valid_bank_code_length?
       return unless valid_country_code?
+      return true if structure[:bank_code_length] == 0
 
-      if swift_bank_code.nil? || swift_bank_code.length == 0
+      if swift_bank_code.nil? || swift_bank_code.empty?
         @errors[:bank_code] = Ibandit.translate(:is_required)
         return false
       end
@@ -153,7 +166,7 @@ module Ibandit
       if structure[:branch_code_length] == 0
         @errors[:branch_code] = Ibandit.translate(:not_used_in_country,
                                                   country_code: country_code)
-      elsif swift_branch_code.nil? || swift_branch_code.length == 0
+      elsif swift_branch_code.nil? || swift_branch_code.empty?
         @errors[:branch_code] = Ibandit.translate(:is_required)
       else
         @errors[:branch_code] =
@@ -186,7 +199,7 @@ module Ibandit
       if iban.scan(/[^A-Z0-9]/).any?
         @errors[:characters] =
           Ibandit.translate(:non_alphanumeric_characters,
-                            characters: iban.scan(/[^A-Z\d]/).join(' '))
+                            characters: iban.scan(/[^A-Z\d]/).join(" "))
         false
       else
         true
@@ -195,6 +208,7 @@ module Ibandit
 
     def valid_format?
       return unless valid_country_code?
+      return unless structure[:bban_format]
 
       if bban =~ Regexp.new(structure[:bban_format])
         true
@@ -207,6 +221,7 @@ module Ibandit
 
     def valid_bank_code_format?
       return unless valid_bank_code_length?
+      return true if structure[:bank_code_length] == 0
 
       if swift_bank_code =~ Regexp.new(structure[:bank_code_format])
         true
@@ -252,21 +267,21 @@ module Ibandit
       return unless valid_country_code?
 
       case country_code
-      when 'DE' then supports_iban_determination?
-      when 'SE' then valid_swedish_details?
+      when "DE" then supports_iban_determination?
+      when "SE" then valid_swedish_details?
       else true
       end
     end
 
     def supports_iban_determination?
       return unless valid_format?
-      return true unless country_code == 'DE'
+      return true unless country_code == "DE"
 
       begin
         GermanDetailsConverter.convert(
           country_code: country_code,
           bank_code: swift_bank_code,
-          account_number: swift_account_number
+          account_number: swift_account_number,
         )
         true
       rescue UnsupportedAccountDetails
@@ -276,7 +291,7 @@ module Ibandit
     end
 
     def valid_swedish_details?
-      return true unless country_code == 'SE'
+      return true unless country_code == "SE"
 
       if branch_code
         valid_swedish_local_details?
@@ -296,7 +311,7 @@ module Ibandit
       length_valid =
         Sweden::Validator.account_number_length_valid_for_bank_code?(
           bank_code: swift_bank_code,
-          account_number: swift_account_number
+          account_number: swift_account_number,
         )
 
       unless length_valid
@@ -315,7 +330,7 @@ module Ibandit
 
       valid_serial_number = Sweden::Validator.valid_serial_number_length?(
         clearing_code: branch_code,
-        serial_number: account_number
+        serial_number: account_number,
       )
 
       unless valid_serial_number
@@ -332,6 +347,38 @@ module Ibandit
 
     private
 
+    def valid_pseudo_iban?
+      [
+        valid_pseudo_iban_check_digits?,
+        valid_country_code?,
+        valid_bank_code_length?,
+        valid_branch_code_length?,
+        valid_account_number_length?,
+        valid_bank_code_format?,
+        valid_branch_code_format?,
+        valid_account_number_format?,
+        passes_country_specific_checks?,
+      ].all?
+    end
+
+    def valid_iban?
+      [
+        valid_country_code?,
+        valid_characters?,
+        valid_check_digits?,
+        valid_length?,
+        valid_bank_code_length?,
+        valid_branch_code_length?,
+        valid_account_number_length?,
+        valid_format?,
+        valid_bank_code_format?,
+        valid_branch_code_format?,
+        valid_account_number_format?,
+        valid_local_modulus_check?,
+        passes_country_specific_checks?,
+      ].all?
+    end
+
     def decomposable?
       [iban, country_code, swift_bank_code, swift_account_number].none?(&:nil?)
     end
@@ -349,7 +396,12 @@ module Ibandit
       @swift_bank_code      = try_dup(local_details[:swift_bank_code])
 
       @iban                 = IBANAssembler.assemble(swift_details)
-      @check_digits         = @iban.slice(2, 2) unless @iban.nil?
+
+      if source == :pseudo_iban
+        @check_digits       = try_dup(local_details[:check_digits])
+      else
+        @check_digits       = @iban.slice(2, 2) unless @iban.nil?
+      end
     end
 
     def extract_swift_details_from_iban!
@@ -363,7 +415,7 @@ module Ibandit
       @swift_account_number = swift_details[:account_number]
 
       return if Constants::PSEUDO_IBAN_COUNTRY_CODES.
-                include?(@country_code)
+          include?(@country_code)
 
       @bank_code      = swift_details[:bank_code]
       @branch_code    = swift_details[:branch_code]
@@ -410,11 +462,12 @@ module Ibandit
         country_code: @country_code,
         account_number: @swift_account_number,
         branch_code: @swift_branch_code,
-        bank_code: @swift_bank_code
+        bank_code: @swift_bank_code,
       }
     end
 
     def pseudo_iban?(input)
+      return false if input.nil?
       input.slice(2, 2) == Constants::PSEUDO_IBAN_CHECK_DIGITS
     end
   end

data/lib/ibandit/iban_assembler.rb

@@ -1,6 +1,6 @@
 module Ibandit
   module IBANAssembler
-    EXCEPTION_COUNTRY_CODES = %w(IT SM BE).freeze
+    EXCEPTION_COUNTRY_CODES = %w[IT SM BE].freeze
 
     def self.assemble(local_details)
       country_code = local_details[:country_code]
@@ -47,7 +47,7 @@ module Ibandit
       partial_bban = [
         opts[:bank_code],
         opts[:branch_code],
-        opts[:account_number]
+        opts[:account_number],
       ].join
 
       check_digit = opts[:check_digit] || CheckDigit.italian(partial_bban)
@@ -69,7 +69,8 @@ module Ibandit
     end
 
     def self.supported_country_code?(local_details)
-      Constants::SUPPORTED_COUNTRY_CODES.include?(local_details[:country_code])
+      Constants::CONSTRUCTABLE_IBAN_COUNTRY_CODES.
+        include?(local_details[:country_code])
     end
 
     def self.valid_arguments?(local_details)
@@ -86,22 +87,23 @@ module Ibandit
 
     def self.required_fields(country_code)
       case country_code
-      when *%w(AT CY CZ DE DK EE FI HR IS LT LU LV NL NO PL RO SE SI SK)
-        %i(bank_code account_number)
-      when 'BE'
-        %i(account_number)
+      when "AT", "CY", "CZ", "DE", "DK", "EE", "FI", "HR", "IS", "LT", "LU",
+           "LV", "NL", "NO", "PL", "RO", "SE", "SI", "SK"
+        %i[bank_code account_number]
+      when "BE"
+        %i[account_number]
       else
-        %i(bank_code branch_code account_number)
+        %i[bank_code branch_code account_number]
       end
     end
 
     def self.allowed_fields(country_code)
       # Some countries have additional optional fields
       case country_code
-      when 'BE' then %i(bank_code account_number)
-      when 'CY' then %i(bank_code branch_code account_number)
-      when 'IT' then %i(bank_code branch_code account_number check_digit)
-      when 'CZ', 'SK' then %i(bank_code account_number account_number_prefix)
+      when "BE" then %i[bank_code account_number]
+      when "CY" then %i[bank_code branch_code account_number]
+      when "IT" then %i[bank_code branch_code account_number check_digit]
+      when "CZ", "SK" then %i[bank_code account_number account_number_prefix]
       else required_fields(country_code)
       end
     end
@@ -110,7 +112,7 @@ module Ibandit
       [
         country_code,
         CheckDigit.iban(country_code, bban),
-        bban
+        bban,
       ].join
     rescue InvalidCharacterError
       nil

data/lib/ibandit/iban_splitter.rb

@@ -6,7 +6,7 @@ module Ibandit
         check_digits:   check_digits_from(iban),
         bank_code:      bank_code_from(iban),
         branch_code:    branch_code_from(iban),
-        account_number: account_number_from(iban)
+        account_number: account_number_from(iban),
       }
     end
 
@@ -31,7 +31,7 @@ module Ibandit
 
       iban.slice(
         structure(iban)[:bank_code_position] - 1,
-        structure(iban)[:bank_code_length]
+        structure(iban)[:bank_code_length],
       )
     end
 
@@ -42,7 +42,7 @@ module Ibandit
 
       iban.slice(
         structure(iban)[:branch_code_position] - 1,
-        structure(iban)[:branch_code_length]
+        structure(iban)[:branch_code_length],
       )
     end
 
@@ -51,7 +51,7 @@ module Ibandit
 
       iban.slice(
         structure(iban)[:account_number_position] - 1,
-        structure(iban)[:account_number_length]
+        structure(iban)[:account_number_length],
       )
     end
 

data/lib/ibandit/local_details_cleaner.rb

@@ -2,13 +2,13 @@ module Ibandit
   module LocalDetailsCleaner
     def self.clean(local_details)
       country_code = local_details[:country_code]
-
       if can_clean?(country_code, local_details)
         local_details = local_details.merge(
-          public_send(:"clean_#{country_code.downcase}_details", local_details))
+          public_send(:"clean_#{country_code.downcase}_details", local_details),
+        )
       end
 
-      return local_details if explicit_swift_details?(country_code)
+      return local_details if explicit_swift_details?(local_details)
 
       swift_details_for(local_details).merge(local_details)
     end
@@ -22,8 +22,8 @@ module Ibandit
         fields_for?(country_code, local_details)
     end
 
-    def self.explicit_swift_details?(country_code)
-      Constants::PSEUDO_IBAN_COUNTRY_CODES.include?(country_code)
+    def self.explicit_swift_details?(local_details)
+      local_details.include?(:swift_account_number)
     end
 
     def self.fields_for?(country_code, opts)
@@ -32,16 +32,18 @@ module Ibandit
 
     def self.required_fields(country_code)
       case country_code
-      when 'AT', 'CY', 'DE', 'FI', 'LT', 'LU', 'LV', 'NL', 'RO', 'SI', 'SK'
-        %i(bank_code account_number)
-      when 'BE', 'CZ', 'DK', 'EE', 'ES', 'HR', 'HU', 'IS', 'NO', 'PL', 'SE'
-        %i(account_number)
-      when 'GB', 'IE', 'MT'
-        if Ibandit.bic_finder.nil? then %i(bank_code branch_code account_number)
-        else %i(branch_code account_number)
+      when "AT", "CY", "DE", "FI", "LT", "LU", "LV", "NL", "RO", "SI", "SK"
+        %i[bank_code account_number]
+      when "BE", "CZ", "DK", "EE", "ES", "HR", "HU", "IS", "NO", "PL", "SE"
+        %i[account_number]
+      when "GB", "IE", "MT"
+        if Ibandit.bic_finder.nil? then %i[bank_code branch_code account_number]
+        else %i[branch_code account_number]
         end
+      when "AU"
+        %i[branch_code account_number]
       else
-        %i(bank_code branch_code account_number)
+        %i[bank_code branch_code account_number]
       end
     end
 
@@ -55,16 +57,23 @@ module Ibandit
       return {} unless local_details[:account_number].length >= 4
       {
         bank_code:      local_details[:bank_code],
-        account_number: local_details[:account_number].rjust(11, '0')
+        account_number: local_details[:account_number].rjust(11, "0"),
+      }
+    end
+
+    def self.clean_au_details(local_details)
+      {
+        branch_code:    local_details[:branch_code].delete("-"),
+        account_number: local_details[:account_number],
       }
     end
 
     def self.clean_be_details(local_details)
-      account_number = local_details[:account_number].tr('-', '')
+      account_number = local_details[:account_number].tr("-", "")
 
       {
         bank_code:      local_details[:bank_code] || account_number.slice(0, 3),
-        account_number: account_number
+        account_number: account_number,
       }
     end
 
@@ -76,7 +85,7 @@ module Ibandit
     def self.clean_cy_details(local_details)
       # Account number may be 7-16 digits long.
       # Add leading zeros to account number if < 16 digits.
-      cleaned_bank_code = local_details[:bank_code].gsub(/[-\s]/, '')
+      cleaned_bank_code = local_details[:bank_code].gsub(/[-\s]/, "")
 
       bank_code      = cleaned_bank_code.slice(0, 3)
       branch_code    =
@@ -87,7 +96,7 @@ module Ibandit
         end
       account_number =
         if local_details[:account_number].length >= 7
-          local_details[:account_number].rjust(16, '0')
+          local_details[:account_number].rjust(16, "0")
         else
           local_details[:account_number]
         end
@@ -95,7 +104,7 @@ module Ibandit
       {
         bank_code:      bank_code,
         branch_code:    branch_code,
-        account_number: account_number
+        account_number: account_number,
       }
     end
 
@@ -106,16 +115,16 @@ module Ibandit
       account_number =
         if local_details.include?(:account_number_prefix)
           [
-            local_details[:account_number_prefix].rjust(6, '0'),
-            local_details[:account_number].rjust(10, '0')
+            local_details[:account_number_prefix].rjust(6, "0"),
+            local_details[:account_number].rjust(10, "0"),
           ].join
         else
-          local_details[:account_number].tr('-', '').rjust(16, '0')
+          local_details[:account_number].tr("-", "").rjust(16, "0")
         end
 
       {
         bank_code:      local_details[:bank_code],
-        account_number: account_number
+        account_number: account_number,
       }
     end
 
@@ -137,7 +146,7 @@ module Ibandit
 
       {
         bank_code:      converted_details[:bank_code],
-        account_number: converted_details[:account_number].rjust(10, '0')
+        account_number: converted_details[:account_number].rjust(10, "0"),
       }
     end
 
@@ -145,12 +154,12 @@ module Ibandit
       # This method supports being passed the component IBAN parts, as defined
       # by SWIFT, or a single traditional-format string split by a '-'.
       if local_details[:bank_code]
-        bank_code      = local_details[:bank_code].rjust(4, '0')
-        account_number = local_details[:account_number].rjust(10, '0')
-      elsif local_details[:account_number].include?('-')
-        bank_code, account_number = local_details[:account_number].split('-', 2)
-      elsif local_details[:account_number].gsub(/\s/, '').length == 14
-        cleaned_account_number = local_details[:account_number].gsub(/\s/, '')
+        bank_code      = local_details[:bank_code].rjust(4, "0")
+        account_number = local_details[:account_number].rjust(10, "0")
+      elsif local_details[:account_number].include?("-")
+        bank_code, account_number = local_details[:account_number].split("-", 2)
+      elsif local_details[:account_number].gsub(/\s/, "").length == 14
+        cleaned_account_number = local_details[:account_number].gsub(/\s/, "")
         bank_code      = cleaned_account_number.slice(0, 4)
         account_number = cleaned_account_number.slice(4, 10)
       else
@@ -158,8 +167,8 @@ module Ibandit
       end
 
       {
-        bank_code:      bank_code.rjust(4, '0'),
-        account_number: account_number.delete('-').rjust(10, '0')
+        bank_code:      bank_code.rjust(4, "0"),
+        account_number: account_number.delete("-").rjust(10, "0"),
       }
     end
 
@@ -171,16 +180,16 @@ module Ibandit
       # account number and converted using the rules at
       # http://www.pangaliit.ee/en/settlements-and-standards/bank-codes-of-estonian-banks
       domestic_bank_code =
-        local_details[:account_number].gsub(/\A0+/, '').slice(0, 2)
+        local_details[:account_number].gsub(/\A0+/, "").slice(0, 2)
 
       iban_bank_code =
         case domestic_bank_code
-        when '11' then '22'
-        when '93' then '00'
+        when "11" then "22"
+        when "93" then "00"
         else domestic_bank_code
         end
 
-      account_number = local_details[:account_number].rjust(14, '0')
+      account_number = local_details[:account_number].rjust(14, "0")
 
       { bank_code: iban_bank_code, account_number: account_number }
     end
@@ -191,10 +200,10 @@ module Ibandit
       if local_details[:bank_code] && local_details[:branch_code]
         bank_code      = local_details[:bank_code]
         branch_code    = local_details[:branch_code]
-        account_number = local_details[:account_number].gsub(/[-\s]/, '')
+        account_number = local_details[:account_number].gsub(/[-\s]/, "")
       else
         cleaned_account_number =
-          local_details[:account_number].gsub(/[-\s]/, '')
+          local_details[:account_number].gsub(/[-\s]/, "")
 
         bank_code      = cleaned_account_number.slice(0, 4)
         branch_code    = cleaned_account_number.slice(4, 4)
@@ -204,7 +213,7 @@ module Ibandit
       {
         bank_code:      bank_code,
         branch_code:    branch_code,
-        account_number: account_number
+        account_number: account_number,
       }
     end
 
@@ -213,18 +222,18 @@ module Ibandit
       #   by adding zero-padding. The expansion method depends on the first
       #   character of the bank code.
       account_number =
-        if %w(4 5 6).include?(local_details[:bank_code][0])
+        if %w[4 5 6].include?(local_details[:bank_code][0])
           [
             local_details[:account_number][0],
-            local_details[:account_number][1..-1].rjust(7, '0')
+            local_details[:account_number][1..-1].rjust(7, "0"),
           ].join
         else
-          local_details[:account_number].rjust(8, '0')
+          local_details[:account_number].rjust(8, "0")
         end
 
       {
         bank_code:      local_details[:bank_code],
-        account_number: account_number
+        account_number: account_number,
       }
     end
 
@@ -232,29 +241,29 @@ module Ibandit
       {
         bank_code:      local_details[:bank_code],
         branch_code:    local_details[:branch_code],
-        account_number: local_details[:account_number].gsub(/[-\s]/, '')
+        account_number: local_details[:account_number].gsub(/[-\s]/, ""),
       }
     end
 
     def self.clean_gb_details(local_details)
       # Account number may be 6-8 digits
       # Add leading zeros to account number if < 8 digits.
-      branch_code = local_details[:branch_code].gsub(/[-\s]/, '')
+      branch_code = local_details[:branch_code].gsub(/[-\s]/, "")
 
       if local_details[:bank_code]
         bank_code = local_details[:bank_code]
       else
-        bic = Ibandit.find_bic('GB', branch_code)
+        bic = Ibandit.find_bic("GB", branch_code)
         bank_code = bic.nil? ? nil : bic.slice(0, 4)
       end
 
-      account_number = local_details[:account_number].gsub(/[-\s]/, '')
-      account_number = account_number.rjust(8, '0') if account_number.length > 5
+      account_number = local_details[:account_number].gsub(/[-\s]/, "")
+      account_number = account_number.rjust(8, "0") if account_number.length > 5
 
       {
         bank_code:      bank_code,
         branch_code:    branch_code,
-        account_number: account_number
+        account_number: account_number,
       }
     end
 
@@ -268,13 +277,13 @@ module Ibandit
       # This method supports being passed the component IBAN parts, as defined
       # by SWIFT, or a single traditional-format string split by a '-'.
       return local_details if local_details[:bank_code]
-      return local_details unless local_details[:account_number].include?('-')
+      return local_details unless local_details[:account_number].include?("-")
 
-      bank_code, account_number = local_details[:account_number].split('-', 2)
+      bank_code, account_number = local_details[:account_number].split("-", 2)
 
       {
         bank_code:      bank_code,
-        account_number: account_number
+        account_number: account_number,
       }
     end
 
@@ -285,20 +294,20 @@ module Ibandit
         return local_details
       end
 
-      cleaned_acct_number = local_details[:account_number].gsub(/[-\s]/, '')
+      cleaned_acct_number = local_details[:account_number].gsub(/[-\s]/, "")
 
       case cleaned_acct_number.length
       when 16
         {
           bank_code:      cleaned_acct_number.slice(0, 3),
           branch_code:    cleaned_acct_number.slice(3, 4),
-          account_number: cleaned_acct_number.slice(7, 9).ljust(17, '0')
+          account_number: cleaned_acct_number.slice(7, 9).ljust(17, "0"),
         }
       when 24
         {
           bank_code:      cleaned_acct_number.slice(0, 3),
           branch_code:    cleaned_acct_number.slice(3, 4),
-          account_number: cleaned_acct_number.slice(7, 17)
+          account_number: cleaned_acct_number.slice(7, 17),
         }
       else local_details
       end
@@ -306,39 +315,39 @@ module Ibandit
 
     def self.clean_ie_details(local_details)
       # Ireland uses the same local details as the United Kingdom
-      branch_code = local_details[:branch_code].gsub(/[-\s]/, '')
+      branch_code = local_details[:branch_code].gsub(/[-\s]/, "")
 
       if local_details[:bank_code]
         bank_code = local_details[:bank_code]
       else
-        bic = Ibandit.find_bic('IE', branch_code)
+        bic = Ibandit.find_bic("IE", branch_code)
         bank_code = bic.nil? ? nil : bic.slice(0, 4)
       end
 
-      account_number = local_details[:account_number].gsub(/[-\s]/, '')
-      account_number = account_number.rjust(8, '0') if account_number.length > 5
+      account_number = local_details[:account_number].gsub(/[-\s]/, "")
+      account_number = account_number.rjust(8, "0") if account_number.length > 5
 
       {
         bank_code:      bank_code,
         branch_code:    branch_code,
-        account_number: account_number
+        account_number: account_number,
       }
     end
 
     def self.clean_is_details(local_details)
       if local_details[:bank_code]
         bank_code = local_details[:bank_code]
-        parts = local_details[:account_number].split('-')
-      elsif local_details[:account_number].include?('-')
-        bank_code, *parts = local_details[:account_number].split('-')
+        parts = local_details[:account_number].split("-")
+      elsif local_details[:account_number].include?("-")
+        bank_code, *parts = local_details[:account_number].split("-")
       else
         bank_code = local_details[:account_number].slice(0, 4)
         parts = Array(local_details[:account_number][4..-1])
       end
 
       {
-        bank_code:      bank_code.rjust(4, '0'),
-        account_number: pad_is_account_number(parts)
+        bank_code:      bank_code.rjust(4, "0"),
+        account_number: pad_is_account_number(parts),
       }
     end
 
@@ -347,7 +356,7 @@ module Ibandit
       {
         bank_code:      local_details[:bank_code],
         branch_code:    local_details[:branch_code],
-        account_number: local_details[:account_number].rjust(12, '0')
+        account_number: local_details[:account_number].rjust(12, "0"),
       }
     end
 
@@ -378,17 +387,17 @@ module Ibandit
       if local_details[:bank_code]
         bank_code = local_details[:bank_code]
       else
-        bic = Ibandit.find_bic('MT', branch_code)
+        bic = Ibandit.find_bic("MT", branch_code)
         bank_code = bic.nil? ? nil : bic.slice(0, 4)
       end
 
-      account_number = local_details[:account_number].gsub(/[-\s]/, '')
-      account_number = account_number.rjust(18, '0')
+      account_number = local_details[:account_number].gsub(/[-\s]/, "")
+      account_number = account_number.rjust(18, "0")
 
       {
         bank_code:      bank_code,
         branch_code:    branch_code,
-        account_number: account_number
+        account_number: account_number,
       }
     end
 
@@ -396,7 +405,7 @@ module Ibandit
       # Add leading zeros to account number if < 10 digits.
       {
         bank_code:      local_details[:bank_code],
-        account_number: local_details[:account_number].rjust(10, '0')
+        account_number: local_details[:account_number].rjust(10, "0"),
       }
     end
 
@@ -407,7 +416,7 @@ module Ibandit
         bank_code      = local_details[:bank_code]
         account_number = local_details[:account_number]
       else
-        cleaned_acct_number = local_details[:account_number].gsub(/[-.\s]/, '')
+        cleaned_acct_number = local_details[:account_number].gsub(/[-.\s]/, "")
 
         bank_code      = cleaned_acct_number.slice(0, 4)
         account_number = cleaned_acct_number[4..-1]
@@ -415,7 +424,7 @@ module Ibandit
 
       {
         bank_code:      bank_code,
-        account_number: account_number
+        account_number: account_number,
       }
     end
 
@@ -426,7 +435,7 @@ module Ibandit
         bank_code      = local_details[:bank_code]
         account_number = local_details[:account_number]
       else
-        cleaned_acct_number = local_details[:account_number].gsub(/[\s]/, '')
+        cleaned_acct_number = local_details[:account_number].gsub(/[\s]/, "")
 
         bank_code      = cleaned_acct_number.slice(2, 8)
         account_number = cleaned_acct_number[10..-1]
@@ -434,7 +443,7 @@ module Ibandit
 
       {
         bank_code:      bank_code,
-        account_number: account_number
+        account_number: account_number,
       }
     end
 
@@ -451,14 +460,14 @@ module Ibandit
       if local_details[:bank_code]
         # If a bank_code was provided without a branch code we're (probably)
         # dealing with SWIFT details and should just return them.
-        return {
+        {
           swift_account_number: local_details[:account_number],
-          swift_bank_code: local_details[:bank_code]
+          swift_bank_code: local_details[:bank_code],
         }
       else
         Sweden::LocalDetailsConverter.new(
           branch_code: local_details[:branch_code],
-          account_number: local_details[:account_number]
+          account_number: local_details[:account_number],
         ).convert
       end
     end
@@ -467,7 +476,7 @@ module Ibandit
       # Add leading zeros to account number if < 10 digits.
       {
         bank_code:      local_details[:bank_code],
-        account_number: local_details[:account_number].rjust(10, '0')
+        account_number: local_details[:account_number].rjust(10, "0"),
       }
     end
 
@@ -482,12 +491,12 @@ module Ibandit
     end
 
     def self.pad_is_account_number(parts)
-      hufo           = parts[0].nil? ? '' : parts[0].rjust(2, '0')
-      reikningsnumer = parts[1].nil? ? '' : parts[1].rjust(6, '0')
-      ken_1          = parts[2].nil? ? '' : parts[2].rjust(6, '0')
-      ken_2          = parts[3].nil? ? '' : parts[3].rjust(4, '0')
+      hufo           = parts[0].nil? ? "" : parts[0].rjust(2, "0")
+      reikningsnumer = parts[1].nil? ? "" : parts[1].rjust(6, "0")
+      ken_1          = parts[2].nil? ? "" : parts[2].rjust(6, "0")
+      ken_2          = parts[3].nil? ? "" : parts[3].rjust(4, "0")
 
-      kennitala      = ken_1.empty? ? '' : (ken_1 + ken_2).rjust(10, '0')
+      kennitala      = ken_1.empty? ? "" : (ken_1 + ken_2).rjust(10, "0")
 
       hufo + reikningsnumer + kennitala
     end
@@ -497,7 +506,7 @@ module Ibandit
       {
         swift_bank_code:      local_details[:bank_code],
         swift_branch_code:    local_details[:branch_code],
-        swift_account_number: local_details[:account_number]
+        swift_account_number: local_details[:account_number],
       }
     end
     private_class_method :swift_details_for