algorand / algorand/algorand-sdk-testing

Use data tables

オープン
#152 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る
Team Lamprey tech debt
主要言語
Gherkin
スター
21
フォーク
35
平均マージ
15分
マージ済み PR(30日)
1

説明

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.

コントリビューションガイド

このリポジトリのコントリビューションガイドは索引されていません

評価

この issue はまだ評価されていません。

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。