Fixed #32: Script with dash in filename doesn't work

Author: holgerbrandl

kscript

@@ -1,7 +1,7 @@
 #!/bin/bash
 
 ## note: always use a patch release even if it's 0
-KSCRIPT_VERSION=1.5.1
+KSCRIPT_VERSION=1.5.2
 
 ## make sure that all requirements (java, kotlinc, maven) to run kscript are met
 assertInPath(){ if [ -z "$(which $1)" ]; then echo "[ERROR] Could not locate '$1'" 1>&2; exit 1; fi }
@@ -122,6 +122,7 @@ fi
 
 ## ... or use a cached version of it if possible
 #scriptFile="./test.kts"
+#scriptFile="test-script.kts"
 scriptFile="$1"
 shift
 
@@ -250,8 +251,8 @@ scriptCheckSum=$(kscript_md5 ${scriptFile})
 jarFile=${KSCRIPT_CACHE_DIR}/$(basename ${scriptFile} .kts).${scriptCheckSum}.jar
 
 
-## capitalize first letter (since this is what kotlin compiler is doing for the wrapper
-className=$(basename ${scriptFile} .kts)
+## capitalize first letter and get rid of dashes (since this is what kotlin compiler is doing for the wrapper to create a valid java class name)
+className=$(basename ${scriptFile} .kts | tr '-' '_')
 #className="${className^}" ## disabled because requires bash4 and thus not on macos
 className=$(echo ${className:0:1} | tr  '[a-z]' '[A-Z]')${className:1}
 

test/resources/dash-test.kts

@@ -0,0 +1 @@
+println("dash alarm!")

test/test_suite.sh

@@ -42,6 +42,10 @@ assert "source ${KSCRIPT_HOME}/test/resources/local_script_file.sh" "kscript roc
 ## make sure that it runs with local script files
 assert "kscript ${KSCRIPT_HOME}/test/resources/multi_line_deps.kts" "kscript is  cool!"
 
+## scripts with dashes in the file name should work as well
+assert "kscript ${KSCRIPT_HOME}/test/resources/dash-test.kts" "dash alarm!"
+
+
 ## missing script
 assert_raises "kscript i_do_not_exist.kts" 1
 assert "kscript i_do_not_exist.kts 2>&1" "[ERROR] Could not open script file 'i_do_not_exist.kts'"