vpm 0.0.1 → 0.0.2

data/spec/tasks/create_directory_spec.rb

@@ -0,0 +1,33 @@
+require 'spec_helper'
+
+describe CreateDirectory do
+  before do
+    @dir  = File.expand_path "temp"
+    @task = CreateDirectory.new @dir
+  end
+
+  context "#perform" do
+    it "creates the directory if it doesn't already exist" do
+      Dir.exists?(@dir).should be_false
+      @task.perform
+      Dir.exists?(@dir).should be_true
+    end
+
+    it "works even if the directory already exists" do
+      Dir.exists?(@dir).should be_true
+      @task.perform
+      Dir.exists?(@dir).should be_true
+    end
+  end
+
+  context "#undo" do
+    it "removes #{@dir} if this task created it" do
+      FileUtils.rmdir @dir
+      Dir.exists?(@dir).should be_false
+      @task.perform
+      Dir.exists?(@dir).should be_true
+      @task.undo
+      Dir.exists?(@dir).should be_false
+    end
+  end
+end

data/spec/tasks/create_our_rc_file_spec.rb

@@ -0,0 +1,31 @@
+require 'spec_helper'
+
+describe CreateOurRCFile do
+  before do
+    @task = CreateOurRCFile.new
+  end
+
+  context "#perform" do
+    it "creates the file if it doesn't exist" do
+      File.exists?(VPM.vpmrc_path).should be_false
+      @task.perform
+      File.exists?(VPM.vpmrc_path).should be_true
+    end
+
+    it "inserts the call to pathogen#infect into the file" do
+      @task.perform
+      File.open(VPM.vpmrc_path, "r") do |file|
+        file.read.should include "call pathogen#infect()"
+      end
+    end
+  end
+
+  context "#undo" do
+    it "removes the rc file" do
+      @task.perform
+      File.exists?(VPM.vpmrc_path).should be_true
+      @task.undo
+      File.exists?(VPM.vpmrc_path).should be_false
+    end
+  end
+end

data/spec/tasks/inject_our_load_script_spec.rb

@@ -0,0 +1,39 @@
+require 'spec_helper'
+
+describe InjectOurLoadScript do
+  context "#perform" do
+    it "sources our file at the top of the user's vimrc" do
+      Dir.mktmpdir do |temp_dir|
+        @task = InjectOurLoadScript.new(temp_dir)
+        FileUtils.touch File.expand_path(VPM.vimrc_path)
+        File.read(VPM.vimrc_path).should_not include "source #{VPM.vpmrc_path}"
+        @task.perform
+        File.read(VPM.vimrc_path).start_with?("source #{VPM.vpmrc_path}").should be_true
+      end
+    end
+
+    it "retains exactly the same text as the old file" do
+      Dir.mktmpdir do |temp_dir|
+        @task = InjectOurLoadScript.new(temp_dir)
+        File.open(VPM.vimrc_path, "w") {|file| file.puts "test\nother"}
+        File.read(VPM.vimrc_path).should_not include "source #{VPM.vpmrc_path}"
+        @task.perform
+        File.read(VPM.vimrc_path).should == "source #{VPM.vpmrc_path}\ntest\nother\n"
+      end
+    end
+  end
+
+  context "#undo" do
+    it "replaces the old file" do
+      Dir.mktmpdir do |temp_dir|
+        @task = InjectOurLoadScript.new(temp_dir)
+        File.open(VPM.vimrc_path, "w") {|file| file.print "test\nother"}
+        File.read(VPM.vimrc_path).should_not include "source #{VPM.vpmrc_path}"
+        @task.perform
+        File.read(VPM.vimrc_path).should == "source #{VPM.vpmrc_path}\ntest\nother\n"
+        @task.undo
+        File.read(VPM.vimrc_path).should == "test\nother"
+      end
+    end
+  end
+end

data/spec/tasks/move_pathogen_file_spec.rb

@@ -0,0 +1,42 @@
+require 'spec_helper'
+
+describe MovePathogenFile do
+  before do
+    @temp_dir         = File.expand_path "temp"
+    @source_path      = File.join(@temp_dir, "pathogen", "autoload", "pathogen.vim")
+    FileUtils.mkdir_p File.join(@temp_dir, "pathogen", "autoload")
+    FileUtils.touch   @source_path
+    @autoload_path    = File.join(VPM.vim_dir_path, "autoload")
+    @destination_path = File.join(@autoload_path, "pathogen.vim")
+    FileUtils.mkdir_p @autoload_path
+    @task             = MovePathogenFile.new @temp_dir
+  end
+
+  context "#perform" do
+    it "moves the pathogen.vim file from its current location to vim's autoload directory" do
+      File.exists?(@destination_path).should be_false
+      @task.perform
+      File.exists?(@destination_path).should be_true
+    end
+  end
+
+  context "#undo" do
+    it "removes pathogen.vim if the task installed it" do
+      FileUtils.rm @destination_path
+      File.exists?(@destination_path).should be_false
+      @task.perform
+      File.exists?(@destination_path).should be_true
+      @task.undo
+      File.exists?(@destination_path).should be_false
+    end
+
+    it "does not remove the file if it already existed" do
+      FileUtils.touch @destination_path
+      File.exists?(@destination_path).should be_true
+      @task.perform
+      File.exists?(@destination_path).should be_true
+      @task.undo
+      File.exists?(@destination_path).should be_true
+    end
+  end
+end

data/spec/transaction_spec.rb

@@ -0,0 +1,65 @@
+require 'spec_helper'
+
+describe Transaction do
+  context "#perform" do
+    before do
+      @transaction = Transaction.new
+      @task_one = double "Task One"
+      @task_two = double "Task Two"
+      @transaction << @task_one
+      @transaction << @task_two
+    end
+
+    it 'runs each task when the transaction is peformed' do
+      @task_one.should_receive("perform").and_return true
+      @task_two.should_receive("perform").and_return true
+      @transaction.perform
+    end
+
+    it 'undoes every task that has been run so far if any of them return false' do
+      @task_one.should_receive("perform").and_return true
+      @task_one.should_receive "undo"
+      @task_two.should_receive("perform").and_return false
+      @task_two.should_receive "undo"
+      @transaction.perform
+    end
+
+    it 'skips any tasks following a failing one' do
+      @task_one.should_receive("perform").and_return false
+      @task_one.should_receive "undo"
+      @task_two.should_not_receive "peform"
+      @transaction.perform
+    end
+
+    it 'undoes the tasks in the opposite order they were performed' do
+      @transaction = Transaction.new
+      result = []
+      @task_one = Object.new
+      @task_two = Object.new
+      @task_one.stub(:perform).and_return true
+      @task_one.stub(:undo) do result << "task one" end
+      @task_two.stub(:perform).and_return false
+      @task_two.stub(:undo) do result << "task two" end
+      @transaction << @task_one
+      @transaction << @task_two
+      @transaction.perform
+      result.should == ["task two", "task one"]
+    end
+
+    it 'undoes all of the tasks if one raises an execption' do
+      @task_one.should_receive("perform").and_return true
+      @task_one.should_receive "undo"
+      @task_two.should_receive("perform") { raise "an error" }
+      @task_two.should_receive "undo"
+      expect { @transaction.perform }.to raise_error
+    end
+
+    it 'rethrows exceptions raised by a task' do
+      @task_one.should_receive("perform").and_return true
+      @task_one.should_receive "undo"
+      @task_two.should_receive("perform") { raise "an error" }
+      @task_two.should_receive "undo"
+      expect { @transaction.perform }.to raise_error
+    end
+  end
+end

data/vpm.gemspec

@@ -5,10 +5,11 @@ require "vpm/version"
 Gem::Specification.new do |s|
   s.name        = "vpm"
   s.version     = VPM::VERSION
-  s.authors     = ["Jingwen Owen Ou"]
+  s.authors     = ["Jingwen Owen Ou", "Michael Baker"]
   s.email       = ["jingweno@gmail.com"]
-  s.homepage    = ""
-  s.summary     = %q{An awesome utility for managing VIM plugins}
+  s.homepage    = "https://github.com/jingweno/vpm"
+  s.summary     = %q{An awesome utility for managing Vim plugins}
+  s.description = %q{An awesome utility for managing Vim plugins}
 
   s.rubyforge_project = "vpm"
 
@@ -19,6 +20,7 @@ Gem::Specification.new do |s|
 
   # specify any dependencies here; for example:
   s.add_development_dependency "rake"
-  s.add_development_dependency "turn"
+  s.add_development_dependency "rspec"
+  s.add_development_dependency "fakefs"
   # s.add_runtime_dependency "rest-client"
 end

metadata

@@ -1,19 +1,20 @@
 --- !ruby/object:Gem::Specification
 name: vpm
 version: !ruby/object:Gem::Version
-  version: 0.0.1
+  version: 0.0.2
   prerelease: 
 platform: ruby
 authors:
 - Jingwen Owen Ou
+- Michael Baker
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-01-14 00:00:00.000000000 Z
+date: 2012-01-21 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake
-  requirement: &70192619467680 !ruby/object:Gem::Requirement
+  requirement: &2154096600 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -21,10 +22,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70192619467680
+  version_requirements: *2154096600
 - !ruby/object:Gem::Dependency
-  name: turn
-  requirement: &70192619467260 !ruby/object:Gem::Requirement
+  name: rspec
+  requirement: &2154096180 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -32,8 +33,19 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70192619467260
-description: 
+  version_requirements: *2154096180
+- !ruby/object:Gem::Dependency
+  name: fakefs
+  requirement: &2154095760 !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: *2154095760
+description: An awesome utility for managing Vim plugins
 email:
 - jingweno@gmail.com
 executables:
@@ -42,22 +54,50 @@ extensions: []
 extra_rdoc_files: []
 files:
 - .gitignore
+- .rvmrc
 - Gemfile
 - Gemfile.lock
 - README.md
 - Rakefile
 - bin/vpm
+- lib/transaction.rb
 - lib/vpm.rb
-- lib/vpm/git_plugin.rb
+- lib/vpm/command_options.rb
+- lib/vpm/command_options/abstract_command_options.rb
+- lib/vpm/command_options/install.rb
+- lib/vpm/command_options/list.rb
+- lib/vpm/commands.rb
+- lib/vpm/commands/install.rb
+- lib/vpm/commands/install_all_plugins.rb
+- lib/vpm/commands/invalid_command.rb
+- lib/vpm/commands/list.rb
+- lib/vpm/commands/setup.rb
+- lib/vpm/git.rb
 - lib/vpm/manifest_parser.rb
 - lib/vpm/plugin.rb
+- lib/vpm/plugins.rb
+- lib/vpm/runner.rb
+- lib/vpm/tasks.rb
+- lib/vpm/tasks/clone_pathogen.rb
+- lib/vpm/tasks/create_autoload_dir.rb
+- lib/vpm/tasks/create_bundle_dir.rb
+- lib/vpm/tasks/create_directory.rb
+- lib/vpm/tasks/create_our_rc_file.rb
+- lib/vpm/tasks/inject_our_load_script.rb
+- lib/vpm/tasks/move_pathogen_file.rb
 - lib/vpm/version.rb
-- spec/git_plugin_spec.rb
+- spec/commands/install_spec.rb
 - spec/manifest_parser_spec.rb
+- spec/plugins_spec.rb
 - spec/spec_helper.rb
-- spec/vpm_test.rb
+- spec/tasks/create_bundle_dir_spec.rb
+- spec/tasks/create_directory_spec.rb
+- spec/tasks/create_our_rc_file_spec.rb
+- spec/tasks/inject_our_load_script_spec.rb
+- spec/tasks/move_pathogen_file_spec.rb
+- spec/transaction_spec.rb
 - vpm.gemspec
-homepage: ''
+homepage: https://github.com/jingweno/vpm
 licenses: []
 post_install_message: 
 rdoc_options: []
@@ -71,7 +111,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 1524590733277353739
+      hash: -1969408185855622509
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
@@ -80,15 +120,21 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 1524590733277353739
+      hash: -1969408185855622509
 requirements: []
 rubyforge_project: vpm
 rubygems_version: 1.8.10
 signing_key: 
 specification_version: 3
-summary: An awesome utility for managing VIM plugins
+summary: An awesome utility for managing Vim plugins
 test_files:
-- spec/git_plugin_spec.rb
+- spec/commands/install_spec.rb
 - spec/manifest_parser_spec.rb
+- spec/plugins_spec.rb
 - spec/spec_helper.rb
-- spec/vpm_test.rb
+- spec/tasks/create_bundle_dir_spec.rb
+- spec/tasks/create_directory_spec.rb
+- spec/tasks/create_our_rc_file_spec.rb
+- spec/tasks/inject_our_load_script_spec.rb
+- spec/tasks/move_pathogen_file_spec.rb
+- spec/transaction_spec.rb

data/lib/vpm/git_plugin.rb

@@ -1,22 +0,0 @@
-module VPM
-  class GitPlugin < Plugin
-    attr_accessor :git_commander
-
-    def initialize(name, options)
-      super
-      @git_commander = GitCommander
-    end
-
-    def install
-      @git_commander.clone(options[:git], VPM.plugin_dir)
-    end
-
-    module GitCommander
-      def self.clone(repo, destination)
-        Dir.chdir destination do
-          `git clone #{repo}`
-        end
-      end
-    end
-  end
-end

data/spec/git_plugin_spec.rb

@@ -1,12 +0,0 @@
-require 'spec_helper'
-
-describe VPM::GitPlugin do
-  it "intalls the plugin if it hasn't" do
-    mock_git_command = MiniTest::Mock.new
-    mock_git_command.expect(:clone, true, ["https://xxx.git", "#{ENV['HOME']}/.vim/bundle"])
-    plugin = VPM::GitPlugin.new("command-t", :git => "https://xxx.git")
-    plugin.git_commander = mock_git_command
-    plugin.install
-    mock_git_command.verify
-  end
-end

data/spec/vpm_test.rb

@@ -1,13 +0,0 @@
-require 'test_helper'
-
-describe VPM do
-  it "parses install command with args" do
-    parser = MiniTest::Mock.new
-
-    VPM.run(['intstall', 'xxx.git'])
-    # ['<command'>, args]
-    # InstallCommand.run(args)
-    # RemoveCommadn.run(args)
-    VPM.parse
-  end
-end