linera-io/linera-protocol

Split `test_simple_user_operation` unit test into smaller unit tests

Chiusa

#1484 aperta il 12 gen 2024

Vedi su GitHubGuida contributor
 (0 commenti) (0 reazioni) (0 assegnatari)Rust (2354 fork)batch import
good first issuerefactortesting

Metriche repository

Star
 (32.149 stelle)
Metriche merge PR
 (Merge medio 2g 22h) (111 PR mergiate in 30 g)

Descrizione

Motivation

The test_simple_user_operation is an integration test that covers a few different functionalities, like:

  • Handling an operation
  • Forwarding authentication
  • Using the runtime storage
  • Performing a cross-application call
  • Using a session
  • Performing a query

Partly this was because it relied on a custom TestApplication type before #1482 was merged replacing it with a more customizable MockApplication. With the MockApplication it is now possible to split the test into simpler tests that are more like unit tests, which would help with regressions by making it simpler to isolate the issues.

Proposal

Split the test into a few smaller tests. One suggestion is to have at least:

  • a test for handling an operation that uses the runtime storage
  • a test that performs a query
  • a test that performs a cross-application call
  • a test to ensure authentication is forwarded correctly

New tests can also be added, like for example:

  • a test that performs a cross-application query
  • a test that calls a session more than once

Guida contributor

Direzione di ricerca
Esamina la funzione `test simple user operation` in `linera execution/tests/test execution.rs`. Identifica le funzionalità distinte elencate nella proposta (gestione dell'operazione, inoltro dell'autenticazione, storage di runtime, chiamata cross application, sessione, query). Quindi crea nuove funzioni di test per ciascuna, utilizzando `MockApplication` per isolare ogni scenario. Assicurati che i nuovi test siano posizionati nello stesso file e seguano i pattern di test esistenti.
Tech stack
rust
Dominio
testing
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Rust testing experienceKnowledge of linera execution test framework
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
85