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

Hamcrest falls into stack overflow when missing attribute is addressed #820

Closed
mximp opened this issue Jul 5, 2022 · 3 comments
Closed

Comments

@mximp
Copy link
Contributor

mximp commented Jul 5, 2022

Test below test fails with stack overflow:

[] > invalid-test
  assert-that > @
    TRUE
    $.equals-to TRUE
[ERROR] EOorg.EOeolang.EOinvalid_testTest.testWorks  Time elapsed: 0.01 s  <<< ERROR!
java.lang.StackOverflowError

[ERROR]   EOinvalid_testTest>PhDefault.attr:237->PhDefault.attr:237->PhDefault.attr:237->PhDefault.attr:237->PhDefault.attr:237->PhDefault.attr:237->PhDefault.attr:237->PhDefault.attr:237->PhDefault.attr:237->PhDefault.attr:237->PhDefault.attr:237->PhDefault.attr:237->PhDefault.attr:237->PhDefault.attr:237->PhDefault.attr:237->PhDefault.attr:237->PhDefault.attr:237->PhDefault.attr:237->PhDefault.attr:237->PhDefault.attr:237->PhDefault.attr:237->PhDefault.attr:237->PhDefault.attr:237->PhDefault.attr:237->PhDefault.attr:237->PhDefault.attr:237->PhDefault.attr:237->PhDefault.attr:237->PhDefault.attr:237->PhDefault.attr:237->PhDefault.attr:237->PhDefault.attr:237->PhDefault.attr:237->PhDefault.attr:237->PhDefault.attr:237->PhDefault.attr:237->PhDefault.attr:237->PhDefault.attr:237->PhDefault.attr:237->PhDefault.attr:237->PhDefault.attr:237->PhDefault.attr:237->PhDefault.attr:237->PhDefault.attr:237->PhDefault.attr:237->PhDefault.attr:237->PhDefault.attr:237->PhDefault.attr:237 » StackOverflow

Expected behaviour: something like Can't get(), attribute "equals-to" is absent among other attrs (ρ, minus, σ, lt, eq, gt, as-hash, plus, div, neg, times, Δ, as-bytes, gte, lte) and φ is absent

@mximp
Copy link
Contributor Author

mximp commented Jul 5, 2022

@Graur please check

@Graur
Copy link
Contributor

Graur commented Jul 5, 2022

@mximp Thanks! I moved this issue to eo-hamcrest. So, I think you can close this issue now.

@mximp mximp closed this as completed Jul 6, 2022
@mximp
Copy link
Contributor Author

mximp commented Jul 6, 2022

Closing as will be tracked in eo-hamcrest

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants