Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add null or date assert #17

Merged
merged 3 commits into from
May 25, 2015
Merged

Conversation

nunofgs
Copy link
Contributor

@nunofgs nunofgs commented May 25, 2015

  • Adds a NullOrDate assert.
  • Fixes an issue with the DateDiffGreaterThan and DateDiffLessThan asserts that were incorrectly calling a dependent assert which was never properly initialized.

R=@ruimarinho

Assert.prototype.NullOrDate = assert;
});

it('should throw an error if the input value is not a `null` or a date', function() {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Date?

@nunofgs nunofgs force-pushed the enhancement/add-null-or-date-assert branch 3 times, most recently from c2a5d77 to 8dec7a5 Compare May 25, 2015 15:26
@nunofgs nunofgs force-pushed the enhancement/add-null-or-date-assert branch from 8dec7a5 to eb38dbf Compare May 25, 2015 15:30
ruimarinho added a commit that referenced this pull request May 25, 2015
@ruimarinho ruimarinho merged commit 541444d into master May 25, 2015
@ruimarinho ruimarinho deleted the enhancement/add-null-or-date-assert branch May 25, 2015 15:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants