[IMPROVEMENT] Add integration test for FlowYieldVaultsEVM
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 35/100
- Tipo de issue
- Nueva funcionalidad
- Claridad
- Bastante claro
- Estado de actividad
- Estancado
- Stack tecnológico
- blockchain
- Área
- blockchain, testing
Línea de trabajo
Start by reviewing the existing basic integration test in FlowYieldVaultsEVM and the draft implementation in FlowYieldVaults pull request #89. Compare the possible approaches for adding coverage in FlowYieldVaults, reusing the EVM test, or removing the submodule dependency. Done means the repository has an integration test that confirms FlowYieldVaultsEVM functionality without introducing the noted submodule import loop.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Instructions
FlowYieldVaultsEVM was added as a submodule recently. We skipped writing new integration tests as the current FlowYieldVaultsEVM repository provides a basic integration test.
Issue To Be Solved
We still think a test suite is needed in theFlowYieldVaults repo to confirm correct functionality of submodule.
Solution
Multiple ways to solve this issue:
- Write a new integration test suite in
FlowYieldVaults(there's a draft here) - Update the integration test in
FlowYieldVaultsEVMto be re-usable. Call it fromFlowYieldVaults. - Copy the code from
FlowYieldVaultsEVMintoFlowYieldVaultsinstead of using submodules. The original integration test will then serve as the only correct one. This can also solve the current potential submodule import loop (FlowYieldVaultshasFlowYieldVaultsEVMandFlowYieldVaultsEVMhasFlowYieldVaultsas submodules)
- Lenguaje dominante
- Cadence
- Estrellas
- 41
- Forks
- 0
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
- Incluye un Dockerfile o un archivo de Docker Compose
- Tiene una plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de onflow/FlowYieldVaults
-
Clarify contract version namingAbierto
Dificultad 4/5 3-5 días Aptitud para principiantes 25/100
onflow/FlowYieldVaults#274 ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
onflow/FlowYieldVaults#273 ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 48/100
onflow/FlowYieldVaults#272 ·
-
Remove outdated documentationAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 35/100
onflow/FlowYieldVaults#271 ·
-
Move all test files into `./cadence/tests`Quizá libre de nuevo Un pull request para esta issue se cerró sin fusionarse. Abierto
Dificultad 3/5 1-2 días Aptitud para principiantes 35/100
onflow/FlowYieldVaults#270 ·
Todos los issues de onflow/FlowYieldVaults
Issues similares
-
documentation good first issue help wanted
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
Sorogate/example-consumer#17 ·
-
documentation good first issue help wanted
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
bsv-blockchain/teranode#1918 ·
Los mantenedores suelen responder en 2 días
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 66/100
paritytech/individuality-community#191 ·
Los mantenedores suelen responder en 6 días
-
hook: V23Hook (robinhood)Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
Los mantenedores suelen responder en 1 día