Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

test: terraform plan parsing #1736

Merged
merged 1 commit into from
Mar 18, 2021
Merged

test: terraform plan parsing #1736

merged 1 commit into from
Mar 18, 2021

Conversation

rontalx
Copy link
Contributor

@rontalx rontalx commented Mar 17, 2021

What does this PR do?

Adds unit-tests for Terraform Plan parsing in IaC local exec

@rontalx rontalx requested review from a team as code owners March 17, 2021 12:49
@rontalx rontalx force-pushed the test/iac-terraform-plan-tests branch from d2d313f to 37134c7 Compare March 17, 2021 12:52
@rontalx rontalx requested review from a team and removed request for a team March 17, 2021 12:52
Base automatically changed from fix/jest-config-not-provided to master March 17, 2021 15:26
@rontalx rontalx requested a review from p0tr3c March 17, 2021 16:04
@rontalx rontalx force-pushed the test/iac-terraform-plan-tests branch from 37134c7 to afa9d68 Compare March 17, 2021 21:29
@github-actions
Copy link
Contributor

github-actions bot commented Mar 17, 2021 •
edited
Loading

Warnings
⚠️

Looks like you added a new Tap test. Consider making it a Jest test instead. See files like test/*.spec.ts for examples. Files found:

  • test/jest/unit/iac-unit-tests/terraform-plan-parser.fixtures.ts
Messages
📖

This PR will not trigger a new version. It doesn't include any commit message with feat or fix.

Generated by 🚫 dangerJS against d534127

@rontalx rontalx force-pushed the test/iac-terraform-plan-tests branch 2 times, most recently from 2962f6d to 478f72b Compare March 18, 2021 13:57
@rontalx rontalx force-pushed the test/iac-terraform-plan-tests branch from 478f72b to d534127 Compare March 18, 2021 18:14
@rontalx rontalx merged commit 2f1326c into master Mar 18, 2021
@rontalx rontalx deleted the test/iac-terraform-plan-tests branch March 18, 2021 19:08
This was referenced May 4, 2021
Open
Open
Open
Open
@snyk-bot snyk-bot mentioned this pull request Jun 3, 2021
Open
@swinton swinton mentioned this pull request Aug 2, 2022
Closed
@snyk-bot snyk-bot mentioned this pull request Sep 29, 2022
Open
@nathan-snyk nathan-snyk mentioned this pull request Oct 5, 2022
Open
@nathan-snyk nathan-snyk mentioned this pull request Jan 10, 2023
Open
@cloudgeek7 cloudgeek7 mentioned this pull request Feb 25, 2023
Merged
@snyk-bot snyk-bot mentioned this pull request Apr 27, 2023
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@p0tr3c p0tr3c p0tr3c approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@rontalx @p0tr3c