Closed (duplicate)
Project:
Drupal core
Version:
8.5.x-dev
Component:
node system
Priority:
Major
Category:
Bug report
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
7 Sep 2016 at 21:11 UTC
Updated:
6 Mar 2022 at 21:54 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
dawehnerIs there a way to adapt our tests to not fail in that weird way?
Comment #3
catchNot sure why DrupalCI isn't recognising the patch, trying an extra character.
Comment #5
catchYeah yeah.
Comment #7
alexpottI thought I fixed this in #2614824: Tests fail when version constant is stable - ho hum.
Comment #8
xjm@alexpott, @catch, @effulgentsia, @cilefen, and I discussed this issue today and agreed it is a major bug. It is not critical because the test failure only disrupts core when HEAD is on a tagged version constant instead of a dev version constant -- normally only for a few seconds each month -- and the release managers already are aware of the issue.
Moving to the Node component since that is where the test lives.
Comment #9
xjmComment #12
mile23NodeTypeTranslationTest was moved to be a functional test in #2770921: Feb 21st: Convert chunk of WTB to BTB by just moving classes, changing use statements adding traits
It still has the same problem. Removing -dev from
\Drupal::VERSIONresults in a failed test:That's because the user interface is in what looks like French, even though only Spanish and English are installed on my site. The line in question in the test is:
The submit button is unavailable because it says 'Enregistrer et continuer' when
\Drupal::VERSIONdoes not include -dev.Change
\Drupal::VERSIONto include -dev and we see this:This is all local testing on a dev site.
Comment #13
mile23It matters what the '-dev' string is. Unfortunately, it's pretty hard to search all of core for 'dev' and get useful results.
Comment #16
alexpottThis will be fixed by #2955869: Fix multilingual install on Drupal dev version for CLI utilities as that addresses the root cause of the problem - ie. different behaviour in the installer depending on the version string. Postponing on that issue. Not marking closed because the issue still exists until that is fixed.
Comment #17
alexpottIf the patch attached is green we can close this.
Comment #18
alexpottYay - this is no longer an issue.
Comment #19
xjm