fog-cloudatcost 0.2.3 → 0.4.0

data/lib/fog/cloudatcost/requests/delete_server.rb

@@ -1,29 +1,31 @@
+# frozen_string_literal: true
+
 module Fog
   module Compute
     class CloudAtCost
       class Real
         def delete_server(id)
-          body = { sid: "#{id}" }
+          body = { sid: id.to_s }
           request(
-            :expects => [200],
-            :method  => 'POST',
-            :path    => 'api/v1/cloudpro/delete.php',
-            :body    => body,
+            expects: [200],
+            method: 'POST',
+            path: 'api/v1/cloudpro/delete.php',
+            body: body
           )
         end
       end
 
       class Mock
-        def delete_server(id)
-          response         = Excon::Response.new
-          response.status  = 200
-          response.body    = {
+        def delete_server(_id)
+          response = Excon::Response.new
+          response.status = 200
+          response.body = {
             'result'     => 'successful',
             'api'        => 'v1',
             'action'     => 'delete',
             'status'     => 'ok',
-            'taskid'     => 123123123123,
-            'time'       => 12312323,
+            'taskid'     => 123_123_123_123,
+            'time'       => 12_312_323
           }
           response
         end

data/lib/fog/cloudatcost/requests/list_servers.rb

@@ -1,12 +1,14 @@
+# frozen_string_literal: true
+
 module Fog
   module Compute
     class CloudAtCost
       class Real
         def list_servers
           request(
-            :expects => [200],
-            :method => 'GET',
-            :path => '/api/v1/listservers.php'
+            expects: [200],
+            method: 'GET',
+            path: '/api/v1/listservers.php'
           )
         end
       end
@@ -17,8 +19,8 @@ module Fog
           response = Excon::Response.new
           response.status = 200
           response.body = {
-              'status' => 'OK',
-              'servers'  => self.data[:data]
+            'status' => 'OK',
+            'servers' => data[:data]
           }
           response
         end

data/lib/fog/cloudatcost/requests/list_tasks.rb

@@ -1,12 +1,14 @@
+# frozen_string_literal: true
+
 module Fog
   module Compute
     class CloudAtCost
       class Real
         def list_tasks
           request(
-            :expects => [200],
-            :method => 'GET',
-            :path => '/api/v1/listtasks.php'
+            expects: [200],
+            method: 'GET',
+            path: '/api/v1/listtasks.php'
           )
         end
       end
@@ -17,8 +19,8 @@ module Fog
           response = Excon::Response.new
           response.status = 200
           response.body = {
-              'status' => 'OK',
-              'servers'  => self.data[:data]
+            'status' => 'OK',
+            'servers' => data[:data]
           }
           response
         end

data/lib/fog/cloudatcost/requests/list_templates.rb

@@ -1,12 +1,14 @@
+# frozen_string_literal: true
+
 module Fog
   module Compute
     class CloudAtCost
       class Real
         def list_templates
           request(
-            :expects => [200],
-            :method => 'GET',
-            :path => '/api/v1/listtemplates.php'
+            expects: [200],
+            method: 'GET',
+            path: '/api/v1/listtemplates.php'
           )
         end
       end
@@ -17,8 +19,8 @@ module Fog
           response = Excon::Response.new
           response.status = 200
           response.body = {
-              'status' => 'OK',
-              'servers'  => self.data[:data]
+            'status' => 'OK',
+            'servers' => data[:data]
           }
           response
         end

data/lib/fog/cloudatcost/requests/power_off.rb

@@ -1,30 +1,32 @@
+# frozen_string_literal: true
+
 module Fog
   module Compute
     class CloudAtCost
       class Real
         def power_off(id)
-          body = { :sid => "#{id}", :action => 'poweroff' }
+          body = { sid: id.to_s, action: 'poweroff' }
           request(
-            :expects => [200],
-            :method  => 'POST',
-            :path    => 'api/v1/powerop.php',
-            :body    => body,
+            expects: [200],
+            method: 'POST',
+            path: 'api/v1/powerop.php',
+            body: body
           )
         end
       end
 
       class Mock
-        def power_off(id)
-          response         = Excon::Response.new
-          response.status  = 200
-          response.body    = {
-            'server_id'   => Fog::Mock.random_numbers(1).to_i,
+        def power_off(_id)
+          response = Excon::Response.new
+          response.status = 200
+          response.body = {
+            'server_id' => Fog::Mock.random_numbers(1).to_i,
             'api'        => 'v1',
             'status'     => 'ok',
             'result'     => 'successful',
             'action'     => 'poweroff',
-            'time'       => 12312323,
-            'taskid'     => 123123123123
+            'time'       => 12_312_323,
+            'taskid'     => 123_123_123_123
           }
           response
         end

data/lib/fog/cloudatcost/requests/power_on.rb

@@ -1,30 +1,32 @@
+# frozen_string_literal: true
+
 module Fog
   module Compute
     class CloudAtCost
       class Real
         def power_on(id)
-          body = { :sid => "#{id}", :action => 'poweron' }
+          body = { sid: id.to_s, action: 'poweron' }
           request(
-            :expects => [200],
-            :method  => 'POST',
-            :path    => 'api/v1/powerop.php',
-            :body    => body,
+            expects: [200],
+            method: 'POST',
+            path: 'api/v1/powerop.php',
+            body: body
           )
         end
       end
 
       class Mock
-        def power_on(id)
-          response         = Excon::Response.new
-          response.status  = 200
-          response.body    = {
-            'server_id'   => Fog::Mock.random_numbers(1).to_i,
+        def power_on(_id)
+          response = Excon::Response.new
+          response.status = 200
+          response.body = {
+            'server_id' => Fog::Mock.random_numbers(1).to_i,
             'api'        => 'v1',
             'status'     => 'ok',
             'result'     => 'successful',
             'action'     => 'poweron',
-            'time'       => 12312323,
-            'taskid'     => 123123123123
+            'time'       => 12_312_323,
+            'taskid'     => 123_123_123_123
           }
           response
         end

data/lib/fog/cloudatcost/requests/rename_server.rb

@@ -1,28 +1,30 @@
+# frozen_string_literal: true
+
 module Fog
   module Compute
     class CloudAtCost
       class Real
         def rename_server(id, name)
-          body = { :sid => "#{id}", :name => "#{name}" }
+          body = { sid: id.to_s, name: name.to_s }
           request(
-            :expects => [200],
-            :method  => 'POST',
-            :path    => 'api/v1/renameserver.php',
-            :body    => body,
+            expects: [200],
+            method: 'POST',
+            path: 'api/v1/renameserver.php',
+            body: body
           )
         end
       end
 
       class Mock
-        def rename_server(id, name)
-          response         = Excon::Response.new
-          response.status  = 200
-          response.body    = {
-            'server_id'   => Fog::Mock.random_numbers(1).to_i,
-            'api'        => "v1",
-            'status'     => "ok",
-            'result'     => "successful",
-            'time'       => 12312323,
+        def rename_server(_id, _name)
+          response = Excon::Response.new
+          response.status = 200
+          response.body = {
+            'server_id' => Fog::Mock.random_numbers(1).to_i,
+            'api'        => 'v1',
+            'status'     => 'ok',
+            'result'     => 'successful',
+            'time'       => 12_312_323
           }
           response
         end

data/lib/fog/cloudatcost/requests/reset.rb

@@ -1,30 +1,32 @@
+# frozen_string_literal: true
+
 module Fog
   module Compute
     class CloudAtCost
       class Real
         def reset(id)
-          body = { :sid => "#{id}", :action => 'reset' }
+          body = { sid: id.to_s, action: 'reset' }
           request(
-            :expects => [200],
-            :method  => 'POST',
-            :path    => "api/v1/powerop.php",
-            :body    => body,
+            expects: [200],
+            method: 'POST',
+            path: 'api/v1/powerop.php',
+            body: body
           )
         end
       end
 
       class Mock
-        def reset(id)
-          response         = Excon::Response.new
-          response.status  = 200
-          response.body    = {
-            'server_id'   => Fog::Mock.random_numbers(1).to_i,
+        def reset(_id)
+          response = Excon::Response.new
+          response.status = 200
+          response.body = {
+            'server_id' => Fog::Mock.random_numbers(1).to_i,
             'api'        => 'v1',
             'status'     => 'ok',
             'result'     => 'successful',
             'action'     => 'reset',
-            'time'       => 12312323,
-            'taskid'     => 123123123123
+            'time'       => 12_312_323,
+            'taskid'     => 123_123_123_123
           }
           response
         end

data/lib/fog/cloudatcost/requests/reverse_dns.rb

@@ -1,28 +1,30 @@
+# frozen_string_literal: true
+
 module Fog
   module Compute
     class CloudAtCost
       class Real
         def reverse_dns(id, host_name)
-          body = { :sid => "#{id}", :hostname => "#{host_name}" }
+          body = { sid: id.to_s, hostname: host_name.to_s }
           request(
-            :expects => [200],
-            :method  => 'POST',
-            :path    => 'api/v1/rdns.php',
-            :body    => body,
+            expects: [200],
+            method: 'POST',
+            path: 'api/v1/rdns.php',
+            body: body
           )
         end
       end
 
       class Mock
-        def reverse_dns(id, hostname)
-          response         = Excon::Response.new
-          response.status  = 200
-          response.body    = {
-            'server_id'   => Fog::Mock.random_numbers(1).to_i,
+        def reverse_dns(_id, _hostname)
+          response = Excon::Response.new
+          response.status = 200
+          response.body = {
+            'server_id' => Fog::Mock.random_numbers(1).to_i,
             'api'        => 'v1',
             'status'     => 'ok',
             'result'     => 'successful',
-            'time'       => 12312323,
+            'time'       => 12_312_323
           }
           response
         end

data/lib/fog/cloudatcost/requests/run_mode.rb

@@ -1,28 +1,30 @@
+# frozen_string_literal: true
+
 module Fog
   module Compute
     class CloudAtCost
       class Real
         def run_mode(id, action)
-          body = { :sid => "#{id}", :mode => "#{action}" }
+          body = { sid: id.to_s, mode: action.to_s }
           request(
-            :expects => [200],
-            :method  => 'POST',
-            :path    => 'api/v1/runmode.php',
-            :body    => body,
+            expects: [200],
+            method: 'POST',
+            path: 'api/v1/runmode.php',
+            body: body
           )
         end
       end
 
       class Mock
-        def run_mode(id, action)
-          response         = Excon::Response.new
-          response.status  = 200
-          response.body    = {
-            'server_id'   => Fog::Mock.random_numbers(1).to_i,
+        def run_mode(_id, _action)
+          response = Excon::Response.new
+          response.status = 200
+          response.body = {
+            'server_id' => Fog::Mock.random_numbers(1).to_i,
             'api'        => 'v1',
             'status'     => 'ok',
             'result'     => 'successful',
-            'time'       => 12312323,
+            'time'       => 12_312_323
           }
           response
         end

data/lib/fog/cloudatcost/version.rb

@@ -1,5 +1,7 @@
+# frozen_string_literal: true
+
 module Fog
   module CloudAtCost
-    VERSION = '0.2.3'
+    VERSION = '0.4.0'
   end
 end

data/spec/fog/models/server_spec.rb

@@ -1,14 +1,15 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
-require 'fog/cloudatcost/models/server'
 
-describe Fog::Compute::CloudAtCost::Server do
+RSpec.describe Fog::Compute::CloudAtCost::Server do
   let(:server) { Fog::Compute::CloudAtCost::Server.new }
 
-  server_attributes = %w(sid id CustID packageid label servername 
+  server_attributes = %w[sid id CustID packageid label servername
                          vmname ip netmask gateway hostname rootpass
                          vncport vncpass servertype template cpu cpuusage
                          ram ramusage storage hdusage sdate status panel_note
-                         mode uid rdns rdnsdefault template_id)
+                         mode uid rdns rdnsdefault template_id]
 
   it 'tests attributes' do
     server_attributes.each do |attribute|

data/spec/fog/models/servers_spec.rb

@@ -1,11 +1,11 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
-require 'fog/cloudatcost/models/servers'
 
-describe Fog::Compute::CloudAtCost::Servers do
+RSpec.describe Fog::Compute::CloudAtCost::Servers do
   subject { Fog::Compute::CloudAtCost::Servers }
 
   it 'should have a collection of servers' do
     expect(subject.model).to eq(Fog::Compute::CloudAtCost::Server)
   end
 end
-

data/spec/fog/models/task_spec.rb

@@ -1,7 +1,8 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
-require 'fog/cloudatcost/models/task'
 
-describe Fog::Compute::CloudAtCost::Task do
+RSpec.describe Fog::Compute::CloudAtCost::Task do
   let(:server) { Fog::Compute::CloudAtCost::Task.new }
 
   it 'respond to #id' do