The interactive installer tests cannot use the Testing profile in HEAD, because it is not visible in the UI.

Therefore, they need to use the slower Minimal profile, which is very time-consuming (especially when running tests locally).

But we can easily fix that. :-)

CommentFileSizeAuthor
test.interactive.0.patch1.53 KBsun

Comments

tstoeckler’s picture

tstoeckler
he/him
Primary language German
Location Essen, Germany
commented
Status: Needs review » Reviewed & tested by the community

Makes a lot of sense.

I originally wanted to suggest to add an explicit check for 'testing' in there, but then I realized: If people for some reason want to provide hidden profiles in contrib or on custom sites they should be able to test the installer with those profiles as well.

Verified that the Testing profile does not show up in the installer.

Status: Reviewed & tested by the community » Needs work

The last submitted patch, test.interactive.0.patch, failed testing.

tstoeckler’s picture

tstoeckler
he/him
Primary language German
Location Essen, Germany
commented

#2261725: Random test failure in ConfigImportUITest

tstoeckler’s picture

tstoeckler
he/him
Primary language German
Location Essen, Germany
commented
Status: Needs work » Needs review

test.interactive.0.patch queued for re-testing.

sun’s picture

sun
Primary language German
Location Karlsruhe
commented
Status: Needs review » Reviewed & tested by the community
catch’s picture

catch
he/him
Primary language English
commented
Status: Reviewed & tested by the community » Fixed

Committed/pushed to 8.x, thanks!

  • Commit 7c3e637 on 8.x by catch: 
    Issue #2260321 by sun: Use Testing profile instead of Minimal profile in...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.