Validation proposals

[ndrwrbgs]

Paths

make sure work here includes thoughts about how to handle the case where Directory method is called but a File exists at the path

• DirectoryExists
• DirectoryDoesNotExist
• FileExists
• FileDoesNotExist

Object

• IsNull (though Is(null) probably suffices)
• Implements/Extends
• IsAnyOf (analogous to Is)
• IsNotAnyOf

Number

• IsApproximately(accuracy/within)
• IsPositive
• IsNegative
• IsNotNegative

Collection

• SizeIsGt/Lt/Gte/Lte
• SizeIsNot
• Contains
• DoesNotContain
• ^variation of above | HasNoNull
• ContainsExactly(2, [optional] where: item => item == "foo")
• IsEmpty (.SizeIs(0))
• IsInAscendingOrder (IComparable only)
• IsInDescendingOrder (IComparable only)

Dictionary

• DoesNotContainKey
• ContainsValue
• DoesNotContainsValue

String

• StartsWith StartsWith validation for Ensure.That #97
• EndsWith
• Contains
• DoesNotContain
• IsNullOrWhiteSpace
• IsNullOrEmpty
• IsEmpty
• SizeIsGt/Lt/Gte/Lte
• SizeIsNot
• DoesNotMatch

[ndrwrbgs]

For the String and Collection SizeIs methods, they seem redundant since users could use Ensure.That(str.Length).IsGt(4). However I would suggest either feature parity (if exposing SizeIs, also expose Gt/Lt/!=), or not exposing any of them and forcing access via string.Length

[ndrwrbgs]

@danielwertheim I'm game to work on any/all of these if/as you approve of them, just let me know if you think they'd be valuable contributions.

[danielwertheim]

Haven't had time to look through this yet. Want to finalize 7.1 first so that it gets merged to master.

Also, before doing this, a decision to revoke the Ensure.That(...,...).Foo() or not need to be taken.

[dmarciano]

Hi...I'm interested in contributing to this, specifically working on the numbers (IsApproximately, IsPosition, etc.). I didn't see any PR guidelines in the repo, so I was just wondering, would it be preferred that each implementation has its own PR or could I implement all four methods under the Number category and submit a single pull request for all of them?

[danielwertheim]

@dmarciano the smaller the better. Preferably one PR per feature if OK.

[dmarciano]

I have implemented the IsPositive, IsNegative, IsNotNegative, and IsApproximately methods and extension methods, along with 16 new unit tests to cover these methods (all of which have passed successfully).

The project's coding style is a little different to how I usually code, but I tried to match the pattern/style as closely as possible with my new code. If there is anything off, or you would prefer I change, please let me know.

[ndrwrbgs]

Most of these implemented in the -Experimental library
RE my comment in #125 I worked around it by allowing the user to specify how to handle 0, if you wanted to do similarly @dmarciano

[dmarciano]

@ndrwrbgs Yes...I saw your comment but haven't had a chance to review the link you provided yet as I'm traveling today. I should be able to look at it first thing tomorrow morning and either give feedback or start work on a change for it. I'll keep you posted.

[ndrwrbgs]

Linked in the other convo (was trying to avoid 'advertising' as that isn't the intention here) but will link just for your ease since you're traveling :-P
I implemented samples here: https://github.com/ndrwrbgs/Ensure.That.Experimental

[danielwertheim]

@ndrwrbgs what's in experimental that can't be in the public repo?