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.

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

Code Generation: Initial tests #1583

Merged
merged 1 commit into from
May 27, 2024
Merged

Conversation

julienduchesne
Copy link
Member

Depends on #1582

Implements a full integration test using the dashboard example file: This test applies the dashboard shown in the TF docs and tests that the expected files are there after generating the TF config from the instance

Note: The generated files are not optimal. This PR is not meant to modify behaviour. It just sets up a base testing framework that we can build upon

@julienduchesne julienduchesne requested a review from a team as a code owner May 24, 2024 18:49
Copy link

In order to lower resource usage and have a faster runtime, PRs will not run Cloud tests automatically.
To do so, a Grafana Labs employee must trigger the cloud acceptance tests workflow manually.

Base automatically changed from julienduchesne/generate-sort to main May 27, 2024 01:05
Implements a full integration test using the dashboard example file:
This test applies the dashboard shown in the TF docs and tests that the expected files are there after generating the TF config from the instance

To make this test pass reliably, I had to also make sure the files being generated are properly sorted. This is also tested

Note: The generated files are not optimal. This PR is not meant to modify behaviour
@julienduchesne julienduchesne force-pushed the julienduchesne/generate-test branch from 4f74e86 to a384fa4 Compare May 27, 2024 01:06
@julienduchesne julienduchesne merged commit 89caddb into main May 27, 2024
23 of 24 checks passed
@julienduchesne julienduchesne deleted the julienduchesne/generate-test branch May 27, 2024 01:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants