Add a test for number of queries in courses api by annagav · Pull Request #3361 · mitodl/mitxonline

annagav · 2026-03-07T00:58:13Z

What are the relevant tickets?

Related to https://github.com/mitodl/hq/issues/10189

Description (What does it do?)

Testing prefetch for course runs and products

How can this be tested?

github-actions · 2026-03-07T00:58:30Z

OpenAPI Changes

Show/hide ## Changes for v0.yaml:

## Changes for v0.yaml:
22 changes: 0 error, 21 warning, 1 info
warning	[response-optional-property-removed] at head/openapi/specs/v0.yaml	
	in API GET /api/v2/course_certificates/{cert_uuid}/
		removed the optional property 'course_run/course/allOf[#/components/schemas/V2Course]/programs/items/display_mode' from the response with the '200' status

warning	[response-optional-property-removed] at head/openapi/specs/v0.yaml	
	in API GET /api/v2/courses/
		removed the optional property 'results/items/programs/items/display_mode' from the response with the '200' status

warning	[response-optional-property-removed] at head/openapi/specs/v0.yaml	
	in API GET /api/v2/courses/{id}/
		removed the optional property 'programs/items/display_mode' from the response with the '200' status

warning	[response-optional-property-removed] at head/openapi/specs/v0.yaml	
	in API GET /api/v2/enrollments/
		removed the optional property '/items/run/allOf[#/components/schemas/V2CourseRunWithCourse]/course/allOf[#/components/schemas/V2Course]/programs/items/display_mode' from the response with the '200' status

warning	[response-optional-property-removed] at head/openapi/specs/v0.yaml	
	in API POST /api/v2/enrollments/
		removed the optional property 'run/allOf[#/components/schemas/V2CourseRunWithCourse]/course/allOf[#/components/schemas/V2Course]/programs/items/display_mode' from the response with the '201' status

warning	[response-optional-property-removed] at head/openapi/specs/v0.yaml	
	in API GET /api/v2/pages/?fields=*&type=cms.coursepage
		removed the optional property 'items/items/course_details/programs/items/display_mode' from the response with the '200' status

warning	[response-optional-property-removed] at head/openapi/specs/v0.yaml	
	in API GET /api/v2/pages/?fields=*&type=cms.programpage
		removed the optional property 'items/items/program_details/display_mode' from the response with the '200' status

warning	[response-optional-property-removed] at head/openapi/specs/v0.yaml	
	in API GET /api/v2/pages/{id}/
		removed the optional property '/oneOf[#/components/schemas/CoursePageItem]/course_details/programs/items/display_mode' from the response with the '200' status

warning	[response-optional-property-removed] at head/openapi/specs/v0.yaml	
	in API GET /api/v2/pages/{id}/
		removed the optional property '/oneOf[#/components/schemas/ProgramPageItem]/program_details/display_mode' from the response with the '200' status

warning	[response-optional-property-removed] at head/openapi/specs/v0.yaml	
	in API GET /api/v2/program_certificates/{cert_uuid}/
		removed the optional property 'program/display_mode' from the response with the '200' status

warning	[response-optional-property-removed] at head/openapi/specs/v0.yaml	
	in API GET /api/v2/program_enrollments/
		removed the optional property '/items/enrollments/items/run/allOf[#/components/schemas/V2CourseRunWithCourse]/course/allOf[#/components/schemas/V2Course]/programs/items/display_mode' from the response with the '200' status

warning	[response-optional-property-removed] at head/openapi/specs/v0.yaml	
	in API GET /api/v2/program_enrollments/
		removed the optional property '/items/program/display_mode' from the response with the '200' status

warning	[response-optional-property-removed] at head/openapi/specs/v0.yaml	
	in API DELETE /api/v2/program_enrollments/{id}/
		removed the optional property '/items/enrollments/items/run/allOf[#/components/schemas/V2CourseRunWithCourse]/course/allOf[#/components/schemas/V2Course]/programs/items/display_mode' from the response with the '200' status

warning	[response-optional-property-removed] at head/openapi/specs/v0.yaml	
	in API DELETE /api/v2/program_enrollments/{id}/
		removed the optional property '/items/program/display_mode' from the response with the '200' status

warning	[response-optional-property-removed] at head/openapi/specs/v0.yaml	
	in API GET /api/v2/program_enrollments/{id}/
		removed the optional property 'enrollments/items/run/allOf[#/components/schemas/V2CourseRunWithCourse]/course/allOf[#/components/schemas/V2Course]/programs/items/display_mode' from the response with the '200' status

warning	[response-optional-property-removed] at head/openapi/specs/v0.yaml	
	in API GET /api/v2/program_enrollments/{id}/
		removed the optional property 'program/display_mode' from the response with the '200' status

warning	[response-optional-property-removed] at head/openapi/specs/v0.yaml	
	in API GET /api/v2/programs/
		removed the optional property 'results/items/display_mode' from the response with the '200' status

warning	[response-optional-property-removed] at head/openapi/specs/v0.yaml	
	in API GET /api/v2/programs/{id}/
		removed the optional property 'display_mode' from the response with the '200' status

warning	[response-optional-property-removed] at head/openapi/specs/v0.yaml	
	in API POST /api/v2/verified_program_enrollments/{program_id}/{courserun_id}/
		removed the optional property 'run/allOf[#/components/schemas/V2CourseRunWithCourse]/course/allOf[#/components/schemas/V2Course]/programs/items/display_mode' from the response with the '201' status

warning	[response-optional-property-removed] at head/openapi/specs/v0.yaml	
	in API GET /api/v3/program_enrollments/
		removed the optional property '/items/program/allOf[#/components/schemas/V3SimpleProgram]/display_mode' from the response with the '200' status

warning	[response-optional-property-removed] at head/openapi/specs/v0.yaml	
	in API GET /api/v3/program_enrollments/{program_id}/
		removed the optional property 'program/allOf[#/components/schemas/V3SimpleProgram]/display_mode' from the response with the '200' status

info	[api-schema-removed] 	
	in components/schemas
		removed the schema 'DisplayModeEnum'



## Changes for v1.yaml:
22 changes: 0 error, 21 warning, 1 info
warning	[response-optional-property-removed] at head/openapi/specs/v1.yaml	
	in API GET /api/v2/course_certificates/{cert_uuid}/
		removed the optional property 'course_run/course/allOf[#/components/schemas/V2Course]/programs/items/display_mode' from the response with the '200' status

warning	[response-optional-property-removed] at head/openapi/specs/v1.yaml	
	in API GET /api/v2/courses/
		removed the optional property 'results/items/programs/items/display_mode' from the response with the '200' status

warning	[response-optional-property-removed] at head/openapi/specs/v1.yaml	
	in API GET /api/v2/courses/{id}/
		removed the optional property 'programs/items/display_mode' from the response with the '200' status

warning	[response-optional-property-removed] at head/openapi/specs/v1.yaml	
	in API GET /api/v2/enrollments/
		removed the optional property '/items/run/allOf[#/components/schemas/V2CourseRunWithCourse]/course/allOf[#/components/schemas/V2Course]/programs/items/display_mode' from the response with the '200' status

warning	[response-optional-property-removed] at head/openapi/specs/v1.yaml	
	in API POST /api/v2/enrollments/
		removed the optional property 'run/allOf[#/components/schemas/V2CourseRunWithCourse]/course/allOf[#/components/schemas/V2Course]/programs/items/display_mode' from the response with the '201' status

warning	[response-optional-property-removed] at head/openapi/specs/v1.yaml	
	in API GET /api/v2/pages/?fields=*&type=cms.coursepage
		removed the optional property 'items/items/course_details/programs/items/display_mode' from the response with the '200' status

warning	[response-optional-property-removed] at head/openapi/specs/v1.yaml	
	in API GET /api/v2/pages/?fields=*&type=cms.programpage
		removed the optional property 'items/items/program_details/display_mode' from the response with the '200' status

warning	[response-optional-property-removed] at head/openapi/specs/v1.yaml	
	in API GET /api/v2/pages/{id}/
		removed the optional property '/oneOf[#/components/schemas/CoursePageItem]/course_details/programs/items/display_mode' from the response with the '200' status

warning	[response-optional-property-removed] at head/openapi/specs/v1.yaml	
	in API GET /api/v2/pages/{id}/
		removed the optional property '/oneOf[#/components/schemas/ProgramPageItem]/program_details/display_mode' from the response with the '200' status

warning	[response-optional-property-removed] at head/openapi/specs/v1.yaml	
	in API GET /api/v2/program_certificates/{cert_uuid}/
		removed the optional property 'program/display_mode' from the response with the '200' status

warning	[response-optional-property-removed] at head/openapi/specs/v1.yaml	
	in API GET /api/v2/program_enrollments/
		removed the optional property '/items/enrollments/items/run/allOf[#/components/schemas/V2CourseRunWithCourse]/course/allOf[#/components/schemas/V2Course]/programs/items/display_mode' from the response with the '200' status

warning	[response-optional-property-removed] at head/openapi/specs/v1.yaml	
	in API GET /api/v2/program_enrollments/
		removed the optional property '/items/program/display_mode' from the response with the '200' status

warning	[response-optional-property-removed] at head/openapi/specs/v1.yaml	
	in API DELETE /api/v2/program_enrollments/{id}/
		removed the optional property '/items/enrollments/items/run/allOf[#/components/schemas/V2CourseRunWithCourse]/course/allOf[#/components/schemas/V2Course]/programs/items/display_mode' from the response with the '200' status

warning	[response-optional-property-removed] at head/openapi/specs/v1.yaml	
	in API DELETE /api/v2/program_enrollments/{id}/
		removed the optional property '/items/program/display_mode' from the response with the '200' status

warning	[response-optional-property-removed] at head/openapi/specs/v1.yaml	
	in API GET /api/v2/program_enrollments/{id}/
		removed the optional property 'enrollments/items/run/allOf[#/components/schemas/V2CourseRunWithCourse]/course/allOf[#/components/schemas/V2Course]/programs/items/display_mode' from the response with the '200' status

warning	[response-optional-property-removed] at head/openapi/specs/v1.yaml	
	in API GET /api/v2/program_enrollments/{id}/
		removed the optional property 'program/display_mode' from the response with the '200' status

warning	[response-optional-property-removed] at head/openapi/specs/v1.yaml	
	in API GET /api/v2/programs/
		removed the optional property 'results/items/display_mode' from the response with the '200' status

warning	[response-optional-property-removed] at head/openapi/specs/v1.yaml	
	in API GET /api/v2/programs/{id}/
		removed the optional property 'display_mode' from the response with the '200' status

warning	[response-optional-property-removed] at head/openapi/specs/v1.yaml	
	in API POST /api/v2/verified_program_enrollments/{program_id}/{courserun_id}/
		removed the optional property 'run/allOf[#/components/schemas/V2CourseRunWithCourse]/course/allOf[#/components/schemas/V2Course]/programs/items/display_mode' from the response with the '201' status

warning	[response-optional-property-removed] at head/openapi/specs/v1.yaml	
	in API GET /api/v3/program_enrollments/
		removed the optional property '/items/program/allOf[#/components/schemas/V3SimpleProgram]/display_mode' from the response with the '200' status

warning	[response-optional-property-removed] at head/openapi/specs/v1.yaml	
	in API GET /api/v3/program_enrollments/{program_id}/
		removed the optional property 'program/allOf[#/components/schemas/V3SimpleProgram]/display_mode' from the response with the '200' status

info	[api-schema-removed] 	
	in components/schemas
		removed the schema 'DisplayModeEnum'



## Changes for v2.yaml:
22 changes: 0 error, 21 warning, 1 info
warning	[response-optional-property-removed] at head/openapi/specs/v2.yaml	
	in API GET /api/v2/course_certificates/{cert_uuid}/
		removed the optional property 'course_run/course/allOf[#/components/schemas/V2Course]/programs/items/display_mode' from the response with the '200' status

warning	[response-optional-property-removed] at head/openapi/specs/v2.yaml	
	in API GET /api/v2/courses/
		removed the optional property 'results/items/programs/items/display_mode' from the response with the '200' status

warning	[response-optional-property-removed] at head/openapi/specs/v2.yaml	
	in API GET /api/v2/courses/{id}/
		removed the optional property 'programs/items/display_mode' from the response with the '200' status

warning	[response-optional-property-removed] at head/openapi/specs/v2.yaml	
	in API GET /api/v2/enrollments/
		removed the optional property '/items/run/allOf[#/components/schemas/V2CourseRunWithCourse]/course/allOf[#/components/schemas/V2Course]/programs/items/display_mode' from the response with the '200' status

warning	[response-optional-property-removed] at head/openapi/specs/v2.yaml	
	in API POST /api/v2/enrollments/
		removed the optional property 'run/allOf[#/components/schemas/V2CourseRunWithCourse]/course/allOf[#/components/schemas/V2Course]/programs/items/display_mode' from the response with the '201' status

warning	[response-optional-property-removed] at head/openapi/specs/v2.yaml	
	in API GET /api/v2/pages/?fields=*&type=cms.coursepage
		removed the optional property 'items/items/course_details/programs/items/display_mode' from the response with the '200' status

warning	[response-optional-property-removed] at head/openapi/specs/v2.yaml	
	in API GET /api/v2/pages/?fields=*&type=cms.programpage
		removed the optional property 'items/items/program_details/display_mode' from the response with the '200' status

warning	[response-optional-property-removed] at head/openapi/specs/v2.yaml	
	in API GET /api/v2/pages/{id}/
		removed the optional property '/oneOf[#/components/schemas/CoursePageItem]/course_details/programs/items/display_mode' from the response with the '200' status

warning	[response-optional-property-removed] at head/openapi/specs/v2.yaml	
	in API GET /api/v2/pages/{id}/
		removed the optional property '/oneOf[#/components/schemas/ProgramPageItem]/program_details/display_mode' from the response with the '200' status

warning	[response-optional-property-removed] at head/openapi/specs/v2.yaml	
	in API GET /api/v2/program_certificates/{cert_uuid}/
		removed the optional property 'program/display_mode' from the response with the '200' status

warning	[response-optional-property-removed] at head/openapi/specs/v2.yaml	
	in API GET /api/v2/program_enrollments/
		removed the optional property '/items/enrollments/items/run/allOf[#/components/schemas/V2CourseRunWithCourse]/course/allOf[#/components/schemas/V2Course]/programs/items/display_mode' from the response with the '200' status

warning	[response-optional-property-removed] at head/openapi/specs/v2.yaml	
	in API GET /api/v2/program_enrollments/
		removed the optional property '/items/program/display_mode' from the response with the '200' status

warning	[response-optional-property-removed] at head/openapi/specs/v2.yaml	
	in API DELETE /api/v2/program_enrollments/{id}/
		removed the optional property '/items/enrollments/items/run/allOf[#/components/schemas/V2CourseRunWithCourse]/course/allOf[#/components/schemas/V2Course]/programs/items/display_mode' from the response with the '200' status

warning	[response-optional-property-removed] at head/openapi/specs/v2.yaml	
	in API DELETE /api/v2/program_enrollments/{id}/
		removed the optional property '/items/program/display_mode' from the response with the '200' status

warning	[response-optional-property-removed] at head/openapi/specs/v2.yaml	
	in API GET /api/v2/program_enrollments/{id}/
		removed the optional property 'enrollments/items/run/allOf[#/components/schemas/V2CourseRunWithCourse]/course/allOf[#/components/schemas/V2Course]/programs/items/display_mode' from the response with the '200' status

warning	[response-optional-property-removed] at head/openapi/specs/v2.yaml	
	in API GET /api/v2/program_enrollments/{id}/
		removed the optional property 'program/display_mode' from the response with the '200' status

warning	[response-optional-property-removed] at head/openapi/specs/v2.yaml	
	in API GET /api/v2/programs/
		removed the optional property 'results/items/display_mode' from the response with the '200' status

warning	[response-optional-property-removed] at head/openapi/specs/v2.yaml	
	in API GET /api/v2/programs/{id}/
		removed the optional property 'display_mode' from the response with the '200' status

warning	[response-optional-property-removed] at head/openapi/specs/v2.yaml	
	in API POST /api/v2/verified_program_enrollments/{program_id}/{courserun_id}/
		removed the optional property 'run/allOf[#/components/schemas/V2CourseRunWithCourse]/course/allOf[#/components/schemas/V2Course]/programs/items/display_mode' from the response with the '201' status

warning	[response-optional-property-removed] at head/openapi/specs/v2.yaml	
	in API GET /api/v3/program_enrollments/
		removed the optional property '/items/program/allOf[#/components/schemas/V3SimpleProgram]/display_mode' from the response with the '200' status

warning	[response-optional-property-removed] at head/openapi/specs/v2.yaml	
	in API GET /api/v3/program_enrollments/{program_id}/
		removed the optional property 'program/allOf[#/components/schemas/V3SimpleProgram]/display_mode' from the response with the '200' status

info	[api-schema-removed] 	
	in components/schemas
		removed the schema 'DisplayModeEnum'

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

for more information, see https://pre-commit.ci

Add a test for number of queries in courses api

1ba7816

pre-commit-ci bot and others added 3 commits March 7, 2026 01:00

[pre-commit.ci] auto fixes from pre-commit.com hooks

b56acd8

for more information, see https://pre-commit.ci

reducing products queries

e743fdc

is_upgradable

b0c01b8

annagav force-pushed the ag/optimise_prefetch branch from 4bc4512 to b0c01b8 Compare March 11, 2026 01:49

annagav marked this pull request as ready for review March 11, 2026 01:49

[pre-commit.ci] auto fixes from pre-commit.com hooks

3f21621

for more information, see https://pre-commit.ci

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Add a test for number of queries in courses api#3361

Add a test for number of queries in courses api#3361
annagav wants to merge 5 commits intomainfrom
ag/optimise_prefetch

annagav commented Mar 7, 2026

Uh oh!

github-actions bot commented Mar 7, 2026 •

edited

Loading

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

Conversation

annagav commented Mar 7, 2026

What are the relevant tickets?

Description (What does it do?)

How can this be tested?

Uh oh!

github-actions bot commented Mar 7, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

OpenAPI Changes

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

github-actions bot commented Mar 7, 2026 •

edited

Loading