Problem/Motivation
Since testbot only tests issues with the vendor versions in the composer lock file, we need a way to test against recent vendor updates in order to catch issues like #3039408: Updating twig/twig to v1.38.0 or v1.38.1 causes fatal error and #3039584: The "Symfony\Bridge\PsrHttpMessage\Factory\DiactorosFactory" class is deprecated since symfony/psr-http-message-bridge 1.2, use PsrHttpFactory instead. and #3045349: Lazy started sessions with session data are not saved with symfony/http-foundation 3.4.24. If we enable Symfony 4 support in 8.8, we will need a way to test against that as well
Proposed resolution
Keep an RTBC issue with a patch to run composer update against the codebase prior to running the test suite.
Remaining tasks
User interface changes
API changes
Data model changes
Release notes snippet
| Comment | File | Size | Author |
|---|---|---|---|
| #29 | 3044155-11.no-commit.patch | 573 bytes | mondrake |
Comments
Comment #2
mikelutzComment #3
mikelutzComment #4
mikelutzComment #6
mikelutzComment #7
mikelutzComment #8
xjmLet's close this so it's not in the RTEBC queue. :) Thanks!
Comment #9
mikelutzI'm happy to respect your decision, but I want to point out that the reason this was set to RTBC was to have the tests run daily, and email the followers if they failed. This allowed us to catch #3045349: Lazy started sessions with session data are not saved with symfony/http-foundation 3.4.24 almost a full day before anyone was really complaining about it. Gabor had suggested and I had hoped that putting it in the support request category would have removed it from your RTBC queue view. I certainly understand having the issue in RTBC might be annoying for the committers, but I wanted to make sure you were aware of why it was set to RTBC.
Comment #10
berdirI agree that this is useful to have, our dependencies seem to break stuff frequently atm: #3051269: Updating Drupal 8.6.x to twig 1.40.0 causes twig exceptions
Comment #11
alexpottDealing with coder and composer :)
Comment #12
mikelutzComment #14
mikelutzTestbot hiccup..
Comment #16
mikelutzComment #18
mikelutzre-uploading the patch from #11, just to clean up the failures.
Comment #19
xjmComment #21
mikelutzComment #23
mikelutzComment #25
alexpottFix for latest fail - #3086005: Make DrupalMinkClientTest::followMetaRefresh() compatible with max dependencies
Comment #26
mikelutzFix in, re-queuing
Comment #28
mile23Child issue addressing fail in #27: #3086557: Make DrupalMinkClient::request() compatible with max dependencies
Comment #29
mondrakeReuploading patch #11 to cleanup from failed tests. #3086332: Explicitly require-dev symfony/browser-kit: 3.4.0 is in so this should be green now since symfony/browser-toolkit is now constrained to ^3.4 and won't upgrade to ^4
Comment #32
mikelutzComment #38
cilefen commented