Merge in tests from Comfortstat

.github/workflows/main.yml

@@ -28,6 +28,8 @@ jobs:
         uses: actions/setup-go@v5
         with:
           go-version: 1.23
+      - name: Install dependencies
+        run: make deps
       - name: Run tests
         run: make test
       - name: Report stats

Comfortstat/Comfortstat.go

@@ -31,6 +31,7 @@ func main() {
 
 	// instantiate a template unit asset
 	assetTemplate := initTemplate()
+	initAPI()
 	assetName := assetTemplate.GetName()
 	sys.UAssets[assetName] = &assetTemplate
 
@@ -45,8 +46,8 @@ func main() {
 		if err := json.Unmarshal(raw, &uac); err != nil {
 			log.Fatalf("Resource configuration error: %+v\n", err)
 		}
-		ua, cleanup := newUnitAsset(uac, &sys, servsTemp)
-		defer cleanup()
+		ua, startup := newUnitAsset(uac, &sys, servsTemp)
+		startup()
 		sys.UAssets[ua.GetName()] = &ua
 	}
 
@@ -66,7 +67,6 @@ func main() {
 	time.Sleep(2 * time.Second) // allow the go routines to be executed, which might take more time than the main routine to end
 }
 
-// TODO: change the namne, will get one function for each of the four cases
 // Serving handles the resources services. NOTE: it exepcts those names from the request URL path
 func (t *UnitAsset) Serving(w http.ResponseWriter, r *http.Request, servicePath string) {
 	switch servicePath {
@@ -97,13 +97,18 @@ func (rsc *UnitAsset) set_SEKprice(w http.ResponseWriter, r *http.Request) {
 	}
 }
 
-// TODO: split up this function to two sepreate function that sets on max and min price.
+// All these functions below handles HTTP "PUT" or "GET" requests to modefy or retrieve the MAX/MIN temprature/price and desierd temprature
+// For the PUT case - the "HTTPProcessSetRequest(w, r)" is called to prosses the data given from the user and if no error,
+// call the set functions in things.go with the value witch updates the value in the struct
 func (rsc *UnitAsset) set_minTemp(w http.ResponseWriter, r *http.Request) {
 	switch r.Method {
 	case "PUT":
 		sig, err := usecases.HTTPProcessSetRequest(w, r)
 		if err != nil {
-			log.Println("Error with the setting request of the position ", err)
+			//log.Println("Error with the setting request of the position ", err)
+			http.Error(w, "request incorreclty formated", http.StatusBadRequest)
+			return
+
 		}
 		rsc.setMin_temp(sig)
 	case "GET":
@@ -118,7 +123,9 @@ func (rsc *UnitAsset) set_maxTemp(w http.ResponseWriter, r *http.Request) {
 	case "PUT":
 		sig, err := usecases.HTTPProcessSetRequest(w, r)
 		if err != nil {
-			log.Println("Error with the setting request of the position ", err)
+			//log.Println("Error with the setting request of the position ", err)
+			http.Error(w, "request incorreclty formated", http.StatusBadRequest)
+			return
 		}
 		rsc.setMax_temp(sig)
 	case "GET":
@@ -129,16 +136,14 @@ func (rsc *UnitAsset) set_maxTemp(w http.ResponseWriter, r *http.Request) {
 	}
 }
 
-// LOOK AT: I guess that we probable only need to if there is a PUT from user?
-// LOOK AT: so not the GET!
-// For PUT - the "HTTPProcessSetRequest(w, r)" is called to prosses the data given from the user and if no error, call set_minMaxprice with the value
-// wich updates the value in thge struct
 func (rsc *UnitAsset) set_minPrice(w http.ResponseWriter, r *http.Request) {
 	switch r.Method {
 	case "PUT":
 		sig, err := usecases.HTTPProcessSetRequest(w, r)
 		if err != nil {
-			log.Println("Error with the setting request of the position ", err)
+			//log.Println("Error with the setting request of the position ", err)
+			http.Error(w, "request incorreclty formated", http.StatusBadRequest)
+			return
 		}
 		rsc.setMin_price(sig)
 	case "GET":
@@ -155,7 +160,9 @@ func (rsc *UnitAsset) set_maxPrice(w http.ResponseWriter, r *http.Request) {
 	case "PUT":
 		sig, err := usecases.HTTPProcessSetRequest(w, r)
 		if err != nil {
-			log.Println("Error with the setting request of the position ", err)
+			//log.Println("Error with the setting request of the position ", err)
+			http.Error(w, "request incorreclty formated", http.StatusBadRequest)
+			return
 		}
 		rsc.setMax_price(sig)
 	case "GET":
@@ -172,7 +179,9 @@ func (rsc *UnitAsset) set_desiredTemp(w http.ResponseWriter, r *http.Request) {
 	case "PUT":
 		sig, err := usecases.HTTPProcessSetRequest(w, r)
 		if err != nil {
-			log.Println("Error with the setting request of the position ", err)
+			//log.Println("Error with the setting request of the position ", err)
+			http.Error(w, "request incorreclty formated", http.StatusBadRequest)
+			return
 		}
 		rsc.setDesired_temp(sig)
 	case "GET":
@@ -181,4 +190,5 @@ func (rsc *UnitAsset) set_desiredTemp(w http.ResponseWriter, r *http.Request) {
 	default:
 		http.Error(w, "Method is not supported.", http.StatusNotFound)
 	}
+
 }