source-academy / frontend / 33820839998

45%

Build Type

push

github

Committed by web-flow

Commit Message

Apply the course-configured default language in the Playground (#4337)

* Apply the course-configured default language in the Playground

Ground Control's "Default language" selector already persisted a course's
default sublanguage (source_chapter), but nothing under Conductor ever
read it back - the Playground always fell back to the first language
directory entry, so admins changing the default had no visible effect.

LanguageDirectorySaga now derives the course default from source_chapter
(via the existing deriveLanguageFromChapter) and applies it whenever the
directory or the course config load, in whichever order they arrive. A
new isDefaultSelection flag on the languageDirectory slice stops a late
course config from clobbering a language the user (or a share link,
textbook route, or assessment) already picked deliberately.

Playground.tsx's legacy course-language effect is now gated off when
Conductor is enabled - it duplicated this under Conductor and could
throw for course chapters above Source §4 (Python Full, Scheme), which
have no Source-chapter equivalent.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01CYixG1H9x1C9m3KbqR846j

* Wire changeSublanguage into the mock backend; drop unused test import

Ground Control's "Default language" selector dispatches
WorkspaceActions.changeSublanguage, which was only handled by the real
BackendSaga - under the mock backend (REACT_APP_USE_BACKEND=FALSE) the
control silently no-op'd, making this fix impossible to demo without a
live backend. Mirrors the real handler's effect on course config, minus
the network round-trip.

Also drops an unused `expect` import left over in the saga test file.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01CYixG1H9x1C9m3KbqR846j

* Fall back to the first directory entry on an invalid course default too

setCourseConfiguration previously bailed out when the ne... (continued)

Coverage Stats

3132 of 7871 branches covered (39.79%)

Branch coverage included in aggregate %.

28 of 32 new or added lines in 5 files covered. (87.5%)

2 existing lines in 2 files now uncovered.

7107 of 14648 relevant lines covered (48.52%)

31.14 hits per line