Complete Refactor (#9)

Author: KyleKotowick

.github/workflows/CICD.yml

@@ -0,0 +1,45 @@
+name: "Build"
+
+on: [push, pull_request]
+
+jobs:
+  Matrix:
+    runs-on: ubuntu-latest
+
+    steps:
+      - name: Generate Matrix
+        id: matrix
+        uses: Invicton-Labs/terraform-module-testing/matrix@main
+        with:
+          minimum_tf_version: '0.13.1'
+          # 0.15.0 - 0.15.3 has a bug where it doesn't like sensitive-marked outputs
+          excluded_tf_versions: '0.15.0,0.15.1,0.15.2,0.15.3,1.3.0'
+
+      - name: Output Matrix
+        run: |
+          echo "Strategy: ${{ steps.matrix.outputs.strategy }}"
+
+    outputs:
+      strategy: ${{ steps.matrix.outputs.strategy }}
+          
+  Test:
+    needs: [Matrix]
+    strategy: ${{ fromJSON(needs.Matrix.outputs.strategy)}}
+    runs-on: ${{ matrix.runs-on }}
+    container: ${{ matrix.container }}
+    steps:
+      - name: Run Tests
+        id: tests
+        uses: Invicton-Labs/terraform-module-testing/test@main
+        with:
+          testing_path: tester
+          
+  # This job just waits for all other jobs to pass. We have it here
+  # so our branch protection rule can reference a single job, instead
+  # of needing to list every matrix value of every job above.
+  Passed:
+    runs-on: ubuntu-latest
+    needs: [Test]
+    steps:
+    - name: Mark tests as passed
+      run: echo "🎉"

dynamic-depends-on/main.tf

@@ -1,24 +1,107 @@
+//==================================================
+//     Outputs that match the input variables
+//==================================================
+output "command_unix" {
+  description = "The value of the `command_unix` input variable, or the default value if the input was `null`, with all carriage returns removed."
+  value       = local.command_create_unix
+}
+output "command_windows" {
+  description = "The value of the `command_windows` input variable, or the default value if the input was `null`, with all carriage returns removed."
+  value       = local.command_create_windows
+}
+output "command_destroy_unix" {
+  description = "The value of the `command_destroy_unix` input variable, or the default value if the input was `null`, with all carriage returns removed."
+  value       = local.command_destroy_unix
+}
+output "command_destroy_windows" {
+  description = "The value of the `command_destroy_windows` input variable, or the default value if the input was `null`, with all carriage returns removed."
+  value       = local.command_destroy_windows
+}
+output "environment" {
+  description = "The value of the `environment` input variable, or the default value if the input was `null`, with all carriage returns removed."
+  value       = local.env_vars
+}
+output "environment_sensitive" {
+  description = "The value of the `environment_sensitive` input variable, or the default value if the input was `null`, with all carriage returns removed."
+  value       = local.env_vars_sensitive
+}
+output "environment_triggerless" {
+  description = "The value of the `environment_triggerless` input variable, or the default value if the input was `null`, with all carriage returns removed."
+  value       = local.env_vars_triggerless
+}
+output "triggers" {
+  description = "The value of the `triggers` input variable, or the default value if the input was `null`."
+  value       = local.var_triggers
+}
+output "working_dir" {
+  description = "The value of the `working_dir` input variable."
+  value       = local.var_working_dir
+}
+output "fail_create_on_nonzero_exit_code" {
+  description = "The value of the `fail_create_on_nonzero_exit_code` input variable, or the default value if the input was `null`."
+  value       = local.var_fail_create_on_nonzero_exit_code
+}
+output "fail_create_on_stderr" {
+  description = "The value of the `fail_create_on_stderr` input variable, or the default value if the input was `null`."
+  value       = local.var_fail_create_on_stderr
+}
+output "fail_create_on_timeout" {
+  description = "The value of the `fail_create_on_timeout` input variable, or the default value if the input was `null`."
+  value       = local.var_fail_create_on_timeout
+}
+output "fail_destroy_on_nonzero_exit_code" {
+  description = "The value of the `fail_destroy_on_nonzero_exit_code` input variable, or the default value if the input was `null`."
+  value       = local.var_fail_create_on_nonzero_exit_code
+}
+output "fail_destroy_on_stderr" {
+  description = "The value of the `fail_destroy_on_stderr` input variable, or the default value if the input was `null`."
+  value       = local.var_fail_create_on_stderr
+}
+output "fail_destroy_on_timeout" {
+  description = "The value of the `fail_destroy_on_timeout` input variable, or the default value if the input was `null`."
+  value       = local.var_fail_create_on_timeout
+}
+output "timeout_create" {
+  description = "The value of the `timeout_create` input variable, or the default value if the input was `null`."
+  value       = local.var_timeout_create
+}
+output "timeout_destroy" {
+  description = "The value of the `timeout_destroy` input variable, or the default value if the input was `null`."
+  value       = local.var_timeout_destroy
+}
+output "suppress_console" {
+  description = "The value of the `suppress_console` input variable, or the default value if the input was `null`."
+  value       = local.var_suppress_console
+}
+output "unix_interpreter" {
+  description = "The value of the `unix_interpreter` input variable, or the default value if the input was `null`."
+  value       = local.var_unix_interpreter
+}
+
+//==================================================
+//       Outputs generated by this module
+//==================================================
 output "id" {
   description = "A unique ID for this resource. The ID does not change when the shell command is re-run."
-  value       = random_uuid.uuid.result
+  value       = local.execution_id
 }
-
 output "stdout" {
-  description = "The stdout output of the shell command."
-  value       = local.outputs.stdout
+  description = "The stdout output of the shell command, with all carriage returns and trailing newlines removed."
+  value       = local.stdout_censored
 }
-
 output "stderr" {
-  description = "The stderr output of the shell command."
-  value       = local.outputs.stderr
+  description = "The stderr output of the shell command, with all carriage returns and trailing newlines removed."
+  value       = local.stderr_censored
 }
-
 output "exit_code" {
-  description = "The exit status code of the shell command."
-  value       = tonumber(local.outputs.exit_code)
+  description = "The exit status code of the shell command. If the `timeout` input variable was provided and the command timed out, this will be `null`."
+  value       = local.outputs.exit_code == null ? null : tonumber(local.outputs.exit_code)
 }
-
-output "version" {
-  description = "The version of the shell resource (see the `track_version` input variable)."
-  value       = var.track_version ? module.state_keeper[0].output : null
+output "null_command_windows" {
+  description = "A command that does nothing on Windows systems. You can use this output as the value for the `command_windows` input if you don't want the module to do anything on Windows systems."
+  value       = local.null_command_windows
+}
+output "null_command_unix" {
+  description = "A command that does nothing on Unix-based systems. You can use this output as the value for the `command_unix` input if you don't want the module to do anything on Unix-based systems."
+  value       = local.null_command_unix
 }