Skip to content

Commit

Permalink
Merge pull request #4257 from minrk/App.argv
Browse files Browse the repository at this point in the history
fix unicode argv parsing
  • Loading branch information
takluyver committed Sep 23, 2013
2 parents 62e35db + 77b0752 commit 09f6b40
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 1 deletion.
2 changes: 1 addition & 1 deletion IPython/config/application.py
Original file line number Diff line number Diff line change
Expand Up @@ -144,7 +144,7 @@ class Application(SingletonConfigurable):
version = Unicode(u'0.0')

# the argv used to initialize the application
argv = List(Unicode)
argv = List()

# The log level for the application
log_level = Enum((0,10,20,30,40,50,'DEBUG','INFO','WARN','ERROR','CRITICAL'),
Expand Down
5 changes: 5 additions & 0 deletions IPython/config/tests/test_application.py
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
# coding: utf-8
"""
Tests for IPython.config.application.Application
Expand Down Expand Up @@ -185,4 +186,8 @@ def test_extra_args(self):
self.assertEqual(app.bar.b, 5)
self.assertEqual(app.extra_args, ['extra', '--disable', 'args'])

def test_unicode_argv(self):
app = MyApp()
app.parse_command_line(['ünîcødé'])


0 comments on commit 09f6b40

Please sign in to comment.