Skip to content

Add --make-codes flag to b2b_courseware call in test_datagen and language fixes#3552

Open
dsubak wants to merge 2 commits intomainfrom
dansubak/202605_make_codes_like_we_used_to
Open

Add --make-codes flag to b2b_courseware call in test_datagen and language fixes#3552
dsubak wants to merge 2 commits intomainfrom
dansubak/202605_make_codes_like_we_used_to

Conversation

@dsubak
Copy link
Copy Markdown
Contributor

@dsubak dsubak commented May 6, 2026

Description (What does it do?)

This script has been busted for a bit - there's been a fair number of changes around language handling, filtering out source runs in some cases and changing the implicit provisioning of enrollment codes in certain codepaths. This attempts to fix the showstopping errors.

test_datagen is of limited utility at the moment, but it can still be useful to provision a fresh database with some structured data. It did provision data for use with e2e testing over in learn, but unfortunately since that doesn't run anywhere, both the tests and this script have not been kept up to date w/ behavioral changes. These changes probably aren't perfect and make no attempt to verify and fix the e2e tests, but they unblock use for bootstrapping local devs.

How can this be tested?

  • On a fresh local, run test_datagen. You should observe that after the queued task runs you have enrollment codes for the provisioned contract. You should be able to see it at /staff-dashboard/discounts if you've set up a user with admin access

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 6, 2026

OpenAPI Changes

Show/hide ## Changes for v0.yaml:
## Changes for v0.yaml:
No changes detected

## Changes for v1.yaml:
No changes detected

## Changes for v2.yaml:
No changes detected

Unexpected changes? Ensure your branch is up-to-date with main (consider rebasing).

@dsubak dsubak changed the title Add --make-codes flag to b2b_courseware call in test_datagen Add --make-codes flag to b2b_courseware call in test_datagen and language fixes May 6, 2026
@dsubak dsubak marked this pull request as ready for review May 6, 2026 20:59
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.

1 participant