added tests, a mock for osascript, and fixed a bug

mocks are still actually really bad in general... looks like it's easy for parameters to get interpreted by echo or print instead of getting captured and written.

Author: NReilingh

archive_fusionvm

@@ -23,7 +23,7 @@ i=0
 running=(${(@f)"$(vmrun list | grep "$SRCPATH")"})
 
 # Save this initial list of .vmx files
-vmxlist=$running
+vmxlist=($running)
 
 while true; do
 	echo "Current running VM list is $running"
@@ -49,7 +49,9 @@ while true; do
 done
 
 # Test that every VMX in the list has no lockfile; otherwise quit Fusion
+echo "Checking for lockfiles for VM list $vmxlist"
 for vmx in $vmxlist; do
+	echo "Testing lockfile $vmx.lck"
 	if [ -f "$vmx.lck" ]; then
 		osascript -e 'tell application "VMware Fusion" to quit'
 		break

tests/_support/mocks/hdiutil

@@ -1,5 +1,5 @@
 #!/usr/bin/env zsh
 
-print -l $@ > $TEMP/hdiutil.calls
+print -l $@ >> $TEMP/hdiutil.calls
 
 exit 0

tests/_support/mocks/osascript

@@ -0,0 +1,5 @@
+#!/usr/bin/env zsh
+
+echo "$@" >> $TEMP/osascript.calls
+
+exit 0

tests/archive_fusionvm.zunit

@@ -2,15 +2,11 @@
 
 # $TEMP is set up in bootstrap
 @setup {
-  [ -d $TEMP ] || mkdir $TEMP
+  rm -r $TEMP && mkdir $TEMP
   [ -f $TEMP/decoy_file ] || touch $TEMP/decoy_file
   [ -d $TEMP/decoy_dir ] || mkdir $TEMP/decoy_dir
 }
 
-@teardown {
-  rm -r $TEMP
-}
-
 @test 'Exit with error when no arguments are supplied' {
   run archive_fusionvm
 
@@ -114,6 +110,57 @@
   fi
 }
 
+@test 'osascript executed when lockfile exists' {
+  vmx="$TEMP/decoy_dir/example vm.vmx"
+
+  cat > $TEMP/vmrun_list_1 <<-EOF
+	$vmx
+	EOF
+
+  touch "$vmx.lck"
+
+  run archive_fusionvm $TEMP/decoy_dir $TEMP/out.dmg
+
+  if [ -f $TEMP/osascript.calls ]; then
+    pass
+  else
+    fail 'osascript was not called'
+  fi
+}
+
+@test 'osascript is not executed if no lockfiles exist' {
+  cat > $TEMP/vmrun_list_1 <<-EOF
+	$TEMP/decoy_dir/example vm.vmx
+	EOF
+
+  run archive_fusionvm $TEMP/decoy_dir $TEMP/out.dmg
+
+  if [ -f $TEMP/osascript.calls ]; then
+    fail 'osascript was called'
+  else
+    pass
+  fi
+}
+
+@test 'osascript is executed only once if multiple lockfiles exist' {
+  vmone="$TEMP/decoy_dir/examplevm one.vmx"
+  vmtwo="$TEMP/decoy_dir/examplevmtwo.vmx"
+
+  cat > $TEMP/vmrun_list_1 <<-EOF
+	$vmone
+	$vmtwo
+	EOF
+
+  touch "$vmone.lck"
+  touch "$vmtwo.lck"
+
+  run archive_fusionvm $TEMP/decoy_dir $TEMP/out.dmg
+
+  osascriptcalls=(${(@f)"$(<$TEMP/osascript.calls)"})
+
+  assert ${#osascriptcalls} equals 1
+}
+
 @test 'hdiutil create is executed' {
   run archive_fusionvm $TEMP/decoy_dir $TEMP/out.dmg