Use set literals

jdufresne · jdufresne · commit 0426748c66c4 · 2017-04-17T20:48:49.000-07:00
Avoids unnecessary, temporary list.
diff --git a/debug_toolbar/settings.py b/debug_toolbar/settings.py
@@ -15,7 +15,7 @@
 
 CONFIG_DEFAULTS = {
     # Toolbar options
-    'DISABLE_PANELS': set(['debug_toolbar.panels.redirects.RedirectsPanel']),
+    'DISABLE_PANELS': {'debug_toolbar.panels.redirects.RedirectsPanel'},
     'INSERT_BEFORE': '</body>',
     'JQUERY_URL': '//ajax.googleapis.com/ajax/libs/jquery/2.2.4/jquery.min.js',
     'RENDER_PANELS': None,
diff --git a/docs/configuration.rst b/docs/configuration.rst
@@ -52,7 +52,7 @@ Toolbar options
 
 * ``DISABLE_PANELS``
 
-  Default: ``set(['debug_toolbar.panels.redirects.RedirectsPanel'])``
+  Default: ``{'debug_toolbar.panels.redirects.RedirectsPanel'}``
 
   This setting is a set of the full Python paths to each panel that you
   want disabled (but still displayed) by default.
diff --git a/tests/panels/test_sql.py b/tests/panels/test_sql.py
@@ -26,7 +26,7 @@ def tearDown(self):
 
     def test_disabled(self):
         config = {
-            'DISABLE_PANELS': set(['debug_toolbar.panels.sql.SQLPanel'])
+            'DISABLE_PANELS': {'debug_toolbar.panels.sql.SQLPanel'}
         }
         self.assertTrue(self.panel.enabled)
         with self.settings(DEBUG_TOOLBAR_CONFIG=config):
diff --git a/tests/panels/test_template.py b/tests/panels/test_template.py
@@ -76,8 +76,7 @@ def test_custom_context_processor(self):
 
     def test_disabled(self):
         config = {
-            'DISABLE_PANELS': set([
-                'debug_toolbar.panels.templates.TemplatesPanel'])
+            'DISABLE_PANELS': {'debug_toolbar.panels.templates.TemplatesPanel'}
         }
         self.assertTrue(self.panel.enabled)
         with self.settings(DEBUG_TOOLBAR_CONFIG=config):

Original file line number	Diff line number	Diff line change
`@@ -26,7 +26,7 @@ def tearDown(self):`
`26`	`26`
`27`	`27`	`def test_disabled(self):`
`28`	`28`	`config = {`
`29`		`- 'DISABLE_PANELS': set(['debug_toolbar.panels.sql.SQLPanel'])`
	`29`	`+ 'DISABLE_PANELS': {'debug_toolbar.panels.sql.SQLPanel'}`
`30`	`30`	`}`
`31`	`31`	`self.assertTrue(self.panel.enabled)`
`32`	`32`	`with self.settings(DEBUG_TOOLBAR_CONFIG=config):`
Original file line number	Diff line number	Diff line change
`@@ -76,8 +76,7 @@ def test_custom_context_processor(self):`
`76`	`76`
`77`	`77`	`def test_disabled(self):`
`78`	`78`	`config = {`
`79`		`- 'DISABLE_PANELS': set([`
`80`		`- 'debug_toolbar.panels.templates.TemplatesPanel'])`
	`79`	`+ 'DISABLE_PANELS': {'debug_toolbar.panels.templates.TemplatesPanel'}`
`81`	`80`	`}`
`82`	`81`	`self.assertTrue(self.panel.enabled)`
`83`	`82`	`with self.settings(DEBUG_TOOLBAR_CONFIG=config):`