rush 0.1 → 0.2

data/spec/local_spec.rb

@@ -27,6 +27,11 @@ describe Rush::Connection::Local do
 		@con.receive(:action => 'destroy', :full_path => 'file')
 	end
 
+	it "receive -> purge(dir)" do
+		@con.should_receive(:purge).with('dir')
+		@con.receive(:action => 'purge', :full_path => 'dir')
+	end
+
 	it "receive -> create_dir(path)" do
 		@con.should_receive(:create_dir).with('dir')
 		@con.receive(:action => 'create_dir', :full_path => 'dir')
@@ -79,7 +84,12 @@ describe Rush::Connection::Local do
 
 	it "receive -> kill_process" do
 		@con.should_receive(:kill_process).with(123).and_return(true)
-		@con.receive(:action => 'kill_process', :pid => 123)
+		@con.receive(:action => 'kill_process', :pid => '123')
+	end
+
+	it "receive -> bash" do
+		@con.should_receive(:bash).with('cmd').and_return('output')
+		@con.receive(:action => 'bash', :payload => 'cmd').should == 'output'
 	end
 
 	it "receive -> unknown action exception" do
@@ -99,6 +109,11 @@ describe Rush::Connection::Local do
 		@con.file_contents(fname).should == "stuff\n"
 	end
 
+	it "file_contents raises DoesNotExist if the file does not exist" do
+		fname = "#{@sandbox_dir}/does_not_exist"
+		lambda { @con.file_contents(fname) }.should raise_error(Rush::DoesNotExist, fname)
+	end
+
 	it "destroy to destroy a file or dir" do
 		fname = "#{@sandbox_dir}/delete_me"
 		system "touch #{fname}"
@@ -106,6 +121,20 @@ describe Rush::Connection::Local do
 		File.exists?(fname).should be_false
 	end
 
+	it "purge to purge a dir" do
+		system "cd #{@sandbox_dir}; touch {1,2}; mkdir 3; touch 3/4"
+		@con.purge(@sandbox_dir)
+		File.exists?(@sandbox_dir).should be_true
+		Dir.glob("#{@sandbox_dir}/*").should == []
+	end
+
+	it "purge kills hidden (dotfile) entries too" do
+		system "cd #{@sandbox_dir}; touch .killme"
+		@con.purge(@sandbox_dir)
+		File.exists?(@sandbox_dir).should be_true
+		`cd #{@sandbox_dir}; ls -lA | wc -l`.to_i.should == 0
+	end
+
 	it "create_dir creates a directory" do
 		fname = "#{@sandbox_dir}/a/b/c/"
 		@con.create_dir(fname)
@@ -127,6 +156,14 @@ describe Rush::Connection::Local do
 		File.exists?("#{@sandbox_dir}/subdir/a").should be_true
 	end
 
+	it "copy raises DoesNotExist with source path if it doesn't exist or otherwise can't be accessed" do
+		lambda { @con.copy('/does/not/exist', '/tmp') }.should raise_error(Rush::DoesNotExist, '/does/not/exist')
+	end
+
+	it "copy raises DoesNotExist with destination path if it can't access the destination" do
+		lambda { @con.copy('/tmp', '/does/not/exist') }.should raise_error(Rush::DoesNotExist, '/does/not')
+	end
+
 	it "read_archive to pull an archive of a dir into a byte stream" do
 		system "touch #{@sandbox_dir}/a"
 		@con.read_archive(@sandbox_dir).size.should > 50
@@ -150,11 +187,20 @@ describe Rush::Connection::Local do
 		@con.index(@sandbox_dir, '*.rb').should == [ 'a.rb' ]
 	end
 
+	it "index raises DoesNotExist when the base path is invalid" do
+		lambda { @con.index('/does/not/exist', '*') }.should raise_error(Rush::DoesNotExist, '/does/not/exist')
+	end
+
 	it "stat gives file stats like size and timestamps" do
 		@con.stat(@sandbox_dir).should have_key(:ctime)
 		@con.stat(@sandbox_dir).should have_key(:size)
 	end
 
+	it "stat raises DoesNotExist if the entry does not exist" do
+		fname = "#{@sandbox_dir}/does_not_exist"
+		lambda { @con.stat(fname) }.should raise_error(Rush::DoesNotExist, fname)
+	end
+
 	if !RUBY_PLATFORM.match(/darwin/)   # doesn't work on OS X 'cause du switches are different
 		it "size gives size of a directory and all its contents recursively" do
 			system "mkdir -p #{@sandbox_dir}/a/b/; echo 1234 > #{@sandbox_dir}/a/b/c"
@@ -163,11 +209,12 @@ describe Rush::Connection::Local do
 	end
 
 	it "parses ps output on os x" do
-		@con.parse_ps("21712   501   1236   0 /usr/bin/vi somefile.rb").should == {
+		@con.parse_ps("21712   501   21711   1236   0 /usr/bin/vi somefile.rb").should == {
 			:pid => "21712",
 			:uid => "501",
-			:rss => "1236",
-			:cpu => "0",
+			:parent_pid => 21711,
+			:mem => 1236,
+			:cpu => 0,
 			:command => '/usr/bin/vi',
 			:cmdline => '/usr/bin/vi somefile.rb',
 		}
@@ -175,22 +222,42 @@ describe Rush::Connection::Local do
 
 	it "gets the list of processes on os x via the ps command" do
 		@con.should_receive(:os_x_raw_ps).and_return <<EOPS
-PID UID   RSS  CPU COMMAND
-1     0   1111   0 cmd1 args
-2   501    222   1 cmd2
+PID UID   PPID  RSS  CPU COMMAND
+1     0      1 1111   0 cmd1 args
+2   501      1  222   1 cmd2
 EOPS
 		@con.os_x_processes.should == [
-			{ :pid => "1", :uid => "0", :rss => "1111", :cpu => "0", :command => "cmd1", :cmdline => "cmd1 args" },
-			{ :pid => "2", :uid => "501", :rss => "222", :cpu => "1", :command => "cmd2", :cmdline => "cmd2" },
+			{ :pid => "1", :uid => "0", :parent_pid => 1, :mem => 1111, :cpu => 0, :command => "cmd1", :cmdline => "cmd1 args" },
+			{ :pid => "2", :uid => "501", :parent_pid => 1, :mem => 222, :cpu => 1, :command => "cmd2", :cmdline => "cmd2" },
 		]
 	end
 
-	it "checks whether a given process is alive by pid" do
-		@con.process_alive(Process.pid).should == true
+	it "the current process should be alive" do
+		@con.process_alive(Process.pid).should be_true
+	end
+
+	it "a made-up process should not be alive" do
+		@con.process_alive(99999).should be_false
 	end
 
 	it "kills a process by pid" do
-		::Process.should_receive(:kill).with('TERM', 123)
-		@con.kill_process('123')
+		::Process.should_receive(:kill).at_least(:once)
+		@con.kill_process(123)
+	end
+
+	it "executes a bash command, returning stdout when successful" do
+		@con.bash("echo test").should == "test\n"
+	end
+
+	it "executes a bash command, raising and error (with stderr as the message) when return value is nonzero" do
+		lambda { @con.bash("no_such_bin") }.should raise_error(Rush::BashFailed, /command not found/)
+	end
+
+	it "ensure_tunnel to match with remote connection" do
+		@con.ensure_tunnel
+	end
+
+	it "always returns true on alive?" do
+		@con.should be_alive
 	end
 end

data/spec/process_spec.rb

@@ -12,6 +12,13 @@ describe Rush::Process do
 		system "kill -9 #{@pid}"
 	end
 
+	if !RUBY_PLATFORM.match(/darwin/)   # OS x reports pids weird
+		it "knows all its child processes" do
+			parent = Rush::Process.all.detect { |p| p.pid == Process.pid }
+			parent.children.should == [ @process ]
+		end
+	end
+
 	it "gets the list of all processes" do
 		list = Rush::Process.all
 		list.size.should > 5
@@ -34,11 +41,33 @@ describe Rush::Process do
 		@process.cmdline.should match(/process_spec.rb/)
 	end
 
+	it "knows the memory used" do
+		@process.mem.should > 0
+	end
+
+	it "knows the cpu used" do
+		@process.cpu.should >= 0
+	end
+
+	it "knows the parent process pid" do
+		@process.parent_pid.should == Process.pid
+	end
+
+	it "knows the parent process" do
+		this = Rush::Box.new.processes.select { |p| p.pid == Process.pid }.first
+		@process.parent.should == this
+	end
+
 	it "can kill itself" do
 		system "sleep 30 &"
-		@process = Rush::Process.all.detect { |p| p.command == "sleep" }
-		@process.kill
+		process = Rush::Process.all.detect { |p| p.command == "sleep" }
+		process.kill
 		sleep 0.1
-		@process.alive?.should be_false
+		process.alive?.should be_false
+	end
+
+	it "if box and pid are the same, process is equal" do
+		other = Rush::Process.new({ :pid => @process.pid }, @process.box)
+		@process.should == other
 	end
 end

data/spec/remote_spec.rb

@@ -27,6 +27,11 @@ describe Rush::Connection::Local do
 		@con.destroy('file')
 	end
 
+	it "transmits purge" do
+		@con.should_receive(:transmit).with(:action => 'purge', :full_path => 'dir')
+		@con.purge('dir')
+	end
+
 	it "transmits create_dir" do
 		@con.should_receive(:transmit).with(:action => 'create_dir', :full_path => 'file')
 		@con.create_dir('file')
@@ -81,4 +86,45 @@ describe Rush::Connection::Local do
 		@con.should_receive(:transmit).with(:action => 'kill_process', :pid => 123)
 		@con.kill_process(123)
 	end
+
+	it "transmits bash" do
+		@con.should_receive(:transmit).with(:action => 'bash', :payload => 'cmd').and_return('output')
+		@con.bash('cmd').should == 'output'
+	end
+
+	it "an http result code of 401 raises NotAuthorized" do
+		lambda { @con.process_result("401", "") }.should raise_error(Rush::NotAuthorized)
+	end
+
+	it "an http result code of 400 raises the exception passed in the result body" do
+		@con.stub!(:parse_exception).and_return(Rush::DoesNotExist, "message")
+		lambda { @con.process_result("400", "") }.should raise_error(Rush::DoesNotExist)
+	end
+
+	it "an http result code of 501 (or anything other than the other defined codes) raises FailedTransmit" do
+		lambda { @con.process_result("501", "") }.should raise_error(Rush::FailedTransmit)
+	end
+
+	it "parse_exception takes the class from the first line and the message from the second" do
+		@con.parse_exception("Rush::DoesNotExist\nthe message\n").should == [ Rush::DoesNotExist, "the message" ]
+	end
+
+	it "parse_exception rejects unrecognized exceptions" do
+		lambda { @con.parse_exception("NotARushException\n") }.should raise_error
+	end
+
+	it "passes through ensure_tunnel" do
+		@con.tunnel.should_receive(:ensure_tunnel)
+		@con.ensure_tunnel
+	end
+
+	it "is alive if the box is responding to commands" do
+		@con.should_receive(:index).and_return(:dummy)
+		@con.should be_alive
+	end
+
+	it "not alive if an attempted command throws an exception" do
+		@con.should_receive(:index).and_raise(RuntimeError)
+		@con.should_not be_alive
+	end
 end

data/spec/shell_spec.rb

@@ -7,6 +7,6 @@ describe Rush::Shell do
 	end
 
 	it "matches open path commands for readline tab completion" do
-		@shell.path_parts("dir['app").should == [ "dir", "'", "app" ]
+		@shell.path_parts("dir['app").should == [ "dir", "'", "app", "" ]
 	end
 end

data/spec/ssh_tunnel_spec.rb

@@ -58,19 +58,35 @@ describe Rush::SshTunnel do
 		@tunnel.tunnel_options.should == {
 			:local_port => 1234,
 			:remote_port => 7770,
-			:ssh_host => 'spec.example.com',
-			:stall_command => 'sleep 9000'
+			:ssh_host => 'spec.example.com'
 		}
 	end
 
-	it "constructs the bash ssh command from the options hash" do
+	it "ssh_stall_command uses an infinite loop for :timeout => :infinite" do
+		@tunnel.ssh_stall_command(:timeout => :infinite).should match(/while .* sleep .* done/)
+	end
+
+	it "ssh_stall_command sleeps for the number of seconds given as the :timeout option" do
+		@tunnel.ssh_stall_command(:timeout => 123).should == "sleep 123"
+	end
+
+	it "ssh_stall_command uses the default timeout when no options are given" do
+		@tunnel.ssh_stall_command.should == "sleep 9000"
+	end
+
+	it "constructs the ssh tunnel command (everything but stall) from the options hash" do
 		@tunnel.should_receive(:tunnel_options).at_least(:once).and_return(
 			:local_port => 123,
 			:remote_port => 456,
-			:ssh_host => 'example.com',
-			:stall_command => 'stall'
+			:ssh_host => 'example.com'
 		)
-		@tunnel.ssh_tunnel_command.should == "ssh -f -L 123:127.0.0.1:456 example.com \"stall\""
+		@tunnel.ssh_tunnel_command_without_stall.should == "ssh -f -L 123:127.0.0.1:456 example.com"
+	end
+
+	it "combines the tunnel command without stall and the stall command into the final command" do
+		@tunnel.should_receive(:ssh_tunnel_command_without_stall).and_return('ssh command')
+		@tunnel.should_receive(:ssh_stall_command).and_return('sleep 123')
+		@tunnel.ssh_tunnel_command.should == 'ssh command "sleep 123"'
 	end
 
 	it "ssh_tunnel_command request that the port be set" do

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: rush
 version: !ruby/object:Gem::Version 
-  version: "0.1"
+  version: "0.2"
 platform: ruby
 authors: 
 - Adam Wiggins
@@ -9,10 +9,36 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2008-02-20 00:00:00 -08:00
+date: 2008-03-12 00:00:00 -07:00
 default_executable: 
-dependencies: []
-
+dependencies: 
+- !ruby/object:Gem::Dependency 
+  name: mongrel
+  version_requirement: 
+  version_requirements: !ruby/object:Gem::Requirement 
+    requirements: 
+    - - ">="
+      - !ruby/object:Gem::Version 
+        version: "0"
+    version: 
+- !ruby/object:Gem::Dependency 
+  name: rspec
+  version_requirement: 
+  version_requirements: !ruby/object:Gem::Requirement 
+    requirements: 
+    - - ">="
+      - !ruby/object:Gem::Version 
+        version: "0"
+    version: 
+- !ruby/object:Gem::Dependency 
+  name: session
+  version_requirement: 
+  version_requirements: !ruby/object:Gem::Requirement 
+    requirements: 
+    - - ">="
+      - !ruby/object:Gem::Version 
+        version: "0"
+    version: 
 description: A Ruby replacement for bash+ssh, providing both an interactive shell and a library.  Manage both local and remote unix systems from a single client.
 email: adam@heroku.com
 executables: 
@@ -33,7 +59,9 @@ files:
 - lib/rush/config.rb
 - lib/rush/dir.rb
 - lib/rush/entry.rb
+- lib/rush/exceptions.rb
 - lib/rush/file.rb
+- lib/rush/find_by.rb
 - lib/rush/fixnum_ext.rb
 - lib/rush/head_tail.rb
 - lib/rush/local.rb
@@ -53,6 +81,7 @@ files:
 - spec/dir_spec.rb
 - spec/entry_spec.rb
 - spec/file_spec.rb
+- spec/find_by_spec.rb
 - spec/fixnum_ext_spec.rb
 - spec/local_spec.rb
 - spec/process_spec.rb