-
-
Notifications
You must be signed in to change notification settings - Fork 768
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
Unable to export test (version 3.17.2) #1839
Comments
Thats a good question. V3 is kinda hitting EOL. I have a recommendation that may help, or may hinder. In v4, I've cut the code export utilities into a standalone CLI tool using npm. Do you have npm on your computer? Basically, you can run code export as it's own utility this way if you do:
The advantage to doing this is that the stack trace is viewable and generally there is a lot less technology in the way.
|
Thanks, I'll look into that |
I still have the same error doing that but we do have more detail: CMD PROMPT CMD PROMPT OUTPUT RangeError: Maximum call stack size exceeded Node.js v20.12.2 |
Any update on this? |
Ah shoot, no sorry. I first have to figure out a har file thing. However, fixing this should be a lot easier. The stack trace is right there. Can you give me a minimal side file that makes this issue? That will help me fix it faster. I'm sorry, you've caught me in the middle of basically a vacation from OSS. |
Here is a test side file that uses accounts.google.com with just 1 test |
Oh neat, basic recursion. Okay, I'll look into it! New core test unlocked |
@TJSWANS300 - New version going out now, I think it is quite good at recursion but please let me know what you think. |
@toddtarsi Tried using the TestProject file above: CMD PROMPT CMD OUTPUT RangeError: Maximum call stack size exceeded Node.js v20.12.2 Does something need updating on my end? |
@TJSWANS300 - what do you get for this?
|
@toddtarsi |
Huh! I'm stumped. I will try with the public binary in tonight. Sorry dude. I mean, there's even a command in the local codebase |
No worries at all. |
Any update on this? |
Ah, not yet. I'll take a look this weekend |
@toddtarsi |
I'm taking a look at this this weekend, I'm starting to use a 'top priority' label to prioritize what comes first, and this is on that list. |
@TJSWANS300 - Okay, just to be safe, I incremented again and tested locally using the recursion side file here: https://github.com/SeleniumHQ/selenium-ide/blob/trunk/tests/examples/simple-parent.side#L85
Seems to transpile fine on latest! |
@toddtarsi Tried using the TestProject file above: CMD PROMPT CMD OUTPUT RangeError: Maximum call stack size exceeded Node.js v20.12.2 side-code-export --version = 4.0.12 Does something need updating on my end? |
Damn, @TJSWANS300 - Can you post a side file reproduction? I'm a bit confused because yeah this is now part of the core CI: https://github.com/SeleniumHQ/selenium-ide/blob/trunk/package.json#L25 Basically, this has a recursive test that does some dynamic stuff, and I don't think yours does. I need to figure out that difference so I can make it a core test too. |
Here is a test side file that uses accounts.google.com with just 1 test |
@TJSWANS300 - Thank you, if I'm caught up on work tonight, I'll take a look. I want recursion to be fully functional in code-export, and will make sure of it. |
Any update on this? |
Nah, sorry. It's one of like four issues with top priority (key-chord support in sendKeys, nested variable paths in sendKeys, multi layer recursion, break-fixing javascript-mocha code-export). It's getting worked on, but I'm kinda shelved for a week or two unfortunately. |
No worries |
Thank you |
Hi.
For Selenium IDE (version 3.17.2), I have a test called Misc/Cleanup.
It is made up of 3 steps:
When trying to export the Test to C# NUnit, the following error shows:
Could this be down to the Run command inside a test?
The text was updated successfully, but these errors were encountered: