@scottish-government/designsystem-react 0.6.0 → 0.7.1

package/src/components/{hide-this-page/hide-this-page.test.tsx → HideThisPage/HideThisPage.test.tsx}

@@ -1,6 +1,6 @@
 import { test, expect } from 'vitest';
 import { render, screen, within } from '@testing-library/react';
-import HideThisPage from './hide-this-page';
+import HideThisPage from './HideThisPage';
 
 test('hide this page renders correctly', () => {
     render(
@@ -39,16 +39,16 @@ test('hide this page renders correctly', () => {
 });
 
 test('custom escape URL', () => {
-    const escapeUrl = 'https://www.mygov.scot';
+    const ESCAPE_URL = 'https://www.mygov.scot';
 
     render(
-        <HideThisPage data-testid="htp" escapeUrl={escapeUrl} />
+        <HideThisPage data-testid="htp" escapeUrl={ESCAPE_URL} />
     );
 
     const hideThisPageElement = screen.getByTestId('htp');
     const link = within(hideThisPageElement).getByRole('link');
 
-    expect(link).toHaveAttribute('href', escapeUrl);
+    expect(link).toHaveAttribute('href', ESCAPE_URL);
 })
 
 test('passing additional props', () => {

package/src/components/{hide-this-page/hide-this-page.tsx → HideThisPage/HideThisPage.tsx}

@@ -13,10 +13,10 @@ const HideThisPage: React.FC<SGDS.Component.HideThisPage> = ({
         if (ref.current) {
             new DSHideThisPage().init();
 
-            const htpDivId = 'hide-this-page-instruction';
-            if (!document.getElementById(htpDivId)) {
+            const HIDE_THIS_PAGE_DIV_ID = 'hide-this-page-instruction';
+            if (!document.getElementById(HIDE_THIS_PAGE_DIV_ID)) {
                 const htpDiv = document.createElement('div');
-                htpDiv.id = htpDivId;
+                htpDiv.id = HIDE_THIS_PAGE_DIV_ID;
                 htpDiv.classList.add('visually-hidden');
                 htpDiv.classList.add('ds_hide-page');
                 htpDiv.innerHTML = '<p>To leave the page quickly, press the escape key.</p>';

package/src/components/{inset-text/inset-text.test.tsx → InsetText/InsetText.test.tsx}

@@ -1,6 +1,6 @@
 import { test, expect } from 'vitest';
 import { render } from '@testing-library/react';
-import InsetText from './inset-text';
+import InsetText from './InsetText';
 
 const text = `You may be able to apply for free school meals at the same
     time as you apply for the clothing grant.`;

package/src/components/{notification-banner/notification-banner.test.tsx → NotificationBanner/NotificationBanner.test.tsx}

@@ -5,9 +5,9 @@
 
 import { test, expect } from 'vitest';
 import { render, screen, within } from '@testing-library/react';
-import NotificationBanner from './notification-banner';
+import NotificationBanner from './NotificationBanner';
 
-const text = 'We need to tell you about something';
+const BANNER_TEXT = 'We need to tell you about something';
 
 /**
  * NotificationBanner has the ds_reversed class
@@ -15,7 +15,7 @@ const text = 'We need to tell you about something';
 test('notification banner renders correctly', () => {
     render(
         <NotificationBanner>
-            {text}
+            {BANNER_TEXT}
         </NotificationBanner>
     );
 
@@ -30,7 +30,7 @@ test('notification banner renders correctly', () => {
 test('notification banner with icon', () => {
     render(
         <NotificationBanner icon>
-            {text}
+            {BANNER_TEXT}
         </NotificationBanner>
     );
 

package/src/components/{notification-banner/notification-banner.tsx → NotificationBanner/NotificationBanner.tsx}

@@ -1,5 +1,5 @@
 import { useEffect, useRef } from 'react';
-import AbstractNotificationBanner from '../../common/abstract-notification-banner';
+import AbstractNotificationBanner from '../../common/AbstractNotificationBanner';
 // @ts-ignore
 import DSNotificationBanner from '@scottish-government/design-system/src/components/notification-banner/notification-banner';
 

package/src/components/{notification-panel/notification-panel.test.tsx → NotificationPanel/NotificationPanel.test.tsx}

@@ -1,14 +1,14 @@
 import { test, expect } from 'vitest';
 import { render, screen } from '@testing-library/react';
-import NotificationPanel from './notification-panel';
+import NotificationPanel from './NotificationPanel';
 
-const headingText = 'Thank you';
-const text = 'Your Saltire Scholarship Application form has been successfully submitted.';
+const HEADING_TEXT = 'Thank you';
+const TEXT = 'Your Saltire Scholarship Application form has been successfully submitted.';
 
 test('notification banner renders correctly', () => {
     render(
-        <NotificationPanel title={headingText}>
-            {text}
+        <NotificationPanel title={HEADING_TEXT}>
+            {TEXT}
         </NotificationPanel>
     );
 
@@ -18,56 +18,57 @@ test('notification banner renders correctly', () => {
 
     expect(notificationPanel).toHaveClass('ds_notification-panel');
     expect(notificationPanelHeading).toHaveClass('ds_notification-panel__title');
-    expect(notificationPanelHeading.textContent).toEqual(headingText);
+    expect(notificationPanelHeading.textContent).toEqual(HEADING_TEXT);
     expect(notificationPanelHeading.tagName).toEqual('H1');
     expect(notificationPanelContent).toHaveClass('ds_notification-panel__content');
-    expect(notificationPanelContent?.textContent).toEqual(text);
+    expect(notificationPanelContent?.textContent).toEqual(TEXT);
 });
 
 test('notification banner with custom heading level', () => {
-    const headerLevel = 'h2';
+    const HEADING_LEVEL = 'h2';
 
     render(
-        <NotificationPanel headerLevel={headerLevel} title={headingText}>
-            {text}
+        <NotificationPanel headingLevel={HEADING_LEVEL} title={HEADING_TEXT}>
+            {TEXT}
         </NotificationPanel>
     );
 
     const notificationPanelHeading = screen.getByRole('heading');
-    expect(notificationPanelHeading.tagName).toEqual(headerLevel.toUpperCase());
+    expect(notificationPanelHeading.tagName).toEqual(HEADING_LEVEL.toUpperCase());
 });
 
 test('notification banner with aria-live', () => {
-    const ariaLive = 'polite';
+    const ARIA_LIVE = 'polite';
 
     render(
-        <NotificationPanel ariaLive={ariaLive} title={headingText}>
-            {text}
+        <NotificationPanel ariaLive={ARIA_LIVE} title={HEADING_TEXT}>
+            {TEXT}
         </NotificationPanel>
     );
 
     const notificationPanelHeading = screen.getByRole('heading');
     const notificationPanel = notificationPanelHeading.parentElement;
 
-    expect(notificationPanel).toHaveAttribute('aria-live', ariaLive);
+    expect(notificationPanel).toHaveAttribute('aria-live', ARIA_LIVE);
 });
 
 test('notification banner with nonsense heading level reverts to H1', () => {
-    const headerLevel = 'h2';
+    const HEADING_LEVEL = 'h2';
+
     render(
-        <NotificationPanel headerLevel={headerLevel} title={headingText}>
-            {text}
+        <NotificationPanel headingLevel={HEADING_LEVEL} title={HEADING_TEXT}>
+            {TEXT}
         </NotificationPanel>
     );
 
     const notificationPanelHeading = screen.getByRole('heading');
-    expect(notificationPanelHeading.tagName).toEqual(headerLevel.toUpperCase());
+    expect(notificationPanelHeading.tagName).toEqual(HEADING_LEVEL.toUpperCase());
 });
 
 test('passing additional props', () => {
     render(
-        <NotificationPanel title={headingText} data-test="foo">
-            {text}
+        <NotificationPanel title={HEADING_TEXT} data-test="foo">
+            {TEXT}
         </NotificationPanel>
     )
 
@@ -78,8 +79,8 @@ test('passing additional props', () => {
 
 test('passing additional CSS classes', () => {
     render(
-        <NotificationPanel title={headingText} className="foo">
-            {text}
+        <NotificationPanel title={HEADING_TEXT} className="foo">
+            {TEXT}
         </NotificationPanel>
     )
 

package/src/components/{notification-panel/notification-panel.tsx → NotificationPanel/NotificationPanel.tsx}

@@ -1,10 +1,10 @@
-import WrapperTag from '../../common/wrapper-tag';
+import WrapperTag from '../../common/WrapperTag';
 
 const NotificationPanel: React.FC<SGDS.Component.NotificationPanel> = function ({
     ariaLive,
     children,
     className,
-    headerLevel = 'h1',
+    headingLevel = 'h1',
     title,
     ...props
 }) {
@@ -19,7 +19,7 @@ const NotificationPanel: React.FC<SGDS.Component.NotificationPanel> = function (
         >
             <WrapperTag
                 className="ds_notification-panel__title"
-                tagName={headerLevel}
+                tagName={headingLevel}
             >
                 {title}
             </WrapperTag>

package/src/components/{page-header/page-header.test.tsx → PageHeader/PageHeader.test.tsx}

@@ -1,13 +1,13 @@
 import { test, expect } from 'vitest';
 import { render, screen, within } from '@testing-library/react';
-import PageHeader from './page-header';
+import PageHeader from './PageHeader';
 
-const labelText = 'Guide';
-const titleText = 'Apply for or renew a disabled parking permit';
+const LABEL_TEXT = 'Guide';
+const TITLE_TEXT = 'Apply for or renew a disabled parking permit';
 
 test('notification banner renders correctly', () => {
     render(
-        <PageHeader label={labelText} title={titleText}/>
+        <PageHeader label={LABEL_TEXT} title={TITLE_TEXT}/>
     );
 
     const header = screen.getByRole('banner');
@@ -18,17 +18,17 @@ test('notification banner renders correctly', () => {
     expect(header.tagName).toEqual('HEADER');
 
     expect(label).toHaveClass('ds_page-header__label', 'ds_content-label');
-    expect(label?.textContent).toEqual(labelText);
+    expect(label?.textContent).toEqual(LABEL_TEXT);
     expect(label?.tagName).toEqual('SPAN');
 
     expect(title).toHaveClass('ds_page-header__title');
-    expect(title.textContent).toEqual(titleText);
+    expect(title.textContent).toEqual(TITLE_TEXT);
     expect(title.tagName).toEqual('H1');
 });
 
 test('header with no label', () => {
     render(
-        <PageHeader title={titleText}/>
+        <PageHeader title={TITLE_TEXT}/>
     );
 
     const header = screen.getByRole('banner');
@@ -40,7 +40,7 @@ test('header with no label', () => {
 
 test('passing additional props', () => {
     render(
-       <PageHeader data-test="foo" label={labelText} title={titleText}/>
+       <PageHeader data-test="foo" label={LABEL_TEXT} title={TITLE_TEXT}/>
     )
 
     const header = screen.getByRole('banner');
@@ -49,7 +49,7 @@ test('passing additional props', () => {
 
 test('passing additional CSS classes', () => {
     render(
-       <PageHeader className="foo" label={labelText} title={titleText}/>
+       <PageHeader className="foo" label={LABEL_TEXT} title={TITLE_TEXT}/>
     )
 
     const header = screen.getByRole('banner');

package/src/components/{page-metadata/page-metadata.test.tsx → PageMetadata/PageMetadata.test.tsx}

@@ -1,15 +1,15 @@
 import { test, expect } from 'vitest';
 import { render, screen } from '@testing-library/react';
-import Metadata from './page-metadata';
+import Metadata from './PageMetadata';
 
-const name = 'Directorate';
-const value = 'Equality, Inclusion and Human Rights Directorate';
+const NAME = 'Directorate';
+const VALUE = 'Equality, Inclusion and Human Rights Directorate';
 
 test('metadata renders correctly', () => {
     render(
         <Metadata>
-            <Metadata.Item name={name}>
-                {value}
+            <Metadata.Item name={NAME}>
+                {VALUE}
             </Metadata.Item>
         </Metadata>
     );
@@ -22,16 +22,16 @@ test('metadata renders correctly', () => {
     expect(metadata).toBeInTheDocument();
     expect(metadataItem).toBeInTheDocument();
     expect(metadataItemKey).toHaveClass('ds_metadata__key');
-    expect(metadataItemKey.textContent).toEqual(name);
+    expect(metadataItemKey.textContent).toEqual(NAME);
     expect(metadataItemValue).toHaveClass('ds_metadata__value');
-    expect(metadataItemValue.textContent).toEqual(value);
+    expect(metadataItemValue.textContent).toEqual(VALUE);
 });
 
 test('inline metadata', () => {
     render(
         <Metadata inline>
-            <Metadata.Item name={name}>
-                {value}
+            <Metadata.Item name={NAME}>
+                {VALUE}
             </Metadata.Item>
         </Metadata>
     );

package/src/components/{pagination/pagination.test.tsx → Pagination/Pagination.test.tsx}

@@ -1,39 +1,39 @@
 import { test, expect, vi } from 'vitest';
 import { render, screen, within, fireEvent } from '@testing-library/react';
-import Pagination, { Page, Ellipsis } from './pagination';
+import Pagination, { Page, Ellipsis } from './Pagination';
 
-const pageAriaLabel = 'Page 1';
-const pageHref = '#foo';
-const pageText = '1';
-const currentPage = 10;
-const totalPages = 21;
+const PAGE_ARIA_LABEL = 'Page 1';
+const PAGE_HREF = '#foo';
+const PAGE_TEXT = '1';
+const CURRENT_PAGE = 10;
+const TOTAL_PAGES = 21;
 
 test('pagination page renders correctly', () => {
     render(
         <Page
-            ariaLabel={pageAriaLabel}
-            href={pageHref}
-            text={pageText}
+            ariaLabel={PAGE_ARIA_LABEL}
+            href={PAGE_HREF}
+            text={PAGE_TEXT}
         />
     );
 
     const item = screen.getByRole('listitem');
     const link = within(item).getByRole('link');
-    const span = within(link).getByText(pageText);
+    const span = within(link).getByText(PAGE_TEXT);
 
     expect(item).toHaveClass('ds_pagination__item');
     expect(link).toHaveClass('ds_pagination__link');
-    expect(link).toHaveAttribute('aria-label', pageAriaLabel);
-    expect(link).toHaveAttribute('href', pageHref);
+    expect(link).toHaveAttribute('aria-label', PAGE_ARIA_LABEL);
+    expect(link).toHaveAttribute('href', PAGE_HREF);
     expect(span).toHaveClass('ds_pagination__link-label');
 });
 
 test('current pagination page', () => {
     render(
         <Page
-            ariaLabel={pageAriaLabel}
-            href={pageHref}
-            text={pageText}
+            ariaLabel={PAGE_ARIA_LABEL}
+            href={PAGE_HREF}
+            text={PAGE_TEXT}
             current
         />
     );
@@ -46,14 +46,14 @@ test('current pagination page', () => {
 });
 
 test('pagination page with click event', () => {
-    const onClickFn = vi.fn();
+    const ONCLICK_FUNCTION = vi.fn();
 
     render(
         <Page
-            ariaLabel={pageAriaLabel}
-            href={pageHref}
-            text={pageText}
-            onClick={onClickFn}
+            ariaLabel={PAGE_ARIA_LABEL}
+            href={PAGE_HREF}
+            text={PAGE_TEXT}
+            onClick={ONCLICK_FUNCTION}
         />
     );
 
@@ -62,7 +62,7 @@ test('pagination page with click event', () => {
 
     fireEvent.click(link);
 
-    expect(onClickFn).toHaveBeenCalled();
+    expect(ONCLICK_FUNCTION).toHaveBeenCalled();
 });
 
 test('Ellipsis item renders correctly', () => {
@@ -79,11 +79,11 @@ test('Ellipsis item renders correctly', () => {
 });
 
 test('pagination renders correctly', () => {
-    const currentPage = 10;
-    const totalPages = 21;
+    const CURRENT_PAGE = 10;
+    const TOTAL_PAGES = 21;
 
     render(
-        <Pagination page={currentPage} totalPages={totalPages} />
+        <Pagination page={CURRENT_PAGE} totalPages={TOTAL_PAGES} />
     );
 
     const paginationNav = screen.getByRole('navigation');
@@ -100,7 +100,7 @@ test('pagination renders correctly', () => {
     const firstPageLabel = within(paginationList).getByText('1');
     const firstPageLink = firstPageLabel.parentElement;
     const firstPageItem = firstPageLink?.parentElement;
-    const lastPageLabel = within(paginationList).getByText(totalPages);
+    const lastPageLabel = within(paginationList).getByText(TOTAL_PAGES);
     const lastPageLink = lastPageLabel.parentElement;
     const lastPageItem = lastPageLink?.parentElement;
 
@@ -125,7 +125,7 @@ test('pagination renders correctly', () => {
     expect(prevItem?.parentElement).toEqual(paginationList);
     expect(prevLink).toHaveClass('ds_pagination__link', 'ds_pagination__link--text', 'ds_pagination__link--icon')
     expect(prevLink).toHaveAttribute('aria-label', 'Previous page');
-    expect(prevLink).toHaveAttribute('href', `/search?page=${currentPage - 1}`);
+    expect(prevLink).toHaveAttribute('href', `/search?page=${CURRENT_PAGE - 1}`);
     expect(prevLink?.tagName).toEqual('A');
     expect(prevIcon).toHaveClass('ds_icon');
     expect(prevIcon).toHaveAttribute('aria-hidden', 'true')
@@ -138,7 +138,7 @@ test('pagination renders correctly', () => {
     expect(nextItem?.parentElement).toEqual(paginationList);
     expect(nextLink).toHaveClass('ds_pagination__link', 'ds_pagination__link--text', 'ds_pagination__link--icon')
     expect(nextLink).toHaveAttribute('aria-label', 'Next page');
-    expect(nextLink).toHaveAttribute('href', `/search?page=${currentPage + 1}`);
+    expect(nextLink).toHaveAttribute('href', `/search?page=${CURRENT_PAGE + 1}`);
     expect(nextLink?.tagName).toEqual('A');
     expect(nextIcon).toHaveClass('ds_icon');
     expect(nextIcon).toHaveAttribute('aria-hidden', 'true')
@@ -160,8 +160,8 @@ test('pagination renders correctly', () => {
     expect(lastPageItem?.tagName).toEqual('LI');
     expect(lastPageItem?.parentElement).toEqual(paginationList);
     expect(lastPageLink).toHaveClass('ds_pagination__link')
-    expect(lastPageLink).toHaveAttribute('aria-label', `Page ${totalPages}`);
-    expect(lastPageLink).toHaveAttribute('href', `/search?page=${totalPages}`);
+    expect(lastPageLink).toHaveAttribute('aria-label', `Page ${TOTAL_PAGES}`);
+    expect(lastPageLink).toHaveAttribute('href', `/search?page=${TOTAL_PAGES}`);
     expect(lastPageLink?.tagName).toEqual('A');
     expect(lastPageLabel).toHaveClass('ds_pagination__link-label');
     expect(lastPageLabel?.tagName).toEqual('SPAN');
@@ -184,7 +184,7 @@ test('pagination renders correctly', () => {
 
     expect(currentPageItem).toHaveClass('ds_pagination__item');
     expect(currentPageLink).toHaveClass('ds_pagination__link', 'ds_current');
-    expect(currentPageLink?.textContent).toEqual(currentPage.toString());
+    expect(currentPageLink?.textContent).toEqual(CURRENT_PAGE.toString());
 
     // expect one link either side of the current (default padding)
     expect(currentPageItem?.previousElementSibling?.querySelector('a')).toHaveAttribute('aria-label', 'Page 9');
@@ -198,13 +198,13 @@ test('pagination renders correctly', () => {
 });
 
 test('pagination with 2 padding', () => {
-    const padding = 2;
+    const PADDING = 2;
 
     render(
         <Pagination
-            page={currentPage}
-            totalPages={totalPages}
-            padding={padding}
+            page={CURRENT_PAGE}
+            totalPages={TOTAL_PAGES}
+            padding={PADDING}
         />
     );
 
@@ -217,29 +217,29 @@ test('pagination with 2 padding', () => {
 });
 
 test('pagination with custom aria label', () => {
-    const ariaLabel = 'My label';
+    const ARIA_LABEL = 'My label';
 
     render(
         <Pagination
-            page={currentPage}
-            totalPages={totalPages}
-            ariaLabel={ariaLabel}
+            page={CURRENT_PAGE}
+            totalPages={TOTAL_PAGES}
+            ariaLabel={ARIA_LABEL}
         />
     );
 
     const paginationNav = screen.getByRole('navigation');
 
-    expect(paginationNav).toHaveAttribute('aria-label', ariaLabel);
+    expect(paginationNav).toHaveAttribute('aria-label', ARIA_LABEL);
 });
 
 test('pagination passes onclick event to child links', () => {
-    const onClickFn = vi.fn();
+    const ONCLICK_FUNCTION = vi.fn();
 
     render(
         <Pagination
-            page={currentPage}
-            totalPages={totalPages}
-            onClick={onClickFn}
+            page={CURRENT_PAGE}
+            totalPages={TOTAL_PAGES}
+            onClick={ONCLICK_FUNCTION}
         />
     );
 
@@ -249,14 +249,14 @@ test('pagination passes onclick event to child links', () => {
 
     fireEvent.click(link);
 
-    expect(onClickFn).toHaveBeenCalled();
+    expect(ONCLICK_FUNCTION).toHaveBeenCalled();
 });
 
 test('pagination modifies an input pattern for its link format', () => {
     render(
         <Pagination
-            page={currentPage}
-            totalPages={totalPages}
+            page={CURRENT_PAGE}
+            totalPages={TOTAL_PAGES}
             pattern='My/Link/Format?Page=$1#foo'
         />
     );
@@ -271,7 +271,7 @@ test('pagination at an early link in the list', () => {
     render(
         <Pagination
             page="1"
-            totalPages={totalPages}
+            totalPages={TOTAL_PAGES}
         />
     );
 
@@ -289,7 +289,7 @@ test('pagination at an early link in the list, increased padding', () => {
         <Pagination
             padding={2}
             page="1"
-            totalPages={totalPages}
+            totalPages={TOTAL_PAGES}
         />
     );
 
@@ -305,8 +305,8 @@ test('pagination at an early link in the list, increased padding', () => {
 test('pagination at late link in the list', () => {
     render(
         <Pagination
-            page={totalPages}
-            totalPages={totalPages}
+            page={TOTAL_PAGES}
+            totalPages={TOTAL_PAGES}
         />
     );
 
@@ -323,8 +323,8 @@ test('pagination at late link in the list, increased padding', () => {
     render(
         <Pagination
             padding={2}
-            page={totalPages}
-            totalPages={totalPages}
+            page={TOTAL_PAGES}
+            totalPages={TOTAL_PAGES}
         />
     );
 
@@ -340,8 +340,8 @@ test('pagination at late link in the list, increased padding', () => {
 test('passing additional props', () => {
     render(
         <Pagination
-            page={currentPage}
-            totalPages={totalPages}
+            page={CURRENT_PAGE}
+            totalPages={TOTAL_PAGES}
             data-test="foo"
         />
     );
@@ -353,8 +353,8 @@ test('passing additional props', () => {
 test('passing additional CSS classes', () => {
     render(
         <Pagination
-            page={currentPage}
-            totalPages={totalPages}
+            page={CURRENT_PAGE}
+            totalPages={TOTAL_PAGES}
             className="foo"
         />
     )

package/src/components/{pagination/pagination.tsx → Pagination/Pagination.tsx}

@@ -1,4 +1,4 @@
-import Icon from "../../common/icon";
+import Icon from "../../common/Icon";
 
 export const Page: React.FC<SGDS.Component.Pagination.Page> = ({
     ariaLabel,

package/src/components/{phase-banner/phase-banner.test.tsx → PhaseBanner/PhaseBanner.test.tsx}

@@ -1,14 +1,14 @@
 import { test, expect } from 'vitest';
 import { render } from '@testing-library/react';
-import PhaseBanner from './phase-banner';
+import PhaseBanner from './PhaseBanner';
 
-const text = 'This is a new service. Your feedback will help us to improve it.';
-const defaultText = 'This is a new service';
+const BANNER_TEXT = 'This is a new service. Your feedback will help us to improve it.';
+const DEFAULT_TEXT = 'This is a new service';
 
 test('phase banner renders correctly', () => {
     render(
         <PhaseBanner>
-            {text}
+            {BANNER_TEXT}
         </PhaseBanner>
     );
 
@@ -23,7 +23,7 @@ test('phase banner renders correctly', () => {
     expect(phaseBannerWrapper?.tagName).toEqual('DIV');
     expect(phaseBannerContent?.tagName).toEqual('P');
     expect(phaseBannerText?.tagName).toEqual('SPAN');
-    expect(phaseBannerText?.textContent).toEqual(text);
+    expect(phaseBannerText?.textContent).toEqual(BANNER_TEXT);
 });
 
 test('phase banner with default text', () => {
@@ -36,14 +36,14 @@ test('phase banner with default text', () => {
     const phaseBannerContent = phaseBanner?.querySelector('.ds_phase-banner__content');
     const phaseBannerText = phaseBannerContent?.querySelector('.ds_phase-banner__text');
 
-    expect(phaseBannerText?.textContent).toEqual(defaultText);
+    expect(phaseBannerText?.textContent).toEqual(DEFAULT_TEXT);
 });
 
 test('phase banner with phase tag', () => {
-    const phase = 'Beta';
+    const PHASE_TEXT = 'Beta';
 
     render(
-        <PhaseBanner phaseName={phase}></PhaseBanner>
+        <PhaseBanner phaseName={PHASE_TEXT}></PhaseBanner>
     );
 
     const phaseBanner = document.querySelector('.ds_phase-banner');
@@ -52,7 +52,7 @@ test('phase banner with phase tag', () => {
     expect(phaseBannerTag).toBeInTheDocument();
     expect(phaseBannerTag?.tagName).toEqual('SPAN');
     expect(phaseBannerTag).toHaveClass('ds_tag', 'ds_phase-banner__tag');
-    expect(phaseBannerTag?.textContent).toEqual(phase);
+    expect(phaseBannerTag?.textContent).toEqual(PHASE_TEXT);
 });
 
 test('passing additional props', () => {

package/src/components/{phase-banner/phase-banner.tsx → PhaseBanner/PhaseBanner.tsx}

@@ -1,4 +1,4 @@
-import Tag from "../tag/tag";
+import Tag from "../Tag/Tag";
 
 const PhaseBanner: React.FC<SGDS.Component.PhaseBanner> = ({
     children,