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

RangeError with correct solution on Waypoint: Passing Values to Functions with Arguments #5560

Closed
BKinahan opened this issue Dec 30, 2015 · 27 comments

Comments

@BKinahan
Copy link
Contributor

Challenge Waypoint: Passing Values to Functions with Arguments has an issue.
User Agent is: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_11_2) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/47.0.2526.106 Safari/537.36.

Running tests outputs RangeError: Maximum call stack size exceeded and no tests pass.

My code:

// Example
function ourFunction(a, b) {
  console.log(a - b);
}
ourFunction(10, 5); // Outputs 5

// Only change code below this line.

function myFunction(c, d){
  console.log(c + d);
}
myFunction(2, 3);
@ltegman
Copy link
Member

ltegman commented Dec 30, 2015

Huh, I'm unable to repro this using the exact same code. Anyone else seeing this error?

@BKinahan
Copy link
Contributor Author

Just noticed it's mentioned in #5499 so it's not just me.. However I just reopened the waypoint page now and the tests were passed before running the tests, and running the tests was successful. Not sure what's going on there.

@nicolefffe
Copy link

I am having this problem too.

// Example
function ourFunction(a, b) {
  console.log(a - b);
}
ourFunction(10, 5); // Outputs 5

// Only change code below this line.
function myFunction(arg1, arg2) {
  console.log(arg1 + arg2);
}

myFunction(5, 4);

No test cases pass and I get the error: RangeError: Maximum call stack size exceeded

@nicolefffe
Copy link

I also re-entered the waypoint from the map and the same code passed the tests, so it is a mystery :)

@lucabarbetti
Copy link

Test successfully passed after reloading the page.

@masondz
Copy link

masondz commented Dec 30, 2015

Reloading the page also worked for me.

@harwluk
Copy link

harwluk commented Dec 30, 2015

having same issue. reloaded the page worked for me, too.

@ngorton91
Copy link

Getting this same error

@danjm75
Copy link

danjm75 commented Dec 31, 2015

same problem. Is there an actual fix, or did I miss it?

@abhisekp
Copy link
Member

Works for me now. Was getting range stack error previously.

@danjm75
Copy link

danjm75 commented Dec 31, 2015

man I wish I would have refreshed like 15 minutes ago. that was a waste of time. oh well, I'll just subtract that time from my retirement!

@atan77
Copy link

atan77 commented Dec 31, 2015

I got the same error initially and then with a reload from the map it validated ok with no change to solution

@SaintPeter
Copy link
Member

@BerkeleyTrue I have periodically been able to reproduce this, but refreshing the browser makes it go away. Any ideas what might be going on here? I'm presuming it's related to the test architecture.

@BerkeleyTrue
Copy link
Contributor

Does it only happen on page initial page load (with default code) then on refresh with completed code it is fixed?

@SaintPeter
Copy link
Member

It appears to happen on initial page load, then after refreshing it is fixed . . . but if you load the page, paste in the solution and try to run it, it fails.

@AkiraLaine
Copy link
Member

Same problem here. With @SaintPeter's solution, it passed.

@KrisVos130
Copy link

I can reproduce this by being in an anonymous tab on Chrome, going to the challenge from the map and then just clicking submit. After that, when you clear your Local Storage and do the same thing, then the error happens again. You don't need to modify any code for the error to happen either.

@DDaems
Copy link
Contributor

DDaems commented Dec 31, 2015

You can just do a reset, change something in the challenge field. adding a random space or whatever. And hit refresh ;-). Still i think the reset button should reset the localstorage item too. Wich it atm doesn't seem to do.

Anyway i created a PR to fix this issue and will create a issue for clearing localstorage on reset

@teal18aa9f
Copy link

Refresh the page.It worked for me!

@bssatya
Copy link

bssatya commented Jan 6, 2016

Same here. Page refresh worked...

@onetallorder
Copy link

Reloading the page resolved this issue for me.

@ArielLeslie
Copy link
Contributor

I ran into this a couple days ago as well. Sometimes I had to refresh several times before it would work.

@denmch
Copy link

denmch commented Jan 6, 2016

I had this happen with several of the new JS Waypoints over the last few days. In each case a hard refresh cleared it up.

@SimpliiD
Copy link

SimpliiD commented Jan 9, 2016

I just reproduced this error after initially loading using the 'learn' button, but refreshing fixed it. Also running in Chrome.

@ViktorDimitrievski
Copy link

reloading page work for me.

@shahh
Copy link

shahh commented Jan 30, 2016

Refreshing the page worked for me too.

@estraneoDev
Copy link

Page refresh solve the problem

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