algorand / algorand/go-algorand

Add E2E test verifying behavior when transactions are not gossiped

Ouverte
#4,538 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub
Team Scytale
Langage dominant
Go
Étoiles
1.4k
Forks
537
Merge moyen
1 j 7 h
PR mergées (30 j)
18

Description

## Problem

In https://github.com/algorand/go-algorand/pull/4371#discussion_r940496735, we discovered an untested code path in `ledger/internal/eval.go`. Although a unit test was added to fix the thread's issue, it's problematic because there exist other untested scenarios.

It'd be ideal to create a go-algorand E2E test verifying correct behavior + defending against regressions. The challenge is to trigger the code path by adding a node that does _not_ see gossiped transactions.

## Solution

## Dependencies

## Urgency

Guide de contribution

Ouvrir le guide de contribution

Évaluation

Cette issue n'a pas encore été évaluée.

Recevez les nouvelles issues par e-mail

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