From 926e90132dc15d76bb8d16e2f9f1279566cac3c3 Mon Sep 17 00:00:00 2001
From: jpic <jamespic@gmail.com>
Date: Mon, 25 Jan 2016 05:15:42 +0100
Subject: Fixed #25731 -- Removed unused choices kwarg for Select.render()

---
 tests/forms_tests/widget_tests/test_select.py      | 44 +++++++---------------
 .../widget_tests/test_selectmultiple.py            | 20 +++++-----
 2 files changed, 24 insertions(+), 40 deletions(-)

(limited to 'tests/forms_tests')

diff --git a/tests/forms_tests/widget_tests/test_select.py b/tests/forms_tests/widget_tests/test_select.py
index 86ef355c62..185352cda3 100644
--- a/tests/forms_tests/widget_tests/test_select.py
+++ b/tests/forms_tests/widget_tests/test_select.py
@@ -10,14 +10,14 @@ from .base import WidgetTest
 
 
 class SelectTest(WidgetTest):
-    widget = Select()
+    widget = Select
     nested_widget = Select(choices=(
         ('outer1', 'Outer 1'),
         ('Group "1"', (('inner1', 'Inner 1'), ('inner2', 'Inner 2'))),
     ))
 
     def test_render(self):
-        self.check_html(self.widget, 'beatle', 'J', choices=self.beatles, html=(
+        self.check_html(self.widget(choices=self.beatles), 'beatle', 'J', html=(
             """<select name="beatle">
             <option value="J" selected="selected">John</option>
             <option value="P">Paul</option>
@@ -30,7 +30,7 @@ class SelectTest(WidgetTest):
         """
         If the value is None, none of the options are selected.
         """
-        self.check_html(self.widget, 'beatle', None, choices=self.beatles, html=(
+        self.check_html(self.widget(choices=self.beatles), 'beatle', None, html=(
             """<select name="beatle">
             <option value="J">John</option>
             <option value="P">Paul</option>
@@ -44,7 +44,7 @@ class SelectTest(WidgetTest):
         If the value corresponds to a label (but not to an option value), none
         of the options are selected.
         """
-        self.check_html(self.widget, 'beatle', 'John', choices=self.beatles, html=(
+        self.check_html(self.widget(choices=self.beatles), 'beatle', 'John', html=(
             """<select name="beatle">
             <option value="J">John</option>
             <option value="P">Paul</option>
@@ -59,7 +59,7 @@ class SelectTest(WidgetTest):
         """
         choices = [('0', '0'), ('1', '1'), ('2', '2'), ('3', '3'), ('0', 'extra')]
 
-        self.check_html(self.widget, 'choices', '0', choices=choices, html=(
+        self.check_html(self.widget(choices=choices), 'choices', '0', html=(
             """<select name="choices">
             <option value="0" selected="selected">0</option>
             <option value="1">1</option>
@@ -90,8 +90,8 @@ class SelectTest(WidgetTest):
         The value is compared to its str().
         """
         self.check_html(
-            self.widget, 'num', 2,
-            choices=[('1', '1'), ('2', '2'), ('3', '3')],
+            self.widget(choices=[('1', '1'), ('2', '2'), ('3', '3')]),
+            'num', 2,
             html=(
                 """<select name="num">
                 <option value="1">1</option>
@@ -101,8 +101,8 @@ class SelectTest(WidgetTest):
             ),
         )
         self.check_html(
-            self.widget, 'num', '2',
-            choices=[(1, 1), (2, 2), (3, 3)],
+            self.widget(choices=[(1, 1), (2, 2), (3, 3)]),
+            'num', '2',
             html=(
                 """<select name="num">
                 <option value="1">1</option>
@@ -112,8 +112,8 @@ class SelectTest(WidgetTest):
             ),
         )
         self.check_html(
-            self.widget, 'num', 2,
-            choices=[(1, 1), (2, 2), (3, 3)],
+            self.widget(choices=[(1, 1), (2, 2), (3, 3)]),
+            'num', 2,
             html=(
                 """<select name="num">
                 <option value="1">1</option>
@@ -162,25 +162,9 @@ class SelectTest(WidgetTest):
             </select>"""
         ))
 
-    def test_choices_constuctor_and_render(self):
-        """
-        If 'choices' is passed to both the constructor and render(), then
-        they'll both be in the output.
-        """
-        widget = Select(choices=[(1, 1), (2, 2), (3, 3)])
-        self.check_html(widget, 'num', 2, choices=[(4, 4), (5, 5)], html=(
-            """<select name="num">
-            <option value="1">1</option>
-            <option value="2" selected="selected">2</option>
-            <option value="3">3</option>
-            <option value="4">4</option>
-            <option value="5">5</option>
-            </select>"""
-        ))
-
     def test_choices_escaping(self):
         choices = (('bad', 'you & me'), ('good', mark_safe('you &gt; me')))
-        self.check_html(self.widget, 'escape', None, choices=choices, html=(
+        self.check_html(self.widget(choices=choices), 'escape', None, html=(
             """<select name="escape">
             <option value="bad">you &amp; me</option>
             <option value="good">you &gt; me</option>
@@ -189,8 +173,8 @@ class SelectTest(WidgetTest):
 
     def test_choices_unicode(self):
         self.check_html(
-            self.widget, 'email', 'ŠĐĆŽćžšđ',
-            choices=[('ŠĐĆŽćžšđ', 'ŠĐabcĆŽćžšđ'), ('ćžšđ', 'abcćžšđ')],
+            self.widget(choices=[('ŠĐĆŽćžšđ', 'ŠĐabcĆŽćžšđ'), ('ćžšđ', 'abcćžšđ')]),
+            'email', 'ŠĐĆŽćžšđ',
             html=(
                 """<select name="email">
                 <option value="\u0160\u0110\u0106\u017d\u0107\u017e\u0161\u0111" selected="selected">
diff --git a/tests/forms_tests/widget_tests/test_selectmultiple.py b/tests/forms_tests/widget_tests/test_selectmultiple.py
index 502aba3bf4..3430d68113 100644
--- a/tests/forms_tests/widget_tests/test_selectmultiple.py
+++ b/tests/forms_tests/widget_tests/test_selectmultiple.py
@@ -4,11 +4,11 @@ from .base import WidgetTest
 
 
 class SelectMultipleTest(WidgetTest):
-    widget = SelectMultiple()
+    widget = SelectMultiple
     numeric_choices = (('0', '0'), ('1', '1'), ('2', '2'), ('3', '3'), ('0', 'extra'))
 
     def test_render_selected(self):
-        self.check_html(self.widget, 'beatles', ['J'], choices=self.beatles, html=(
+        self.check_html(self.widget(choices=self.beatles), 'beatles', ['J'], html=(
             """<select multiple="multiple" name="beatles">
             <option value="J" selected="selected">John</option>
             <option value="P">Paul</option>
@@ -18,7 +18,7 @@ class SelectMultipleTest(WidgetTest):
         ))
 
     def test_render_multiple_selected(self):
-        self.check_html(self.widget, 'beatles', ['J', 'P'], choices=self.beatles, html=(
+        self.check_html(self.widget(choices=self.beatles), 'beatles', ['J', 'P'], html=(
             """<select multiple="multiple" name="beatles">
             <option value="J" selected="selected">John</option>
             <option value="P" selected="selected">Paul</option>
@@ -31,7 +31,7 @@ class SelectMultipleTest(WidgetTest):
         """
         If the value is None, none of the options are selected.
         """
-        self.check_html(self.widget, 'beatles', None, choices=self.beatles, html=(
+        self.check_html(self.widget(choices=self.beatles), 'beatles', None, html=(
             """<select multiple="multiple" name="beatles">
             <option value="J">John</option>
             <option value="P">Paul</option>
@@ -45,7 +45,7 @@ class SelectMultipleTest(WidgetTest):
         If the value corresponds to a label (but not to an option value), none
         of the options are selected.
         """
-        self.check_html(self.widget, 'beatles', ['John'], choices=self.beatles, html=(
+        self.check_html(self.widget(choices=self.beatles), 'beatles', ['John'], html=(
             """<select multiple="multiple" name="beatles">
             <option value="J">John</option>
             <option value="P">Paul</option>
@@ -58,7 +58,7 @@ class SelectMultipleTest(WidgetTest):
         """
         Multiple options with the same value can be selected (#8103).
         """
-        self.check_html(self.widget, 'choices', ['0'], choices=self.numeric_choices, html=(
+        self.check_html(self.widget(choices=self.numeric_choices), 'choices', ['0'], html=(
             """<select multiple="multiple" name="choices">
             <option value="0" selected="selected">0</option>
             <option value="1">1</option>
@@ -73,7 +73,7 @@ class SelectMultipleTest(WidgetTest):
         If multiple values are given, but some of them are not valid, the valid
         ones are selected.
         """
-        self.check_html(self.widget, 'beatles', ['J', 'G', 'foo'], choices=self.beatles, html=(
+        self.check_html(self.widget(choices=self.beatles), 'beatles', ['J', 'G', 'foo'], html=(
             """<select multiple="multiple" name="beatles">
             <option value="J" selected="selected">John</option>
             <option value="P">Paul</option>
@@ -85,7 +85,7 @@ class SelectMultipleTest(WidgetTest):
     def test_compare_string(self):
         choices = [('1', '1'), ('2', '2'), ('3', '3')]
 
-        self.check_html(self.widget, 'nums', [2], choices=choices, html=(
+        self.check_html(self.widget(choices=choices), 'nums', [2], html=(
             """<select multiple="multiple" name="nums">
             <option value="1">1</option>
             <option value="2" selected="selected">2</option>
@@ -93,7 +93,7 @@ class SelectMultipleTest(WidgetTest):
             </select>"""
         ))
 
-        self.check_html(self.widget, 'nums', ['2'], choices=choices, html=(
+        self.check_html(self.widget(choices=choices), 'nums', ['2'], html=(
             """<select multiple="multiple" name="nums">
             <option value="1">1</option>
             <option value="2" selected="selected">2</option>
@@ -101,7 +101,7 @@ class SelectMultipleTest(WidgetTest):
             </select>"""
         ))
 
-        self.check_html(self.widget, 'nums', [2], choices=choices, html=(
+        self.check_html(self.widget(choices=choices), 'nums', [2], html=(
             """<select multiple="multiple" name="nums">
             <option value="1">1</option>
             <option value="2" selected="selected">2</option>
-- 
cgit v1.3