This should fix the HEAD test fails, the deprecated update tests is broken due to some update changes.

Note that testing each update function separately makes the test 4x as slow, 2min instead of 30s on my system. And each update runs a full update, so it's not really separated anyway, I'd suggest to merge that together, but not here.

CommentFileSizeAuthor
#2 linkit-update-test-3052546-2.patch635 bytesberdir

Comments

Berdir created an issue. See original summary.

berdir’s picture

berdir
Primary language German
Location Switzerland
commented
StatusFileSize
new linkit-update-test-3052546-2.patch635 bytes
chris burge’s picture

chris burge commented
Status: Needs review » Reviewed & tested by the community

+1 for committing

I had the testbot run tests against D8.7 and D8.6, too. The failure ("Composer require failure") is a fluke and is unrelated to the patch.

anon’s picture

anon
Primary language Swedish
Location Gothenburg
commented

Note that testing each update function separately makes the test 4x as slow,

Good point. I will try to fix this.

Currently running tests local to see that everything still works, if it does, I will commit the patch in #2 ASAP.

  • anon committed 8593473 on 8.x-5.x authored by Berdir 
    Issue #3052546 by Berdir, anon: Convert LinkitUpdateTest to phpunit
anon’s picture

anon
Primary language Swedish
Location Gothenburg
commented
Status: Reviewed & tested by the community » Fixed

Thanks for the patch.

Status: Fixed » Closed (fixed)

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