Fix failing cron job

[rahulporuri]

No description provided.

[mdickinson]

@rahulporuri Do you happen to know which upstream change triggered the need for this one?

[mdickinson]

I'm a bit puzzled by the remaining failure. I get:

mdickinson@mirzakhani envisage % python -m unittest envisage.ui.action.tests.test_action_manager_builder.ActionManagerBuilderTestCase.test_duplicate_menu
F
======================================================================
FAIL: test_duplicate_menu (envisage.ui.action.tests.test_action_manager_builder.ActionManagerBuilderTestCase)
duplicate menu
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/Users/mdickinson/Enthought/ETS/envisage/envisage/ui/action/tests/test_action_manager_builder.py", line 634, in test_duplicate_menu
    ["NewGroup", "ExitGroup", "ExtraGroup", "additions"], ids
AssertionError: Lists differ: ['NewGroup', 'ExitGroup', 'ExtraGroup', 'additions'] != ['NewGroup', 'ExtraGroup', 'ExitGroup', 'additions']

First differing element 1:
'ExitGroup'
'ExtraGroup'

- ['NewGroup', 'ExitGroup', 'ExtraGroup', 'additions']
?             -------------

+ ['NewGroup', 'ExtraGroup', 'ExitGroup', 'additions']
?                            +++++++++++++


----------------------------------------------------------------------

But as far as I can see, this test hasn't changed recently, so I'm not sure what caused it to start failing.

[rahulporuri]

@rahulporuri Do you happen to know which upstream change triggered the need for this one?

Nope, I didn't try tracking that down, I blindly started to try and fix things.

But as far as I can see, this test hasn't changed recently

But I did change the test in this PR -

envisage/envisage/ui/action/tests/test_action_manager_builder.py

Lines 584 to 591 in 92d827e

	groups=[
	Group(
	id="NewGroup",
	path="MenuBar/File",
	before="ExitGroup",
	),
	Group(id="ExitGroup", path="MenuBar/File"),
	],

and

envisage/envisage/ui/action/tests/test_action_manager_builder.py

Lines 600 to 606 in 92d827e

	groups=[
	Group(
	id="ExtraGroup",
	path="MenuBar/File",
	before="ExitGroup",
	),
	],

[mdickinson]

@rahulporuri Okay, I think this is ready to merge, if the EDM builds pass. Does it look okay to you in its current form?

We have what looks like a brand new issue to deal with with the PyPI build.

[rahulporuri]

Does it look okay to you in its current form?

Yes it does. Thanks @mdickinson !

[mdickinson]

Merging. The PyPI failures are due to #469, and are independent of the changes in this PR. I'll tackle those next.

Thanks, @rahulporuri!

[mdickinson]

Fake LGTM, so that this can be merged.

(Real LGTM on @rahulporuri's changes, and @rahulporuri has in effect approved my subsequent changes.)