From a28615e59ad1eb7a4eefb80cdadf413c5491c8fb Mon Sep 17 00:00:00 2001
From: Tom Walker <tomwalker0472@gmail.com>
Date: Wed, 26 Nov 2014 11:55:50 +0000
Subject: [PATCH] small cleanup, mostly pep8 related modifications

---
 essay/models.py       |  2 +-
 multichoice/models.py |  5 ++---
 quiz/models.py        | 10 +++-------
 quiz/tests.py         | 13 +++----------
 true_false/models.py  |  2 +-
 5 files changed, 10 insertions(+), 22 deletions(-)

diff --git a/essay/models.py b/essay/models.py
index de8dcb1..af02e7a 100644
--- a/essay/models.py
+++ b/essay/models.py
@@ -1,4 +1,4 @@
-from __future__ import unicode_literals # for Py2 & Py3 compatibility
+from __future__ import unicode_literals
 from django.utils.encoding import python_2_unicode_compatible
 from django.utils.translation import ugettext as _
 from quiz.models import Question
diff --git a/multichoice/models.py b/multichoice/models.py
index f7f3d1a..22ca026 100644
--- a/multichoice/models.py
+++ b/multichoice/models.py
@@ -1,5 +1,5 @@
-from __future__ import unicode_literals # for Py2 & Py3 compatibility
-from django.utils.encoding import python_2_unicode_compatible # ditto
+from __future__ import unicode_literals
+from django.utils.encoding import python_2_unicode_compatible
 from django.utils.translation import ugettext as _
 from django.db import models
 from quiz.models import Question
@@ -75,4 +75,3 @@ class Answer(models.Model):
     class Meta:
         verbose_name = _("Answer")
         verbose_name_plural = _("Answers")
-
diff --git a/quiz/models.py b/quiz/models.py
index f2a4974..64913d2 100644
--- a/quiz/models.py
+++ b/quiz/models.py
@@ -1,4 +1,4 @@
-from __future__ import unicode_literals # for Py2 & Py3 compatibility
+from __future__ import unicode_literals
 import re
 import json
 
@@ -36,7 +36,6 @@ class Category(models.Model):
         verbose_name = _("Category")
         verbose_name_plural = _("Categories")
 
-    # Changed __unicode__() to __str__() for Py2 & Py3 compatibility
     def __str__(self):
         return self.category
 
@@ -57,8 +56,7 @@ class SubCategory(models.Model):
     class Meta:
         verbose_name = _("Sub-Category")
         verbose_name_plural = _("Sub-Categories")
-    
-    # Changed __unicode__() to __str__() for Py2 & Py3 compatibility
+
     def __str__(self):
         return self.sub_category + " (" + self.category.category + ")"
 
@@ -153,7 +151,6 @@ class Quiz(models.Model):
         verbose_name = _("Quiz")
         verbose_name_plural = _("Quizzes")
 
-    # Changed __unicode__() to __str__() for Py2 & Py3 compatibility
     def __str__(self):
         return self.title
 
@@ -526,6 +523,7 @@ class Sitting(models.Model):
         total = self.get_max_score
         return answered, total
 
+
 @python_2_unicode_compatible
 class Question(models.Model):
     """
@@ -572,7 +570,5 @@ class Question(models.Model):
         verbose_name_plural = _("Questions")
         ordering = ['category']
 
-    # Changed __unicode__() to __str__() for Py2 & Py3 compatibility
     def __str__(self):
         return self.content
-
diff --git a/quiz/tests.py b/quiz/tests.py
index 50caa80..1b18905 100644
--- a/quiz/tests.py
+++ b/quiz/tests.py
@@ -22,20 +22,13 @@ from essay.models import Essay_Question
 
 class TestCategory(TestCase):
     def setUp(self):
-        self.c1 = Category.objects.new_category(category='elderberries')
-        self.c2 = Category.objects.new_category(category='straw.berries')
-        self.c3 = Category.objects.new_category(category='black berries')
-        self.c4 = Category.objects.new_category(category='squishy   berries')
+        self.c1 = Category.objects.new_category(category='squishy   berries')
 
         self.sub1 = SubCategory.objects.create(sub_category='Red',
                                                category=self.c1)
 
     def test_categories(self):
-
-        self.assertEqual(self.c1.category, 'elderberries')
-        self.assertEqual(self.c2.category, 'straw.berries')
-        self.assertEqual(self.c3.category, 'black-berries')
-        self.assertEqual(self.c4.category, 'squishy-berries')
+        self.assertEqual(self.c1.category, 'squishy-berries')
 
     def test_sub_categories(self):
         self.assertEqual(self.sub1.category, self.c1)
@@ -148,7 +141,7 @@ class TestProgress(TestCase):
     def test_subcategory_all_empty(self):
         SubCategory.objects.create(sub_category='pickles',
                                    category=self.c1)
-        self.p1.list_all_cat_scores
+        # self.p1.list_all_cat_scores
         # self.assertIn('pickles', self.p1.score)
         # TODO: test after implementing subcategory scoring on progress page
 
diff --git a/true_false/models.py b/true_false/models.py
index 159c973..7bf1adc 100644
--- a/true_false/models.py
+++ b/true_false/models.py
@@ -1,4 +1,4 @@
-from __future__ import unicode_literals # for Py2 & Py3 compatibility
+from __future__ import unicode_literals
 from django.utils.encoding import python_2_unicode_compatible
 from django.utils.translation import ugettext as _
 from django.db import models
-- 
2.39.5