diff options
| author | Alexis Metaireau <alexis@notmyidea.org> | 2012-09-02 13:53:01 +0200 |
|---|---|---|
| committer | Alexis Metaireau <alexis@notmyidea.org> | 2012-09-02 13:53:01 +0200 |
| commit | 6a544855d7f04b3ed5cdc4465bdbd44ae98e8be2 (patch) | |
| tree | 575130ddd8e63cacb62868a67f135856f69e36bc | |
| parent | e75809dddf7947e2db0d7eab7e5b14775ed6dba0 (diff) | |
| download | ihatemoney-mirror-6a544855d7f04b3ed5cdc4465bdbd44ae98e8be2.zip ihatemoney-mirror-6a544855d7f04b3ed5cdc4465bdbd44ae98e8be2.tar.gz ihatemoney-mirror-6a544855d7f04b3ed5cdc4465bdbd44ae98e8be2.tar.bz2 | |
Fix project deletion. fix #87
| -rw-r--r-- | budget/templates/edit_project.html | 2 | ||||
| -rw-r--r-- | budget/tests.py | 18 | ||||
| -rw-r--r-- | budget/web.py | 5 |
3 files changed, 22 insertions, 3 deletions
diff --git a/budget/templates/edit_project.html b/budget/templates/edit_project.html index 585d231..0240eff 100644 --- a/budget/templates/edit_project.html +++ b/budget/templates/edit_project.html @@ -3,7 +3,7 @@ {% block js %} $('#delete-project').click(function () { - $(this).html("<a style='color:red; ' href='{{ url_for('.remove_project') }}' >{{_("you sure?")}}</a>"); + $(this).html("<a style='color:red; ' href='{{ url_for('.delete_project') }}' >{{_("you sure?")}}</a>"); }); {% endblock %} diff --git a/budget/tests.py b/budget/tests.py index cbcefec..d127b06 100644 --- a/budget/tests.py +++ b/budget/tests.py @@ -147,6 +147,24 @@ class BudgetTestCase(TestCase): # no new project added self.assertEqual(len(models.Project.query.all()), 1) + def test_project_deletion(self): + + with run.app.test_client() as c: + c.post("/create", data={ + 'name': 'raclette party', + 'id': 'raclette', + 'password': 'party', + 'contact_email': 'raclette@notmyidea.org' + }) + + # project added + self.assertEqual(len(models.Project.query.all()), 1) + + c.get('/raclette/delete') + + # project removed + self.assertEqual(len(models.Project.query.all()), 0) + def test_membership(self): self.post_project("raclette") self.login("raclette") diff --git a/budget/web.py b/budget/web.py index 6097d80..489874c 100644 --- a/budget/web.py +++ b/budget/web.py @@ -205,9 +205,10 @@ def edit_project(): return render_template("edit_project.html", form=form) -@main.route("/<project_id>/delete", methods=["POST"]) -def remove_project(): +@main.route("/<project_id>/delete") +def delete_project(): g.project.remove_project() + flash(_('Project successfully deleted')) return redirect(url_for(".home")) |
