unit-hosting 0.3.1 → 0.4.0

data/spec/unit-hosting/cache_spec.rb

@@ -0,0 +1,33 @@
+# -*- coding: utf-8 -*-
+require 'spec_helper'
+require 'unit-hosting/cache'
+
+describe UnitHosting::Cache do
+  before{
+    @cache_file = Tempfile.new('cache')
+    @cache_file.close
+    @cache = UnitHosting::Cache.new(@cache_file.path)
+  }
+  after {
+    @cache_file.unlink
+  }
+  describe "#update_group!" do
+    context "when there are three gropus in cache." do
+      before{
+        @group222 = UnitHosting::Group.new("test-sg-222")
+        @cache.transaction { |ps|
+          ps["groups"] = [
+                          UnitHosting::Group.new("test-sg-1"),
+                          UnitHosting::Group.new("test-sg-2"),
+                          UnitHosting::Group.new("test-sg-3"),
+                          @group222
+                         ].extend(UnitHosting::Groups)
+        }
+      }
+      it "update given group" do
+        expect(@group222).to receive(:update)
+        @cache.update_group!(@group222)
+      end
+    end
+  end
+end

data/spec/unit-hosting/cli_spec.rb

@@ -0,0 +1,23 @@
+# -*- coding: utf-8 -*-
+require 'spec_helper'
+require 'unit-hosting/cli'
+
+describe UnitHosting::CLI do
+  before {
+    @cli = UnitHosting::CLI.new
+  }
+  describe "#version" do
+    it "print version string" do
+      expect(capture(:stdout) {
+               @cli.version}).to match /\d+\.\d+\.\d+/
+    end
+  end
+  describe "#dispatch" do
+    context "foo:bar" do
+      it "call @commands.foo_bar" do
+        expect(@cli.commands).to receive(:foo_bar).once
+        @cli.dispatch("foo:bar",[])
+      end
+    end
+  end
+end

data/spec/unit-hosting/commands_spec.rb

@@ -0,0 +1,214 @@
+# -*- coding: utf-8 -*-
+require 'spec_helper'
+require 'unit-hosting/commands'
+describe UnitHosting::Commands do
+  before {
+    @endpoint = "https://example.net"
+    @commands = UnitHosting::Commands.new(@endpoint)
+    @commands.keyname = "rspec-unit-hosting-test"
+  }
+  describe "#login" do
+    before{
+      @commands.agent.stub(:login).and_return(nil)
+      @user1 = { :username =>"test-user-1", :password =>"correct-for-1", :bad_password =>"bad-for-1"}
+      expect(Keystorage).to receive(:set).
+      with(@commands.keyname,@user1[:username],@user1[:password]).once
+    }
+    context "put collect username and password." do
+      before {
+        @commands.agent.stub(:login?).and_return(true)
+      }
+      context "answer collect username and password" do
+        before { }
+        it "returns true." do
+          @commands.stub(:ask) { |question, &details|
+            HighLine::Question.new(question, String, &details)
+            case question
+            when "Enter user: "
+              "test-user-1"
+            when "Enter your password: "
+              "correct-for-1"
+            end
+          }
+          $stderr.should_receive(:puts).with("login OK").once
+          @commands.login
+        end
+      end
+      
+    end
+    context "put bad pair of username and password." do
+      before{
+        @commands.agent.stub(:login?).and_return(false,true)
+      }
+      it "asks username and password again." do
+        $stderr.should_receive(:puts).with("login OK").once
+        $stderr.should_receive(:puts).with("password mismatch").once
+        expect(@commands).to receive(:ask).
+          with("Enter user: ").and_return(@user1[:username]).twice
+        expect(@commands).to receive(:ask).
+          with("Enter your password: ").and_return(@user1[:password]).twice
+        @commands.login
+      end
+    end
+  end
+  
+  describe "#logout" do
+    it "removes Keychain entry." do
+      expect(Keystorage).to receive(:delete).with(@commands.keyname).once
+      @commands.logout
+    end
+  end
+  
+  describe "#groups" do
+    it "returns groups." do
+      expect(@commands.cache).to receive(:groups).with().once
+      expect($stdout).to receive(:puts).once
+      @commands.groups
+    end
+  end
+  
+  describe "#group" do
+    it "asks group andreturns group." do
+      expect(@commands).to receive(:ask_group).once
+      @commands.group
+    end
+  end
+  describe "#update" do
+    before{
+      @cache_file = Tempfile.new('cache')
+      @cache_file.close
+      @commands.stub(:cache_file).and_return(@cache_file.path)
+      @commands.stub(:start).and_return true
+    }
+    after {
+      @cache_file.unlink
+    }
+    context "when --all is set" do
+      before{
+        @groups = [].extend(UnitHosting::Groups)
+        (1..10).each { |n|
+          group = UnitHosting::Group.new("test-sg-#{n}")
+          group.key = "dummy-key-#{n}"
+          # group.stub(:update).and_return([])
+          @groups << group
+        }
+        @commands.agent.stub(:groups).and_return(@groups)
+
+        # curl -X POST "https://cloud.unit-hosting.com/xmlrpc"
+        xml = <<XML
+<?xml version="1.0" encoding="UTF-8"?>
+ <methodResponse>
+  <params>
+   <param><value><array><data>
+    <value><struct>
+     <member><name>instance_id</name><value><string>test-vm-1280</string></value></member>
+     <member><name>display_name</name><value><string>test-server-1</string></value></member>
+     <member><name>api_key</name><value><string>dummy-api-key-00398342190234</string></value></member>
+     <member><name>status</name><value><string>halted</string></value></member>
+    </struct></value>
+    <value><struct>
+     <member><name>instance_id</name><value><string>test-vm-1645</string></value></member>
+     <member><name>display_name</name><value><string>test-server-2</string></value></member>
+     <member><name>api_key</name><value><string/></value></member>
+     <member><name>status</name><value><string>halted</string></value></member>
+    </struct></value>
+   </data></array></value></param>
+  </params>
+ </methodResponse>
+XML
+        stub_request(:post, "https://cloud.unit-hosting.com/xmlrpc").
+        to_return(:status =>200, :body => xml, :headers => {:content_type =>'text/xml; charset=utf-8'})
+      }
+      it "updates all cache" do
+        @commands.update(true)
+      end
+    end
+    context "when --all is not set" do
+      before {
+        @group = UnitHosting::Group.new("test-sg-9")
+        @group.key = "dummy-key-9"
+        @commands.stub(:ask_group).and_return(@group)
+      }
+      it "updates cache of specified group" do
+        @group.should_receive(:update)
+        @commands.update
+      end
+    end
+  end
+
+  describe "#start" do
+    context "when still logged in" do
+      before{
+        @commands.agent.stub(:login?).and_return(true)
+      }
+      it "returns true" do
+        expect(@commands.send(:start)).to be_true
+      end
+    end
+    context "failed login" do
+      before{
+        @commands.agent.stub(:login?).and_return(false)
+        @commands.agent.stub(:login).and_return()
+        expect(@commands).to receive(:login)
+      }
+      it "raises LoginError exception." do
+        expect{
+          @commands.send(:start)
+        }.to raise_error(UnitHosting::Commands::LoginError)
+      end
+    end
+    context "when not logged in" do
+      before{
+        @commands.agent.stub(:login?).and_return(false,true)
+      }
+      context "when there is a user-entry in keystorage" do
+        before {
+          Keystorage.stub(:list).and_return(["test-user-1"])
+          Keystorage.stub(:get).with(@commands.keyname,"test-user-1").and_return("test-password-1")
+        }
+        it "try to login by using username" do
+          expect(@commands.agent).to receive(:login).with("test-user-1","test-password-1").once
+          @commands.send(:start)
+        end
+      end
+      context "when there is no user-entry in keystorage" do
+        before {
+          Keystorage.stub(:list).and_return([])
+        }
+        it "asks username" do
+          expect(@commands).to receive(:login)
+          @commands.send(:start)
+        end
+      end
+      #it "returns true" do
+      #   expect(@commands.send(:start)).to be_true
+      #end
+    end
+  end
+  describe "#ask_group" do
+    before{
+      @groups = ['test-sg-1','test-sg-2','test-sg-3','test-123'].extend(UnitHosting::Groups)
+      @commands.stub(:ask) { |question, &details|
+        HighLine::Question.new(question, String, &details)
+        "test-sg-2"
+      }
+    }
+    context "when group_id is not given" do
+      it "asks group_id" do
+        groups = [UnitHosting::Group.new('test-sg-1'),
+                  UnitHosting::Group.new('test-sg-2'),
+                  UnitHosting::Group.new('test-sg-3')].extend(UnitHosting::Groups)
+        expect(capture(:stdout) {
+                 expect(@commands.send(:ask_group,nil,groups).instance_id).to eq "test-sg-2"
+                 }).to match /test\-sg\-3/
+
+      end
+    end
+  end
+  describe "#cache_file" do
+    it "returns cache file path" do
+      expect(@commands.send(:cache_file,"foo.cache")).to match /foo\.cache$/
+    end
+  end
+  
+end

data/spec/unit-hosting/group_spec.rb

@@ -0,0 +1,64 @@
+# -*- coding: utf-8 -*-
+require 'spec_helper'
+require 'unit-hosting/group'
+
+describe UnitHosting::Groups do
+  before {
+    @groups = [
+               UnitHosting::Group.new("test-sg-123"),
+               UnitHosting::Group.new("test-sg-789"),
+              ].extend(UnitHosting::Groups)
+  }
+  describe "#ids" do
+    it "returns array of instance_id" do
+     expect(@groups.ids).to eq ["test-sg-123","test-sg-789"]
+    end
+  end
+  describe "#tablize" do
+    context "no groups" do
+      before {
+        @groups = [].extend(UnitHosting::Groups)
+      }
+      it "returns 'no groups'" do
+        expect(@groups.tablize).to match /no groups/
+      end
+    end
+    context "have 2 groups" do
+      it "returns table string" do
+        expect(@groups.tablize).to match /test-sg-789/
+      end
+    end    
+  end
+end
+
+describe UnitHosting::Group do
+  before {
+    @group = UnitHosting::Group.new("test-sg-123")
+  }
+  describe "#tablize" do
+    context "no vms" do
+      it "returns 'no vms'" do
+        expect(@group.tablize).to match /no vms/
+      end
+    end
+    context "have 2 vms" do
+      before{
+        @group.vms = [
+               {
+                 "instance_id" => "test-vm-1",
+                 "status" =>"halted",
+                 "display_name" =>"test-server-1"
+               },
+               {
+                 "instance_id" => "test-vm-2",
+                 "status" =>"halted",
+                 "display_name" =>"test-server-2"
+               }
+              ]
+      }
+      it "returns table string" do
+        expect(@group.tablize).to match /test-server-2/
+      end
+    end
+  end
+end

data/unit-hosting.gemspec

@@ -2,16 +2,16 @@
 # DO NOT EDIT THIS FILE DIRECTLY
 # Instead, edit Jeweler::Tasks in Rakefile, and run 'rake gemspec'
 # -*- encoding: utf-8 -*-
-# stub: unit-hosting 0.3.1 ruby lib
+# stub: unit-hosting 0.4.0 ruby lib
 
 Gem::Specification.new do |s|
   s.name = "unit-hosting"
-  s.version = "0.3.1"
+  s.version = "0.4.0"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.require_paths = ["lib"]
   s.authors = ["Yoshihiro TAKAHARA"]
-  s.date = "2014-02-02"
+  s.date = "2014-02-08"
   s.description = "This is a command to manage virtual servers on UnitHosting(http://www.unit-hosting.com)."
   s.email = "y.takahara@gmail.com"
   s.executables = ["unit-hosting"]
@@ -20,6 +20,7 @@ Gem::Specification.new do |s|
     "README.md"
   ]
   s.files = [
+    ".coveralls.yml",
     ".document",
     ".travis.yml",
     "Gemfile",
@@ -39,8 +40,16 @@ Gem::Specification.new do |s|
     "lib/unit-hosting/cli.rb",
     "lib/unit-hosting/commands.rb",
     "lib/unit-hosting/group.rb",
-    "test/helper.rb",
-    "test/test_unit-hosting.rb",
+    "spec/spec_helper.rb",
+    "spec/unit-hosting/agent_spec.rb",
+    "spec/unit-hosting/api/base_spec.rb",
+    "spec/unit-hosting/api/vm_group_spec.rb",
+    "spec/unit-hosting/api/vm_recipe_spec.rb",
+    "spec/unit-hosting/api/vm_spec.rb",
+    "spec/unit-hosting/cache_spec.rb",
+    "spec/unit-hosting/cli_spec.rb",
+    "spec/unit-hosting/commands_spec.rb",
+    "spec/unit-hosting/group_spec.rb",
     "unit-hosting.gemspec"
   ]
   s.homepage = "http://github.com/tumf/unit-hosting"
@@ -54,34 +63,58 @@ Gem::Specification.new do |s|
     if Gem::Version.new(Gem::VERSION) >= Gem::Version.new('1.2.0') then
       s.add_runtime_dependency(%q<mutter>, [">= 0"])
       s.add_runtime_dependency(%q<keystorage>, ["~> 0.4.13"])
-      s.add_runtime_dependency(%q<mechanize>, [">= 0"])
       s.add_runtime_dependency(%q<highline>, ["> 1.6"])
       s.add_runtime_dependency(%q<progressbar>, [">= 0.9.0"])
       s.add_runtime_dependency(%q<httpclient>, [">= 2.1.6.1"])
       s.add_runtime_dependency(%q<command-line-utils>, [">= 0.0.1"])
-      s.add_development_dependency(%q<bundler>, [">= 0"])
+      s.add_runtime_dependency(%q<active_support>, [">= 0"])
+      s.add_runtime_dependency(%q<i18n>, [">= 0"])
+      s.add_runtime_dependency(%q<nokogiri>, ["~> 1.5.9"])
+      s.add_runtime_dependency(%q<mechanize>, ["= 2.6.0"])
       s.add_development_dependency(%q<jeweler>, [">= 0"])
+      s.add_development_dependency(%q<simplecov>, [">= 0"])
+      s.add_development_dependency(%q<coveralls>, [">= 0"])
+      s.add_development_dependency(%q<bundler>, [">= 0"])
+      s.add_development_dependency(%q<rake>, [">= 0"])
+      s.add_development_dependency(%q<rspec>, [">= 0"])
+      s.add_development_dependency(%q<webmock>, [">= 0"])
     else
       s.add_dependency(%q<mutter>, [">= 0"])
       s.add_dependency(%q<keystorage>, ["~> 0.4.13"])
-      s.add_dependency(%q<mechanize>, [">= 0"])
       s.add_dependency(%q<highline>, ["> 1.6"])
       s.add_dependency(%q<progressbar>, [">= 0.9.0"])
       s.add_dependency(%q<httpclient>, [">= 2.1.6.1"])
       s.add_dependency(%q<command-line-utils>, [">= 0.0.1"])
-      s.add_dependency(%q<bundler>, [">= 0"])
+      s.add_dependency(%q<active_support>, [">= 0"])
+      s.add_dependency(%q<i18n>, [">= 0"])
+      s.add_dependency(%q<nokogiri>, ["~> 1.5.9"])
+      s.add_dependency(%q<mechanize>, ["= 2.6.0"])
       s.add_dependency(%q<jeweler>, [">= 0"])
+      s.add_dependency(%q<simplecov>, [">= 0"])
+      s.add_dependency(%q<coveralls>, [">= 0"])
+      s.add_dependency(%q<bundler>, [">= 0"])
+      s.add_dependency(%q<rake>, [">= 0"])
+      s.add_dependency(%q<rspec>, [">= 0"])
+      s.add_dependency(%q<webmock>, [">= 0"])
     end
   else
     s.add_dependency(%q<mutter>, [">= 0"])
     s.add_dependency(%q<keystorage>, ["~> 0.4.13"])
-    s.add_dependency(%q<mechanize>, [">= 0"])
     s.add_dependency(%q<highline>, ["> 1.6"])
     s.add_dependency(%q<progressbar>, [">= 0.9.0"])
     s.add_dependency(%q<httpclient>, [">= 2.1.6.1"])
     s.add_dependency(%q<command-line-utils>, [">= 0.0.1"])
-    s.add_dependency(%q<bundler>, [">= 0"])
+    s.add_dependency(%q<active_support>, [">= 0"])
+    s.add_dependency(%q<i18n>, [">= 0"])
+    s.add_dependency(%q<nokogiri>, ["~> 1.5.9"])
+    s.add_dependency(%q<mechanize>, ["= 2.6.0"])
     s.add_dependency(%q<jeweler>, [">= 0"])
+    s.add_dependency(%q<simplecov>, [">= 0"])
+    s.add_dependency(%q<coveralls>, [">= 0"])
+    s.add_dependency(%q<bundler>, [">= 0"])
+    s.add_dependency(%q<rake>, [">= 0"])
+    s.add_dependency(%q<rspec>, [">= 0"])
+    s.add_dependency(%q<webmock>, [">= 0"])
   end
 end
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: unit-hosting
 version: !ruby/object:Gem::Version
-  version: 0.3.1
+  version: 0.4.0
 platform: ruby
 authors:
 - Yoshihiro TAKAHARA
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-02-02 00:00:00.000000000 Z
+date: 2014-02-08 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: mutter
@@ -38,20 +38,6 @@ dependencies:
     - - ~>
       - !ruby/object:Gem::Version
         version: 0.4.13
-- !ruby/object:Gem::Dependency
-  name: mechanize
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - '>='
-      - !ruby/object:Gem::Version
-        version: '0'
-  type: :runtime
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - '>='
-      - !ruby/object:Gem::Version
-        version: '0'
 - !ruby/object:Gem::Dependency
   name: highline
   requirement: !ruby/object:Gem::Requirement
@@ -108,6 +94,104 @@ dependencies:
     - - '>='
       - !ruby/object:Gem::Version
         version: 0.0.1
+- !ruby/object:Gem::Dependency
+  name: active_support
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: i18n
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: nokogiri
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: 1.5.9
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: 1.5.9
+- !ruby/object:Gem::Dependency
+  name: mechanize
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - '='
+      - !ruby/object:Gem::Version
+        version: 2.6.0
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - '='
+      - !ruby/object:Gem::Version
+        version: 2.6.0
+- !ruby/object:Gem::Dependency
+  name: jeweler
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: simplecov
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: coveralls
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - '>='
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
@@ -123,7 +207,35 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: jeweler
+  name: rake
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: rspec
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: webmock
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - '>='
@@ -145,6 +257,7 @@ extra_rdoc_files:
 - LICENSE.txt
 - README.md
 files:
+- .coveralls.yml
 - .document
 - .travis.yml
 - Gemfile
@@ -164,8 +277,16 @@ files:
 - lib/unit-hosting/cli.rb
 - lib/unit-hosting/commands.rb
 - lib/unit-hosting/group.rb
-- test/helper.rb
-- test/test_unit-hosting.rb
+- spec/spec_helper.rb
+- spec/unit-hosting/agent_spec.rb
+- spec/unit-hosting/api/base_spec.rb
+- spec/unit-hosting/api/vm_group_spec.rb
+- spec/unit-hosting/api/vm_recipe_spec.rb
+- spec/unit-hosting/api/vm_spec.rb
+- spec/unit-hosting/cache_spec.rb
+- spec/unit-hosting/cli_spec.rb
+- spec/unit-hosting/commands_spec.rb
+- spec/unit-hosting/group_spec.rb
 - unit-hosting.gemspec
 homepage: http://github.com/tumf/unit-hosting
 licenses: