feat/dataverse-data #1589

merunga · 2023-09-23T14:28:40Z

Addresses #1576

projects/02-dataverse-explorers/tests-read-only/oa/oa-prompting.spec.js

unjust · 2023-09-23T14:48:27Z

projects/02-dataverse-explorers/tests-read-only/oa/oa-prompting.spec.js

+        });
+
+        it('Los nombres de todas las propiedades deben tener formato camelCase', () => {
+          expect.assertions();


deberia decir cuantas assertions? (Object.keys(item.facts)).length o no importa?

hay assertions(number) y hasAssertions() https://jestjs.io/docs/expect#assertion-count

tienes razon, uso hasAssertions

projects/02-dataverse-explorers/tests-read-only/oa/oa-prompting.spec.js

unjust · 2023-09-23T14:53:49Z

projects/02-dataverse-explorers/tests-read-only/oa/oa-prompting.spec.js

+    it.skip('Los elementos comparten en sus "facts" al menos 3 propiedades', () => {
+      // FIXME
+    });
+  });


Suggested change

it.skip('Los elementos comparten en sus "facts" al menos 3 propiedades', () => {

// FIXME

});

});

it('Los elementos comparten en sus "facts" al menos 3 propiedades', () => {

expect.assertions(data.length);

data.forEach(obj => {

expect(Object.keys(obj.facts).length).toBeGreaterThanOrEqual(3);

})

});

});

Aquí no se está chequeando q sean compartidos, no?

… in README

…rt-1 (#1604) * boilerplate for data lovers makeover * rename and reorder contents, add section objetivos aprendizaje al readme * renderitems * cambios del feedback en tests e2e, oas, boilerplate, y readme * mata errores mdlint * Ultimos cambios feedback 1 (#1541) * quita package.lock * viewFunctions -> view * move file * Aplicamos cambios de prompting al readme * Borrado de data de ejemplo y agregado de data generada por AI y test * Dentro del resumen del proyecto: 'Construirás una página web' primero * Cambio de 'cálculo agregado' por 'estadística' * Cambio de orden entre Consideraciones Generales y Funcionalidades * Cambio en el diagrama de boilerplate, dejo carpeta tests-read-only vacía * sección Prompt Utilizado agregada * Acento en propósito * Agregado de wireframes mobile de ejemplo * Referencia al screenshot del prompt utilizado dentro de Consideraciones Generales * Cambio en tiempo máximo de tiempo para generar los datos * Cambio de usuarios por usuarias * Cambio de extensión archivos de datos data.json a .js * Cambio de texto haciendo referencia al archivo javascript en vez de .json * usuario -> usuaria en todo el readme * Cambio en texto dentro de parentesis en Historias de Usuaria * Varios cambios en redacción sugeridos por Sergio * Requisito de mínimo 100 palabras agregado a descripcion * agregado de descripcion larga a los datos de ejemplo * agregado de -que vas a generar con prompting- en Resumen del proyecto * acento agregado en la palabra técnicas * Agregado de petición de captura de prompt en Consideraciones para pedir PF * Revisión general de todo el readme * Cambios pequeños - revisión Sergio * cambios en archivo oas-js-datatypes.js * Redacción recomendando páginas sobre generación de imágenes con IA * corrección en número de assertions * historias de usuaria -> usuario * explicación de chat con data en -Generar los datos- * Cambio img -> imgSrc * Corrección: data ahora tiene descripciones con más de 100 palabras * Agregado: tests para verificar que la longitud de la descipción sea mas de 100 palabras y verificacion de URL * Agregado: expect.assertions en los tests de las propiedades -value- y -label- * agregado de assertions en todos los tests * Apply suggestions from code review * Cambiando sugerencia de generador de imagenes a Bing * Pull con cambio pequeño de de Ivy * moving files into 02-dataverse-explorers, checkout 02-data-lovers from main (#3) * Update README.md for mdlint errors * fix: cambios con feedback DEV011 - resaltar root, modificar instrucciones Addresses #1577 (#1587) * resaltar root, modificar instruccion de view * cambio de instrucciones en dataFunctions, cambiar el console.log para ejecutar las funciones, añadir parametro en view.js * mejorar instrucciones en dataFunctions * espacios despues comments * Quita prototipo de baja, reordenar indice --------- Co-authored-by: Ivy Feraco <[email protected]> Co-authored-by: Ivy Feraco <[email protected]> * fix: remove extra docs in Dataverse and edit title (#1586) * remove EXTRA and old data lovers README in pt * move some resources to main README, delete FAQ * Apply suggestions from code review, acentos etc. Co-authored-by: David Ruiz <[email protected]> * Update README.md quita card class --------- Co-authored-by: David Ruiz <[email protected]> * fix: adds prompting objectives to project.yml (#1585) * fix: dataset changes, diversificacion y tests addresses #1576 (#1589) * Dataverse: Data spec * Dataverse: Nueva data * Dataverse: Data test * dataverse: apply comments * dataverse: fix asasertions call. remove old tests. reference new test in README * fix: renaming dataverse project (no explorers) * fix: script test in package.json fail in windows (#1592) * fix: dataverse images y tests de facts #1580 (#1599) * Dataverse: Data spec * Dataverse: Nueva data * Dataverse: Data test * dataverse: apply comments * dataverse: fix asasertions call. remove old tests. reference new test in README * dataverse: test para 3 propiedades compartidas en facts * dataverse: dataset images --------- Co-authored-by: merunga <[email protected]> --------- Co-authored-by: Davichobits <[email protected]> Co-authored-by: DeysiVergara <[email protected]> Co-authored-by: David Ruiz <[email protected]> Co-authored-by: Mariano Crowe <[email protected]> Co-authored-by: AlexisJoselyn <[email protected]>

merunga added 3 commits September 23, 2023 09:27

Dataverse: Data spec

5835931

Dataverse: Nueva data

b30d101

Dataverse: Data test

b876170

merunga changed the title ~~Feat/dataverse part 1~~ feat/dataverse-data Sep 23, 2023

merunga requested a review from unjust September 23, 2023 14:29

dataverse: apply comments

3c7e469

unjust assigned merunga Sep 23, 2023

unjust suggested changes Sep 23, 2023

View reviewed changes

dataverse: fix asasertions call. remove old tests. reference new test…

745ba94

… in README

unjust marked this pull request as ready for review September 23, 2023 15:24

unjust approved these changes Sep 23, 2023

View reviewed changes

unjust linked an issue Sep 23, 2023 that may be closed by this pull request

Dataverse: dataset consideraciones con funcionalidades de proyecto #1576

Closed

unjust merged commit e4bf6ff into Laboratoria:feat/dataverse-part-1 Sep 23, 2023
4 checks passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat/dataverse-data #1589

feat/dataverse-data #1589

merunga commented Sep 23, 2023 •

edited by unjust

Loading

unjust Sep 23, 2023

merunga Sep 23, 2023

unjust Sep 23, 2023

merunga Sep 23, 2023

-    it.skip('Los elementos comparten en sus "facts" al menos 3 propiedades', () => {
-      // FIXME
-    });
-  });
+    it('Los elementos comparten en sus "facts" al menos 3 propiedades', () => {
+      expect.assertions(data.length);
+      data.forEach(obj => {
+            expect(Object.keys(obj.facts).length).toBeGreaterThanOrEqual(3);
+      })
+    });
+  });

feat/dataverse-data #1589

feat/dataverse-data #1589

Conversation

merunga commented Sep 23, 2023 • edited by unjust Loading

unjust Sep 23, 2023

Choose a reason for hiding this comment

merunga Sep 23, 2023

Choose a reason for hiding this comment

unjust Sep 23, 2023

Choose a reason for hiding this comment

merunga Sep 23, 2023

Choose a reason for hiding this comment

merunga commented Sep 23, 2023 •

edited by unjust

Loading