Throttle project saves on dance party levels

[davidsbailey]

Background

work item: https://trello.com/c/P4Ge3bbu/28-turn-off-autosave-for-dance-lab-levels

Description

This implements both flags in the trello item. Once this change reaches levelbuilder, I'll turn the appropriate flags on for the dance levels via the levelbuilder machine to minimize merge conflicts.

This also stops calling onContinue twice when the user clicks Continue on levels without the new skip_run_save flag. I'm relying on existing test coverage to confirm that this doesn't break anything.

[davidsbailey]

CC @sureshc

[joshlory]

Test coverage?

[davidsbailey]

Can you be more specific? Any other feedback?

[joshlory]

Is there existing test coverage for this change, or should we track adding test coverage?

I haven't had a chance to review fully yet — sorry!

[davidsbailey]

Good question! This existing test verifies that basic "continue" functionality still works via dismissing the dialog:

code-dot-org/dashboard/test/ui/features/hour_of_code.feature

Line 30 in beb4517

And block "6" is child of block "5"

I've added a new test which verifies that existing continue behavior works when clicking the continue button.

There is no test coverage for the new functionality yet. I will add something to this PR.

[joshlory]

LGTM!

[davidsbailey]

merging without tests for new functionality because (1) we want some bake time to see if this breaks "continue" behavior for any existing level types, (2) I need to put this down to work on other priorities, and (3) we may be going with a level_sources approach instead of skip_run_save anyway.

I'll revisit writing tests for this before enabling it for any /s/dance levels.