-
Notifications
You must be signed in to change notification settings - Fork 0
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
delete categorizations in test setup #156
Conversation
Thanks. |
@ZhongweiL , please watch the tests and let me know if they pass. |
@khatchad They shouldn't pass because we're writing tests before implementing the changes, but I don't think I understand the error message, they seem like some low-level issues with Django. |
What changes haven't you implemented?
Raffi Khatchadourian ***@***.***>
Assistant Professor, Computer Science, Hunter College
Doctoral Faculty of the Graduate School and University Center's Ph.D. Program in Computer Science
City University of New York (CUNY)
http://cs.hunter.cuny.edu/~Raffi.Khatchadourian99
On Dec 7, 2021 5:51 PM, Zhongwei Li ***@***.***> wrote:
@khatchad<https://github.com/khatchad> They shouldn't pass because we're writing tests before implementing the changes, but I don't think I understand the error message, they seem like some low-level issues with Django.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub<#156 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AAPUGP37GTFSUFUNSCKLVATUP2FXXANCNFSM5JSLMNHA>.
|
Please investigate.
Raffi Khatchadourian ***@***.***>
Assistant Professor, Computer Science, Hunter College
Doctoral Faculty of the Graduate School and University Center's Ph.D. Program in Computer Science
City University of New York (CUNY)
http://cs.hunter.cuny.edu/~Raffi.Khatchadourian99
On Dec 7, 2021 5:51 PM, Zhongwei Li ***@***.***> wrote:
@khatchad<https://github.com/khatchad> They shouldn't pass because we're writing tests before implementing the changes, but I don't think I understand the error message, they seem like some low-level issues with Django.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub<#156 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AAPUGP37GTFSUFUNSCKLVATUP2FXXANCNFSM5JSLMNHA>.
|
@khatchad Never mind, the changes for this test have been implemented in the model and merged, but I don't think it has been migrated to the remote database, which may indicate that the automatic deployment has failed. Or it might be because that the automatic deployment was added after the change was merged. |
resolves the bug in PR #148 with deleting categorizers by making sure the categorization table is empty in test setup