v0.4.12: LatestRelease <- Development

Merge pull request #41 from NineNineFive/Development

Author: lmbek

launcher/chromeLauncher.go

@@ -65,9 +65,12 @@ func (launcher *ChromeLauncher) launchForWindows() bool {
 
 			// Start frontend by starting a new Chrome process
 			path := "C:\\Program Files\\Google\\Chrome\\Application\\chrome.exe"
-			// TODO: see if --user-data-dir can be removed
+
 			cmd := exec.Command(path, "--app=http://"+fileserver.GetServerAddress(), "--user-data-dir="+launcher.FrontendInstallLocation)
-			cmd.Start()
+			err := cmd.Start()
+			if err != nil {
+				println("Warning: Chrome could not start, is it installed?")
+			}
 
 			// Set up a signal handler to gracefully shutdown the program, when it should shutdown
 			signalHandler := make(chan os.Signal, 1)

tests/main/main.go

@@ -12,7 +12,7 @@ import (
 var organisationName = "NewOrganisationName" // put in organisation name
 var projectName = "NewProjectName"           // put in project name
 
-var frontendPath = "./frontend" // this should be set to where frontend files is (frontend folder: html, css, javascript...)
+var frontendPath = "./tests/main/frontend" // this should be set to where frontend files is (frontend folder: html, css, javascript...)
 
 var chromeLauncher = launcher.ChromeLauncher{
 	Location:                "C:\\Program Files\\Google\\Chrome\\Application\\chrome.exe",
@@ -25,15 +25,6 @@ var chromeLauncher = launcher.ChromeLauncher{
 }
 
 var chromiumLauncher = launcher.DefaultChromiumLauncher // default chrome or chromium launcher settings can be used like this
-/* // Otherwise they can also be customized like this
-var chromiumLauncher = launcher.ChromiumLauncher{
-	Location:      "/var/lib/snapd/desktop/applications/chromium_chromium.desktop", // TODO: check if better location or can be customised
-	Domain:        "localhost",
-	PortMin:       11430,
-	PreferredPort: 11451,
-	PortMax:       11500,
-}
-*/
 
 func main() {
 	launchApp()

tests/main/main_test.go

@@ -0,0 +1,16 @@
+package main
+
+import (
+	"testing"
+)
+
+func TestLaunchApp(test *testing.T) {
+	var err error
+
+	// TODO: let launchApp return an error, if it could not open app correctly
+	launchApp()
+	if err != nil {
+		test.Errorf("failed")
+	}
+
+}

tests/utils/net/port_test.go

@@ -0,0 +1,25 @@
+package utils
+
+import (
+	"github.com/NineNineFive/go-local-web-gui/utils/net"
+	"strconv"
+	"testing"
+)
+
+func BenchmarkGetInt(benchmark *testing.B) {
+	// Reset the timer before running the code
+	benchmark.ResetTimer()
+
+	// Run the code b.N times
+	for i := 0; i < benchmark.N; i++ {
+		for j := 11450; j < 11453; j++ {
+			net.IsPortUsed("localhost", strconv.Itoa(j))
+		}
+	}
+}
+
+func TestPortIsUsed(test *testing.T) {
+	for i := 11450; i < 11453; i++ {
+		net.IsPortUsed("localhost", strconv.Itoa(i))
+	}
+}

tests/utils/random/random_test.go

@@ -0,0 +1,23 @@
+package random
+
+import (
+	"github.com/NineNineFive/go-local-web-gui/utils/random"
+	"testing"
+)
+
+func TestGetInt(test *testing.T) {
+	// Set the random seed to a fixed value to make the test deterministic
+	random.SetRandomSeed(0)
+
+	for i := 0; i < 100; i++ { // modify amount of tries
+		// Call the function that returns the value to be tested
+		value := random.GetInt(10000, 10100)
+
+		// Check if the value is within the range of 10000 and 10100
+		if value < 10000 || value > 10100 {
+			test.Errorf("Value returned is outside the range of 10000 and 10100. Value: %d", value)
+		}
+		//println(value) // uncomment to check values
+	}
+
+}

tests/utils_test.go

@@ -0,0 +1,25 @@
+package utils
+
+import (
+	"github.com/NineNineFive/go-local-web-gui/utils"
+	"reflect"
+	"testing"
+)
+
+func BenchmarkToString(benchmark *testing.B) {
+	// Reset the timer before running the code
+	benchmark.ResetTimer()
+
+	// Run the code b.N times
+	for i := 0; i < benchmark.N; i++ {
+		utils.IntegerToString(50)
+	}
+}
+
+func TestIntegerToString(test *testing.T) {
+	newString := utils.IntegerToString(50)
+	if reflect.TypeOf(newString).Kind() != reflect.String {
+		test.Error("not a string")
+	}
+	println(newString)
+}