Merge pull request #202 from gnawux/test_hostname

test case for hostname field in pod spec

Author: feiskyer

hack/lib/test.sh

@@ -51,6 +51,19 @@ hyper::test::run_attached_pod() {
   hyper run --rm -a -p $1
 }
 
+hyper::test::hostname() {
+  hname=$(hyper::test::run_attached_pod ${HYPER_ROOT}/hack/pods/hostname.pod | tr -d '\r')
+  echo "hostname is ${hname}, expected myname"
+  [ "${hname}x" != "mynamex" ] && return 1
+
+  echo "check long hostname"
+  hyper::test::run_attached_pod ${HYPER_ROOT}/hack/pods/hostname-err-long.pod && return 1
+  echo "check invalid hostname"
+  hyper::test::run_attached_pod ${HYPER_ROOT}/hack/pods/hostname-err-char.pod && return 1
+
+  return 0
+}
+
 hyper::test::insert_file() {
   res=0
   count=5

hack/pods/hostname-err-char.pod

@@ -0,0 +1,16 @@
+{
+	"id": "host-name-tester",
+	"hostname": "/myname",
+	"containers" : [{
+	    "name": "file-tester",
+	    "image": "busybox:latest",
+	    "workdir": "/",
+	    "command": ["hostname"]
+	}],
+	"resource": {
+	    "vcpu": 1,
+	    "memory": 512
+	},
+	"volumes": [],
+	"tty": true
+}

hack/pods/hostname-err-long.pod

@@ -0,0 +1,16 @@
+{
+	"id": "host-name-tester",
+	"hostname": "nnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnn",
+	"containers" : [{
+	    "name": "file-tester",
+	    "image": "busybox:latest",
+	    "workdir": "/",
+	    "command": ["hostname"]
+	}],
+	"resource": {
+	    "vcpu": 1,
+	    "memory": 512
+	},
+	"volumes": [],
+	"tty": true
+}

hack/pods/hostname.pod

@@ -0,0 +1,16 @@
+{
+	"id": "host-name-tester",
+	"hostname": "myname",
+	"containers" : [{
+	    "name": "file-tester",
+	    "image": "busybox:latest",
+	    "workdir": "/",
+	    "command": ["hostname"]
+	}],
+	"resource": {
+	    "vcpu": 1,
+	    "memory": 512
+	},
+	"volumes": [],
+	"tty": true
+}

hack/test-cmd.sh

@@ -156,6 +156,7 @@ __EOF__
 
   hyper::test::exitcode
   hyper::test::exec
+  hyper::test::hostname
   hyper::test::insert_file
   hyper::test::map_file
   hyper::test::with_volume