Fix file deletion strategy

Add better reporting on unit tests

See merge request devs/moth!178

.gitlab-ci.yml

@@ -11,8 +11,14 @@ test:
       - main
       - merge_requests
   script:
-    - go test -race ./...
+    - go test -coverprofile=coverage.txt -covermode=atomic -race ./...
-
+    - go get github.com/boumenot/gocover-cobertura
+    - go run github.com/boumenot/gocover-cobertura < coverage.txt > coverage.xml
+  artifacts:
+    reports:
+      coverage_report:
+        coverage_format: cobertura
+        path: coverage.xml
 push:
   stage: push
   rules:

cmd/mothd/state_filesystem_test.go

@@ -627,8 +627,32 @@ func TestStateTeamIDs(t *testing.T) {
 			t.Errorf("Expected to find 0 team ID, found %d (%s), instead", len(teamIDs), teamIDs)
 		} 
 	}
+}
 
+func TestStateDeleteTeamIDList(t *testing.T) {
+	s := NewTestState()
+	s.refresh()
 
+	s.Fs.Remove("teamids.txt")
+
+	teamIDs, err := s.TeamIDs()
+
+	if len(teamIDs) != 0 {
+		t.Errorf("Expected to find 0 team IDs, found %d (%s), instead", len(teamIDs), teamIDs)
+	}
+
+	if err == nil {
+		t.Errorf("Did not receive expected error for non-existent teamids.txt")
+	}
+}
+
+func TestStateTeamNames(t *testing.T) {
+	s := NewTestState()
+	s.refresh()
+
+	if teamNames := s.TeamNames(); len(teamNames) != 0 {
+		t.Errorf("Expected to find 0 registered teams, found %d (%s), instead", len(teamNames), teamNames)
+	}
 }
 
 func TestDevelState(t *testing.T) {