solving assertion failure when parsing fetch request

[brightredchilli]

Hi rentzsch, im referencing:#81

I ran into this problem with one of my fetch requests, and I propose the fix above. What was happening is that the _resolveKeyPathType: fails when the key path contains an attribute. I hope this is helpful. Let me know if I can clarify anything.

[rentzsch]

Looking good, thanks. One request, though: could you add an example fetch request to the mogeneratorTestMule that needs this fix? That will help me understand its use-case and ensure it doesn't get broken in the future.

[brightredchilli]

I would be happy to except im quite confused by the mogeneratortestmule. At
no point does it actually seem to include the files generated by
mogenerator, and at no point does it seem to run mogenerator itself. Im not
even sure where the test cases are run-is it just in the app delegate?
There seems to be only one test case now.

On Tue, Jun 5, 2012 at 11:21 AM, Jonathan 'Wolf' Rentzsch <
[email protected]

wrote:

Looking good, thanks. One request, though: could you add an example fetch
request to the mogeneratorTestMule that needs this fix? That will help me
understand its use-case and ensure it doesn't get broken in the future.

---

Reply to this email directly or view it on GitHub:
#114 (comment)

[rentzsch]

Ah, mogeneratorTestMule uses the old include.m mechanism -- see its mogenerate.command file for the invocation and notice the project includes the generated include.m file.

Yeah, the "test case" is just to get the app compiled has have it successfully run.

Thanks!

[brightredchilli]

thanks, ill take a look into it.

On Tue, Jun 5, 2012 at 11:56 PM, Jonathan 'Wolf' Rentzsch <
[email protected]

wrote:

Ah, mogeneratorTestMule uses the old include.m mechanism -- see its
mogenerate.command file for the invocation and notice the project
includes the generated include.m file.

Yeah, the "test case" is just to get the app compiled has have it
successfully run.

Thanks!

---

Reply to this email directly or view it on GitHub:
#114 (comment)

[rentzsch]

Dup of #15