Skip to content

Fix for GH9992 - use create_tempfile to remove permission-related errors when running tests #10080

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.

Sign up for GitHub

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
wants to merge 5 commits into from
Closed

Fix for GH9992 - use create_tempfile to remove permission-related errors when running tests #10080

Changes from all commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
ccdb73d
ENH: Add StringMethods.partition and rpartition
sinhrks Apr 1, 2015
ae3e70d
Corrected typo in Grammar of Graphics
FvD May 7, 2015
3c7438b
Use temp files to fix GH9992.
May 7, 2015
e6c2bab
Use temp files to fix GH9992.
May 7, 2015
9695069
Merge branch 'master' of github.com:vikramls/pandas
May 7, 2015
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
42 changes: 22 additions & 20 deletions pandas/io/tests/test_pytables.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -156,50 +156,51 @@ def tearDown(self):
pass

def test_factory_fun(self):
path = create_tempfile(self.path)
try:
with get_store(self.path) as tbl:
with get_store(path) as tbl:
raise ValueError('blah')
except ValueError:
pass
finally:
safe_remove(self.path)
safe_remove(path)

try:
with get_store(self.path) as tbl:
with get_store(path) as tbl:
tbl['a'] = tm.makeDataFrame()

with get_store(self.path) as tbl:
with get_store(path) as tbl:
self.assertEqual(len(tbl), 1)
self.assertEqual(type(tbl['a']), DataFrame)
finally:
safe_remove(self.path)

def test_context(self):
path = create_tempfile(self.path)
try:
with HDFStore(self.path) as tbl:
with HDFStore(path) as tbl:
raise ValueError('blah')
except ValueError:
pass
finally:
safe_remove(self.path)
safe_remove(path)

try:
with HDFStore(self.path) as tbl:
with HDFStore(path) as tbl:
tbl['a'] = tm.makeDataFrame()

with HDFStore(self.path) as tbl:
with HDFStore(path) as tbl:
self.assertEqual(len(tbl), 1)
self.assertEqual(type(tbl['a']), DataFrame)
finally:
safe_remove(self.path)
safe_remove(path)

def test_conv_read_write(self):

path = create_tempfile(self.path)
try:

def roundtrip(key, obj,**kwargs):
obj.to_hdf(self.path, key,**kwargs)
return read_hdf(self.path, key)
obj.to_hdf(path, key,**kwargs)
return read_hdf(path, key)

o = tm.makeTimeSeries()
assert_series_equal(o, roundtrip('series',o))
Expand All @@ -215,12 +216,12 @@ def roundtrip(key, obj,**kwargs):

# table
df = DataFrame(dict(A=lrange(5), B=lrange(5)))
df.to_hdf(self.path,'table',append=True)
result = read_hdf(self.path, 'table', where = ['index>2'])
df.to_hdf(path,'table',append=True)
result = read_hdf(path, 'table', where = ['index>2'])
assert_frame_equal(df[df.index>2],result)

finally:
safe_remove(self.path)
safe_remove(path)

def test_long_strings(self):

Expand Down Expand Up @@ -4329,13 +4330,14 @@ def do_copy(f = None, new_f = None, keys = None, propindexes = True, **kwargs):
df = tm.makeDataFrame()

try:
st = HDFStore(self.path)
path = create_tempfile(self.path)
st = HDFStore(path)
st.append('df', df, data_columns = ['A'])
st.close()
do_copy(f = self.path)
do_copy(f = self.path, propindexes = False)
do_copy(f = path)
do_copy(f = path, propindexes = False)
finally:
safe_remove(self.path)
safe_remove(path)

def test_legacy_table_write(self):
raise nose.SkipTest("cannot write legacy tables")
Expand Down