Commit a2716608 authored by catch's avatar catch

Issue #2553553 by burkeker, alvar0hurtad0, meramo: Remove references to...

Issue #2553553 by burkeker, alvar0hurtad0, meramo: Remove references to system.module.css and system.theme.css in core
parent 2846b529
......@@ -198,7 +198,7 @@ public function testCurrentPathChange() {
* Tests that overridden CSS files are not added during lazy load.
*/
public function testLazyLoadOverriddenCSS() {
// The test theme overrides system.module.css without an implementation,
// The test theme overrides js.module.css without an implementation,
// thereby removing it.
\Drupal::service('theme_handler')->install(array('test_theme'));
$this->config('system.theme')
......@@ -214,6 +214,6 @@ public function testLazyLoadOverriddenCSS() {
// information about the file; we only really care about whether it appears
// in a LINK or STYLE tag, for which Drupal always adds a query string for
// cache control.
$this->assertNoText('system.module.css?', 'Ajax lazy loading does not add overridden CSS files.');
$this->assertNoText('js.module.css?', 'Ajax lazy loading does not add overridden CSS files.');
}
}
......@@ -175,7 +175,7 @@ function testCSSOverride() {
$config->set('css.preprocess', 0);
$config->save();
$this->drupalGet('theme-test/suggestion');
$this->assertNoText('system.module.css', "The theme's .info.yml file is able to remove a module CSS file from being added to the page.");
$this->assertNoText('js.module.css', 'The theme\'s .info.yml file is able to override a module CSS file from being added to the page.');
// Also test with aggregation enabled, simply ensuring no PHP errors are
// triggered during drupal_build_css_cache() when a source file doesn't
......
......@@ -15,7 +15,7 @@ version: VERSION
base theme: classy
core: 8.x
stylesheets-remove:
- '@system/css/system.module.css'
- '@system/css/js.module.css'
libraries:
- test_theme/global-styling
libraries-override:
......
......@@ -46,15 +46,15 @@ function testGrouper() {
'browsers' => array('IE' => TRUE, '!IE' => TRUE),
'basename' => 'system.base.css',
),
'system.theme.css' => array(
'js.module.css' => array(
'group' => -100,
'type' => 'file',
'weight' => 0.013,
'media' => 'all',
'preprocess' => TRUE,
'data' => 'core/modules/system/system.theme.css',
'data' => 'core/modules/system/js.module.css',
'browsers' => array('IE' => TRUE, '!IE' => TRUE),
'basename' => 'system.theme.css',
'basename' => 'js.module.css',
),
'jquery.ui.core.css' => array(
'group' => -100,
......@@ -120,7 +120,7 @@ function testGrouper() {
$this->assertSame($group['preprocess'], TRUE);
$this->assertSame(count($group['items']), 3);
$this->assertContains($css_assets['system.base.css'], $group['items']);
$this->assertContains($css_assets['system.theme.css'], $group['items']);
$this->assertContains($css_assets['js.module.css'], $group['items']);
// Check group 2.
$group = $groups[1];
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment