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

test: reduce duration of E2E tests #2562

Closed
lucasrod16 opened this issue May 30, 2024 · 1 comment
Closed

test: reduce duration of E2E tests #2562

lucasrod16 opened this issue May 30, 2024 · 1 comment

Comments

@lucasrod16
Copy link
Contributor

lucasrod16 commented May 30, 2024

Describe what should be investigated or refactored

Our E2E tests currently take about 20 minutes to run. We should aim to speed up this feedback loop by removing low-value tests and parallelizing where possible.

@lucasrod16
Copy link
Contributor Author

lucasrod16 commented May 30, 2024

lucasrod16 pushed a commit that referenced this issue May 30, 2024
## Description
Removes `TestMismatchedArchitectures` e2e test and refactors
`TestValidatePackageArchitecture` unit test

## Related Issue

Relates to #2562

## Checklist before merging

- [x] Test, docs, adr added or updated as needed
- [x] [Contributor Guide
Steps](https://github.com/defenseunicorns/zarf/blob/main/.github/CONTRIBUTING.md#developer-workflow)
followed
lucasrod16 pushed a commit that referenced this issue May 31, 2024
## Description
Removes `TestMismatchedVersions` and adds `TestPrintBreakingChanges`
unit test

## Related Issue

Relates to #2562

## Checklist before merging

- [x] Test, docs, adr added or updated as needed
- [x] [Contributor Guide
Steps](https://github.com/defenseunicorns/zarf/blob/main/.github/CONTRIBUTING.md#developer-workflow)
followed
lucasrod16 pushed a commit that referenced this issue May 31, 2024
## Description

I believe this test is no longer necessary since we've added unit tests
to the agent

Relates to #2562

## Checklist before merging

- [ ] Test, docs, adr added or updated as needed
- [ ] [Contributor Guide
Steps](https://github.com/defenseunicorns/zarf/blob/main/.github/CONTRIBUTING.md#developer-workflow)
followed
@lucasrod16 lucasrod16 mentioned this issue Jun 7, 2024
2 tasks
AustinAbro321 pushed a commit that referenced this issue Jun 10, 2024
## Description
Simplify `TestMain` and update the `build-examples` make target to build
only the binary for the local system.

## Related Issue

Relates to #2562

## Checklist before merging

- [x] Test, docs, adr added or updated as needed
- [x] [Contributor Guide
Steps](https://github.com/defenseunicorns/zarf/blob/main/.github/CONTRIBUTING.md#developer-workflow)
followed
@salaxander salaxander added this to Zarf Jul 22, 2024
@github-project-automation github-project-automation bot moved this to Backlog in Zarf Jul 22, 2024
@salaxander salaxander removed the status in Zarf Jul 22, 2024
AustinAbro321 pushed a commit that referenced this issue Jul 23, 2024
## Description
Removes `TestMismatchedArchitectures` e2e test and refactors
`TestValidatePackageArchitecture` unit test

## Related Issue

Relates to #2562

## Checklist before merging

- [x] Test, docs, adr added or updated as needed
- [x] [Contributor Guide
Steps](https://github.com/defenseunicorns/zarf/blob/main/.github/CONTRIBUTING.md#developer-workflow)
followed

Signed-off-by: Austin Abro <[email protected]>
AustinAbro321 pushed a commit that referenced this issue Jul 23, 2024
## Description
Removes `TestMismatchedVersions` and adds `TestPrintBreakingChanges`
unit test

## Related Issue

Relates to #2562

## Checklist before merging

- [x] Test, docs, adr added or updated as needed
- [x] [Contributor Guide
Steps](https://github.com/defenseunicorns/zarf/blob/main/.github/CONTRIBUTING.md#developer-workflow)
followed

Signed-off-by: Austin Abro <[email protected]>
AustinAbro321 added a commit that referenced this issue Jul 23, 2024
## Description

I believe this test is no longer necessary since we've added unit tests
to the agent

Relates to #2562

## Checklist before merging

- [ ] Test, docs, adr added or updated as needed
- [ ] [Contributor Guide
Steps](https://github.com/defenseunicorns/zarf/blob/main/.github/CONTRIBUTING.md#developer-workflow)
followed

Signed-off-by: Austin Abro <[email protected]>
AustinAbro321 pushed a commit that referenced this issue Jul 23, 2024
## Description
Simplify `TestMain` and update the `build-examples` make target to build
only the binary for the local system.

## Related Issue

Relates to #2562

## Checklist before merging

- [x] Test, docs, adr added or updated as needed
- [x] [Contributor Guide
Steps](https://github.com/defenseunicorns/zarf/blob/main/.github/CONTRIBUTING.md#developer-workflow)
followed

Signed-off-by: Austin Abro <[email protected]>
@salaxander salaxander moved this to Triage in Zarf Sep 10, 2024
@lucasrod16 lucasrod16 closed this as not planned Won't fix, can't repro, duplicate, stale Dec 1, 2024
@github-project-automation github-project-automation bot moved this from Triage to Done in Zarf Dec 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

No branches or pull requests

2 participants