Coverage increased (+0.0%) to 90.14% when pulling ac65121 on giggio:redundant-field into f689d48 on code-cracker:master.

Coverage increased (+0.0%) to 90.17% when pulling e73ef1e on giggio:redundant-field into 69418ae on code-cracker:master.

@giggio This test is breaking: public async Task ComputeExpression(string original, string fix) =>
await VerifyCSharpFixAsync(original.WrapInCSharpMethod(), fix.WrapInCSharpMethod());

Hm... same problem of globalization again, I will check it out.

@carloscds This test ComputeExpression is not from this Pull Request. This PR is about redundant fields, and I did not change the CompureExpression files, as you can see from the commits.
You had already caught this error on #312, and I had already fixed it. We even worked on #313 because of it, remember?
Are you up to date with the master branch?

@giggio Yes, I know this. My suggestion is open a bug to solve this.

@carloscds I can't repro. As I told you, I already fixed that bug. See this line:
https://github.com/code-cracker/code-cracker/blob/master/src/CSharp/CodeCracker/Refactoring/ComputeExpressionCodeFixProvider.cs#L51
Is this PR ok?

@giggio I remember this, but now in RC I have same bug when I run tests.

Ok @carloscds, bug #339 opened for that unrelated issue.

How about this specific PR? Is it ok?