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: reduce duration of E2E tests #2562

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

test: reduce duration of E2E tests #2562

lucasrod16 opened this issue May 30, 2024 · 1 comment
Labels
tech-debt 💳 Debt that the team has charged and needs to repay

Comments

@lucasrod16
Copy link
Contributor

lucasrod16 commented May 30, 2024 •
edited

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 lucasrod16 added the tech-debt 💳 Debt that the team has charged and needs to repay label May 30, 2024
@lucasrod16
Copy link
Contributor Author

lucasrod16 commented May 30, 2024 •
edited

@lucasrod16 lucasrod16 mentioned this issue May 30, 2024
Merged
2 tasks
lucasrod16 pushed a commit that referenced this issue May 30, 2024
test: remove validate pkg arch e2e test (#2563)
ff83e19 
## 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 lucasrod16 mentioned this issue May 30, 2024
Merged
2 tasks
@AustinAbro321 AustinAbro321 mentioned this issue May 31, 2024
Merged
2 tasks
lucasrod16 pushed a commit that referenced this issue May 31, 2024
test: remove TestMismatchedVersions e2e test (#2564)
69a8507 
## 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
@AustinAbro321
test: delete agent e2e label test (#2568)
0c02643 
## 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 May 31, 2024
Closed
@lucasrod16 lucasrod16 mentioned this issue Jun 7, 2024
Merged
2 tasks
AustinAbro321 pushed a commit that referenced this issue Jun 10, 2024
test: cleanup e2e tests (#2601)
fd45291 
## 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
@lucasrod16 lucasrod16 mentioned this issue Jun 14, 2024
Merged
2 tasks
@lucasrod16 lucasrod16 mentioned this issue Jul 12, 2024
Closed
AustinAbro321 pushed a commit that referenced this issue Jul 23, 2024
@AustinAbro321
test: remove validate pkg arch e2e test (#2563)
d34345d 
## 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 <AustinAbro321@gmail.com>
AustinAbro321 pushed a commit that referenced this issue Jul 23, 2024
@AustinAbro321
test: remove TestMismatchedVersions e2e test (#2564)
aeb2367 
## 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 <AustinAbro321@gmail.com>
AustinAbro321 added a commit that referenced this issue Jul 23, 2024
@AustinAbro321
test: delete agent e2e label test (#2568)
1de0918 
## 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 <AustinAbro321@gmail.com>
AustinAbro321 pushed a commit that referenced this issue Jul 23, 2024
@AustinAbro321
test: cleanup e2e tests (#2601)
1a0b131 
## 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 <AustinAbro321@gmail.com>
@schristoff schristoff mentioned this issue Aug 22, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
tech-debt 💳 Debt that the team has charged and needs to repay
Projects
Zarf
Status: No status
Milestone
No milestone
Development

No branches or pull requests
1 participant
@lucasrod16