From 8836acd4691590699cbebf60e5e1fad4c849337b Mon Sep 17 00:00:00 2001
From: escuriola <escuriola@gmail.com>
Date: Fri, 8 Jul 2022 15:24:50 +0200
Subject: [PATCH] OEL-1823: Fixing tests after BCL upgrade.
---
.../tests/src/Functional/ListPagesTest.php | 2 +-
.../src/Kernel/Paragraphs/ContentRowTest.php | 6 ++---
.../Kernel/Paragraphs/FactsFiguresTest.php | 14 +++++------
.../Paragraphs/TimelineParagraphTest.php | 24 +++++++++----------
.../src/Functional/ContentEventRenderTest.php | 2 +-
tests/src/Kernel/FacetsFormTest.php | 2 +-
.../InPageNavigationAssert.php | 2 +-
7 files changed, 26 insertions(+), 26 deletions(-)
diff --git a/modules/oe_whitelabel_list_pages/tests/src/Functional/ListPagesTest.php b/modules/oe_whitelabel_list_pages/tests/src/Functional/ListPagesTest.php
index 477b6f07..e04aa618 100644
--- a/modules/oe_whitelabel_list_pages/tests/src/Functional/ListPagesTest.php
+++ b/modules/oe_whitelabel_list_pages/tests/src/Functional/ListPagesTest.php
@@ -54,7 +54,7 @@ class ListPagesTest extends WhitelabelBrowserTestBase {
// Assert offcanvas.
$offcanvas = $left_column->find('css', 'div.bcl-offcanvas');
- $title = $offcanvas->find('css', 'h3.offcanvas-title');
+ $title = $offcanvas->find('css', 'h2.offcanvas-title');
$this->assertSame('Filter options', $title->getText());
$offcanvas->hasField('Title');
$offcanvas->hasButton('Search');
diff --git a/modules/oe_whitelabel_paragraphs/tests/src/Kernel/Paragraphs/ContentRowTest.php b/modules/oe_whitelabel_paragraphs/tests/src/Kernel/Paragraphs/ContentRowTest.php
index 6c5734de..b0bb03a5 100644
--- a/modules/oe_whitelabel_paragraphs/tests/src/Kernel/Paragraphs/ContentRowTest.php
+++ b/modules/oe_whitelabel_paragraphs/tests/src/Kernel/Paragraphs/ContentRowTest.php
@@ -144,11 +144,11 @@ class ContentRowTest extends ParagraphsTestBase {
$this->assertCount(1, $crawler->filter('div.row > div > nav.bcl-inpage-navigation'));
$this->assertCount(8, $crawler->filter('p'));
// Assert the left column navigation.
- $left = $crawler->filter('div.col-md-3.d-none.d-md-block');
+ $left = $crawler->filter('div.bcl-sidebar');
$nav = $left->filter('nav.bcl-inpage-navigation');
$this->assertCount(1, $nav);
- $h3 = $nav->filter('h3');
- $this->assertSame('Page content', $h3->text());
+ $h2 = $nav->filter('h2');
+ $this->assertSame('Page content', $h2->text());
$ul = $left->filter('ul.nav.nav-pills.flex-column');
$this->assertCount(1, $ul);
$links = $ul->filter('li.nav-item a.nav-link');
diff --git a/modules/oe_whitelabel_paragraphs/tests/src/Kernel/Paragraphs/FactsFiguresTest.php b/modules/oe_whitelabel_paragraphs/tests/src/Kernel/Paragraphs/FactsFiguresTest.php
index 49daf942..050b1535 100644
--- a/modules/oe_whitelabel_paragraphs/tests/src/Kernel/Paragraphs/FactsFiguresTest.php
+++ b/modules/oe_whitelabel_paragraphs/tests/src/Kernel/Paragraphs/FactsFiguresTest.php
@@ -83,7 +83,7 @@ class FactsFiguresTest extends ParagraphsTestBase {
$this->assertCount(1, $crawler->filter('h2.bcl-heading'));
$this->assertCount(1, $crawler->filter('div.row-cols-md-3.row'));
$this->assertCount(6, $crawler->filter('svg.bi.icon--l'));
- $this->assertCount(6, $crawler->filter('div.fs-3'));
+ $this->assertCount(6, $crawler->filter('div.fs-4'));
$this->assertCount(6, $crawler->filter('div.fs-5'));
$this->assertCount(6, $crawler->filter('div.col'));
@@ -93,7 +93,7 @@ class FactsFiguresTest extends ParagraphsTestBase {
$link->html()
);
- $title_fact = $crawler->filter('div.row-cols-md-3.row > div.col:nth-child(1) > div.fs-3');
+ $title_fact = $crawler->filter('div.row-cols-md-3.row > div.col:nth-child(1) > div.fs-4');
$this->assertStringContainsString(
'1529 JIRA Ticket',
$title_fact->html()
@@ -109,7 +109,7 @@ class FactsFiguresTest extends ParagraphsTestBase {
$description_fact->html()
);
- $title_fact = $crawler->filter('div.row-cols-md-3.row > div.col:nth-child(2) > div.fs-3');
+ $title_fact = $crawler->filter('div.row-cols-md-3.row > div.col:nth-child(2) > div.fs-4');
$this->assertStringContainsString(
'337 Features',
$title_fact->html()
@@ -125,7 +125,7 @@ class FactsFiguresTest extends ParagraphsTestBase {
$description_fact->html()
);
- $title_fact = $crawler->filter('div.row-cols-md-3.row > div.col:nth-child(3) > div.fs-3');
+ $title_fact = $crawler->filter('div.row-cols-md-3.row > div.col:nth-child(3) > div.fs-4');
$this->assertStringContainsString(
'107 Tests',
$title_fact->html()
@@ -141,7 +141,7 @@ class FactsFiguresTest extends ParagraphsTestBase {
$description_fact->html()
);
- $title_fact = $crawler->filter('div.row-cols-md-3.row > div.col:nth-child(4) > div.fs-3');
+ $title_fact = $crawler->filter('div.row-cols-md-3.row > div.col:nth-child(4) > div.fs-4');
$this->assertStringContainsString(
'5670 Variants',
$title_fact->html()
@@ -157,7 +157,7 @@ class FactsFiguresTest extends ParagraphsTestBase {
$description_fact->html()
);
- $title_fact = $crawler->filter('div.row-cols-md-3.row > div.col:nth-child(5) > div.fs-3');
+ $title_fact = $crawler->filter('div.row-cols-md-3.row > div.col:nth-child(5) > div.fs-4');
$this->assertStringContainsString(
'345 Dev Ticket',
$title_fact->html()
@@ -173,7 +173,7 @@ class FactsFiguresTest extends ParagraphsTestBase {
$description_fact->html()
);
- $title_fact = $crawler->filter('div.row-cols-md-3.row > div.col:nth-child(6) > div.fs-3');
+ $title_fact = $crawler->filter('div.row-cols-md-3.row > div.col:nth-child(6) > div.fs-4');
$this->assertStringContainsString(
'43 Components',
$title_fact->html()
diff --git a/modules/oe_whitelabel_paragraphs/tests/src/Kernel/Paragraphs/TimelineParagraphTest.php b/modules/oe_whitelabel_paragraphs/tests/src/Kernel/Paragraphs/TimelineParagraphTest.php
index 490a6598..79389047 100644
--- a/modules/oe_whitelabel_paragraphs/tests/src/Kernel/Paragraphs/TimelineParagraphTest.php
+++ b/modules/oe_whitelabel_paragraphs/tests/src/Kernel/Paragraphs/TimelineParagraphTest.php
@@ -119,25 +119,25 @@ class TimelineParagraphTest extends ParagraphsTestBase {
$this->assertCount(7, $crawler->filter('ol.bcl-timeline li'));
$this->assertCount(1, $crawler->filter('ol.bcl-timeline li .label-collapsed'));
$this->assertCount(1, $crawler->filter('ol.bcl-timeline li.bcl-timeline__item--toggle'));
- $this->assertEquals('Label 1', trim($crawler->filter('ol.bcl-timeline li:nth-child(1) h5')->html()));
- $this->assertEquals('Title 1', trim($crawler->filter('ol.bcl-timeline li:nth-child(1) h6')->html()));
+ $this->assertEquals('Label 1', trim($crawler->filter('ol.bcl-timeline li:nth-child(1) p.fs-5.fw-medium')->html()));
+ $this->assertEquals('Title 1', trim($crawler->filter('ol.bcl-timeline li:nth-child(1) p.fs-6.fw-medium')->html()));
$this->assertEquals('Description 1', trim($crawler->filter('ol.bcl-timeline li:nth-child(1) div')->html()));
- $this->assertEquals('Label 2', trim($crawler->filter('ol.bcl-timeline li:nth-child(2) h5')->html()));
- $this->assertEquals('Title 2', trim($crawler->filter('ol.bcl-timeline li:nth-child(2) h6')->html()));
+ $this->assertEquals('Label 2', trim($crawler->filter('ol.bcl-timeline li:nth-child(2) p.fs-5.fw-medium')->html()));
+ $this->assertEquals('Title 2', trim($crawler->filter('ol.bcl-timeline li:nth-child(2) p.fs-6.fw-medium')->html()));
// Explicit format "plain_text" specified.
$this->assertEquals('<p>Description 2</p>', trim($crawler->filter('ol.bcl-timeline li:nth-child(2) div')->html()));
- $this->assertEquals('Label 3', trim($crawler->filter('ol.bcl-timeline li:nth-child(3) h5')->html()));
- $this->assertEquals('Title 3', trim($crawler->filter('ol.bcl-timeline li:nth-child(3) h6')->html()));
+ $this->assertEquals('Label 3', trim($crawler->filter('ol.bcl-timeline li:nth-child(3) p.fs-5.fw-medium')->html()));
+ $this->assertEquals('Title 3', trim($crawler->filter('ol.bcl-timeline li:nth-child(3) p.fs-6.fw-medium')->html()));
$this->assertEquals('<p>Description <strong>3</strong></p>', trim($crawler->filter('ol.bcl-timeline li:nth-child(3) div')->html()));
- $this->assertEquals('Label 4', trim($crawler->filter('ol.bcl-timeline li:nth-child(4) h5')->html()));
- $this->assertEquals('Title 4', trim($crawler->filter('ol.bcl-timeline li:nth-child(4) h6')->html()));
+ $this->assertEquals('Label 4', trim($crawler->filter('ol.bcl-timeline li:nth-child(4) p.fs-5.fw-medium')->html()));
+ $this->assertEquals('Title 4', trim($crawler->filter('ol.bcl-timeline li:nth-child(4) p.fs-6.fw-medium')->html()));
// Explicit format "filtered_html" specified.
$this->assertEquals('Description <strong>4</strong>', trim($crawler->filter('ol.bcl-timeline li:nth-child(4) div')->html()));
$this->assertEquals('<p>Description <strong>5</strong></p>', trim($crawler->filter('ol.bcl-timeline li:nth-child(5) div')->html()));
- $this->assertEquals('Label 5', trim($crawler->filter('ol.bcl-timeline li:nth-child(5) h5')->html()));
- $this->assertEquals('Title 5', trim($crawler->filter('ol.bcl-timeline li:nth-child(5) h6')->html()));
- $this->assertEquals('Label 6', trim($crawler->filter('ol.bcl-timeline li:nth-child(6) h5')->html()));
- $this->assertEquals('Title 6', trim($crawler->filter('ol.bcl-timeline li:nth-child(6) h6')->html()));
+ $this->assertEquals('Label 5', trim($crawler->filter('ol.bcl-timeline li:nth-child(5) p.fs-5.fw-medium')->html()));
+ $this->assertEquals('Title 5', trim($crawler->filter('ol.bcl-timeline li:nth-child(5) p.fs-6.fw-medium')->html()));
+ $this->assertEquals('Label 6', trim($crawler->filter('ol.bcl-timeline li:nth-child(6) p.fs-5.fw-medium')->html()));
+ $this->assertEquals('Title 6', trim($crawler->filter('ol.bcl-timeline li:nth-child(6) p.fs-6.fw-medium')->html()));
// Explicit format "full_html" specified.
$this->assertEquals('Description 6', trim($crawler->filter('ol.bcl-timeline li:nth-child(6) div')->html()));
$this->assertEquals('Show more 3 items', trim($crawler->filter('button .label-collapsed')->text()));
diff --git a/tests/src/Functional/ContentEventRenderTest.php b/tests/src/Functional/ContentEventRenderTest.php
index 7b20eb84..7097c9ea 100644
--- a/tests/src/Functional/ContentEventRenderTest.php
+++ b/tests/src/Functional/ContentEventRenderTest.php
@@ -123,7 +123,7 @@ class ContentEventRenderTest extends WhitelabelBrowserTestBase {
// Assert in-page navigation title.
$this->assertEquals(
'Page content',
- trim($crawler->filter('nav.bcl-inpage-navigation > h3')->text())
+ trim($crawler->filter('nav.bcl-inpage-navigation > h2')->text())
);
// Assert in-page navigation links.
diff --git a/tests/src/Kernel/FacetsFormTest.php b/tests/src/Kernel/FacetsFormTest.php
index 9ccfea9a..ba726108 100644
--- a/tests/src/Kernel/FacetsFormTest.php
+++ b/tests/src/Kernel/FacetsFormTest.php
@@ -96,7 +96,7 @@ class FacetsFormTest extends KernelTestBase {
$offcanvas = $crawler->filter('div#bcl-offcanvas');
$header = $offcanvas->filter('div.offcanvas-header');
$this->assertCount(1, $header);
- $title = $header->filter('h3.offcanvas-title');
+ $title = $header->filter('h2.offcanvas-title');
$this->assertSame('Facets form', $title->text());
$button = $header->filter('button');
$this->assertSame('offcanvas', $button->attr('data-bs-dismiss'));
diff --git a/tests/src/PatternAssertions/InPageNavigationAssert.php b/tests/src/PatternAssertions/InPageNavigationAssert.php
index a367f489..a5c3bada 100644
--- a/tests/src/PatternAssertions/InPageNavigationAssert.php
+++ b/tests/src/PatternAssertions/InPageNavigationAssert.php
@@ -18,7 +18,7 @@ class InPageNavigationAssert extends BasePatternAssert {
return [
'title' => [
[$this, 'assertElementText'],
- 'h3',
+ 'h2',
],
'links' => [
[$this, 'assertList'],
--
GitLab