InfiniteLoopException - Limit Depth? about createandfake

[Werebunny]

Hi, sorry you're facing issues.

It seems silly to ask since I haven't updated the library in awhile, but are you on the latest version 1.8.2? I ask because I added recursion loops in my last commit. To try and reproduce the issue, I created a sample object via properties A.B.C.A and could randomize it without issue. The mechanism for this is on the RandomizerChainer, which will add a recursive reference if the previous type has already been created.

Theoretically you shouldn't ever see the InfiniteLoopException, it means I missed a link in the creation chain and didn't set the parent correctly. I've located all the spots where that is seemingly possible, and have attempted a fix.

Nuget should have version 1.8.3 here soon, I'm hoping it fixes your issue. If not, I'll need more details over how your type chain that's triggering the exception appears. Like are collections involved, generics, abstract classes, delegates... ect. As long as I can reproduce the issue with a test model, I'll be able to get it fixed here for you.

[ibebbs]

Hey, thanks for getting back to me. .

To answer your questions: Yes, I'm using version 1.8.2 and, while a rich object model, it is fairly simple so it does not use any (custom) generics, inheritance or delegates. It uses collections extensively and the issue I'm seeing involves a collection like so: a->ICollection<b>->c->a.

Hope that will provide some clues as to the nature of the issue. If not, let me know and I'll endeavour to reproduce the issue in a repository I can share.

Thanks again,
Ian

[Werebunny]

Did you see if version 1.8.3 worked for you?

I made another branch here to attempt to replicate and solve the issue, you can view the test here:
https://github.com/CreateAndFake/CreateAndFake/blob/Fix-65-InfiniteChainFix/tests/CreateAndFakeTests/IssueReplication/InfiniteChainTests.cs

I did encounter an issue with cloning objects, but not in creating them. So if you don't mind checking the model and comparing with yours I would be grateful.

[ibebbs]

Hey, just updated to 1.8.3 and the InfiniteLoopException issue is gone. Great work, thanks!

I'll close this issue now but any interest in a PR to support Uri and DateTimeOffset creation / duplication? I'm using these types extensively and currently need to add custom Create/CopyHints to support them.

[Werebunny]

I have added Uri and DateTimeOffset support in the default hints released on version 1.8.4.