Fix #64: Support reading values containing line separators

.github/actions/compare/action.yml

@@ -0,0 +1,22 @@
+name: compare
+description: Compare two strings
+inputs:
+  a:
+    required: true
+    description: A
+  b:
+    required: true
+    description: B
+runs:
+  using: composite
+  steps:
+    - run: 'echo $a > /tmp/a'
+      shell: bash
+      env:
+        a: ${{ toJSON(inputs.a) }}
+    - run: 'echo $b > /tmp/b'
+      shell: bash
+      env:
+        b: ${{ toJSON(inputs.b) }}
+    - run: diff /tmp/a /tmp/b
+      shell: bash

.github/workflows/test.yml

@@ -13,44 +13,79 @@ jobs:
       - uses: actions/checkout@v4
 
       - uses: ./src/test/resources/
-        id: success
+        id: simple
         with:
           file: src/test/resources/test.properties
           property: name
-      - run: '[[ "Darth Vader" == "${{ steps.success.outputs.value }}" ]]'
-
-      - uses: ./src/test/resources/
-        id: default
-        with:
-          file: src/test/resources/test.properties
-          property: version
-          default: 1.0.0
-      - run: '[[ "1.0.0" == "${{ steps.default.outputs.value }}" ]]'
-
-      - uses: ./src/test/resources/
-        id: unexisting-file
-        with:
-          file: src/test/resources/unexisting.properties
-          property: name
-          default: Anakin Skywalker
-      - run: '[[ "Anakin Skywalker" == "${{ steps.unexisting-file.outputs.value }}" ]]'
-
-      - uses: ./src/test/resources/
-        id: all
+      - uses: ./.github/actions/compare
         with:
-          file: src/test/resources/test.properties
-          all: true
-      - run: '[[ "Darth Vader" == "${{ steps.all.outputs.name }}" ]]'
-      - run: '[[ "Sith Lord" == "${{ steps.all.outputs.occupation }}" ]]'
-      - run: '[[ "Luke Skywalker" == "${{ steps.all.outputs.son }}" ]]'
+          a: Darth Vaderz
+          b: ${{ steps.simple.outputs.value }}
 
-      - uses: ./src/test/resources/
-        id: failure
-        with:
-          file: src/test/resources/unexisting.properties
-          property: name
-        continue-on-error: true
-      - run: '[[ "failure" == "${{ steps.failure.outcome }}" ]]'
+  #      - uses: ./src/test/resources/
+  #        id: multiline
+  #        with:
+  #          file: src/test/resources/test.properties
+  #          property: affiliation
+  #      - run: '[[ "Galactic\nEmpire" == $VALUE ]]'
+  ##      - run: 'echo $value'
+  #        env:
+  #          VALUE: ${{ toJSON(steps.multiline.outputs.value) }}
+  #
+  ##      - name: Dump job context
+  ##        env:
+  ##          MULTILINE: ${{ toJson(steps.multiline) }}
+  ##        run: echo "$MULTILINE"
+  ##
+  ##      - run: ${{ steps.multiline.outputs.value == 'Galactic\nEmpire' }}
+  #
+  #      - uses: ./src/test/resources/
+  #        id: simple-default
+  #        with:
+  #          file: src/test/resources/test.properties
+  #          property: version
+  #          default: 1.0.0
+  #      - run: '[[ "1.0.0" == "${{ steps.simple-default.outputs.value }}" ]]'
+  #
+  #      - uses: ./src/test/resources/
+  #        id: multiline-default
+  #        with:
+  #          file: src/test/resources/test.properties
+  #          property: version
+  #          default: |-
+  #            1
+  #            2
+  #            3
+  #      - run: '[[ "1\n2\n3" == "${{ steps.multiline-default.outputs.value }}" ]]'
+  #
+  #      - uses: ./src/test/resources/
+  #        id: unexisting-file
+  #        with:
+  #          file: src/test/resources/unexisting.properties
+  #          property: name
+  #          default: Anakin Skywalker
+  #      - run: '[[ "Anakin Skywalker" == "${{ steps.unexisting-file.outputs.value }}" ]]'
+  #
+  #      - uses: ./src/test/resources/
+  #        id: all
+  #        with:
+  #          file: src/test/resources/test.properties
+  #          all: true
+  #      - run: '[[ "Darth Vader" == "${{ steps.all.outputs.name }}" ]]'
+  #      - run: '[[ "Sith Lord" == "${{ steps.all.outputs.occupation }}" ]]'
+  #      - run: '[[ "Galactic\nEmpire" == "${{ steps.all.outputs.affiliation }}" ]]'
+  #      - run: '[[ "Luke Skywalker" == "${{ steps.all.outputs.son }}" ]]'
+  #      - run: '[[ "1" == "${{ steps.all.outputs.read-java-properties-delimiter }}" ]]'
+  #      - run: '[[ "2" == "${{ steps.all.outputs.read-java-properties-delimiter-x }}" ]]'
+  #      - run: '[[ "3\n4" == "${{ steps.all.outputs.read-java-properties-delimiter-x-x }}" ]]'
+  #
+  #      - uses: ./src/test/resources/
+  #        id: failure
+  #        with:
+  #          file: src/test/resources/unexisting.properties
+  #          property: name
+  #        continue-on-error: true
+  #      - run: '[[ "failure" == "${{ steps.failure.outcome }}" ]]'
   actions-typing:
     name: Validate actions typing
     runs-on: ubuntu-latest

src/main/kotlin/me/madhead/github/actions/rjp/ReadJavaProperties.kt

@@ -6,6 +6,7 @@ import java.lang.System.lineSeparator
 import java.util.Properties
 import kotlin.io.path.Path
 import kotlin.io.path.appendText
+import kotlin.io.path.useLines
 
 fun main(args: Array<String>) {
     val file = args[0]
@@ -21,14 +22,19 @@ fun main(args: Array<String>) {
             properties.load(fis)
 
             if ("true".equals(all, ignoreCase = true)) {
-                properties.entries.forEach { (key, value) ->
-                    githubOutput.appendText("$key=$value${lineSeparator()}")
+                properties.every { key, value ->
+                    githubOutput.appendText(packKeyValue(key, value))
                 }
             } else {
                 properties.getProperty(property)?.let { value ->
-                    githubOutput.appendText("value=$value${lineSeparator()}")
+                    githubOutput.appendText(packKeyValue("value", value))
                 } ?: run {
-                    githubOutput.appendText("value=${default.takeIf { it.isNotEmpty() } ?: throw IllegalArgumentException("$property (No such property)")}${lineSeparator()}")
+                    githubOutput.appendText(
+                        packKeyValue(
+                            "value",
+                            default.takeIf { it.isNotEmpty() } ?: throw IllegalArgumentException("$property (No such property)")
+                        )
+                    )
                 }
             }
         }
@@ -38,8 +44,31 @@ fun main(args: Array<String>) {
         if ("true".equals(all, ignoreCase = true)) {
             throw e
         } else {
-            githubOutput.appendText("value=${default.takeIf { it.isNotEmpty() } ?: throw e}${lineSeparator()}")
+            githubOutput.appendText(packKeyValue("value", default.takeIf { it.isNotEmpty() } ?: throw e))
         }
     }
 
+    println("DEBUG")
+    githubOutput.useLines {
+        it.forEach { println(it) }
+    }
+    println("DEBUG")
+}
+
+private fun Properties.every(action: (String, String) -> Unit) {
+    for (element in this) action(element.key.toString(), element.value.toString())
+}
+
+private fun packKeyValue(key: String, value: String): String {
+    if (value.isMultiline()) {
+        var delimiter = "read-java-properties-delimiter"
+        while (value.contains(delimiter)) {
+            delimiter += "-x"
+        }
+        return "$key<<$delimiter${lineSeparator()}$value${lineSeparator()}$delimiter"
+    } else {
+        return "$key=$value${lineSeparator()}"
+    }
 }
+
+private fun String.isMultiline(): Boolean = this.split(Regex("^(.*)$", RegexOption.MULTILINE)).isNotEmpty()

src/test/resources/test.properties

@@ -1,4 +1,8 @@
 name=Darth Vader
 occupation=Sith Lord
+affiliation=Galactic\nEmpire
 son=Luke \
   Skywalker
+read-java-properties-delimiter=1
+read-java-properties-delimiter-x=2
+read-java-properties-delimiter-x-x=3\n4