search

kscripting / kscript

Scripting enhancements for Kotlin
MIT License
2.07k stars 124 forks source link

Fix and expand manual tests with auth repo #403

Closed rocketraman closed 1 year ago

rocketraman commented 1 year ago

The manual tests for an authenticated repo were not complete.

Fixed the test artifactory server startup.

Added a test for environment variable substitution (test 4). Note that this test follows the current documentation in the README of using {{FOO}} style env var substitutions, but that will not currently work. What will work is $FOO. This should be updated as part of the fix for https://github.com/kscripting/kscript/issues/402.

Added a test for environment variable substitution with packaging (test 5). This test will not currently work as the substituted credentials are not passed to Gradle properly by kscript.

NOTE: Instead of this being a manual test, it could be done in an automated unit test with Testcontainers.

rocketraman commented 1 year ago

Closing, will submit a single PR with all related changes.