Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Questionnaire freetext question crashes with "int" validation and non-integer input #100

Open
markkuriekkinen opened this issue Jan 12, 2021 · 2 comments
Open

Questionnaire freetext question crashes with "int" validation and non-integer input #100

markkuriekkinen opened this issue Jan 12, 2021 · 2 comments
Assignees
@markkuriekkinen
Labels
area: grading quizzes Quiz feature effort: hours The issue seems to be small and should be doable in hours or few days. experience: good first issue Good for newcomers priority: high Important issue requester: internal The issue is raised by a person inside the A+ team type: bug This is a bug.

Comments

@markkuriekkinen
Copy link
Contributor

Make an RST questionnaire with a freetext question that has the int validation (.. freetext:: 1 int). Enter non-integer input to the questionnaire field in A+, for example "test". MOOC-Grader crashes:

ValueError invalid literal for int()

This is probably easy to fix. Just a dumb coding mistake. This should be fixed as soon as possible.
@markkuriekkinen markkuriekkinen added type: bug This is a bug. experience: good first issue Good for newcomers priority: high Important issue area: grading quizzes Quiz feature effort: hours The issue seems to be small and should be doable in hours or few days. requester: internal The issue is raised by a person inside the A+ team labels Jan 12, 2021
@jaguarfi
Copy link
Contributor

I was not able to replicate this error. I actually got the expected validation.

image

image

@markkuriekkinen
Copy link
Contributor Author

Let's leave this open and I can try this again myself. Perhaps I mixed up something in my previous testing.

@markkuriekkinen markkuriekkinen removed this from the January 2021 Release v1.8 milestone Jun 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@markkuriekkinen markkuriekkinen

Labels
area: grading quizzes Quiz feature effort: hours The issue seems to be small and should be doable in hours or few days. experience: good first issue Good for newcomers priority: high Important issue requester: internal The issue is raised by a person inside the A+ team type: bug This is a bug.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@markkuriekkinen @jaguarfi