From 6a3ba3ffd8236a01595405795d784ab3c9534be5 Mon Sep 17 00:00:00 2001 From: Holly Cummins Date: Wed, 8 Jan 2025 10:49:53 +0000 Subject: [PATCH] Resolve some warnings --- src/components/List/List.js | 5 +---- src/components/Post/Post.js | 11 +---------- src/components/Post/Share.js | 6 ------ src/pages/contact.test.js | 12 ------------ src/templates/PostTemplate.test.js | 4 ++-- 5 files changed, 4 insertions(+), 34 deletions(-) diff --git a/src/components/List/List.js b/src/components/List/List.js index 275bd9dc..69b04dc9 100644 --- a/src/components/List/List.js +++ b/src/components/List/List.js @@ -5,7 +5,7 @@ import { icon } from "../../utils/type"; import { useTheme } from "../../layouts/theme"; const List = props => { - const { edges, showIcon, useShortDate } = props; + const { edges, showIcon = true, useShortDate } = props; const theme = useTheme(); return ( @@ -91,8 +91,5 @@ List.propTypes = { showIcon: PropTypes.bool }; -List.defaultProps = { - showIcon: true -}; export default List; diff --git a/src/components/Post/Post.js b/src/components/Post/Post.js index f9ad3e5b..d1f72fa6 100644 --- a/src/components/Post/Post.js +++ b/src/components/Post/Post.js @@ -2,21 +2,12 @@ import React from "react"; import PropTypes from "prop-types"; import "prismjs/themes/prism-okaidia.css"; -import asyncComponent from "../AsyncComponent"; import Headline from "../Article/Headline"; import Bodytext from "../Article/Bodytext"; import Meta from "./Meta"; import Author from "./Author"; import NextPrev from "./NextPrev"; - -const Share = asyncComponent(() => - import("./Share") - .then(module => { - return module.default; - }) - .catch(() => { - }) -); +import Share from "./Share"; const Post = props => { const { diff --git a/src/components/Post/Share.js b/src/components/Post/Share.js index 5f994827..7d9f3c28 100644 --- a/src/components/Post/Share.js +++ b/src/components/Post/Share.js @@ -32,9 +32,6 @@ const PostShare = props => { @@ -42,9 +39,6 @@ const PostShare = props => { url={url} title={title} description={excerpt} - additionalProps={{ - "aria-label": "LinkedIn share" - }} > diff --git a/src/pages/contact.test.js b/src/pages/contact.test.js index b8e95dfc..61a4d018 100644 --- a/src/pages/contact.test.js +++ b/src/pages/contact.test.js @@ -4,18 +4,6 @@ import ContactPage from "./contact"; import config from "../utils/configger"; -const layoutData = { - bgDesktop: { - resize: { src: "desktop" } - }, - bgTablet: { - resize: { src: "tablet" } - }, - bgMobile: { - resize: { src: "mobile" } - } -}; - describe("ContactPage", () => { beforeEach(async () => { render(); diff --git a/src/templates/PostTemplate.test.js b/src/templates/PostTemplate.test.js index cf95d730..5f2a1a3a 100644 --- a/src/templates/PostTemplate.test.js +++ b/src/templates/PostTemplate.test.js @@ -35,7 +35,7 @@ describe("PostTemplate", () => { // Why is this each? See https://stackoverflow.com/questions/67669213/react-testing-library-using-beforeall-to-render-cannot-find-item-on-2nd-test beforeEach(() => { - const tree = render(); + render(); }); it("renders the post body", () => { @@ -78,7 +78,7 @@ describe("PostTemplate", () => { // Why is this each? See https://stackoverflow.com/questions/67669213/react-testing-library-using-beforeall-to-render-cannot-find-item-on-2nd-test beforeEach(() => { - const tree = render(); + render(); }); it("renders the title", () => {