tests/icon_tests.py | 33 33 + 0 - 0 !
1 file changed, 33 insertions(+)

 tests: allow one to skip interactive tests via environment variable

Interactive tests cannot be executed in automated test environments.

This adds a way to skip those tests via setting the `TEST_SKIP_INTERACTIVE`
variable to `1`.

[Upstream-Status: submitted(https://github.com/moses-palmer/pystray/pull/133)]

lib/pystray/_base.py | 2 1 + 1 - 0 !
tests/menu_descriptor_tests.py | 3 1 + 2 - 0 !
2 files changed, 2 insertions(+), 3 deletions(-)

 fix test_menu_construct_from_none test case

When `menu(None)` is called, this exception happens:

  File "/tmp/pystray-0.19.4/lib/pystray/_base.py", line 661, in cleaned
    if not i.visible:
  AttributeError: 'NoneType' object has no attribute 'visible'

Adding an `not i or` before that condition will make sure that `None`
will be considered as not visible.

The test itself fails as well:

  File "/tmp/pystray-0.19.4/tests/menu_descriptor_tests.py", line 85, in test_menu_construct_from_none
    self.assertEqual(
  AssertionError: '' != '\n'

Instead just compare if `str(menu(None))` is the empty string.

[Upstream-Status: submitted(https://github.com/moses-palmer/pystray/pull/133)]