Commit 24e1b46b429585a572b8a619ab10d11ece72eb38
1 parent
82344a24
Exists in
master
and in
3 other branches
Added MAX invalid int and added error message
Signed-off-by: Macartur Sousa <macartur.sc@gmail.com>
Showing
2 changed files
with
11 additions
and
2 deletions
Show diff stats
colab_spb/tests/test_get_list.py
@@ -26,4 +26,7 @@ class ColabSPB(TestCase): | @@ -26,4 +26,7 @@ class ColabSPB(TestCase): | ||
26 | def test_getlist_default_MAX(self): | 26 | def test_getlist_default_MAX(self): |
27 | response = self.client.get("/spb/get_list/?list_name=ListA") | 27 | response = self.client.get("/spb/get_list/?list_name=ListA") |
28 | self.assertEqual(7,len(response.context[1]['latest'])) | 28 | self.assertEqual(7,len(response.context[1]['latest'])) |
29 | - pass | 29 | + |
30 | + def test_getlist_invalid_MAX(self): | ||
31 | + response = self.client.get("/spb/get_list/?list_name=ListA&MAX=") | ||
32 | + self.assertEqual(7,len(response.context[1]['latest'])) |
colab_spb/views.py
1 | +# -*- coding: utf-8 -*- | ||
1 | from django.shortcuts import render | 2 | from django.shortcuts import render |
2 | from django.http import HttpResponse | 3 | from django.http import HttpResponse |
3 | from django.utils.translation import ugettext as _ | 4 | from django.utils.translation import ugettext as _ |
@@ -10,6 +11,9 @@ def get_list(request): | @@ -10,6 +11,9 @@ def get_list(request): | ||
10 | list_name = request.GET.get('list_name',None) | 11 | list_name = request.GET.get('list_name',None) |
11 | MAX = request.GET.get('MAX',7) | 12 | MAX = request.GET.get('MAX',7) |
12 | 13 | ||
14 | + if not MAX: | ||
15 | + MAX = 7 | ||
16 | + | ||
13 | context = {} | 17 | context = {} |
14 | 18 | ||
15 | all_privates = {} | 19 | all_privates = {} |
@@ -38,7 +42,9 @@ def get_list(request): | @@ -38,7 +42,9 @@ def get_list(request): | ||
38 | )) | 42 | )) |
39 | 43 | ||
40 | if len(context['lists']) == 0: | 44 | if len(context['lists']) == 0: |
41 | - return HttpResponse('',status=404) | 45 | + return HttpResponse("""Não foi encontrada lista de discussão a está |
46 | + comunidade, para mais detalhes contacte o | ||
47 | + administrador.""",status=404) | ||
42 | 48 | ||
43 | return render(request,"discussion.html",context) | 49 | return render(request,"discussion.html",context) |
44 | 50 |