Skip to content

Commit

Permalink
Merge pull request #1046 from avalentino/bugfix/compatibility-with-nu…
Browse files Browse the repository at this point in the history
…mexpr-2.8.5

Fix compatibility with numexpr v2.8.5
  • Loading branch information
avalentino committed Aug 17, 2023
2 parents 5a3bc7e + 1a23549 commit 7df790d
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 3 deletions.
2 changes: 2 additions & 0 deletions RELEASE_NOTES.rst
Expand Up @@ -19,6 +19,8 @@ Changes from 3.8.0 to 3.9.0
* lz4 v1.9.4
* zlib v1.2.13

- Fix compatibility with numexpr v2.8.5.


Changes from 3.7.0 to 3.8.0
===========================
Expand Down
12 changes: 9 additions & 3 deletions tables/tests/test_queries.py
Expand Up @@ -640,14 +640,20 @@ def test_syntax_error(self):
def test_unsupported_object(self):
"""Using a condition with an unsupported object."""

self.assertRaises(TypeError, self.table.where, '[]')
self.assertRaises(
(TypeError, ValueError), self.table.where, '[]'
)
self.assertRaises(TypeError, self.table.where, 'obj', {'obj': {}})
self.assertRaises(TypeError, self.table.where, 'c_bool < []')
self.assertRaises(
(TypeError, ValueError), self.table.where, 'c_bool < []'
)

def test_unsupported_syntax(self):
"""Using a condition with unsupported syntax."""

self.assertRaises(TypeError, self.table.where, 'c_bool[0]')
self.assertRaises(
(TypeError, ValueError), self.table.where, 'c_bool[0]'
)
self.assertRaises(TypeError, self.table.where, 'c_bool()')
self.assertRaises(NameError, self.table.where, 'c_bool.__init__')

Expand Down

0 comments on commit 7df790d

Please sign in to comment.