FAIL: test that pushed functions have access to globals #386

Closed
fperez opened this Issue Apr 14, 2011 · 4 comments

Projects

None yet

2 participants

@fperez
Member
fperez commented Apr 14, 2011

This failure started a few days ago, but I haven't bisected it yet.

Making it critical because it's in the master branch, and we should try to keep master always failure-clean.

iptest IPython.parallel
======================================================================
FAIL: test that pushed functions have access to globals
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/fperez/usr/lib/python2.6/site-packages/IPython/parallel/tests/test_view.py", line 118, in test_push_function_globals
    self.assertRaisesRemote(NameError, v.execute, 'b=f()')
  File "/home/fperez/usr/lib/python2.6/site-packages/IPython/parallel/tests/clienttest.py", line 91, in assertRaisesRemote
    self.fail("should have raised a RemoteError")
AssertionError: should have raised a RemoteError
    """Fail immediately, with the given message."""
>>  raise self.failureException, 'should have raised a RemoteError'
    

----------------------------------------------------------------------
Ran 58 tests in 34.498s

FAILED (failures=1)
@minrk minrk was assigned Apr 14, 2011
@minrk
Member
minrk commented Apr 14, 2011

I'll look into this - it doesn't fail on my Linux or OSX machines (but does on Windows)

@fperez
Member
fperez commented Apr 14, 2011

On Thu, Apr 14, 2011 at 3:15 PM, minrk
reply@reply.github.com
wrote:

I'll look into this - it doesn't fail on my Linux or OSX machines (but does on Windows)

Very odd that it doesn't fail for you on linux. If you want a login
on my linux desktop so you can test here, ping me off-list.

@minrk
Member
minrk commented Apr 14, 2011

Okay, I believe that I have fixed this in my windows branch by ensuring that every test starts with a clean namespace. I think I've just about finished getting the parallel tests to pass on Windows, and I'll mark there when it's ready to merge.

@minrk
Member
minrk commented Apr 15, 2011

Merging #374 will automatically close this issue.

@minrk minrk closed this in 0d3d807 Apr 21, 2011
@mattvonrocketstein mattvonrocketstein pushed a commit to mattvonrocketstein/ipython that referenced this issue Nov 3, 2014
@minrk minrk Start each test with clear engine namespaces
closes gh-386
e1dae99
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment