test(integration-tests): add multi pathogen revocation test#5616
Merged
fengelniederhammer merged 3 commits intomainfrom Dec 8, 2025
Merged
test(integration-tests): add multi pathogen revocation test#5616fengelniederhammer merged 3 commits intomainfrom
fengelniederhammer merged 3 commits intomainfrom
Conversation
Contributor
Author
|
The test works. It found #5619. |
fengelniederhammer
changed the title
integration-tests/tests/specs/features/multipathogen-submission-flow.spec.ts
Show resolved
Hide resolved
corneliusroemer
force-pushed
the
evSearchTests
branch
from
bdd885d to
da090ca
Compare
|
Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits. |
corneliusroemer
approved these changes
Dec 3, 2025
2 tasks
Contributor
|
should this be rebased on #5626? |
anna-parker
reviewed
Dec 4, 2025
integration-tests/tests/specs/features/multipathogen-submission-flow.spec.ts
Show resolved
Hide resolved
Contributor
|
@fengelniederhammer Shall we merge this one to prevent more conflicts piling up in the future? |
Contributor
Author
I'd delay it until #5382 is merged to main. This PR is 4 layers up in the stack of PR. I'd rather just rebase once more before the final merge and not bother with rebasing those PRs until then. It's a waste of time. |
fengelniederhammer
force-pushed
the
no_waitForLoculusId
branch
from
124ad02 to
d3b6807
Compare
fengelniederhammer
force-pushed
the
evSearchTests
branch
from
da090ca to
1acb7e2
Compare
maverbiest
pushed a commit
that referenced
this pull request
Dec 8, 2025
Expect test failure, see #5619 🚀 Preview: Add `preview` label to enable --------- Co-authored-by: Cornelius Roemer <cornelius.roemer@gmail.com>
fengelniederhammer
added a commit
that referenced
this pull request
Dec 9, 2025
…entries (#5626) resolves #5619 ### Screenshot ### PR Checklist ~~- [ ] All necessary documentation has been adapted.~~ - [x] The implemented feature is covered by appropriate, automated tests. - #5616 - [x] Any manual testing that has been done is documented (i.e. what exactly was tested?) Anya notes: revoked the sequence LOC_003MSV5.1 and confirmed page still works: <img width="2358" height="486" alt="image" src="https://github.com/user-attachments/assets/87c78c53-196e-40de-81a0-bda6021ca713" /> <img width="2582" height="692" alt="image" src="https://github.com/user-attachments/assets/0a4c812f-7835-4c92-8e05-8444a0f90d9c" /> 🚀 Preview: https://5619-missing-genotype-of.loculus.org
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Expect test failure, see #5619
🚀 Preview: Add
previewlabel to enable