Stfaseid http refactor

Description

Draft for new API. Needs extensive manual testing. Only merge if changes in https://code.fbi.h-da.de/danet/api/-/tree/10-refactor-current-proto-definitions-stfahelm and upcoming changes to CLI are approved.

Related Issue

#141 (closed)

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

• My code follows the code style of this project.
• My change requires a change to the documentation.
• I have updated the documentation accordingly.
• I have read the CONTRIBUTING document.
• I have added tests to cover my changes.
• All new and existing tests passed.

http_test.go

 package gosdn
 
 import (
+	"errors"
+	"net"
 	"net/http"
 	"testing"
+	"time"
 )
 
 func Test_httpApi(t *testing.T) {
@@ -34,6 +37,13 @@ func Test_httpApi(t *testing.T) {
 	coreLock.Lock()
 	startHttpServer()
 	coreLock.Unlock()
+
+	err := waitForHTTPServer()
+	if err != nil {
+		t.Errorf("httpApi() error = %v", err)
+		return
+	}
+
 	for _, tt := range tests {
 		t.Run(tt.name, func(t *testing.T) {
 			got, err := http.Get(tt.request)
@@ -47,3 +57,20 @@ func Test_httpApi(t *testing.T) {
 		})
 	}
 }
+
+// see: https://stackoverflow.com/a/56865986
+func waitForHTTPServer() error {
+	for i := 0; i < 10; i++ {
+		conn, err := net.DialTimeout("tcp", ":8080", 1*time.Second)
+		if err != nil {
+			time.Sleep(50 * time.Millisecond)
+			continue
+		}
+		err = conn.Close()
+		if err != nil {
+			return err
+		}
+		return nil
+	}
+	return errors.New("HTTP server could not be reached.")
+}