SRE-2745 Account for Additional Error Cases

[jadabyte256]

Closes https://snapsheettech.atlassian.net/browse/SRE-2745

What

Adds additional test coverage to the cdktf-diff code.

Why

Reduces the likelihood of unintended side effects due to code changes.

How

Added two new tests to account for:

• When the stub_output_file option is passed, it creates a file instead of running cdktf commands.
• Unable to find the workflow run.

Testing

Verify jest tests pass in the NPM Test Coverage / npm_test action or pull this branch down locally and run the entire test suite.

Verify tests exist for the following test case:

• Unable to find the workflow run.
• Unable to find how the node version is specified (file?).
• When the stub_output_file option is passed, it creates a file instead of running cdktf commands.
• Verify that the command string has changed and doesn’t run cdktf diff.
• When the skip_synth option is passed, it adds that to the options for the cdktf command.
• Verify it adds --skip-synth to the command.
• Unable to find the job ID for the current job
• Fetching the job ID from GitHub API results in multiple pages, with the desired job ID coming on a later page.
• When a run of cdktf diff fails.
• When there are no changes for a plan.
• If a plan results in pending changes.
• If we were unable to determine if a diff ran successfully.
• The output written to file matches the expected format.

[jadabyte256]

The Jira story says:

...make any updates to ensure behavior is correct.

Outline the expected cases for behavior in the appropriate sections of the README

No updates were made to functionality, will leave README as is.

[d10zero]

Nice! Looks like most of the tests that were mentioned in the story already existed, and you added the 2 that were missing. Good job 👍