lib/keyring_liberator.rb in cocoapods-keys-1.2.1 vs lib/keyring_liberator.rb in cocoapods-keys-1.3.0

@@ -1,50 +1,48 @@
-require "digest"
-require "yaml"
-require "pathname"
+require 'digest'
+require 'yaml'
+require 'pathname'
 
 module CocoaPodsKeys
   class KeyringLiberator
-
     # Gets given a gives back a Keyring for the project
     # by basically parsing it out of ~/.cocoapods/keys/"pathMD5".yml
 
     def self.keys_dir
-       Pathname.new("~/.cocoapods/keys/").expand_path.to_s
+      Pathname('~/.cocoapods/keys/').expand_path
     end
 
     def self.yaml_path_for_path(path)
-      sha = Digest::MD5.hexdigest(path)
-      File.join(keys_dir, sha + '.yml')
+      sha = Digest::MD5.hexdigest(path.to_s)
+      keys_dir + (sha + '.yml')
     end
 
     def self.get_keyring(path)
       get_keyring_at_path(yaml_path_for_path(path))
     end
-    
+
     def self.get_keyring_named(name)
-      self.get_all_keyrings.find { |k| k.name == name }
+      get_all_keyrings.find { |k| k.name == name }
     end
 
     def self.save_keyring(keyring)
-      `mkdir -p #{keys_dir}`
+      keys_dir.mkpath
 
-      File.open(yaml_path_for_path(keyring.path), 'w') {|f| f.write(YAML::dump(keyring.to_hash)) }
+      yaml_path_for_path(keyring.path).open('w') { |f| f.write(YAML.dump(keyring.to_hash)) }
     end
 
     def self.get_all_keyrings
-      return [] unless Dir.exist? keys_dir
+      return [] unless keys_dir.directory?
       rings = []
-      Dir.glob(keys_dir + "/*.yml").each do |path|
+      Pathname.glob(keys_dir + '*.yml').each do |path|
         rings << get_keyring_at_path(path)
       end
       rings
     end
 
     private
 
     def self.get_keyring_at_path(path)
-      Keyring.from_hash(YAML.load(File.read(path))) if File.exist?(path)
+      Keyring.from_hash(YAML.load(path.read)) if path.file?
     end
-
   end
 end