tanakh / cargo-atcoder

Cargo subcommand for AtCoder
BSD 3-Clause "New" or "Revised" License
397 stars 36 forks source link

Add tests for `new`, `test`, and `gen-binary` #30

Closed qryxip closed 4 years ago

qryxip commented 4 years ago

以前「多数のパッケージを一つのワークスペースで管理することでwarmupを不要にする」という話をしましたが、これ(に限らず)を実装するにあったて既存の使いかたに支障が出ないことをある程度確実に保証した方が良いと思うので。

この間use_crossをぶっ壊しちゃいましたし。

.
├── abc157
│   ├── Cargo.toml
│   └── src
├── abc158
│   ├── Cargo.toml
│   └── src
├── abc159
│   ├── Cargo.toml
│   └── src
├── Cargo.lock
├── Cargo.toml
└── target
    ├── debug
    └── release

一応assert_cmdを使うデメリットとしてcoverageが取れないというのがありますがsrc/下をあれこれ書き換えるのが結構面倒なのもあって使いました。

なんかtestとかが公開コンテストに対してもすべてログインを要求するので一旦draftで。

qryxip commented 4 years ago

あ、しまった。 途中でdraftにはできないんですよね...

qryxip commented 4 years ago

あ、しまった。 途中でdraftにはできないんですよね...

いつの間にできるようになってた...

Still in progress? Convert to draft

qryxip commented 4 years ago

omg https://github.community/t5/GitHub-Actions/Why-is-Docker-not-installed-on-macOS/td-p/39364

qryxip commented 4 years ago

gen-binaryのテストは#[cfg(target_os = "linux")]にしました...

tanakh commented 4 years ago

修正ありがとうございます。マージしました。