- Issue created by @spokje
- 🇬🇧United Kingdom longwave UK
I wrote a test patch a while back over in https://www.drupal.org/files/issues/2022-02-02/btb-visitcount_0.patch → that might be useful to introduce for real once we have solved all these.
- 🇳🇱Netherlands spokje
Nice, but not sure all of the listed ones are solvable, there are already 2 'won't fixes' in the list.
Do we want to force making a HTTP call even when the test doesn't really need it?
Might be a good plan, the extra "effort" might be minimal and we would prevent a lot of Functional-tests-that-are-actually-Kernel-ones to slip in. - 🇬🇧United Kingdom longwave UK
If we use a similar solution to the test patch we could just do
$this->visitCount++;
to fake an HTTP request (a bit like we call$this->addToAssertionCount(1);
which I'm not sure we actually need any more) - 🇬🇧United Kingdom catch
#10 are the won't fixes because those tests do make an http request after all, or for some other reason?
- 🇳🇱Netherlands spokje
#10 are the won't fixes because those tests do make an http request after all, or for some other reason?
Other reason, mostly that the test needs to be an a serialized container.
- 🇳🇱Netherlands spokje
Opened 📌 [PP-1] Reevaluate the break-up of the various testsuites on GitLab Postponed as a follow-up.