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.

Sign up for GitHub

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

Jump to bottom

Bug: Maximum call stack size exceeded #5262

Closed
lneumeier opened this issue Mar 22, 2023 · 5 comments · Fixed by #5310
Closed

Bug: Maximum call stack size exceeded #5262

lneumeier opened this issue Mar 22, 2023 · 5 comments · Fixed by #5310
Assignees
@satanTime
Labels
bug Something isn't working released v14.7.2

Comments

@lneumeier
Copy link

lneumeier commented Mar 22, 2023 •
edited

After the update from 14.5.0 to 14.5.1 (#4564) I'm getting following error

Maximum call stack size exceeded
        at Object.isExcludedDef (node_modules/ng-mocks/index.mjs:1:6637)
        at node_modules/ng-mocks/index.mjs:1:10494
        at t (node_modules/ng-mocks/index.mjs:1:10437)
        at node_modules/ng-mocks/index.mjs:1:10528
        at t (node_modules/ng-mocks/index.mjs:1:10437)
        at node_modules/ng-mocks/index.mjs:1:10528
        at t (node_modules/ng-mocks/index.mjs:1:10437)
        at node_modules/ng-mocks/index.mjs:1:10528
        at t (node_modules/ng-mocks/index.mjs:1:10437)
        at node_modules/ng-mocks/index.mjs:1:10528

on some of my unittests. Do you have any idea what I should look for?

PS:

I'm using

declarations: [MockComponents(Example1Component, Example2Component)],

and

declarations: [MockComponent(ExampleComponent)],
@lneumeier lneumeier added the bug Something isn't working label Mar 22, 2023
@satanTime satanTime mentioned this issue Mar 25, 2023
Merged
satanTime added a commit to satanTime/ng-mocks that referenced this issue Mar 25, 2023
@satanTime
fix(core): detecting self-references in mocks help-me-mom#5262
be540e4
satanTime added a commit to satanTime/ng-mocks that referenced this issue Mar 25, 2023
@satanTime
fix(core): detecting self-references in mocks help-me-mom#5262
ba3e0fc
satanTime added a commit to satanTime/ng-mocks that referenced this issue Mar 25, 2023
@satanTime
fix(core): detecting self-references in mocks help-me-mom#5262
c699d57
satanTime added a commit that referenced this issue Mar 25, 2023
@satanTime
Merge pull request #5310 from satanTime/issues/5262
49ad0b2 
fix(core): detecting self-references in mocks #5262
@satanTime
Copy link
Member

v14.7.2 has been released and contains a fix for the issue. Feel free to reopen the issue or to submit a new one if you meet any problems.

@lneumeier
Copy link
Author

Thank you for the quick fix.

@satanTime
Copy link
Member

Not at all, please let me know if it solves your issue.

@lneumeier
Copy link
Author

Yes, the issue is solved. Thank you.

@satanTime
Copy link
Member

Cool, thanks! Stay in touch!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@satanTime satanTime

Labels
bug Something isn't working released v14.7.2
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix(core): detecting self-references in mocks #5262 satanTime/ng-mocks
2 participants
@satanTime @lneumeier