From 3a6bd88e723d284d51ffc538937c96b090bcfe35 Mon Sep 17 00:00:00 2001
From: Davide Mininni <davide.mininni@finconsgroup.com>
Date: Mon, 9 Sep 2024 14:56:43 +0200
Subject: [PATCH] fix: review

---
 .../header/header/header.visual.spec.ts         |  6 +++++-
 src/elements/signet/signet.ts                   |  2 +-
 src/elements/signet/signet.visual.spec.ts       | 17 ++++++++++-------
 3 files changed, 16 insertions(+), 9 deletions(-)

diff --git a/src/elements/header/header/header.visual.spec.ts b/src/elements/header/header/header.visual.spec.ts
index b823ac0d68..559bbb88a6 100644
--- a/src/elements/header/header/header.visual.spec.ts
+++ b/src/elements/header/header/header.visual.spec.ts
@@ -1,4 +1,4 @@
-import { html, type TemplateResult } from 'lit';
+import { html, nothing, type TemplateResult } from 'lit';
 
 import { describeViewports, visualDiffDefault } from '../../core/testing/private.js';
 
@@ -6,6 +6,7 @@ import './header.js';
 import '../header-link.js';
 import '../header-button.js';
 import '../../menu.js';
+import '../../signet.js';
 
 describe(`sbb-header`, () => {
   const loremIpsumTemplate = `
@@ -37,6 +38,9 @@ describe(`sbb-header`, () => {
         Christina Müller
       </sbb-header-button>
       <sbb-header-button icon-name="globe-small" class="last-element"> English </sbb-header-button>
+      ${size === 's'
+        ? html`<sbb-signet slot="logo" protective-room="none" padded></sbb-signet>`
+        : nothing}
     </sbb-header>
     <div class=${expanded ? 'sbb-page-spacing-expanded' : 'sbb-page-spacing'}>
       ${loremIpsumTemplate}
diff --git a/src/elements/signet/signet.ts b/src/elements/signet/signet.ts
index ee81cb0dcc..1763763a1a 100644
--- a/src/elements/signet/signet.ts
+++ b/src/elements/signet/signet.ts
@@ -33,7 +33,7 @@ export class SbbSignetElement extends LitElement {
             ? html`
                 <svg focusable="false" viewBox="0 0 143 48" xmlns="http://www.w3.org/2000/svg">
                   <title .textContent=${this.accessibilityLabel}></title>
-                  <path id="sbb-signet__panel" stroke-width="1" d="M1 47h141V1H1v46z"></path>
+                  <path id="sbb-signet__panel" d="M1 47h141V1H1v46z"></path>
                   <path
                     id="sbb-signet__icon"
                     d="M87 40h9.3L83.8 27.7h15.5V40h7.4V27.7h15.6L109.7 40h9.3l16-16-15.9-16h-9.3l12.5 12.3h-15.6V8h-7.4v12.3H83.8L96.3 8H87L71 24l16 16z"
diff --git a/src/elements/signet/signet.visual.spec.ts b/src/elements/signet/signet.visual.spec.ts
index 812af2ab97..71e176d9e5 100644
--- a/src/elements/signet/signet.visual.spec.ts
+++ b/src/elements/signet/signet.visual.spec.ts
@@ -1,23 +1,26 @@
 import { html } from 'lit';
 
-import { describeViewports, visualDiffDefault } from '../core/testing/private.js';
+import { describeEach, describeViewports, visualDiffDefault } from '../core/testing/private.js';
 
 import './signet.js';
 
 describe(`sbb-signet`, () => {
-  const protectiveRooms = ['none', 'minimal', 'ideal'];
+  const cases = {
+    protectiveRooms: ['none', 'minimal', 'ideal'],
+    forcedColors: [false, true],
+  };
 
   describeViewports({ viewports: ['zero'] }, () => {
-    for (const protectiveRoom of protectiveRooms) {
+    describeEach(cases, ({ protectiveRooms, forcedColors }) => {
       it(
-        `protectiveRoom=${protectiveRoom}`,
+        ``,
         visualDiffDefault.with(async (setup) => {
           await setup.withFixture(
-            html`<sbb-signet protective-room=${protectiveRoom}></sbb-signet>`,
-            { padding: '0', maxWidth: '300px' },
+            html`<sbb-signet protective-room=${protectiveRooms}></sbb-signet>`,
+            { padding: '0', maxWidth: '300px', forcedColors },
           );
         }),
       );
-    }
+    });
   });
 });