Merge pull request #1 from tilfin/feature/option-and-dryrun

Implement getopts and dry run ENV

Author: tilfin

backup

@@ -5,13 +5,41 @@
 #
 ##############################
 
+#
+# Usage and parse options
+#
+usage() {
+  echo "Usage: $0 [-n] [-f configfile]" 1>&2
+  exit 1
+}
+
+while getopts f:nh OPT
+do
+  case $OPT in
+    f)  cfgfile=$OPTARG
+        ;;
+    n)  DRY_RUN=1
+        echo "Dry run!"
+        export DRY_RUN
+        ;;
+    h)  usage
+        ;;
+    \?) usage
+        ;;
+  esac
+done
+
+shift $((OPTIND - 1))
+
+#
+# Move basedir
+#
 BASEDIR=${0%/*}
 BASEDIR=$(cd $BASEDIR && pwd)
 
 #
 # Load config file and setup env
 #
-cfgfile=$1
 if [ "$cfgfile" == "" ]; then
   cfgfile=$BASEDIR/backup.conf
 fi

sync

@@ -5,4 +5,7 @@
 echo "srcdir : $1"
 echo "bucket : $2"
 echo "destdir: $3"
+if [ "$DRY_RUN" ]; then
+  echo "dry run backup"
+fi
 exit $?

test/bats

@@ -26,7 +26,7 @@ teardown() {
 }
 
 @test "invoking backup with valid config file" {
-  run backup $confdir/valid.conf
+  run backup -f $confdir/valid.conf
   [ "$status" -eq 0 ]
   [ "${lines[2]}" = "[OK] S01command" ]
   [ "${lines[3]}" = "Start to synchronize files to cloud." ]
@@ -37,25 +37,51 @@ teardown() {
 }
 
 @test "invoking backup with config file that does not exist" {
-  run backup notexists.conf
+  run backup -f notexists.conf
   [ "$status" -eq 1 ]
   [ "${lines[0]}" = "Config file not found!" ]
 }
 
 @test "invoking backup with config where buckets is not defined " {
-  run backup $confdir/no-bucket.conf
+  run backup -f $confdir/no-bucket.conf
   [ "$status" -eq 1 ]
   [ "${lines[0]}" = "Buckets not defined!" ]
 }
 
 @test "invoking backup with config where backup temp dir is not defined " {
-  run backup $confdir/no-tmpdir.conf
+  run backup -f $confdir/no-tmpdir.conf
   [ "$status" -eq 1 ]
   [ "${lines[0]}" = "Backup temp dir not defined!" ]
 }
 
 @test "invoking backup with config where backup temp dirname is invalid" {
-  run backup $confdir/invalid-tmpdir.conf
+  run backup -f $confdir/invalid-tmpdir.conf
   [ "$status" -eq 1 ]
   [ "${lines[0]}" = "Backup temp dirname must end with 'temp' or 'tmp'!" ]
 }
+
+@test "invoking dry run backup" {
+  run backup -n
+  [ "$status" -eq 0 ]
+  [ "${lines[0]}" = "Dry run!" ]
+  [ "${lines[3]}" = "[OK] S01command" ]
+  [ "${lines[4]}" = "Start to synchronize files to cloud." ]
+  [ "${lines[5]}" = "srcdir : /tmp/backup-temp" ]
+  [ "${lines[6]}" = "bucket : storage" ]
+  [ "${lines[7]}" = "destdir: destination" ]
+  [ "${lines[8]}" = "dry run backup" ]
+  [ "${lines[9]}" = "[OK] Sync" ]
+}
+
+@test "invoking dry run backup with config file" {
+  run backup -n -f $confdir/valid.conf
+  [ "$status" -eq 0 ]
+  [ "${lines[0]}" = "Dry run!" ]
+  [ "${lines[3]}" = "[OK] S01command" ]
+  [ "${lines[4]}" = "Start to synchronize files to cloud." ]
+  [ "${lines[5]}" = "srcdir : /tmp/backup-temp" ]
+  [ "${lines[6]}" = "bucket : storage" ]
+  [ "${lines[7]}" = "destdir: destination" ]
+  [ "${lines[8]}" = "dry run backup" ]
+  [ "${lines[9]}" = "[OK] Sync" ]
+}