From 78d5279eed5c502f517676174778be78e0ab0680 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Fabrice=20PENHO=C3=8BT?= Date: Thu, 12 Aug 2021 17:57:05 +0200 Subject: [PATCH] =?UTF-8?q?Suppression=20du=20message=20d'erreur=20dans=20?= =?UTF-8?q?la=20page=20en=20cas=20fichier=20sans=20donn=C3=A9es.?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- src/freeCSV2HTML.ts | 9 +++------ tests/freeCSV2HTMLSpec.ts | 8 ++++---- 2 files changed, 7 insertions(+), 10 deletions(-) diff --git a/src/freeCSV2HTML.ts b/src/freeCSV2HTML.ts index a38e594..5d96676 100644 --- a/src/freeCSV2HTML.ts +++ b/src/freeCSV2HTML.ts @@ -110,16 +110,13 @@ export class freeCSV2HTML await this.parse(); - if(this.parseDatas.length === 0 || this.parseMeta!.fields === undefined) // je force avec "!", car l'existence de parseMeta certaine après parse(). - { - this._datasViewElt.eltDOM.innerHTML=errors.datasNotFound; - return false; - } + if(this.parseDatas.length === 0 || this.parseMeta!.fields === undefined) + throw new Error(errors.datasNotFound); else { let converter=this; // Affichage initial des données du fichier - this.datasHTML=this.createDatasHTML(this.parseMeta!.fields, this.parseDatas); + this.datasHTML=this.createDatasHTML(this.parseMeta!.fields, this.parseDatas); // je force avec "!", car l'existence de parseMeta certaine après parse(). this._datasViewElt.eltDOM.innerHTML=this.datasHTML; // Si demandé, création des listes permettant de filter les données diff --git a/tests/freeCSV2HTMLSpec.ts b/tests/freeCSV2HTMLSpec.ts index a904cba..514e02d 100644 --- a/tests/freeCSV2HTMLSpec.ts +++ b/tests/freeCSV2HTMLSpec.ts @@ -101,14 +101,14 @@ describe("freeCSV2HTML", () => converter.datasSourceUrl="http://localhost:9876/datas/datas1.csv"; await expectAsync(converter.run()).toBeResolved(); }); - - it("Doit afficher une erreur dans la page si aucune donnée n'a été trouvée dans le fichier.", async () => + + it("Doit générer une erreur si aucune donnée n'a été trouvée dans le fichier et ne rien afficher dans l'emplacement prévu pour les données.", async () => { converter.datasViewElt={ id:"datas" }; converter.datasSourceUrl="http://localhost:9876/datas/nodatas.csv"; - await converter.run(); + await expectAsync(converter.run()).toBeRejectedWith(new Error(errors.datasNotFound)); let txtDatasViewsElt=document.getElementById("datas").innerHTML; - expect(txtDatasViewsElt).toEqual(errors.datasNotFound); + expect(txtDatasViewsElt).toEqual(""); }); it("Doit afficher un tableau correspondant aux données du fichier csv", async () =>