Closed Bug 996175 Opened 11 years ago Closed 9 years ago

Test for cascade deletes

Categories

(Marketplace Graveyard :: Code Quality, defect, P4)

Product:
Marketplace Graveyard
Component:
Code Quality
Version:
Avenir
Platform:
x86
macOS
Type:
defect

Tracking

(Not tracked)

Status:
RESOLVED WONTFIX

People

(Reporter: robhudson, Assigned: vamsi.lnm)

Details

(Whiteboard: [lang=python][marketplace-transition])

Addons has a test for cascade deletes. We should create one for Webapps. I started one here and then stopped once I collected all the things that can reference the app: https://github.com/robhudson/zamboni/commit/12c1887
Priority: -- → P4
Whiteboard: [contribute]
Whiteboard: [contribute] → [good first bug]
Whiteboard: [good first bug] → [good first bug][lang=python]
Hi Andy/MP-Team, it looks like this might not be very much clear to the newbie how to fix this. Can you put some information here for newbies so that they can pick this bug and fix it. Thanks, Ram
Flags: needinfo?(amckay)
Flags: needinfo?(amckay)
To fix this, one must write new unit tests in https://github.com/mozilla/zamboni/blob/master/mkt/webapps/tests/test_models.py trying to delete Webapps that have things associated to them and making sure everything gets deleted properly. Some tests already exist, but we should add more, testing what happens to Previews and other related objects.
hello sir i am new to open source and i want to work on this and if possible please assign me a mentor
Thanks Assigning this bug to you.
Assignee: nobody → vamsi.lnm
Status: NEW → ASSIGNED
This is not a good first bug, as it is complicated to fix. Please feel free to pick a real goodfirstbug.
Whiteboard: [good first bug][lang=python] → [lang=python]
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → WONTFIX
Whiteboard: [lang=python] → [lang=python][marketplace-transition]
You need to log in before you can comment on or make changes to this bug.