From dc86560fd0a0c111bfde85189ab5a9421ceb1365 Mon Sep 17 00:00:00 2001
From: Hassan Abouelela <47495861+HassanAbouelela@users.noreply.github.com>
Date: Tue, 26 Jan 2021 12:23:52 +0300
Subject: Cleans Up Some Warnings
Makes slight modifications to handle some minor warnings.
Signed-off-by: Hassan Abouelela <47495861+HassanAbouelela@users.noreply.github.com>
---
src/pages/LandingPage.tsx | 2 +-
src/setupTests.ts | 1 -
src/tests/components/FormListing.test.tsx | 16 ++++++++++++----
3 files changed, 13 insertions(+), 6 deletions(-)
(limited to 'src')
diff --git a/src/pages/LandingPage.tsx b/src/pages/LandingPage.tsx
index af968ad..06fef46 100644
--- a/src/pages/LandingPage.tsx
+++ b/src/pages/LandingPage.tsx
@@ -17,7 +17,7 @@ function LandingPage(): JSX.Element {
const fetchForms = async () => {
setForms(await getForms());
};
- fetchForms();
+ fetchForms().then();
}, []);
if (!forms) {
diff --git a/src/setupTests.ts b/src/setupTests.ts
index 5018a42..6a69409 100644
--- a/src/setupTests.ts
+++ b/src/setupTests.ts
@@ -1,2 +1 @@
-
import "@testing-library/jest-dom/extend-expect";
diff --git a/src/tests/components/FormListing.test.tsx b/src/tests/components/FormListing.test.tsx
index f071c33..f269dbf 100644
--- a/src/tests/components/FormListing.test.tsx
+++ b/src/tests/components/FormListing.test.tsx
@@ -56,13 +56,21 @@ test("renders form listing with specified description", () => {
test("renders form listing with background green colour for open", () => {
const { container } = render();
const elem = container.querySelector("a");
- const style = window.getComputedStyle(elem);
- expect(style.backgroundColor).toBe("rgb(67, 181, 129)");
+ expect(elem).not.toBeNull();
+
+ if (elem) {
+ const style = window.getComputedStyle(elem);
+ expect(style.backgroundColor).toBe("rgb(67, 181, 129)");
+ }
});
test("renders form listing with background dark colour for closed", () => {
const { container } = render();
const elem = container.querySelector("a");
- const style = window.getComputedStyle(elem);
- expect(style.backgroundColor).toBe("rgb(44, 47, 51)");
+ expect(elem).not.toBeNull();
+
+ if (elem) {
+ const style = window.getComputedStyle(elem);
+ expect(style.backgroundColor).toBe("rgb(44, 47, 51)");
+ }
});
--
cgit v1.2.3