Skip to content

test_json:test_round_trip_exception_ fails on python3 in windows #5874

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
ghost opened this issue Jan 8, 2014 · 4 comments · Fixed by #5894
Closed

test_json:test_round_trip_exception_ fails on python3 in windows #5874

ghost opened this issue Jan 8, 2014 · 4 comments · Fixed by #5894
Labels
IO JSON read_json, to_json, json_normalize Windows Windows OS
Milestone

Comments

@ghost
Copy link

ghost commented Jan 8, 2014

The API design issue discussed in #5655 again. pd.read_json guesses that a 150KB long string is a filename.

======================================================================
ERROR: test_round_trip_exception_ (pandas.io.tests.test_json.test_pandas.TestPandasContainer)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "C:\workspace\pandas_tests@2\BITS\64\PYTHONVER\33\pandas\util\testing.py", line 1057, in network_wrapper
    return t(*args, **kwargs)
  File "C:\workspace\pandas_tests@2\BITS\64\PYTHONVER\33\pandas\io\tests\test_json\test_pandas.py", line 593, in test_round_trip_exception_
    result = pd.read_json(s)
  File "C:\workspace\pandas_tests@2\BITS\64\PYTHONVER\33\pandas\io\json.py", line 176, in read_json
    if os.path.exists(filepath_or_buffer):
  File "c:\envs\33-64\lib\genericpath.py", line 18, in exists
    os.stat(path)
nose.proxy.ValueError: ValueError: path too long for Windows
-------------------- >> begin captured stdout << ---------------------
Failed: ValueError('path too long for Windows',)

--------------------- >> end captured stdout << ----------------------
@jreback
Copy link
Contributor

jreback commented Jan 9, 2014

@y-p how does the new ci pick up the commits?

@ghost
Copy link
Author

ghost commented Jan 9, 2014

The CI boxes submit the results.

@jreback
Copy link
Contributor

jreback commented Jan 9, 2014

http://scatterci.github.io/ScatterCI-Pandas/

boo ya! fixed

@ghost
Copy link
Author

ghost commented Jan 9, 2014

with eyes closed and one shoe tied behind your back :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
IO JSON read_json, to_json, json_normalize Windows Windows OS
Projects
None yet
1 participant