10/02/2023 - Meeting Notes Libs
- Garcia-Alvarez Roberto
Asistentes
@Garcia-Alvarez Roberto
@Carlos Pastor Matut
@Daniel de la Sota Blanchart
@Alejandro Alfonso
@Alejandro Nieto Gallego
@Antonio González Sanz
@Former user (Deleted)
@Former user (Deleted)
@Manuel Gallego Chinchilla
@Pedro Lopez
@Salustiano Zavalía
PUNTOS DEL DÍA:
@Garcia-Alvarez Roberto crea la sección “Environment requirements” en el readme del repositorio de examples con las versiones de npm y node de las personas que probaron (@Garcia-Alvarez Roberto , @Daniel de la Sota Blanchart y @Alejandro Nieto Gallego) Una vez creada la sección y mergeada a máster, se cierra la tarjeta asociada: https://github.com/alastria/alastria-identity-example/issues/136
Se cierra la pull request https://github.com/alastria/alastria-identity-example/pull/106 ya que el código propuesto se ha quedado obsoleto y ya no tiene sentido. Adicionalmente se cierran las dos tarjetas asociadas al temahttps://github.com/alastria/alastria-identity-lib/issues/118 y https://github.com/alastria/alastria-identity-lib/pull/124
Se cierra el issue abierto en el repositorio de example tras dar solución al problema abierto https://github.com/alastria/alastria-identity-example/issues/146 y queda pendiente cerrar el otro issue para el lunes si no se comenta nada en dicha tarjeta, tras preguntar si ha conseguido solucionar el problema expuesto en el issue: https://github.com/alastria/alastria-identity-example/issues/145
Se han llevado a cabo dos sesiones de trabajo acerca de los test unitarios del publicKeyRegistry. En la primera los asistentes fueron:
@Garcia-Alvarez Roberto
@Daniel de la Sota Blanchart
@Antonio González Sanz
@Carlos Pastor Matut
@Juan Luis Gozalo
@Manuel Gallego Chinchilla
En la segunda sesión los asistentes fueron:
@Garcia-Alvarez Roberto
@Iñigo Garcia
@Daniel de la Sota Blanchart
@Antonio González Sanz
@Alejandro Nieto Gallego
@Juan Luis Gozalo
Tras estas dos sesiones se ha descubierto cual es el problema al ejecutar el truffle test en el repositorio de alastriaID-truffle-contracts. El problema yace en la ejecución del truffle test ya que, truffle utiliza la cuenta admin para ejecutar todo lo relacionado con dicho comando, es decir, la compilación, el despliegue y la posterior ejecución de los test unitarios. En el paso del despliegue de los smart contracts es donde se encuentra el problema, ya que, para desplegar los smart contracts de openzeppeling se necesitan dos cuentas, y truffle no dispone de otra cuenta para realizar esto, por lo que cuando se va a desplegar el smart contract falla si la cuenta es admin, ya que la cuenta admin no puede llamar a los fallback. Se adjunta imagen de la parte del código que da problemas.
Tras las dos sesiones de trabajo se ha determinado los dos siguientes puntos para avanzar:
Migrar todo el repo a hardhat y abandonar tal cual esta el de truffle. Dentro de esta opción, hay dos posibilidades:
1.1 Migrar tal cual estan los contratos ahora mismo.
1.2 Migrar con ciertos cambios que comentó @Iñigo Garcia , como por ejemplo, la eliminación del smart contract alastriaIdentityIssuer.sol, la eliminación del la librería eidas.sol y la unificación del AlastriaCredentialRegistry.sol y del AlastriaPresentationRegistry.sol en un único smart contract AlastriaObjectRegistry.sol.Hacer los test como los hizo @Carlos Pastor Matut del AlastriasCredentialRegistry y del AlastriaPresentationRegistry y ejecutarlos desde remix para probarlos. (Con el problema de que los test estan hechos con ethers y debería de estar hechos con web3 ya que el repositorio es de truffle)
En la reunión se toma la decisión de hacer unos test AlastriaPublicKeyRegistry.ts como los del AlastriasCredentialRegistry.ts y AlastriaPresentationRegistry .ts con el objetivo de cerrar esta versión de smart contracts y este repositori de truffle con el mejor empaque posible. Una vez se tenga esto hecho, se realizaría la migración a hardhat de todo el repositorio que es donde se contuinuaría iterando y dando soporte. Por otro lado, también se ha comentado en la reunión que tras realizar estas pruebas documentar en el README.md cuales serían los pasos a realizar para la ejecución de estos test en remix.
