algorand / algorand/algorand-sdk-testing

Use data tables

Aperta
#152 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub
Team Lamprey tech debt
Lingua principale
Gherkin
Stelle
21
Fork
35
Merge medio
15m
PR unite (30g)
1

Descrizione

Just forwarding a recommendation I got to use [data tables](https://cucumber.io/docs/gherkin/reference/#data-tables):

https://github.com/cucumber-rs/cucumber/issues/183#issuecomment-988979226

> Instead
```
And I build an application transaction with the transient account, the current application, suggested params, operation "create", approval-program "programs/big_app_program.teal.tok", clear-program "programs/big_app_program.teal.tok", global-bytes , global-ints 0, local-bytes , local-ints 0, app-args "", foreign-apps "", foreign-assets "", app-accounts "", extra-pages 3
```
> do (or similar)
```
And I build an application transaction with
| operation | create |
| approval-program | programs/big_app_program.teal.tok |
| clear-program | programs/big_app_program.teal.tok |
| global-bytes | |
| global-ints | 0 |
| local-bytes | |
| local-ints | 0 |
| app-args | |
| foreign-apps | |
| foreign-assets | |
| app-accounts | |
| extra-pages | 3 |
```

> It would be easier both read/match the step and access the table data via gherkin::Step.

I imagine that these kind of changes are difficult to do, since they'd break the tests in all the SDKs, but worth maybe for future features.

Guida per i contributori

Nessuna guida per i contributori indicizzata per questo repository

Valutazione

Questa issue non è ancora stata valutata.

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.