Re: PR #3390 (more options work) merged into devel
On Sat, Jun 17, 2023 at 10:29 AM Thomas Passin wrote: FAILED leo/unittests/core/test_leoImport.py::TestLeoImport::test_recursive_import - AssertionError: 'path: ekr-mypy2/mypy' != 'path: mypy' == 1 failed, 851 passed, 1 skipped, 1 warning in 14.21s == Not a surprise since it's looking for a specific directory on @ekr's > machine. > Well, this failure surprised me :-) The test calls only RecursiveImportController.minimize_headline, and this method does only string (path) computations. The method does not test whether the path exists. However, changing 'ekr-mypy2' to 'non-existent-directory' does in fact fail on my machine. I'm on it. Edward -- You received this message because you are subscribed to the Google Groups "leo-editor" group. To unsubscribe from this group and stop receiving emails from it, send an email to leo-editor+unsubscr...@googlegroups.com. To view this discussion on the web visit https://groups.google.com/d/msgid/leo-editor/CAMF8tS3iViqY4fd-_4jkusZKaNzhiyS5Biy5hS7O%3DRMe%2BzUJxQ%40mail.gmail.com.
Re: PR #3390 (more options work) merged into devel
Run from the leo-editor directory: py -m pytest [snip] leo\unittests\core\test_leoImport.py:106: AssertionError warnings summary leo\plugins\leo_babel\tests\lib_test.py:118 C:\Tom\git\leo-editor\leo\plugins\leo_babel\tests\lib_test.py:118: PytestCollectionWarning: cannot collect test class 'TestCmdr' because it has a __init__ constructor (from: leo/plugins/leo_babel/tests/lib_test.py) class TestCmdr: -- Docs: https://docs.pytest.org/en/stable/how-to/capture-warnings.html short test summary info = FAILED leo/unittests/core/test_leoImport.py::TestLeoImport::test_recursive_import - AssertionError: 'path: ekr-mypy2/mypy' != 'path: mypy' == 1 failed, 851 passed, 1 skipped, 1 warning in 14.21s == Not a surprise since it's looking for a specific directory on @ekr's machine. -- You received this message because you are subscribed to the Google Groups "leo-editor" group. To unsubscribe from this group and stop receiving emails from it, send an email to leo-editor+unsubscr...@googlegroups.com. To view this discussion on the web visit https://groups.google.com/d/msgid/leo-editor/d985d11d-2499-4b4b-bf01-d07ee0151fcbn%40googlegroups.com.
Re: PR #3390 (more options work) merged into devel
On Saturday, June 17, 2023 at 9:44:55 AM UTC-5 Edward K. Ream wrote: >...the new unit tests found more errors in two hours than in the previous four days. The test of LM.scanOptions knows nothing about scanOptions. The test is valid however code works. In other words, this test should have been added 20 years ago! Edward -- You received this message because you are subscribed to the Google Groups "leo-editor" group. To unsubscribe from this group and stop receiving emails from it, send an email to leo-editor+unsubscr...@googlegroups.com. To view this discussion on the web visit https://groups.google.com/d/msgid/leo-editor/22467543-28c3-4fe2-85b9-5288560fcd52n%40googlegroups.com.
Re: PR #3390 (more options work) merged into devel
On Saturday, June 17, 2023 at 9:36:25 AM UTC-5 Edward K. Ream wrote: I've merged this branch early for several reasons I prefer speed over agonizing :-) I want devel to contain the latest code so we can all test it. The devel branch should be safe enough: the new unit tests found more errors in two hours than in the previous four days. Any remaining problems will be fixed in a *small* PR. Edward -- You received this message because you are subscribed to the Google Groups "leo-editor" group. To unsubscribe from this group and stop receiving emails from it, send an email to leo-editor+unsubscr...@googlegroups.com. To view this discussion on the web visit https://groups.google.com/d/msgid/leo-editor/67637131-e44b-4ff2-a94f-348190069e53n%40googlegroups.com.