entrypoint: move go test setup an invocation into a function

A little refactoring to move the setup of the pkg test calls into a
function to ease future changes.

Signed-off-by: John Mulligan <jmulligan@redhat.com>

entrypoint.sh

@@ -51,29 +51,10 @@ while true ; do
     esac
 done
 
-test_go_ceph() {
+test_pkg() {
-    mkdir -p /tmp/ceph
+    local pkg="$1"
-    "${MICRO_OSD_PATH}" /tmp/ceph
-    export CEPH_CONF=/tmp/ceph/ceph.conf
-
-    if [[ ${TEST_RUN} == NONE ]]; then
-        echo "skipping test execution"
-    else
-        go get -t -v ./...
-        diff -u <(echo -n) <(gofmt -d -s .)
-        #go vet ./...
-        #go list ./...
-        echo "mode: count" > "cover.out"
-        P=github.com/ceph/go-ceph
-        pkgs=(\
-            "cephfs" \
-            "errutil" \
-            "rados" \
-            "rbd" \
-            )
-        for pkg in "${pkgs[@]}"; do
     if [[ "$TEST_PKG" && "$TEST_PKG" != "$pkg" ]]; then
-                continue
+        return 0
     fi
     testargs=(\
         "-covermode=count" \
@@ -87,10 +68,35 @@ test_go_ceph() {
 
     go test -v "${testargs[@]}" "./$pkg"
     grep -v "^mode: count" "$pkg.cover.out" >> "cover.out"
+}
+
+test_go_ceph() {
+    mkdir -p /tmp/ceph
+    "${MICRO_OSD_PATH}" /tmp/ceph
+    export CEPH_CONF=/tmp/ceph/ceph.conf
+
+    if [[ ${TEST_RUN} == NONE ]]; then
+        echo "skipping test execution"
+        return 0
+    fi
+
+    go get -t -v ./...
+    diff -u <(echo -n) <(gofmt -d -s .)
+    #go vet ./...
+    #go list ./...
+    echo "mode: count" > "cover.out"
+    P=github.com/ceph/go-ceph
+    pkgs=(\
+        "cephfs" \
+        "errutil" \
+        "rados" \
+        "rbd" \
+        )
+    for pkg in "${pkgs[@]}"; do
+        test_pkg "$pkg"
     done
     mkdir -p /results/coverage
     go tool cover -html=cover.out -o /results/coverage/go-ceph.html
-    fi
 }
 
 pause_if_needed() {