Add directive supporting the use of remote data in doctests #2

Activity

1. drdavella

   added

   enhancement

   on Oct 2, 2017
2. bsipocz commented on Jul 17, 2018

   

   bsipocz

   on Jul 17, 2018

   Member

   More actions

   Related issue: astropy/astropy#2830
3. kakirastern commented on Mar 29, 2019

   

   kakirastern

   on Mar 29, 2019

   More actions

   Hi, is it okay for me to work on this issue?

   👍React with 👍1Reacted by Brigitta Sipőcz
4. kakirastern commented on Mar 29, 2019

   

   kakirastern

   on Mar 29, 2019

   More actions

   Thanks @bsipocz for the approval! Will follow up on this soon after investigating on the issue further.
5. bsipocz commented on Mar 29, 2019

   

   bsipocz

   on Mar 29, 2019

   Member

   More actions

   you don't need to ask for permission to work on issues. If there are work in progress it is most probably cross linked somewhere in the issues.
6. kakirastern commented on Mar 29, 2019

   

   kakirastern

   on Mar 29, 2019

   More actions

   Yup @bsipocz, understood.
7. kakirastern commented on Mar 29, 2019

   

   kakirastern

   on Mar 29, 2019

   Last edited by kakirastern

   More actions

   I am guessing (albeit an educated one) that I will need to modify at least the file at "pytest-doctestplus/pytest_doctestplus/output_checker.py", plus some other tests too...

   Or in hindsight some file in the folder "pytest-doctestplus/pytest_doctestplus"
8. kakirastern commented on Mar 29, 2019

   

   kakirastern

   on Mar 29, 2019

   Last edited by kakirastern

   More actions

   So something in the line of astropy/astropy#2830 (comment) are to be implemented...

   class DoctestRemoteDirective(Directive):
       ... do something with the pytest config.getoption('remote_data') ...
   
   def setup(app):
       app.add_directive('doctest-remote', DoctestRemoteDirective)
       app.add_directive('doctest-remote-all', DoctestRemoteDirective)
   

   • Write a sphinx extension (Reference: http://www.sphinx-doc.org/en/stable/extdev/tutorial.html)
   • Get config into the DoctestRemoteDirective's namespace, where config refers to an object found in "pytest_plugins.py", or some file to the same effect
9. kakirastern commented on Mar 29, 2019

   

   kakirastern

   on Mar 29, 2019

   Last edited by kakirastern

   More actions

   Note: Some other reference which may be useful regarding the OutputChecker and DocTestRunner classes relevant to the doctest.register_optionflag flag option in the proposed PR: https://docs.python.org/3/library/doctest.html#doctest.register_optionflag
10. pllim

    added

    🔥CriticalCritical

    on Aug 3, 2020
11. bsipocz

    closed this as completedin #137on Jul 1, 2021