jmespath / jmespath.py

JMESPath is a query language for JSON.
http://jmespath.org
MIT License
2.19k stars 181 forks source link

0.10.0: pytest warnings #222

Closed kloczek closed 2 years ago

kloczek commented 3 years ago
+ /usr/bin/python3 -Bm pytest -ra
=========================================================================== test session starts ============================================================================
platform linux -- Python 3.8.9, pytest-6.2.3, py-1.10.0, pluggy-0.13.1
rootdir: /home/tkloczko/rpmbuild/BUILD/jmespath.py-0.10.0
plugins: forked-1.3.0, shutil-1.7.0, virtualenv-1.7.0, expect-1.1.0, cov-2.11.1, httpbin-1.0.0, xdist-2.2.1, flake8-1.0.7, timeout-1.4.2, betamax-0.8.1, pyfakefs-4.4.0, freezegun-0.4.2, cases-3.4.6, case-1.5.3, isort-1.3.0, aspectlib-1.5.2, flaky-3.7.0, mock-3.6.0, hypothesis-6.12.0, asyncio-0.15.1
collected 80 items

extra/test_hypothesis.py .......                                                                                                                                     [  8%]
tests/test_compliance.py x                                                                                                                                           [ 10%]
tests/test_custom_functions.py .                                                                                                                                     [ 11%]
tests/test_functions.py .....                                                                                                                                        [ 17%]
tests/test_lexer.py ...................                                                                                                                              [ 41%]
tests/test_parser.py ..........................................                                                                                                      [ 93%]
tests/test_search.py ....s                                                                                                                                           [100%]

============================================================================= warnings summary =============================================================================
tests/test_compliance.py:18
  tests/test_compliance.py:18: PytestCollectionWarning: yield tests were removed in pytest 4.0 - test_compliance will be ignored
    def test_compliance():

tests/test_lexer.py::TestRegexLexer::test_adds_quotes_when_invalid_json
tests/test_lexer.py::TestRegexLexer::test_literal_string
tests/test_lexer.py::TestRegexLexer::test_literal_with_empty_string
  /home/tkloczko/rpmbuild/BUILD/jmespath.py-0.10.0/jmespath/lexer.py:169: PendingDeprecationWarning: deprecated string literal syntax
    warnings.warn("deprecated string literal syntax",

tests/test_lexer.py::TestRegexLexer::test_unknown_character_with_identifier
  /home/tkloczko/rpmbuild/BUILD/jmespath.py-0.10.0/tests/test_lexer.py:155: DeprecationWarning: Please use assertRaisesRegex instead.
    with self.assertRaisesRegexp(LexerError, "Unknown token"):

tests/test_parser.py::TestErrorMessages::test_bad_unicode_string
  /home/tkloczko/rpmbuild/BUILD/jmespath.py-0.10.0/tests/test_parser.py:172: DeprecationWarning: Please use assertRaisesRegex instead.
    with self.assertRaisesRegexp(exceptions.LexerError, error_message):

-- Docs: https://docs.pytest.org/en/stable/warnings.html
========================================================================= short test summary info ==========================================================================
SKIPPED [1] tests/test_search.py:55: Test requires long() type
XFAIL tests/test_compliance.py::test_compliance
  reason: [NOTRUN] yield tests were removed in pytest 4.0 - test_compliance will be ignored
===================================================== 78 passed, 1 skipped, 1 xfailed, 6 warnings in 607.59s (0:10:07) =====================================================
nateprewitt commented 2 years ago

Hi @kloczek, this was fixed with https://github.com/jmespath/jmespath.py/commit/f61b0cd087e965e10602234615ede87d02f7644c.

kloczek commented 2 years ago

That commit do not applies on top of last release cleanly. Will retest on next release. Do you plan maybe make new release soon? 🤔 Thank you 😄