Skip to content

chore: add flipper flag for school - #770

Merged
DNR500 merged 1 commit into
mainfrom
get-test-school-working-with-scratch
Apr 8, 2026
Merged

chore: add flipper flag for school#770
DNR500 merged 1 commit into
mainfrom
get-test-school-working-with-scratch

Conversation

@DNR500

@DNR500 DNR500 commented Apr 8, 2026

Copy link
Copy Markdown
Contributor

Status

Points for consideration:

What's changed?

Sets up the scratch feature on the testing school for the integration test

Copilot AI review requested due to automatic review settings April 8, 2026 09:32
@cla-bot cla-bot Bot added the cla-signed label Apr 8, 2026
@DNR500 DNR500 changed the title chore: add flipper flag for schooko chore: add flipper flag for school Apr 8, 2026
@DNR500
DNR500 enabled auto-merge (squash) April 8, 2026 09:33

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds support for the scratch feature (cat_mode Flipper flag) on the test school used for development and testing. The changes enable the feature during test seed initialization and add a corresponding test to verify the flag is properly set.

Changes:

  • Enable the cat_mode Flipper flag for the test school during seed creation in the rake task
  • Add a test to verify that the cat_mode flag is enabled for the seeded school

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
lib/tasks/test_seeds.rake Adds Flipper.enable_actor :cat_mode, school to enable scratch feature on test school during seed creation
spec/lib/test_seeds_spec.rb Adds test to verify that cat_mode Flipper flag is enabled for the seeded school

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@github-actions

github-actions Bot commented Apr 8, 2026

Copy link
Copy Markdown

Test coverage

90.02% line coverage reported by SimpleCov.
Run: https://github.com/RaspberryPiFoundation/editor-api/actions/runs/24128505846

@DNR500
DNR500 merged commit 1a6106e into main Apr 8, 2026
9 checks passed
@DNR500
DNR500 deleted the get-test-school-working-with-scratch branch April 8, 2026 09:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants