Fix #37 support when the current dir contains special characters

Author: xmik

CHANGELOG.md

@@ -1,3 +1,7 @@
+### 0.12.1 (2024-Feb-05)
+
+* Fix https://github.com/kudulab/dojo/issues/37 - support the case when the current directory contains special characters
+
 ### 0.12.0 (2024-Feb-04)
 
 * Compile Dojo using Golang 1.21.6 thanks to using Docker image kudulab/golang-dojo:2.1.0

utils.go

@@ -8,6 +8,7 @@ import (
 	"log"
 	"math/rand"
 	"os"
+	"regexp"
 	"runtime"
 	"strconv"
 	"strings"
@@ -113,6 +114,9 @@ func getRunIDGenerateFromCurrentDir(currentDirectory string) string {
 	// the currentDirectory contains capital letters and docker-compose project names do not welcome
 	// capital letters
 	runID = strings.ToLower(runID)
+	//runID = strings.ReplaceAll(runID, " ", "")
+	// remove any special characters
+	runID = regexp.MustCompile(`[^a-zA-Z0-9\-]+`).ReplaceAllString(runID, "")
 	return runID
 }
 

utils_test.go

@@ -106,11 +106,21 @@ func Test_getRunID(t *testing.T) {
 }
 
 func Test_getRunIDGenerateFromCurrentDir(t *testing.T) {
+	// lower case letters only
 	runID := getRunIDGenerateFromCurrentDir("mydir")
 	assert.True(t, strings.HasPrefix(runID, "dojo-mydir-"))
 
+	// lower case and upper case letters
 	runID = getRunIDGenerateFromCurrentDir("mYdIR")
 	assert.True(t, strings.HasPrefix(runID, "dojo-mydir-"))
+
+	// lower case and upper case letters and spaces
+	runID = getRunIDGenerateFromCurrentDir("mYdIR with spaces")
+	assert.True(t, strings.HasPrefix(runID, "dojo-mydirwithspaces-"))
+
+	// lower case and upper case letters and spaces and special characters
+	runID = getRunIDGenerateFromCurrentDir("mYdIR wi#th s(3paces")
+	assert.True(t, strings.HasPrefix(runID, "dojo-mydirwiths3paces-"))
 }
 
 func getTestConfig() Config {

version.go

@@ -1,3 +1,3 @@
 package main
-const DojoVersion = "0.12.0"
+const DojoVersion = "0.12.1"