Added documentation to the SharpRaven.Data.Module class
Added documentation to SharpRaven.Logging.IFilter
- Added failing test for the PhoneNumberFilter, proving it doesn't work because replacement isn't done in place, but returned as a new string instance.
- Added failing CreditCardFilter test, unveiling a bug in the filter that leads to an IndexOutOfRangeException.
Fixed the PhoneNumberFilter to use the returned value from Replace(), making PhoneNumberFilterTests.Filter_PhoneNumberIsScrubbed() green.
Some file rejigging
Changed existing CreditCardTest into a negative one due to it using an invalid credit card number and added a positive test for a valid credit card number
Fixed CreditCardFilter so it actually scrubs the data and doesn't suffer from ArgumentOutOfRangeException problems.
Removed the "Filter" prefix from the test names as it's implied in the name of the test fixture
Added a negative PhoneNumberFilter test and changed the test naming to not contain "Filter".
Added ignored tests for SocialSecurityFilter
Added Obsolete attribute to the SocialSecurityFilter.
Added a base class for filter testing
Refactored CreditCardFilterTests to use the abstract FilterTestsBase class.
Refactored PhoneNumberFilterTests to use the abstract FilterTestsBase class.
Refactored SocialSecurityFilterTests to use the abstract FilterTestsBase class.