Join GitHub today
GitHub is home to over 28 million developers working together to host and review code, manage projects, and build software together.
Sign upWrite tests for `#[derive(Associations)] #1484
Comments
weiznich
added this to the 1.2 milestone
Jan 16, 2018
weiznich
added
the
tests
label
Jan 16, 2018
This comment has been minimized.
|
There are already tests in |
weiznich
closed this
Jan 22, 2018
This comment has been minimized.
|
They could stand to be a lot more exhaustive. For starters, there are no tests for having more than one association, using tuple structs, cases where the foreign key field is named differently than the foreign key column. |
sgrif
reopened this
Jan 23, 2018
added a commit
that referenced
this issue
Feb 4, 2018
sgrif
referenced this issue
Feb 4, 2018
Merged
Add some missing test cases for `#[belongs_to]` #1539
added a commit
that referenced
this issue
Feb 5, 2018
Eijebong
closed this
in
#1539
Feb 5, 2018
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
weiznich commentedJan 16, 2018
While porting
#[derive(Associations)]from the old macro_rules based derive to a normal procedural macro derive in #1468 we noticed that there are no tests for this derive.We should definitively write some tests to check if everything works as assumed.