Skip to content

Commit 6b223c1

Browse files
committed
Fixes for tests
Signed-off-by: Marcin Kuszczak <[email protected]>
1 parent 4307bc2 commit 6b223c1

File tree

5 files changed

+16
-10
lines changed

5 files changed

+16
-10
lines changed

src/integration/kotlin/io/github/kscripting/kscript/integration/PackagingTest.kt

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -15,8 +15,8 @@ class PackagingTest : TestBase {
1515
//TODO: doesn't work on msys, cygwin, windows
1616
fun `Packaged script is cached`() {
1717
//@formatter:off
18-
verify("kscript --package \"println(1+1)\"", 0, "", startsWith("[kscript] Packaging script 'scriplet' into standalone executable..."))
19-
verify("kscript --package \"println(1+1)\"", 0, "", startsWith("[kscript] Packaged script 'scriplet' available at path:"))
18+
verify("kscript --package \"println(1+1)\"", 0, "", startsWith("[kscript] Packaging script 'Scriplet' into standalone executable..."))
19+
verify("kscript --package \"println(1+1)\"", 0, "", startsWith("[kscript] Packaged script 'Scriplet' available at path:"))
2020
//@formatter:on
2121
}
2222

src/main/kotlin/io/github/kscripting/kscript/creator/JarArtifactCreator.kt

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,9 @@ data class JarArtifact(val path: OsPath, val execClassName: String)
1111
class JarArtifactCreator(private val executor: Executor) {
1212

1313
fun create(basePath: OsPath, script: Script, resolvedDependencies: Set<OsPath>): JarArtifact {
14-
val execClassName = "${script.entryPoint.value}Kt"
14+
val entryPoint = script.entryPoint?.value ?: "ScripletKt"
15+
val execClassName = script.packageName.value + "." + entryPoint
16+
1517
val jarFile = basePath.resolve("scriplet.jar")
1618
val scriptFile = basePath.resolve("Scriplet.kt")
1719
val execClassNameFile = basePath.resolve("scripletExecClassName.txt")

src/main/kotlin/io/github/kscripting/kscript/model/Script.kt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ data class Script(
88
val resolvedCode: String,
99

1010
val packageName: PackageName,
11-
val entryPoint: Entry,
11+
val entryPoint: Entry?,
1212
val importNames: Set<ImportName>,
1313

1414
val includes: Set<Include>,

src/main/kotlin/io/github/kscripting/kscript/resolver/ScriptResolver.kt

Lines changed: 9 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@ package io.github.kscripting.kscript.resolver
22

33
import io.github.kscripting.kscript.model.*
44
import io.github.kscripting.kscript.parser.LineParser.extractValues
5+
import io.github.kscripting.kscript.util.Logger
56
import io.github.kscripting.kscript.util.ScriptUtils
67
import io.github.kscripting.kscript.util.UriUtils
78
import io.github.kscripting.shell.model.*
@@ -12,7 +13,7 @@ class ScriptResolver(
1213
private val sectionResolver: SectionResolver,
1314
private val scriptingConfig: ScriptingConfig
1415
) {
15-
private val scripletName = "Scriplet"
16+
private val defaultScripletFileName = "Scriplet"
1617

1718
//level parameter - for how many levels should include be resolved
1819
//level 0 - do not resolve includes in base file and any other embedded
@@ -34,7 +35,7 @@ class ScriptResolver(
3435
scriptType,
3536
null,
3637
inputOutputResolver.resolveCurrentDir(),
37-
scripletName
38+
defaultScripletFileName
3839
)
3940

4041
return createScript(
@@ -100,7 +101,7 @@ class ScriptResolver(
100101
content.scriptType,
101102
content.uri,
102103
content.contextUri,
103-
scripletName
104+
defaultScripletFileName
104105
)
105106

106107
return createScript(
@@ -124,7 +125,7 @@ class ScriptResolver(
124125
scriptType,
125126
null,
126127
inputOutputResolver.resolveCurrentDir(),
127-
scripletName
128+
defaultScripletFileName
128129
)
129130

130131
return createScript(
@@ -158,7 +159,9 @@ class ScriptResolver(
158159
resolutionContext.scriptNodes.add(scriptNode)
159160

160161
resolutionContext.packageName = resolutionContext.packageName ?: PackageName("kscript.scriplet")
161-
resolutionContext.entryPoint = resolutionContext.entryPoint ?: Entry("${resolutionContext.packageName!!.value}.Scriplet")
162+
163+
Logger.devMsg("packageName: ${resolutionContext.packageName}")
164+
Logger.devMsg("entryPoint: ${resolutionContext.entryPoint}")
162165

163166
val code = ScriptUtils.resolveCode(resolutionContext.packageName, resolutionContext.importNames, scriptNode)
164167

@@ -174,7 +177,7 @@ class ScriptResolver(
174177
scriptLocation,
175178
code,
176179
resolutionContext.packageName!!,
177-
resolutionContext.entryPoint!!,
180+
resolutionContext.entryPoint,
178181
resolutionContext.importNames,
179182
resolutionContext.includes,
180183
resolutionContext.dependencies,

src/main/kotlin/io/github/kscripting/kscript/util/ScriptUtils.kt

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -54,6 +54,7 @@ object ScriptUtils {
5454

5555
var indent = ""
5656
var shouldPrependEmptyLine = false
57+
5758
if (scriptNode.scriptLocation.scriptType == ScriptType.KTS) {
5859
sb.appendLine("fun main(args: Array<String>) {")
5960
indent = " "

0 commit comments

Comments
 (0)