search

softprops / dynomite

⚡🦀 🧨 make your rust types fit DynamoDB and visa versa
https://docs.rs/dynomite/
MIT License
220 stars 53 forks source link

fix test item-not-on-struct-fail's expected stderr to include error code #153

Closed phrohdoh closed 3 years ago

phrohdoh commented 3 years ago

~Note that this sits on top of #150 (to avoid CI failing due to those issues) and if/once that is merged this should be rebased on top of then-master such that this PR contains only relevant commits. Of course if this is wanted without waiting on #150, those commits can be rebased away from this PR's head repo/branch as desired.~

150 has been merged and this PR has been rebased on top of the new master, so this can be merged.

What did you implement:

Testing compilation error of #[derive(Item)] on a non-struct (item-not-on-struct-fail) failed due to missing error code in the expected stderr content, so I've added the code.

How did you verify your change:

$ cargo t
...
test item-not-on-struct-fail.rs [should fail to compile] ... ok
...
softprops commented 3 years ago

Really appreciate the extra help on this

softprops commented 3 years ago

Merged the other change if you wanted to rebase I'll merge this

phrohdoh commented 3 years ago

Merged the other change if you wanted to rebase I'll merge this

Rebased!