vagrant_cloud 0.4.1 → 0.4.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 6f775de1cb62e00b46563d91fac4dd85dec7c55c
-  data.tar.gz: 3dda5f311191047248d72c56477adfc16ccb425f
+  metadata.gz: 655b657611c9e2117dfba4baf196bf3c0a6bea0f
+  data.tar.gz: ef550822667b5fcb60b2c2b62d8bff3a21abc3f0
 SHA512:
-  metadata.gz: d64b1d9e391b675dd2fd5fc61abe9293558b9f42b30abeb90c3ed8ff761e940e38881f16098860581378ff681b322b2d62d74d77af8a198d2cf2ededc0f7391d
-  data.tar.gz: 58c5acdae88765dff652da9756cab38a79a02d7d7d626ee4df0f84fee1a75401648cef4c2fbf6f9fd6c66f272488a770bff0c291358cbe0f4ab9730b901c8e59
+  metadata.gz: 2599abbfafadf62808ff6753033c1bdd5f12e30261083cca5ecddb41312c2d6a7da6feb905e9d211aba44c9d1e1d6f32cd6c3ccba123e66a0da847e9e92c9276
+  data.tar.gz: e23e005d8a806d6741b3ac913596ff99b557a97dd177495c9393b69edbef6f46b8c6fe63a9e0465e78b459afd65b25265ab95274d23e9ec0eee2c6130dcd5b1a

data/lib/vagrant_cloud/account.rb

@@ -1,7 +1,5 @@
 module VagrantCloud
-
   class Account
-
     attr_accessor :username
     attr_accessor :access_token
 
@@ -26,7 +24,7 @@ module VagrantCloud
       params = box_params(*args)
       params[:name] = name
 
-      data = request('post', '/boxes', {:box => params})
+      data = request('post', '/boxes', box: params)
       get_box(name, data)
     end
 
@@ -47,9 +45,9 @@ module VagrantCloud
 
       # Select elements from params that don't match what we have in the box
       # data. These are changed parameters and should be updated.
-      update_params = params.select { |k,v|
+      update_params = params.select do |k, v|
         box.data[box.param_name(k)] != v
-      }
+      end
 
       # Update the box with any params that had changed.
       box.update(update_params) unless update_params.empty?
@@ -64,14 +62,14 @@ module VagrantCloud
     def request(method, path, params = {})
       params[:access_token] = access_token
       result = RestClient::Request.execute(
-        :method => method,
-        :url => url_base + path,
-        :payload => params,
-        :ssl_version => 'TLSv1'
+        method: method,
+        url: url_base + path,
+        payload: params,
+        ssl_version: 'TLSv1'
       )
       result = JSON.parse(result)
       errors = result['errors']
-      raise(RuntimeError, "Vagrant Cloud returned error: #{errors}") if errors
+      raise "Vagrant Cloud returned error: #{errors}" if errors
       result
     end
 
@@ -103,10 +101,9 @@ module VagrantCloud
       end
 
       # Default boxes to public can be overridden by providing :is_private
-      params[:is_private] = false unless params.has_key?(:is_private)
+      params[:is_private] = false unless params.key?(:is_private)
 
       params
     end
-
   end
 end

data/lib/vagrant_cloud/box.rb

@@ -1,7 +1,5 @@
 module VagrantCloud
-
   class Box
-
     attr_accessor :account
     attr_accessor :name
     attr_accessor :data
@@ -43,7 +41,7 @@ module VagrantCloud
 
     # @param [Hash] args
     def update(args = {})
-      @data = account.request('put', "/box/#{account.username}/#{name}", {:box => args})
+      @data = account.request('put', "/box/#{account.username}/#{name}", box: args)
     end
 
     def delete
@@ -61,9 +59,9 @@ module VagrantCloud
     # @param [String] description
     # @return [Version]
     def create_version(name, description = nil)
-      params = {:version => name}
+      params = { version: name }
       params[:description] = description if description
-      data = account.request('post', "/box/#{account.username}/#{self.name}/versions", {:version => params})
+      data = account.request('post', "/box/#{account.username}/#{self.name}/versions", version: params)
       get_version(data['number'], data)
     end
 
@@ -72,10 +70,8 @@ module VagrantCloud
     # @return [Version]
     def ensure_version(name, description = nil)
       version = versions.select { |version| version.version == name }.first
-      unless version
-        version = create_version(name, description)
-      end
-      if description and (description != version.description)
+      version = create_version(name, description) unless version
+      if description && (description != version.description)
         version.update(description)
       end
       version
@@ -94,8 +90,8 @@ module VagrantCloud
     # Vagrant Cloud returns keys different from what you set for some params.
     # Values in this map should be strings.
     ATTR_MAP = {
-      :is_private  => 'private',
-      :description => 'description_markdown',
-    }
+      is_private: 'private',
+      description: 'description_markdown'
+    }.freeze
   end
 end

data/lib/vagrant_cloud/provider.rb

@@ -1,7 +1,5 @@
 module VagrantCloud
-
   class Provider
-
     attr_accessor :version
     attr_accessor :name
     attr_accessor :data
@@ -32,8 +30,8 @@ module VagrantCloud
 
     # @param [String] url
     def update(url)
-      params = {:url => url}
-      @data = account.request('put', "/box/#{account.username}/#{box.name}/version/#{version.number}/provider/#{name}", {:provider => params})
+      params = { url: url }
+      @data = account.request('put', "/box/#{account.username}/#{box.name}/version/#{version.number}/provider/#{name}", provider: params)
     end
 
     def delete
@@ -51,6 +49,5 @@ module VagrantCloud
     def account
       box.account
     end
-
   end
 end

data/lib/vagrant_cloud/version.rb

@@ -1,7 +1,5 @@
 module VagrantCloud
-
   class Version
-
     attr_accessor :box
     attr_accessor :number
     attr_accessor :data
@@ -47,8 +45,8 @@ module VagrantCloud
 
     # @param [String] description
     def update(description)
-      version = {:description => description}
-      @data = account.request('put', "/box/#{account.username}/#{box.name}/version/#{number}", {:version => version})
+      version = { description: description }
+      @data = account.request('put', "/box/#{account.username}/#{box.name}/version/#{number}", version: version)
     end
 
     def delete
@@ -74,8 +72,8 @@ module VagrantCloud
     # @param [String] url
     # @return [Provider]
     def create_provider(name, url)
-      params = {:name => name, :url => url}
-      data = account.request('post', "/box/#{account.username}/#{box.name}/version/#{self.number}/providers", {:provider => params})
+      params = { name: name, url: url }
+      data = account.request('post', "/box/#{account.username}/#{box.name}/version/#{number}/providers", provider: params)
       get_provider(name, data)
     end
 
@@ -83,13 +81,9 @@ module VagrantCloud
     # @param [String] url
     # @return [Provider]
     def ensure_provider(name, url)
-      provider = providers.select{ |provider| provider.name == name }.first
-      unless provider
-        provider = create_provider(name, url)
-      end
-      if url != provider.url
-        provider.update(url)
-      end
+      provider = providers.select { |provider| provider.name == name }.first
+      provider = create_provider(name, url) unless provider
+      provider.update(url) if url != provider.url
       provider
     end
 
@@ -99,6 +93,5 @@ module VagrantCloud
     def account
       box.account
     end
-
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: vagrant_cloud
 version: !ruby/object:Gem::Version
-  version: 0.4.1
+  version: 0.4.2
 platform: ruby
 authors:
 - Cargo Media
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-02-01 00:00:00.000000000 Z
+date: 2016-10-06 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rest-client
@@ -66,6 +66,20 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '1.21'
+- !ruby/object:Gem::Dependency
+  name: rubocop
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 0.41.2
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 0.41.2
 description: Minimalistic ruby client for the HashiCorp Atlas API (previously Vagrant
   Cloud API)
 email: tech@cargomedia.ch