Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Set::extract with habtm not retrieving correctly and unit test hides #196

Closed
fellars opened this issue Nov 24, 2011 · 0 comments
Closed
Labels
Milestone

Comments

@fellars
Copy link
Contributor

fellars commented Nov 24, 2011

the unit test for testExtractHabtm() line 636 in SetTest shows the results of the extraction for first two tests innacurately.

The last entry in the results should be the data from the Comment with name = jim, but instead it returns (and the expected matches) another comment by 'dan'.

I asked Nate on #li3 and he thought it might be a bug as well.

I'm seeing similar strange behaviors with my extractions as well which led me to dig into the unit tests and find this behavior.

The expected results should be:
$this->assertEqual($result[3]['Comment']['User']['name'], 'jim'); //currently for index 3 the name is 'dan', but should be 'jim'

@jails jails closed this as completed in 719e278 Dec 13, 2012
nateabele added a commit that referenced this issue Dec 13, 2012
Fix `Set::extract()` close #196
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants