From 7255d25f6834a208c0ed44636356cc260f6ab6ba Mon Sep 17 00:00:00 2001
From: Armand Philippot <git@armandphilippot.com>
Date: Wed, 27 Sep 2023 17:38:23 +0200
Subject: refactor(components): rewrite Heading component

* remove `alignment` and `withMargin` props (consumer should handle
that)
* move styles to Sass placeholders to avoid repeats with headings
coming from WordPress
* refactor some other components that depend on Heading to avoid ESlint
errors
---
 src/components/atoms/headings/heading.test.tsx | 57 --------------------------
 1 file changed, 57 deletions(-)
 delete mode 100644 src/components/atoms/headings/heading.test.tsx

(limited to 'src/components/atoms/headings/heading.test.tsx')

diff --git a/src/components/atoms/headings/heading.test.tsx b/src/components/atoms/headings/heading.test.tsx
deleted file mode 100644
index 61d7f8e..0000000
--- a/src/components/atoms/headings/heading.test.tsx
+++ /dev/null
@@ -1,57 +0,0 @@
-import { describe, expect, it } from '@jest/globals';
-import { render, screen } from '../../../../tests/utils';
-import { Heading } from './heading';
-
-describe('Heading', () => {
-  it('renders a h1', () => {
-    render(<Heading level={1}>Level 1</Heading>);
-    expect(screen.getByRole('heading', { level: 1 })).toHaveTextContent(
-      'Level 1'
-    );
-  });
-
-  it('renders a h2', () => {
-    render(<Heading level={2}>Level 2</Heading>);
-    expect(screen.getByRole('heading', { level: 2 })).toHaveTextContent(
-      'Level 2'
-    );
-  });
-
-  it('renders a h3', () => {
-    render(<Heading level={3}>Level 3</Heading>);
-    expect(screen.getByRole('heading', { level: 3 })).toHaveTextContent(
-      'Level 3'
-    );
-  });
-
-  it('renders a h4', () => {
-    render(<Heading level={4}>Level 4</Heading>);
-    expect(screen.getByRole('heading', { level: 4 })).toHaveTextContent(
-      'Level 4'
-    );
-  });
-
-  it('renders a h5', () => {
-    render(<Heading level={5}>Level 5</Heading>);
-    expect(screen.getByRole('heading', { level: 5 })).toHaveTextContent(
-      'Level 5'
-    );
-  });
-
-  it('renders a h6', () => {
-    render(<Heading level={6}>Level 6</Heading>);
-    expect(screen.getByRole('heading', { level: 6 })).toHaveTextContent(
-      'Level 6'
-    );
-  });
-
-  it('renders a text with heading styles', () => {
-    render(
-      <Heading isFake={true} level={2}>
-        Fake heading
-      </Heading>
-    );
-    expect(screen.queryByRole('heading', { level: 2 })).not.toBeInTheDocument();
-    expect(screen.getByText('Fake heading')).toHaveClass('heading');
-  });
-});
-- 
cgit v1.2.3