CalculateCaseTotalsOnSuite causes FormatException

DynamoDS / RevitTestFramework

Unit Testing on Revit

119 stars 81 forks source link

CalculateCaseTotalsOnSuite causes FormatException #52

Open sweco-sejsau-zz opened 8 years ago

sweco-sejsau-zz commented 8 years ago

Hey,

Seems like the CalculateCaseTotalsOnSuite is dependent on what decimal separator you have configured on your Windows (I'm from Sweden and we have ',' as default). Happens on line 351.

Here's a suggestion for solve this problem: suite.time = cases.Sum(x => Convert.ToDouble(x.time, CultureInfo.InvariantCulture)).ToString();

Cheers, Joel

sharadkjaiswal commented 8 years ago

@sweco-sejsau, thanks for looking into it and highlighting the issue. Please propose a pull request for your suggested changes and we would merge it.

sweco-sejsau-zz commented 8 years ago

Ok done!