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

Already on GitHub? Sign in to your account

Ensure that assertions fail when arguments are missing #233

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
2 participants
Contributor

CMTegner commented Oct 1, 2012

  • Ensure that all assertions defined in macros.js throw an
    AssertionException when they are called with no arguments
  • Removing duplicate isDefined test
  • Adding IDEA project files to .gitignore
Ensure that assertions fail when arguments are missing
* Ensure that all assertions defined in macros.js throw an
AssertionException when they are called with no arguments
* Removing duplicate `isDefined` test
* Adding IDEA project files to .gitignore
Owner

indexzero commented Nov 4, 2014

Sounds legitimate and an easy way for people to not stub their toes forgetting an assert argument. I vote for 0.8.x ... @JerrySievert @davglass?

@indexzero indexzero added the v0.8.x label Nov 4, 2014

Owner

indexzero commented Nov 22, 2014

Cherry-picked. Thanks! Sorry it took so long 💯

@indexzero indexzero closed this Nov 22, 2014

Contributor

CMTegner commented Nov 22, 2014

Nice! I could barely remember the context of this change :p

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment