magic_install_default_config test fails when there is no .ipython directory #467

Closed
juliantaylor opened this Issue May 24, 2011 · 1 comment

Comments

Projects
None yet
2 participants
@juliantaylor
Contributor

juliantaylor commented May 24, 2011

when there is no .ipython directory in home and XDG_CONFIG the doctest of
IPython.core.magic.Magic.magic_install_default_config
fails as it outputs

Installing default config file: $HOME/.ipython/ipython_config.py

which is not expected

$ PATH=$PWD/IPython/scripts/:$PATH PYTHONPATH=$PWD IPython/scripts/iptest -v IPython.core
======================================================================
FAIL: Doctest: IPython.core.magic.Magic.magic_install_default_config
----------------------------------------------------------------------
Traceback (most recent call last):
File "/tmp/buildd/ipython-0.11/IPython/testing/plugin/ipdoctest.py", line 265, in runTest
raise self.failureException(self.format_failure(new.getvalue()))
AssertionError: Failed doctest test for IPython.core.magic.Magic.magic_install_default_config
File "/tmp/buildd/ipython-0.11/IPython/core/magic.py", line 3363, in magic_install_default_config

----------------------------------------------------------------------
File "/tmp/buildd/ipython-0.11/IPython/core/magic.py", line 3370, in IPython.core.magic.Magic.magic_install_default_config
Failed example:
    get_ipython().magic(u"install_default_config ")
Expected nothing
Got:
    Installing default config file: /tmp/buildd/.ipython/ipython_config.py

>>  raise self.failureException(self.format_failure(<StringIO.StringIO instance at 0x30c1a70>.getvalue()))

@takluyver takluyver closed this in 34e2283 May 24, 2011

@takluyver

This comment has been minimized.

Show comment
Hide comment
@takluyver

takluyver May 24, 2011

Member

That should sort it - it's not really a test, just an example, so I've made the test machinery ignore it.

Member

takluyver commented May 24, 2011

That should sort it - it's not really a test, just an example, so I've made the test machinery ignore it.

mattvonrocketstein pushed a commit to mattvonrocketstein/ipython that referenced this issue Nov 3, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment