Skip to content
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

Test failures in Sphinx project #316

Open
kgilpin opened this issue May 24, 2024 · 1 comment
Open

Test failures in Sphinx project #316

kgilpin opened this issue May 24, 2024 · 1 comment
Labels
bug Something isn't working investigate needs further investigation

Comments

@kgilpin
Copy link
Contributor

kgilpin commented May 24, 2024

https://github.com/sphinx-doc/sphinx

============================================================= FAILURES =============================================================
_______________________________________________________ test_is_serializable _______________________________________________________

    def test_is_serializable() -> None:
        subject = [1, [2, {3, 'a'}], {'x': {'y': frozenset((4, 5))}}]
        check_is_serializable(subject, circular=False)
    
        a, b = [1], [2]  # type: (CircularList, CircularList)
        a.append(b)
        b.append(a)
>       check_is_serializable(a, circular=True)

tests/test_config/test_config.py:59: 
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ 
venv/lib/python3.10/site-packages/_appmap/wrapt/wrappers.py:622: in __call__
    return self._self_wrapper(self.__wrapped__, self._self_instance,
venv/lib/python3.10/site-packages/_appmap/instrument.py:129: in instrumented_fn
    return call_instrumented(f, instance, args, kwargs)
venv/lib/python3.10/site-packages/_appmap/instrument.py:86: in call_instrumented
    params = CallEvent.set_params(f.params, instance, args, kwargs)
venv/lib/python3.10/site-packages/_appmap/event.py:268: in set_params
    ret.append(p.to_dict(value))
venv/lib/python3.10/site-packages/_appmap/event.py:171: in to_dict
    ret.update(describe_value(self.name, value))
venv/lib/python3.10/site-packages/_appmap/event.py:114: in describe_value
    ret.update(_describe_schema(name, val, 0, max_depth))
venv/lib/python3.10/site-packages/_appmap/event.py:101: in _describe_schema
    schema = [_describe_schema(k, v, depth + 1, max_depth) for k, v in elts]
venv/lib/python3.10/site-packages/_appmap/event.py:101: in <listcomp>
    schema = [_describe_schema(k, v, depth + 1, max_depth) for k, v in elts]
venv/lib/python3.10/site-packages/_appmap/event.py:101: in _describe_schema
    schema = [_describe_schema(k, v, depth + 1, max_depth) for k, v in elts]
venv/lib/python3.10/site-packages/_appmap/event.py:101: in _describe_schema
... many lines
    schema = [_describe_schema(k, v, depth + 1, max_depth) for k, v in elts]
venv/lib/python3.10/site-packages/_appmap/event.py:101: in <listcomp>
    schema = [_describe_schema(k, v, depth + 1, max_depth) for k, v in elts]
venv/lib/python3.10/site-packages/_appmap/event.py:101: in _describe_schema
    schema = [_describe_schema(k, v, depth + 1, max_depth) for k, v in elts]
venv/lib/python3.10/site-packages/_appmap/event.py:101: in <listcomp>
    schema = [_describe_schema(k, v, depth + 1, max_depth) for k, v in elts]
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ 

name = None, val = 2, depth = 476, max_depth = 5

    def _describe_schema(name, val, depth, max_depth):
    
        val_type = type(val)
    
        ret = {}
        if name is not None:
            ret["name"] = name
>       ret["class"] = fqname(val_type)
E       RecursionError: maximum recursion depth exceeded

venv/lib/python3.10/site-packages/_appmap/event.py:88: RecursionError
-------------------------------------------------------- Captured log call ---------------------------------------------------------
INFO     _appmap.recording:recording.py:77 writing /Users/kgilpin/source/land-of-apps/sphinx/tmp/appmap/pytest/test_config_pickle_circular_reference_in_list.appmap.json
============================================================ XFAILURES =============================================================
______________________________________________________ test_restify_Annotated ______________________________________________________

    @pytest.mark.xfail(sys.version_info[:2] <= (3, 11), reason='Needs fixing.')
    def test_restify_Annotated():
>       assert restify(Annotated[str, "foo", "bar"]) == ':py:class:`~typing.Annotated`\\ [:py:class:`str`]'
E       AssertionError: assert ':py:class:`A...:class:`str`]' == ':py:class:`~...:class:`str`]'
E         
E         - :py:class:`~typing.Annotated`\ [:py:class:`str`]
E         ?            --------
E         + :py:class:`Annotated`\ [:py:class:`str`]

tests/test_util/test_util_typing.py:192: AssertionError
-------------------------------------------------------- Captured log call ---------------------------------------------------------
INFO     _appmap.recording:recording.py:77 writing /Users/kgilpin/source/land-of-apps/sphinx/tmp/appmap/pytest/test_restify_Annotated.appmap.json
========================================================= warnings summary =========================================================
venv/lib/python3.10/site-packages/_pytest/config/__init__.py:1285
  /Users/kgilpin/source/land-of-apps/sphinx/venv/lib/python3.10/site-packages/_pytest/config/__init__.py:1285: PytestAssertRewriteWarning: Module already imported so cannot be rewritten: _appmap
    self._mark_plugins_for_rewrite(hook)

venv/lib/python3.10/site-packages/_pytest/config/__init__.py:1285
  /Users/kgilpin/source/land-of-apps/sphinx/venv/lib/python3.10/site-packages/_pytest/config/__init__.py:1285: PytestAssertRewriteWarning: Module already imported so cannot be rewritten: appmap
    self._mark_plugins_for_rewrite(hook)

tests/test_builders/test_build.py: 1 warning
tests/test_builders/test_build_linkcheck.py: 31 warnings
  /Users/kgilpin/source/land-of-apps/sphinx/sphinx/builders/linkcheck.py:86: RemovedInSphinx80Warning: The default value for 'linkcheck_report_timeouts_as_broken' will change to False in Sphinx 8, meaning that request timeouts will be reported with a new 'timeout' status, instead of as 'broken'. This is intended to provide more detail as to the failure mode. See https://github.com/sphinx-doc/sphinx/issues/11868 for details.
    warnings.warn(deprecation_msg, RemovedInSphinx80Warning, stacklevel=1)

-- Docs: https://docs.pytest.org/en/stable/how-to/capture-warnings.html
===================================================== short test summary info ======================================================
SKIPPED [1] tests/test_builders/test_build_epub.py:375: Skipped because DO_EPUBCHECK is not set
SKIPPED [6] tests/test_builders/test_build_latex.py:107: not running latex, the required styles do not seem to be installed
SKIPPED [1] tests/test_builders/test_build_latex.py:1198: not running latex, the required styles do not seem to be installed
SKIPPED [1] tests/test_builders/test_build_latex.py:1610: not running latex, the required styles do not seem to be installed
SKIPPED [1] tests/test_directives/test_directives_no_typesetting.py:44: c:function does not support :no-index: option
SKIPPED [1] tests/test_directives/test_directives_no_typesetting.py:44: cpp:function does not support :no-index: option
SKIPPED [1] tests/test_directives/test_directives_no_typesetting.py:56: c:function does not support :no-index: option
SKIPPED [1] tests/test_directives/test_directives_no_typesetting.py:56: cpp:function does not support :no-index: option
SKIPPED [1] tests/test_directives/test_directives_no_typesetting.py:71: rst:directive does not support :no-index-entry: option
SKIPPED [1] tests/test_directives/test_directives_no_typesetting.py:71: cmdoption does not support :no-index-entry: option
SKIPPED [1] tests/test_directives/test_directives_no_typesetting.py:71: envvar does not support :no-index-entry: option
SKIPPED [1] tests/test_extensions/test_ext_math.py:25: Requires dvipng" binary
SKIPPED [1] tests/test_extensions/test_ext_math.py:45: Requires dvisvgm" binary
SKIPPED [1] tests/test_extensions/test_ext_math.py:66: Requires dvisvgm" binary
XFAIL tests/test_util/test_util_typing.py::test_restify_Annotated - Needs fixing.
FAILED tests/test_config/test_config.py::test_is_serializable - RecursionError: maximum recursion depth exceeded
FAILED tests/test_config/test_config.py::test_config_pickle_circular_reference_in_list - RecursionError: maximum recursion depth exceeded
========================= 2 failed, 2130 passed, 19 skipped, 1 xfailed, 34 warnings in 5464.35s (1:31:04) ==========================
@kgilpin kgilpin added the bug Something isn't working label May 24, 2024
@apotterri
Copy link
Collaborator

This should be rechecked with the latest version of the agent: tests/test_config/test_config.py:59. It's possible that the fix to disable recording when rendering parameters has fixed it.

@apotterri apotterri added the investigate needs further investigation label Jul 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working investigate needs further investigation
Projects
None yet
Development

No branches or pull requests

2 participants