Re: PR #3389 (in devel) concludes work on handling options

[Edward K. Ream]

On Sat, Jun 17, 2023 at 1:13 AM Viktor Ransmayr 
wrote:


I tested the 'leo-editor:devel' branch again in a Fedora 37 VM.
>
> Running Leo's unit tests reports one error:
>

Thanks for this report. I have modified the unit test so it fails on my
machine. I'll fix this issue next.

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/CAMF8tS327gc%2B0J9pDM1_YGp8HENst0zaJfLkhDddw4%3DLy%2BT6xg%40mail.gmail.com.

Re: PR #3389 (in devel) concludes work on handling options

[Viktor Ransmayr]

Hello Edward,

Edward K. Ream schrieb am Freitag, 16. Juni 2023 um 23:37:42 UTC+2:

PR #3389  defines four 
simple helper functions in leoGlobals.py.

Imo these four functions do more than the entire argparse module. They are 
simpler, easier and more flexible.

Thanks again to Thomas for his careful reading of the code.


I tested the 'leo-editor:devel' branch again in a Fedora 37 VM.

Running Leo's unit tests reports one error:

::

(Leo) [user@fedora-leo-study-vm Leo]$ 
(Leo) [user@fedora-leo-study-vm Leo]$ cd leo-editor/leo
(Leo) [user@fedora-leo-study-vm leo]$ python -m unittest















..F.

s...
.
(python:1804): Gdk-CRITICAL **: 08:03:14.903: gdk_atom_intern: 
assertion 'atom_name != NULL' failed

(python:1804): Gdk-CRITICAL **: 08:03:14.903: gdk_atom_intern: 
assertion 'atom_name != NULL' failed

.s..
cursesGui2.py: Tk module required for clipboard 
handling.
s.
==
FAIL: test_recursive_import 
(unittests.core.test_leoImport.TestLeoImport.test_recursive_import)
--
Traceback (most recent call last):
  File 
"/home/user/PyVE/GitHub/Leo/leo-editor/leo/unittests/core/test_leoImport.py", 
line 106, in test_recursive_import
self.assertEqual(root.h, expected)
AssertionError: 'path: ekr-mypy2/mypy' != 'path: mypy'
- path: ekr-mypy2/mypy
+ path: mypy


--
Ran 851 tests in 5.954s

FAILED (failures=1, skipped=3)
(Leo) [user@fedora-leo-study-vm leo]$ 

 With kind regards,

Viktor

-- 
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/6d2f6741-05ef-43ab-b9ea-95e681668fb9n%40googlegroups.com.

Re: PR #3389 (in devel) concludes work on handling options

[Edward K. Ream]

On Friday, June 16, 2023 at 9:32:42 PM UTC-5 Félix wrote:

devel fails this single test on windows (not sure if related or if its a 
real problem) :


The test works for me with python 3.11 on Windows. I have no idea how it 
could fail for you.

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/29a33fe9-28f9-47cb-b879-9be2682a0181n%40googlegroups.com.

Re: PR #3389 (in devel) concludes work on handling options

[Félix]

devel fails this single test on windows (not sure if related or if its a 
real problem) :

./leo/unittests/core/test_leoImport.py::TestLeoImport::test_recursive_import 
Failed: 'path: ekr-mypy2/mypy' != 'path: mypy'
- path: ekr-mypy2/mypy
+ path: mypy

  File "C:\Python311\Lib\unittest\case.py", line 57, in testPartExecutor
yield
  File "C:\Python311\Lib\unittest\case.py", line 623, in run
self._callTestMethod(testMethod)
  File "C:\Python311\Lib\unittest\case.py", line 579, in _callTestMethod
if method() is not None:
   
  File "c:\prog\leo-editor\leo\unittests\core\test_leoImport.py", line 106, 
in test_recursive_import
self.assertEqual(root.h, expected)
AssertionError: 'path: ekr-mypy2/mypy' != 'path: mypy'
- path: ekr-mypy2/mypy
+ path: mypy

On Friday, June 16, 2023 at 5:37:42 PM UTC-4 Edward K. Ream wrote:

> PR #3389  defines 
> four simple helper functions in leoGlobals.py.
>
> Imo these four functions do more than the entire argparse module. They are 
> simpler, easier and more flexible.
>
> Thanks again to Thomas for his careful reading of the code.
>
> 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/e8957739-b6fd-4cea-b440-8f8cff1bcbean%40googlegroups.com.

PR #3389 (in devel) concludes work on handling options

[Edward K. Ream]

PR #3389  defines four 
simple helper functions in leoGlobals.py.

Imo these four functions do more than the entire argparse module. They are 
simpler, easier and more flexible.

Thanks again to Thomas for his careful reading of the code.

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/92052942-d641-40f8-8c1d-16fc1b2e2f85n%40googlegroups.com.