The pytest-markdown-docs from modal-labs

Warning thrown from `pytest` Re: PytestRemovedIn8Warning

Hi there!

Was just trying this plugin for a project I manage, and I came across a few warnings:

=========================================================================================== warnings summary =========================================================================================== 
..\..\..\..\..\Python311\Lib\site-packages\pytest_markdown_docs\plugin.py:234
..\..\..\..\..\Python311\Lib\site-packages\pytest_markdown_docs\plugin.py:234
..\..\..\..\..\Python311\Lib\site-packages\pytest_markdown_docs\plugin.py:234
..\..\..\..\..\Python311\Lib\site-packages\pytest_markdown_docs\plugin.py:234
..\..\..\..\..\Python311\Lib\site-packages\pytest_markdown_docs\plugin.py:234
..\..\..\..\..\Python311\Lib\site-packages\pytest_markdown_docs\plugin.py:234
  C:\Python311\Lib\site-packages\pytest_markdown_docs\plugin.py:234: PytestRemovedIn8Warning: The (fspath: py.path.local) argument to MarkdownTextFile is deprecated. Please use the (path: pathlib.Path) argument instead.
  See https://docs.pytest.org/en/latest/deprecations.html#fspath-argument-for-node-constructors-replaced-with-pathlib-path
    return MarkdownTextFile.from_parent(parent, fspath=path)

..\..\..\..\..\Python311\Lib\site-packages\pytest_markdown_docs\plugin.py:232
..\..\..\..\..\Python311\Lib\site-packages\pytest_markdown_docs\plugin.py:232
  C:\Python311\Lib\site-packages\pytest_markdown_docs\plugin.py:232: PytestRemovedIn8Warning: The (fspath: py.path.local) argument to MarkdownDocstringCodeModule is deprecated. Please use the (path: pathlib.Path) argument instead.
  See https://docs.pytest.org/en/latest/deprecations.html#fspath-argument-for-node-constructors-replaced-with-pathlib-path
    return MarkdownDocstringCodeModule.from_parent(parent, fspath=path)

-- Docs: https://docs.pytest.org/en/stable/how-to/capture-warnings.html
==================================================================================== 2 passed, 8 warnings in 0.65s =====================================================================================

System Information

Operating System: Windows 11
pytest Version: 7.2.0
Python Version: 3.11.0

It seems this might be the result of a pending change in pytest, but perhaps I'm misreading that. Any thoughts?

Thank you for this exciting project!

Broken with pytest v8

This doesn't work with the recent pytest v8. I saw there is a pull request with a tentative fix: #17

Can this be verified and added as a new version that can be pulled via pip?

Additional markers like `notest` are not compatible with MkDocs

Hi,

first of all, thanks for your work on this project! I was really happy when I realized that I could test the code snippets in my docs :)

Unfortunately, I noticed that specifying additional verbs like notest in a code fence, e.g.

```python notest
foobar = "baz"
```

results in detection errors of code blocks when using Material for MkDocs:

This is how it looks without `notest`

A possible workaround would be allowing users to specify notest (or fixtures) within HTML comments above the code block, e.g.

<!-- notest -->
```python
foobar = "baz"
```

If this makes sense for you, I'm happy to give it a try in a PR :)

modal-labs / pytest-markdown-docs Goto Github PK

pytest-markdown-docs's People

Contributors

Stargazers

Watchers

Forkers

pytest-markdown-docs's Issues

Warning thrown from `pytest` Re: PytestRemovedIn8Warning

System Information

Broken with pytest v8

Additional markers like `notest` are not compatible with MkDocs

Recommend Projects

React

Vue.js

Typescript

TensorFlow

Django

Laravel

D3

Recommend Topics

javascript

web

server

Machine learning

Visualization

Game

Recommend Org

Facebook

Microsoft

Google

Alibaba

D3

Tencent

Jobs