Skip to content

Add E2E tests for CfP process#1870

Draft
bouge wants to merge 2 commits intoemfcamp:mainfrom
bouge:main
Draft

Add E2E tests for CfP process#1870
bouge wants to merge 2 commits intoemfcamp:mainfrom
bouge:main

Conversation

@bouge
Copy link
Copy Markdown

@bouge bouge commented Jan 17, 2026

In advance of any changes to CfP -> Content as per issue #1868 - adding E2E testing to CfP to help track any regressions with future changes.

Adds comprehensive end-to-end tests covering the complete CfP lifecycle:

  • Proposal submission - All 6 types via web forms
  • State transitions - Full review workflow from new > finalised
  • Scheduling - CLI commands (schedule, apply_potential_schedule)
  • Favouriting - Add/remove via web routes, schedule visibility
  • Clash detection - overlaps_with(), get_conflicting_content(), ClashFinder tool

Adds CLI wrapper script and additional test container for validation

There is a PR open in the slotmachine repo emfcamp/slotmachine#10 to fix depreciated parameters after a version bump of PuLP.

To test this PR prior to that being merged you can updated pyproject.toml on line 101 to use the forked version from which the PR is raised.

slotmachine = { git = "https://github.com/bouge/slotmachine.git" }

@bouge
Copy link
Copy Markdown
Author

bouge commented Jan 18, 2026

That's my mistake, using default PyCharm linting, will fix

@russss
Copy link
Copy Markdown
Member

russss commented Mar 17, 2026

I will leave @marksteward to pronounce on this one but I fear it may have been missed before the refactor.

@marksteward
Copy link
Copy Markdown
Member

Yeah, I'll rebase this after the merge, sorry @bouge!

@russss
Copy link
Copy Markdown
Member

russss commented Apr 12, 2026

@marksteward are you likely to get around to doing this? Because I reckon it's quickly going to become impossible to get back in sync otherwise

@russss russss marked this pull request as draft April 12, 2026 16:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants