0xMiden / 0xMiden/web-sdk

Redo integration-testing

Ouverte
#375 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub
Langage dominant
TypeScript
Étoiles
1
Forks
21
Merge moyen
12 h 14 min
PR mergées (30 j)
41

Description

cc @WiktorStarczewski
The web-sdk integration tests seem to rely on rust-sdk's test-node script and a manually pinned rust-sdk commit. This is a bit hacky. cc
This also means:

* web-sdk's fee path is not actually tested (it does not use the genesis etc)
* changes to rust-sdk's test setup can break web-sdk CI
* the pin needs to be updated manually when versions change
* running the same setup locally is awkward

We already hit this when rust-sdk enabled fees by default and the web-sdk tests started failing. The workaround was to disable fees again. We should at least:

1. Fund test accounts from the genesis funders and run CI with fees enabled.
2. Move the test-node startup logic into web-sdk.
3. Keep the genesis config in web-sdk instead of depending on rust-sdk's unpublished test crate.
4. Add a simple local command to start the same test node used in CI.

Guide de contribution

Ouvrir le guide de contribution

Piste de recherche

Look at the existing integration tests in the web-sdk to see how they currently invoke the rust-sdk's test-node script. Examine the CI configuration to understand the current setup. The goal is to move the test-node startup logic and genesis configuration into the web-sdk repository, decoupling it from the rust-sdk. A good starting point is to replicate the test node startup in a local script within web-sdk, ensuring test accounts are funded from genesis for fee-enabled tests.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
docker, node.js, rust, typescript
Domaine
ci-cd, testing, tooling
Type d'issue
Refactorisation
Difficulté
4/5
Temps estimé
3-5 jours
Activité
Active
Clarté
Clairement spécifiée
Accessibilité débutants
45/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.