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

Introduce (large) CSV import in e2e tests #2912

Merged
merged 20 commits into from
Nov 7, 2023
Merged

Conversation

jaegeral
Copy link
Collaborator

@jaegeral jaegeral commented Sep 27, 2023

Introducing csv upload end2end tests

they do not trigger
#2796. Fix for that and e2e tests will come with a different PR
Because the file was not touched a for a while, it also contains some formatting.

@jaegeral jaegeral added the Data import All things that are with importing data label Sep 27, 2023
@jaegeral jaegeral requested a review from berggren September 27, 2023 08:27
@jaegeral jaegeral added this to the Release: 20231025 milestone Sep 27, 2023
@jaegeral jaegeral added the Bug label Sep 27, 2023
@berggren berggren changed the title [import_client] Removing a data upload in streamer close Removing a data upload in streamer close Sep 29, 2023
@berggren berggren changed the title Removing a data upload in streamer close Remove data upload in streamer close Sep 29, 2023
@berggren berggren changed the title Remove data upload in streamer close Remove data upload when the streamer closes Sep 29, 2023
@jaegeral jaegeral requested a review from berggren October 4, 2023 10:02
@jaegeral jaegeral changed the title Remove data upload when the streamer closes Introduce (large) CSV import in e2e tests Oct 4, 2023
@jaegeral jaegeral added Code Health Code health improvements and removed Bug labels Oct 4, 2023
@berggren
Copy link
Contributor

Fix the e2e test

@jaegeral
Copy link
Collaborator Author

All green

@berggren berggren removed this from the Release: 20231025 milestone Oct 25, 2023
@berggren berggren added this to the Release: 20231206 milestone Oct 25, 2023
@jaegeral jaegeral merged commit 9d51409 into google:master Nov 7, 2023
24 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Code Health Code health improvements Data import All things that are with importing data
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants