Merge pull request #50 from Nordix/clean_up_unit_test_flow

Fix presubmit unit test run

.prow.yaml

@@ -1,22 +1,11 @@
 presubmits:
   - name: presubmit-nephio-go-test
     decorate: true
-    run_if_changed: "(\\.go|Makefile)$"
+    run_if_changed: "(\\.go|Makefile|\\.mk)$"
     spec:
       containers:
       - image: nephio/gotests:1782782171367346176
         command:
         - make
         args:
         - ci-unit
-  - name: presubmit-porch-e2e
-    decorate: true
-    optional: true
-    run_if_changed: "(\\.go|Makefile)$"
-    spec:
-      containers:
-      - image: nephio/gotests:1782782171367346176
-        command:
-        - make
-        args:
-        - ci-test-porch

Makefile

@@ -159,19 +159,11 @@ generate: generate-api
 tidy:
 	@for f in $(MODULES); do (cd $$f; echo "Tidying $$f"; go mod tidy) || exit 1; done
 
-.PHONY: test-porch
-test-porch:
-	#@for f in $(MODULES); do (cd $$f; echo "Testing $$f"; E2E=1 go test -v -race --count=1 ./...) || exit 1; done
-	@for f in $(MODULES); do (cd $$f; echo "Testing $$f"; go test -v -race --count=1 ./...) || exit 1; done
-
 .PHONY: configure-git
 configure-git:
 	git config --global --add user.name test
 	git config --global --add user.email test@nephio.org
 
-.PHONY: ci-test-porch
-ci-test-porch: configure-git test-porch
-
 .PHONY: ci-unit
 ci-unit: configure-git test
 

default-go-test.mk

@@ -42,4 +42,4 @@ unit-clean: ## Clean up the artifacts created by the unit tests
 ifeq ($(CONTAINER_RUNNABLE), 0)
 		$(CONTAINER_RUNTIME) system prune -f
 endif
-		rm ${TEST_COVERAGE_FILE} ${TEST_COVERAGE_HTML_FILE} ${TEST_COVERAGE_FUNC_FILE} > /dev/null 2>&1
+		rm -f ${TEST_COVERAGE_FILE} ${TEST_COVERAGE_HTML_FILE} ${TEST_COVERAGE_FUNC_FILE} > /dev/null 2>&1