search

bleroy / Nwazet.Commerce

Idiomatic commerce module for Orchard CMS.
BSD 3-Clause "New" or "Revised" License
26 stars 21 forks source link

Update tests #136

Closed MatteoPiovanelli-Laser closed 6 years ago

MatteoPiovanelli-Laser commented 6 years ago

Fixes #109

Some tests required an updated because of the change to decimal for prices.

A couple more corrections ewre required to handle different formatting of numbers in different cultures. I updated UspsService so it always uses the en-US culture to parse strings. However, I don't think the same assumption can be made for ZipCodeTaxPart in the method where it parses the rates.

bleroy commented 6 years ago

Looks good otherwise. Do you want me to wit for you to squash before I merge, or should I merge as is?

MatteoPiovanelli-Laser commented 6 years ago

As you wish. I can squash it tomorrow when I am on my work pc.

bleroy commented 6 years ago

Sounds good. Thanks!