Skip to content

Add lesson to class with remixed project #547

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 10 commits into
base: main
Choose a base branch
from

Conversation

loiswells97
Copy link
Contributor

@loiswells97 loiswells97 commented May 20, 2025

Status

Work in Progress: All the functionality should already be in place, but tests need adding as detailed below.

TODO

Add tests for the following:

  • /lesson/remix request specs:
    • If teacher teaches the class and the original project is public, there is a 200 response
    • If successful, data is returned in the correct format
    • If the teacher does not teach the class, there is a Forbidden response
    • If the original project is not public, there is a Forbidden response
    • If the user is a student, there is a Forbidden response
  • Remix project operation
    • When only the school_id is provided in the remix params, calling the operation increases the project count by 1
    • When only the school_id is provided in the remix params, calling the operation increases the component count by the number of components in the original project
      Note there is already a draft test for the above, but it's failing, so we need to figure out why
  • Remix lesson operation
    • Creates lesson with the correct attributes (name, school, class, user etc.)
    • Remixes the original project and associates the remixed project with the new lesson
    • Errors as expected

@cla-bot cla-bot bot added the cla-signed label May 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants