diff --git a/src/Selector.ts b/src/Selector.ts index 10ebe2a..c08ad40 100644 --- a/src/Selector.ts +++ b/src/Selector.ts @@ -13,8 +13,8 @@ export class Selector implements Selectors private _values: string[]=[]; private _name: string=""; - // Injection de la classe principale, mais uniquement si des données ont été importées - // Le champ duquel le sélecteur tire ses données doit exister ? + // Injection de la classe principale, mais uniquement si des données ont été importées. + // Le champ duquel le sélecteur tire ses données doit exister. constructor(converter: FreeDatas2HTML, datasFieldNb: number, elt: DOMElement, separator?: string) { if(converter.fields.length === 0 || converter.datas.length === 0) @@ -70,13 +70,13 @@ export class Selector implements Selectors // Création du "; // l'option zéro permet d'actualiser l'affichage en ignorant ce filtre + let selectorsHTML=""; - this. _datasViewElt.eltDOM!.innerHTML=selectorsHTML;// "!" car l'existence de "eltDOM" est testé par le constructeur - + this. _datasViewElt.eltDOM!.innerHTML=selectorsHTML;// "!", car l'existence de "eltDOM" est testé par le constructeur. + + // Actualisation de l'affichage lorsqu'une valeur est sélectionnée : const selectElement=document.getElementById("freeDatas2HTML_"+this._datasViewElt.id) as HTMLInputElement, mySelector=this; selectElement.addEventListener("change", function(e) { diff --git a/tests/selectorSpec.ts b/tests/selectorSpec.ts index ababeb4..7d3da94 100644 --- a/tests/selectorSpec.ts +++ b/tests/selectorSpec.ts @@ -29,7 +29,7 @@ describe("Test des sélecteurs de données", () => // Convertisseur non lancé : converter=new FreeDatas2HTML("CSV"); expect(() => { return new Selector(converter, 0, { id:"selector1" }); }).toThrowError(errors.filterNeedDatas); - // Note : les parseurs vont générer une erreur en amont s'ils ne trouvent pas de noms de champs. + // Note : les parseurs vont générer une erreur en amont s'ils ne trouvent pas de noms de champs // Par contre, ils acceptent de ne pas trouver de données : converter.parser.datas2Parse="Z (numéro atomique),Élément,Symbole,Famille,Abondance des éléments dans la croûte terrestre (μg/k)"; await converter.run(); @@ -49,7 +49,7 @@ describe("Test des sélecteurs de données", () => expect(selector.separator).toBeUndefined(); }); - it("Si toutes les paramètres sont correctes, ils doivent être acceptés.", () => + it("Si toutes les paramètres sont valides, ils doivent être acceptés.", () => { expect(() => { selector=new Selector(converter, 2, { id:"selector1" }, ","); return true; }).not.toThrowError(); expect(selector.datasFieldNb).toEqual(2); @@ -198,7 +198,7 @@ describe("Test des sélecteurs de données", () => expect(selector.dataIsOk({ "Famille": "Hallo Eugène !" })).toBeFalse(); }); - it("Doit retourner true, si une donnée testée correspond pas à la valeur sélectionnée pour ce filtre.", () => + it("Doit retourner true, si une donnée testée correspond à la valeur sélectionnée pour ce filtre.", () => { selectElement.value="4"; selectElement.dispatchEvent(new Event('change')); @@ -238,11 +238,11 @@ describe("Test des sélecteurs de données", () => selectElement.value="4"; selectElement.dispatchEvent(new Event('change')); expect(selector.dataIsOk({ "Étiquettes": "Exemple3" })).toBeTrue(); - // Même pas seule : + // Même quand elle n'est pas seule : expect(selector.dataIsOk({ "Étiquettes": "Exemple3|Exemple1|Exemple9" })).toBeTrue(); expect(selector.dataIsOk({ "Étiquettes": "Exemple0|Exemple3|Exemple2" })).toBeTrue(); expect(selector.dataIsOk({ "Étiquettes": "Exemple0|Exemple4|Exemple3" })).toBeTrue(); - // Y compris si entourée d'espaces : + // Ou entourée d'espaces : expect(selector.dataIsOk({ "Étiquettes": "Exemple0|Exemple4| Exemple3 " })).toBeTrue(); expect(selector.dataIsOk({ "Étiquettes": " Exemple3 " })).toBeTrue(); });