Quantcast
Channel: Issues for Drupal core
Viewing all articles
Browse latest Browse all 293221

Color upgrade path test is broken

$
0
0

When working on #1892006: Seven needs a print stylesheet.. The testbot kept failing on the upgrade path test for the color module.
The patch in 4# changed the style.css stylesheet settings from 'screen' to 'all', what made it pass the check on color.module line 67 and crash on the foreach on line 72 when running the upgrade path tests.
The reason why it crashed is that it is expecting an array and a string was given, this string is passed on by the upgrade part test as test content.

This bug doesn't happen on bartik because bartik doesn't use the all media tag.
This bug doesn't happen on stark because there are no upgrade path tests for stark.

So do we need an upgrade path test for color and seven, since color doesn't have an integration with seven?


Viewing all articles
Browse latest Browse all 293221

Trending Articles



<script src="https://jsc.adskeeper.com/r/s/rssing.com.1596347.js" async> </script>