rush2 0.7.0

data/spec/entry_spec.rb

@@ -0,0 +1,133 @@
+require_relative 'base'
+
+describe Rush::Entry do
+	before do
+		@sandbox_dir = "/tmp/rush_spec.#{Process.pid}/"
+		system "rm -rf #{@sandbox_dir}; mkdir -p #{@sandbox_dir}"
+
+		@filename = "#{@sandbox_dir}/test_file"
+		system "touch #{@filename}"
+
+		@entry = Rush::Entry.new(@filename)
+	end
+
+	after do
+		system "rm -rf #{@sandbox_dir}"
+	end
+
+	it "knows its name" do
+		@entry.name.should == File.basename(@filename)
+	end
+
+	it "knows its parent dir" do
+		@entry.parent.should be_kind_of(Rush::Dir)
+		@entry.parent.name.should == File.basename(@sandbox_dir)
+		@entry.parent.full_path.should == @sandbox_dir
+	end
+
+	it "cleans its pathname" do
+		Rush::Entry.new('/a//b//c').full_path.should == '/a/b/c'
+		Rush::Entry.new('/1/2/../3').full_path.should == '/1/3'
+	end
+
+	it "knows its changed_at time" do
+		@entry.changed_at.should == File.stat(@filename).ctime
+	end
+
+	it "knows its last_modified time" do
+		@entry.last_modified.should == File.stat(@filename).mtime
+	end
+
+	it "knows its last_accessed time" do
+		@entry.last_accessed.should == File.stat(@filename).atime
+	end
+
+	it "considers itself equal to other instances with the same full path" do
+		Rush::Entry.new('/not/the/same').should_not == @entry
+		Rush::Entry.new(@entry.full_path).should == @entry
+	end
+
+	it "can rename itself" do
+		new_file = "test2"
+
+		@entry.rename(new_file)
+
+		File.exists?(@filename).should be_false
+		File.exists?("#{@sandbox_dir}/#{new_file}").should be_true
+	end
+
+	it "rename returns the renamed file" do
+		@entry.rename('file2').should == @entry.parent['file2']
+	end
+
+	it "can't rename itself if another file already exists with that name" do
+		new_file = "test3"
+		system "touch #{@sandbox_dir}/#{new_file}"
+
+		lambda { @entry.rename(new_file) }.should raise_error(Rush::NameAlreadyExists, /#{new_file}/)
+	end
+
+	it "can't rename itself to something with a slash in it" do
+		lambda { @entry.rename('has/slash') }.should raise_error(Rush::NameCannotContainSlash, /slash/)
+	end
+
+	it "can duplicate itself within the directory" do
+		@entry.duplicate('newfile').should == Rush::File.new("#{@sandbox_dir}/newfile")
+	end
+
+	it "can move itself to another dir" do
+		newdir = "#{@sandbox_dir}/newdir"
+		system "mkdir -p #{newdir}"
+
+		dst = Rush::Dir.new(newdir)
+		@entry.move_to(dst)
+
+		File.exists?(@filename).should be_false
+		File.exists?("#{newdir}/#{@entry.name}").should be_true
+	end
+
+	it "can copy itself to another directory" do
+		newdir = "#{@sandbox_dir}/newdir"
+		system "mkdir -p #{newdir}"
+
+		dst = Rush::Dir.new(newdir)
+		@copied_dir = @entry.copy_to(dst)
+
+		File.exists?(@filename).should be_true
+		File.exists?("#{newdir}/#{@entry.name}").should be_true
+
+		@copied_dir.full_path.should == "#{@sandbox_dir}newdir/#{@entry.name}"
+	end
+
+	it "considers dotfiles to be hidden" do
+		Rush::Entry.new("#{@sandbox_dir}/show").should_not be_hidden
+		Rush::Entry.new("#{@sandbox_dir}/.dont_show").should be_hidden
+	end
+
+	it "is considered equal to entries with the same full path and on the same box" do
+		same = Rush::Entry.new(@entry.full_path, @entry.box)
+		@entry.should == same
+	end
+
+	it "is considered not equal to entries with the same full path on a different box" do
+		same = Rush::Entry.new(@entry.full_path, Rush::Box.new('dummy'))
+		@entry.should_not == same
+	end
+
+	it "can mimic another entry" do
+		copy = Rush::Entry.new('abc', :dummy)
+		copy.mimic(@entry)
+		copy.path.should == @entry.path
+	end
+
+	it "can update the read access permission" do
+		system "chmod 666 #{@filename}"
+		@entry.access = { :user_can => :read }
+		`ls -l #{@filename}`.should match(/^-r--------/)
+	end
+
+	it "reads the file permissions in the access hash" do
+		system "chmod 640 #{@filename}"
+		@entry.access.should == { :user_can_read => true, :user_can_write => true, :group_can_read => true }
+	end
+end

data/spec/file_spec.rb

@@ -0,0 +1,83 @@
+require_relative 'base'
+
+describe Rush::File do
+	before do
+		@sandbox_dir = "/tmp/rush_spec.#{Process.pid}"
+		system "rm -rf #{@sandbox_dir}; mkdir -p #{@sandbox_dir}"
+
+		@filename = "#{@sandbox_dir}/test_file"
+		@contents = "1234"
+		system "echo #{@contents} > #{@filename}"
+		@contents += "\n"
+
+		@file = Rush::File.new(@filename)
+	end
+
+	after do
+		system "rm -rf #{@sandbox_dir}"
+	end
+
+	it "is a child of Rush::Entry" do
+		@file.should be_kind_of(Rush::Entry)
+	end
+
+	it "is not a dir" do
+		@file.should_not be_dir
+	end
+
+	it "can create itself as a blank file, and return itself" do
+		create_me = Rush::File.new("#{@sandbox_dir}/create_me")
+		create_me.create.should == create_me
+		File.exists?("#{@sandbox_dir}/create_me").should == true
+	end
+
+	it "knows its size in bytes" do
+		@file.size.should == @contents.length
+	end
+
+	it "can read its contents" do
+		@file.contents.should == @contents
+	end
+
+	it "read is an alias for contents" do
+		@file.read.should == @contents
+	end
+
+	it "can write new contents" do
+		@file.write('write test')
+		@file.contents.should == 'write test'
+	end
+
+	it "can count the number of lines it contains" do
+		@file.write("1\n2\n3\n")
+		@file.line_count.should == 3
+	end
+
+	it "searches its contents for matching lines" do
+		@file.write("a\n1\nb\n2\n")
+		@file.search(/\d/).should == %w(1 2)
+	end
+
+	it "search returns nil if no lines match" do
+		@file.write("a\nb\nc\n")
+		@file.search(/\d/).should be_nil
+	end
+
+	it "find-in-file replace" do
+		@file.replace_contents!(/\d/, 'x')
+		@file.contents.should == "xxxx\n"
+	end
+
+	it "can destroy itself" do
+		@file.destroy
+		::File.exists?(@filename).should be_false
+	end
+
+	it "can fetch contents or blank if doesn't exist" do
+		Rush::File.new('/does/not/exist').contents_or_blank.should == ""
+	end
+
+	it "can fetch lines, or empty if doesn't exist" do
+		Rush::File.new('/does/not/exist').lines_or_empty.should == []
+	end
+end

data/spec/find_by_spec.rb

@@ -0,0 +1,58 @@
+require_relative 'base'
+
+describe Rush::FindBy do
+	before do
+		class Foo
+			attr_accessor :bar
+
+			def initialize(bar)
+				@bar = bar
+			end end
+
+		@one = Foo.new('one')
+		@two = Foo.new('two')
+		@three = Foo.new('three')
+
+		@list = [ @one, @two, @three ]
+	end
+
+	it "compare_or_match exact match success" do
+		@list.compare_or_match('1', '1').should == true
+	end
+
+	it "compare_or_match exact match failure" do
+		@list.compare_or_match('1', '2').should == false
+	end
+
+	it "compare_or_match regexp match success" do
+		@list.compare_or_match('123', /2/).should == true
+	end
+
+	it "compare_or_match regexp match failure" do
+		@list.compare_or_match('123', /x/).should == false
+	end
+
+	it "find_by_ extact match" do
+		@list.find_by_bar('two').should == @two
+	end
+
+	it "find_by_ regexp match" do
+		@list.find_by_bar(/.hree/).should == @three
+	end
+
+	it "find_all_by_ exact match" do
+		@list.find_all_by_bar('one').should == [ @one ]
+	end
+
+	it "find_all_by_ regexp match" do
+		@list.find_all_by_bar(/^...$/).should == [ @one, @two ]
+	end
+
+	it "find_by_ with field not recognized by objects raises no errors" do
+		@list.find_by_nothing('x')
+	end
+
+	it "raises NoMethodError for things other than find_by" do
+		lambda { @list.does_not_exist }.should raise_error(NoMethodError)
+	end
+end

data/spec/fixnum_ext_spec.rb

@@ -0,0 +1,19 @@
+require_relative 'base'
+
+describe Fixnum do
+	before do
+		@num = 2
+	end
+
+	it "counts kb" do
+		@num.kb.should == 2*1024
+	end
+
+	it "counts mb" do
+		@num.mb.should == 2*1024*1024
+	end
+
+	it "counts gb" do
+		@num.gb.should == 2*1024*1024*1024
+	end
+end

data/spec/local_spec.rb

@@ -0,0 +1,365 @@
+require_relative 'base'
+
+describe Rush::Connection::Local do
+  before do
+    @sandbox_dir = "/tmp/rush_spec.#{Process.pid}"
+    system "rm -rf #{@sandbox_dir}; mkdir -p #{@sandbox_dir}"
+
+    @con = Rush::Connection::Local.new
+  end
+
+  after do
+    system "rm -rf #{@sandbox_dir}"
+  end
+
+  it "receive -> write_file(file, contents)" do
+    @con.should_receive(:write_file).with('file', 'contents')
+    @con.receive(:action => 'write_file', :full_path => 'file', :payload => 'contents')
+  end
+
+  it "receive -> append_to_file(file, contents)" do
+    @con.should_receive(:append_to_file).with('file', 'contents')
+    @con.receive(:action => 'append_to_file', :full_path => 'file', :payload => 'contents')
+  end
+
+  it "receive -> file_contents(file)" do
+    @con.should_receive(:file_contents).with('file').and_return('the contents')
+    @con.receive(:action => 'file_contents', :full_path => 'file').should == 'the contents'
+  end
+
+  it "receive -> destroy(file or dir)" do
+    @con.should_receive(:destroy).with('file')
+    @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')
+  end
+
+  it "receive -> rename(path, name, new_name)" do
+    @con.should_receive(:rename).with('path', 'name', 'new_name')
+    @con.receive(:action => 'rename', :path => 'path', :name => 'name', :new_name => 'new_name')
+  end
+
+  it "receive -> copy(src, dst)" do
+    @con.should_receive(:copy).with('src', 'dst')
+    @con.receive(:action => 'copy', :src => 'src', :dst => 'dst')
+  end
+
+  it "receive -> read_archive(full_path)" do
+    @con.should_receive(:read_archive).with('full_path').and_return('archive data')
+    @con.receive(:action => 'read_archive', :full_path => 'full_path').should == 'archive data'
+  end
+
+  it "receive -> write_archive(archive, dir)" do
+    @con.should_receive(:write_archive).with('archive', 'dir')
+    @con.receive(:action => 'write_archive', :dir => 'dir', :payload => 'archive')
+  end
+
+  it "receive -> index(base_path, glob)" do
+    @con.should_receive(:index).with('base_path', '*').and_return(%w(1 2))
+    @con.receive(:action => 'index', :base_path => 'base_path', :glob => '*').should == "1\n2\n"
+  end
+
+  it "receive -> stat(full_path)" do
+    @con.should_receive(:stat).with('full_path').and_return(1 => 2)
+    @con.receive(:action => 'stat', :full_path => 'full_path').should == YAML.dump(1 => 2)
+  end
+
+  it "receive -> set_access(full_path, user, group, permissions)" do
+    access = double("access")
+    Rush::Access.should_receive(:from_hash).with(:action => 'set_access', :full_path => 'full_path', :user => 'joe').and_return(access)
+
+    @con.should_receive(:set_access).with('full_path', access)
+    @con.receive(:action => 'set_access', :full_path => 'full_path', :user => 'joe')
+  end
+
+  it "receive -> size(full_path)" do
+    @con.should_receive(:size).with('full_path').and_return("1024")
+    @con.receive(:action => 'size', :full_path => 'full_path').should == "1024"
+  end
+
+  it "receive -> chown(full_path, user, group, options)" do
+    options = { noop: true }
+    @con.should_receive(:chown).with('full_path', 'username', 'groupname', options )
+    @con.receive(:action => 'chown', :full_path => 'full_path', user: 'username', group: 'groupname', options: options)
+  end
+
+  it "receive -> processes" do
+    @con.should_receive(:processes).with().and_return([ { :pid => 1 } ])
+    @con.receive(:action => 'processes').should == YAML.dump([ { :pid => 1 } ])
+  end
+
+  it "receive -> process_alive" do
+    @con.should_receive(:process_alive).with(123).and_return(true)
+    @con.receive(:action => 'process_alive', :pid => 123).should == '1'
+  end
+
+  it "receive -> kill_process" do
+    @con.should_receive(:kill_process).with(123, :wait => 10).and_return(true)
+    @con.receive(:action => 'kill_process', :pid => '123', :payload => YAML.dump(:wait => 10))
+  end
+
+  it "receive -> bash (reset environment)" do
+    @con.should_receive(:bash).with('cmd', 'user', false, true).and_return('output')
+    @con.receive(:action => 'bash', :payload => 'cmd', :user => 'user', :background => 'false', :reset_environment => 'true').should == 'output'
+  end
+
+  it "receive -> bash (foreground)" do
+    @con.should_receive(:bash).with('cmd', 'user', false, false).and_return('output')
+    @con.receive(:action => 'bash', :payload => 'cmd', :user => 'user', :background => 'false').should == 'output'
+  end
+
+  it "receive -> bash (background)" do
+    @con.should_receive(:bash).with('cmd', 'user', true, false).and_return('output')
+    @con.receive(:action => 'bash', :payload => 'cmd', :user => 'user', :background => 'true').should == 'output'
+  end
+
+  it "receive -> unknown action exception" do
+    lambda { @con.receive(:action => 'does_not_exist') }.should raise_error(Rush::Connection::Local::UnknownAction)
+  end
+
+  it "write_file writes contents to a file" do
+    fname = "#{@sandbox_dir}/a_file"
+    data = "some data"
+    @con.write_file(fname, data)
+    File.read(fname).should == data
+  end
+
+  it "append_to_file appends contents to a file" do
+    fname = "#{@sandbox_dir}/a_file"
+    system "echo line1 > #{fname}"
+    @con.append_to_file(fname, 'line2')
+    File.read(fname).should == "line1\nline2"
+  end
+
+  it "file_contents reads a file's contents" do
+    fname = "#{@sandbox_dir}/a_file"
+    system "echo stuff > #{fname}"
+    @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}"
+    @con.destroy(fname)
+    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 | grep -v total | wc -l`.to_i.should == 0
+  end
+
+  it "create_dir creates a directory" do
+    fname = "#{@sandbox_dir}/a/b/c/"
+    @con.create_dir(fname)
+    File.directory?(fname).should be_true
+  end
+
+  it "rename to rename entries within a dir" do
+    system "touch #{@sandbox_dir}/a"
+    @con.rename(@sandbox_dir, 'a', 'b')
+    File.exists?("#{@sandbox_dir}/a").should be_false
+    File.exists?("#{@sandbox_dir}/b").should be_true
+  end
+
+  it "copy to copy an entry to another dir on the same box" do
+    system "mkdir #{@sandbox_dir}/subdir"
+    system "touch #{@sandbox_dir}/a"
+    @con.copy("#{@sandbox_dir}/a", "#{@sandbox_dir}/subdir")
+    File.exists?("#{@sandbox_dir}/a").should be_true
+    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
+  end
+
+  it "read_archive works for paths with spaces" do
+    system "mkdir -p #{@sandbox_dir}/with\\ space; touch #{@sandbox_dir}/with\\ space/a"
+    @con.read_archive("#{@sandbox_dir}/with space").size.should > 50
+  end
+
+  it "write_archive to turn a byte stream into a dir" do
+    system "cd #{@sandbox_dir}; mkdir -p a; touch a/b; tar cf xfer.tar a; mkdir dst"
+    archive = File.read("#{@sandbox_dir}/xfer.tar")
+    @con.write_archive(archive, "#{@sandbox_dir}/dst")
+    File.directory?("#{@sandbox_dir}/dst/a").should be_true
+    File.exists?("#{@sandbox_dir}/dst/a/b").should be_true
+  end
+
+  it "write_archive works for paths with spaces" do
+    system "cd #{@sandbox_dir}; mkdir -p a; touch a/b; tar cf xfer.tar a; mkdir with\\ space"
+    archive = File.read("#{@sandbox_dir}/xfer.tar")
+    @con.write_archive(archive, "#{@sandbox_dir}/with space")
+    File.directory?("#{@sandbox_dir}/with space/a").should be_true
+    File.exists?("#{@sandbox_dir}/with space/a/b").should be_true
+  end
+
+  it "index fetches list of all files and dirs in a dir when pattern is empty" do
+    system "cd #{@sandbox_dir}; mkdir dir; touch file"
+    @con.index(@sandbox_dir, '').should == [ 'dir/', 'file' ]
+  end
+
+  it "index fetches only files with a certain extension with a flat pattern, *.rb" do
+    system "cd #{@sandbox_dir}; touch a.rb; touch b.txt"
+    @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 fetches the octal permissions" do
+    @con.stat(@sandbox_dir)[:mode].should be_kind_of(Fixnum)
+  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
+
+  it "set_access invokes the access object" do
+    access = double("access")
+    access.should_receive(:apply).with('/some/path')
+    @con.set_access('/some/path', access)
+  end
+
+  it "chown should change the ownership of a file"
+
+  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"
+      @con.size(@sandbox_dir).should == (4096*3 + 5)
+    end
+  end
+
+  it "parses ps output on os x" do
+    @con.parse_ps("21712   501   21711   1236   0 /usr/bin/vi somefile.rb").should == {
+      :pid => "21712",
+      :uid => "501",
+      :parent_pid => 21711,
+      :mem => 1236,
+      :cpu => 0,
+      :command => '/usr/bin/vi',
+      :cmdline => '/usr/bin/vi somefile.rb',
+    }
+  end
+
+  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   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", :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 "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 sending a TERM" do
+    @con.stub(:process_alive).and_return(false)
+    ::Process.should_receive(:kill).with('TERM', 123).once
+    @con.kill_process(123)
+  end
+
+  it "kills a process by pid sending a KILL signal if TERM doesn't work" do
+    @con.stub(:process_alive).and_return(true)
+    ::Process.should_receive(:kill).with('TERM', 123).at_least(:twice)
+    ::Process.should_receive(:kill).with('KILL', 123)
+    @con.kill_process(123)
+  end
+
+  it "kills a process by pid without sending TERM if :wait is zero" do
+    ::Process.should_not_receive(:kill).with('TERM', 123)
+    ::Process.should_receive(:kill).with('KILL', 123)
+    @con.kill_process(123, :wait => 0)
+  end
+
+  it "does not raise an error if the process is already dead" do
+    ::Process.should_receive(:kill).and_raise(Errno::ESRCH)
+    lambda { @con.kill_process(123) }.should_not raise_error
+  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 "executes a bash command as another user using sudo" do
+    @con.bash("echo test2", ENV['USER']).should == "test2\n"
+  end
+
+  it "executes a bash command in the background, returning the pid" do
+    @con.bash("true", nil, true).should > 0
+  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
+
+  it "resolves a unix uid to a user" do
+    @con.resolve_unix_uid_to_user(0).should == "root"
+    @con.resolve_unix_uid_to_user('0').should == "root"
+  end
+
+  it "returns nil if the unix uid does not exist" do
+    @con.resolve_unix_uid_to_user(9999).should be_nil
+  end
+
+  it "iterates through a process list and resolves the unix uid for each" do
+    list = [ { :uid => 0, :command => 'pureftpd' }, { :uid => 9999, :command => 'defunk' } ]
+    @con.resolve_unix_uids(list).should == [ { :uid => 0, :user => 'root', :command => 'pureftpd' }, { :uid => 9999, :command => 'defunk', :user => nil } ]
+  end
+end