-
Notifications
You must be signed in to change notification settings - Fork 48
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
[Error] The active test run was aborted. Reason: Test host process crashed : Stack overflow #71
Comments
@orifn1 Could you please share what It seems to be similar what @kaylumah reported at kaylumah/hosting#552 (comment) |
I have a lot - 41
with 'null' value:
|
@orifn1 The ones with I will also check it, but I can do it only tomorrow. |
It has not helped, I commented out both |
@orifn1 do you have a link to the repo containing all these transformations? there might be other problematic ones. For me it was StepTransformation from string to string; but Table transformation to objects is working |
no, there is no public repo I commented out several StepTransformations, and some tests passed, but not all. So, I believe it is a bug that should be fixed |
How did you figure that out? (It might be relevant for the fix as well.) |
I am not aware, I just investigated stack trace) |
you were right - the problem in this transformation
this method returned it recursively because check CanConvert I will try to handle it with your suggestion in the mentioned issue |
@orifn1 thank you for looking into that. I didn't even remember that I made that comment. :) If you come up with any good workaround, please post it here. |
Interessting, that is very similar conversion (so probably same issue) [StepArgumentTransformation]
public static string ToNullableString(string value)
{
return Constants.NullIndicator.Equals(value, System.StringComparison.Ordinal) ? null : value;
} Never knew that |
No this is a regression in SpecFlow v3->v4 or in the Reqnroll porting (unfortunately I had to touch the converters, so it can happen that it was reintroduced during the forking). :( |
If any of you could make a simple check with SpecFlow v4-beta to see if the regression was already present there, it would help the investigation. |
@orifn1 thx. I will try to integrate this to the codebase next week. |
I'm also running into this issue, this transform reproduces it. For us its a blocker to migrate away from SpecFlow. //Replaces placeholders in the input with correct run time values.
[StepArgumentTransformation]
public string Transform(string input) => Parser.Parse(input); |
Reqnroll Version
1.0.1
Which test runner are you using?
MSTest
Test Runner Version Number
3.0.2
.NET Implementation
.NET 7.0
Test Execution Method
Visual Studio Test Explorer
Content of reqnroll.json configuration file
{
"$schema": "https://schemas.reqnroll.net/reqnroll-config-latest.json",
"trace": {
"stepDefinitionSkeletonStyle": "RegexAttribute"
}
}
Issue Description
Steps to Reproduce
just run existing test(migrated from Specflow)
Link to Repro Project
No response
The text was updated successfully, but these errors were encountered: