antonio-hickey / antonio-hickey/tradestation-rs

Fix ignored doc tests

オープン
#15 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る
CI Documentation Good First Issue Help Wanted Refactor Status: Not Started Tests
主要言語
Rust
スター
14
フォーク
8
PR マージ指標
30日以内にマージされた PR はありません

説明

There's a bunch of doc tests which are marked to be ignored, for example:

https://github.com/antonio-hickey/tradestation-rs/blob/c3a9d730398ea83b632177e4ab32b37ce731ad02/src/execution.rs#L74-L107

These need some slight refactoring so that they compile and pass correctly. We still want some partial ignoring like runtime tests for most of them, because they have fake placeholder credentials in the tests. They can't be tested without mocking the HTTP request's which is out of scope for doc examples.

See this commit for an example: https://github.com/antonio-hickey/tradestation-rs/commit/fd79d4c25d70eef9ff5e6b6db66db134ac4fee94

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

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

評価

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

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

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