fix(backend): Fix submitter of revoked sequences to be the revoker

[theosanderson]

Fixes #3244

https://theosanderson-fix-revoked.loculus.org/

Update the submitter of a revoked sequence to the revoker

• Modify backend/src/main/kotlin/org/loculus/backend/service/submission/SubmissionDatabaseService.kt to set the submitterColumn to the revoker's account during the revocation process
• Change the revoke function to use the revoker's username for the submitterColumn instead of cloning the old submitter information

Tested that autoapprove no longer happens:

---

For more details, open the Copilot Workspace session.

[corneliusroemer]

Thanks! Oof, how did this live so long without us noticing 😬

[corneliusroemer]

I'm adding a test for this in #3248

[corneliusroemer]

@fengelniederhammer maybe you have some thoughts on how to better test the created sequence entries - we've now got a bunch of helpers, that's not terrible but maybe there's a neater way you know. Also, we might want to test similar basic things explicitly for other parts of the submission flow.

[fengelniederhammer]

we've now got a bunch of helpers

The helpers already look pretty helpful to me. The tests are nice to read.

[corneliusroemer]

@fengelniederhammer I just wasn't sure whether creating a helper for each property was a good way to go. Could come up with a better way (limited Kotlin knowledge).

[corneliusroemer]

Let's merge? @theosanderson :)