Skip to content

Commit 4197a46

Browse files
committed
bugfix: issue #349 - added test case
feature: improved test suite docs: improved NEWS.md
1 parent 29a984a commit 4197a46

File tree

3 files changed

+185
-174
lines changed

3 files changed

+185
-174
lines changed

NEWS.md

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -4,12 +4,16 @@
44

55
Rewrite by https://github.com/aartiPl
66

7+
Functionality:
8+
* Recursive inclusion of dependant scripts without hardcoded limits
9+
710
Improved test_suite.sh
811
* Automatic setting up of test environment (assert.sh, test directories)
912
* Automatic compilation
1013
* idea - script to help to test idea use cases
1114
* Script setup_environment.sh can be used for local testing
1215
* Some script tests moved to Unit Tests
16+
* Ability to start test suites by name
1317

1418
Improved Unit Tests
1519
* Several new Unit tests
@@ -22,7 +26,6 @@ Modularisation of source code
2226
* Removed duplication
2327
* Code divided in logical pieces and moved to packages
2428
* Script resolution creates immutable objects
25-
2629

2730
Build script
2831
* Updated Gradle to version 7.3 and shadowJar to 6.1.0
@@ -41,12 +44,10 @@ INCOMPATIBLE CHANGES:
4144
* In annotations the only allowed delimiter is coma "," (to allow options with arguments, separated by space)
4245
* Resolution of env variables is more restrictive - only vars expected by kscript can be resolved (for security - it's not good to include arbitrary strings from user env into the script)
4346
* Reworked caching mechanism
44-
* Dropped cache for resolution of artifacts - I can not observe change in resolution time. In fact .m2 repository is already kind of cache, so it should not be needed to add another one.
45-
4647

4748
## 3.2
4849

49-
Release TBD
50+
Not released - incorporated in 4.0
5051

5152
Major changes & enhancements
5253
* Improved for relative script inclusions (Thanks to PR by **aartiPI** [#330](https://github.com/holgerbrandl/kscript/pull/330))

src/main/kotlin/kscript/app/util/Executor.kt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@ class Executor(private val commandResolver: CommandResolver, private val config:
2020
val scriptCompileResult = ShellUtils.evalBash(command)
2121

2222
if (scriptCompileResult.exitCode != 0) {
23-
throw IllegalStateException("compilation of scriplet failed\n$scriptCompileResult.stderr")
23+
throw IllegalStateException("Compilation of scriplet failed:\n$scriptCompileResult")
2424
}
2525
}
2626

0 commit comments

Comments
 (0)