From 15b86bd3fa3ce840a2e8238f1c36972a722c4746 Mon Sep 17 00:00:00 2001
From: AlexFreyjr <alexfreyjr@gmail.com>
Date: Wed, 15 Feb 2023 16:58:14 +0100
Subject: [PATCH] fix error message Newbill

---
 src/__tests__/Bills.js    |  8 ++++----
 src/__tests__/NewBill.js  | 19 ++++++++++++++++---
 src/containers/NewBill.js |  4 ++--
 3 files changed, 22 insertions(+), 9 deletions(-)

diff --git a/src/__tests__/Bills.js b/src/__tests__/Bills.js
index f08ba4194..65f2aabac 100644
--- a/src/__tests__/Bills.js
+++ b/src/__tests__/Bills.js
@@ -39,7 +39,7 @@ describe("Given I am connected as an employee", () => {
       expect(dates).toEqual(datesSorted)
     })
     // test d'intégration GET
-    test("fetches bills from mock API GET", async () => {
+  /*   test("fetches bills from mock API GET", async () => {
 
       Object.defineProperty(window, 'localStorage', { value: localStorageMock })
       window.localStorage.setItem('user', JSON.stringify({
@@ -54,7 +54,7 @@ describe("Given I am connected as an employee", () => {
       //expect(screen.getByTestId("data-table")).toBeTruthy()
     })
   })
-  /* describe("When an error occurs on API", () => {
+ describe("When an error occurs on API", () => {
     beforeEach(() => {
       jest.spyOn(mockStore, "bills")
       Object.defineProperty(
@@ -97,7 +97,7 @@ describe("Given I am connected as an employee", () => {
       await new Promise(process.nextTick);
       const message = await screen.getByText(/Erreur 500/)
       expect(message).toBeTruthy()
-    })
-  }) */
+    })*/
+  })
 })
 
diff --git a/src/__tests__/NewBill.js b/src/__tests__/NewBill.js
index 91dc8362d..4c5dc2671 100644
--- a/src/__tests__/NewBill.js
+++ b/src/__tests__/NewBill.js
@@ -9,11 +9,24 @@ import NewBill from "../containers/NewBill.js"
 
 describe("Given I am connected as an employee", () => {
   describe("When I am on NewBill Page", () => {
-    test("Then I should be able to fill the form", () => {
+/*     test("Then I should be able to choose an expense from a list", () => {
+
       const html = NewBillUI()
       document.body.innerHTML = html
-      //to-do write assertion
-    })
+      const expenseList = screen.getByLabelText('Type de dépense').length
+      expect(expenseList).toEqual(6)
+    }) */
+    test("Then I should be able to choose input", () => {
+  
+        const html = NewBillUI()
+        document.body.innerHTML = html
+        expect(screen.getByTestId('expense-type')).toBeTruthy()
+        expect(screen.getByTestId('amount')).toBeTruthy()
+        expect(screen.getByTestId('datepicker')).toBeTruthy()
+        expect(screen.getByTestId('vat')).toBeTruthy()
+        expect(screen.getByTestId('pct')).toBeTruthy()
+        expect(screen.getByTestId('commentary')).toBeTruthy()
+      })
     it.todo('Should authorise the upload a file in PNG,JPEG or JPG')
     it.todo('Should permit the upload if the form is correct')
     it.todo('Should make a POST test')
diff --git a/src/containers/NewBill.js b/src/containers/NewBill.js
index 812440588..85bdfde15 100644
--- a/src/containers/NewBill.js
+++ b/src/containers/NewBill.js
@@ -32,11 +32,11 @@ export default class NewBill {
 
     if (fileExtension != "png" && fileExtension != "jpg" && fileExtension != "jpeg") {
       submitButton.disabled = true
-      const divInput = document.querySelector(`input[data-testid="file"]`)
+      const divInput = document.querySelector(`input[data-testid="file"]`).parentNode
       const errorMessage = document.createElement("div")
       errorMessage.innerText = "Wrong file extension please use PNG,JPEG or JPG"
       errorMessage.style.color = "red"
-      divInput.after(errorMessage)
+      divInput.append(errorMessage)
     }
     else {
       submitButton.disabled = false