Home Original page

move signin-callout ui tests off of production courses by davidsbailey · Pull Request #71586 · code-dot-org/code-dot-org

Conversation

@davidsbailey

The reason for this change is to make UI tests no longer depend on production courses. For background, see partitioned curriculum data proposal.

Done as part of this PR:

  • added a new ui-test-csf unit / course / course offering via levelbuilder UI
    • mark the unit as 'CSF' to trigger the sign in callout here:
      show_sign_in_callout: csf? || csc?,
  • update signin-callout ui tests to use new course
  • stop seeding the following courses / units in test:
    • express-2017 and pre-express-2017 (already unused)
    • coursea-2020 (freed up by this PR)

Testing story

  • drone shows new ui test is passing
  • the [reset db] commit tag ensures we properly test removal of ui test courses + units from seed.rake

@davidsbailey davidsbailey changed the title remove some csf units from ui test config move signin-callout ui tests off of production courses

Mar 24, 2026

hannahbergam

Choose a reason for hiding this comment

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

Nice- and thank you! What a throwback!

2 participants

@davidsbailey @hannahbergam