Re: [racket-dev] [plt] Push #23053: master branch updated

2011-07-18 Thread Robby Findler
Oh, I see. Yes, indeed, the whole test suite was broken. I've pushed fixes for it and for the things that weren't caught by the old test suite. Thanks, Robby On Sun, Jul 17, 2011 at 8:27 AM, Casey Klein wrote: > Here's what I tried: > > 1. I added this test, which should fail for identifying the

Re: [racket-dev] [plt] Push #23053: master branch updated

2011-07-17 Thread Casey Klein
On Sat, Jul 16, 2011 at 10:52 PM, Robby Findler wrote: > Thanks for fixing the bug! > > On Sat, Jul 16, 2011 at 6:40 PM, Casey Klein > wrote: >> I tried to add a test case to >> collects/tests/racket/contract-test.rktl, but that file seems broken. >> In particular, `test/pos-blame' and `test/neg-

Re: [racket-dev] [plt] Push #23053: master branch updated

2011-07-16 Thread Robby Findler
Thanks for fixing the bug! On Sat, Jul 16, 2011 at 6:40 PM, Casey Klein wrote: > I tried to add a test case to > collects/tests/racket/contract-test.rktl, but that file seems broken. > In particular, `test/pos-blame' and `test/neg-blame' pass strings when > `test/spec-failed' expects a symbol. Fi

Re: [racket-dev] [plt] Push #23053: master branch updated

2011-07-16 Thread Casey Klein
On Sat, Jul 16, 2011 at 6:35 PM, wrote: > > collects/racket/contract/private/arrow.rkt > ~~ > --- OLD/collects/racket/contract/private/arrow.rkt > +++ NEW/collects/racket/contract/private/arrow.rkt > @@ -235,7 +235,7 @@ v4 todo: >