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

Elekto needs comprehensive testing #88

Open
4 tasks
jberkus opened this issue May 10, 2023 · 2 comments
Open
4 tasks

Elekto needs comprehensive testing #88

jberkus opened this issue May 10, 2023 · 2 comments
Assignees
Labels
help wanted Extra attention is needed

Comments

@jberkus
Copy link
Member

jberkus commented May 10, 2023

Currently, elekto has some limited E2E testing, mostly checking that the code loads without errors and that the vote tallying is correct.

This isn't enough. Here's the additional testing we need:

  • unit tests for all modules, particularly ones that run as presubmit tests
  • complete E2E testing that checks that an election can be created, run, and closed
  • permissions testing that checks that pages are accessible in the correct user context
  • container image testing on build
@jberkus jberkus added the help wanted Extra attention is needed label Jul 19, 2023
@nimbinatus
Copy link

Finally got some breathing room, so picking this up as promised (ok, promised nearly a year ago, but still promised). (Do slash commands work in this repo? I'm guessing not.)

@jberkus
Copy link
Member Author

jberkus commented Feb 26, 2024

Nope!

If you wanna deploy Prow for Elekto, I'm fan. But it's probably overkill.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

2 participants