Skip to content
This repository has been archived by the owner on Sep 19, 2018. It is now read-only.

Allow assert_throws to check that it throws without testing more #1

Closed
wants to merge 2 commits into from
Closed

Allow assert_throws to check that it throws without testing more #1

wants to merge 2 commits into from

Conversation

odinho
Copy link
Member

@odinho odinho commented Jun 1, 2012

12:48 < odinho> jgraham, Ms2ger, AryehGregor, other testharness.js people: I kinda want a way to say "WTF don't care" about the exception being thrown in assert_throws().
12:49 < Ms2ger> Why? :)
12:49 < odinho> I have the CORS-tests, and what the XHR throws is really not that important.
12:49 < odinho> In fact, it hasn't anything to do with CORS.
12:49 < odinho> And Mozilla is failing HARD, along with Webkit because of that.
12:49 < odinho> Masking other, possibly real bugs.

12:56 < Ms2ger> odinho, I'd prefer (null, fn), because really I'd rather people don't do it :)
12:56 < odinho> Ms2ger: Okay, fair enough.
12:57 < odinho> Ms2ger: Like, you have to, know what you're doing/really want it, if you do it.
12:57 < Ms2ger> Right

@odinho
Copy link
Member Author

odinho commented Jun 20, 2012

This is pulled into mercurial.

@odinho odinho closed this Jun 20, 2012
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant