The change I made was the minimum delta to make the tests run in DrDr, since the files already had a `main` submodule which ran the actual tests.
Sam On Sun, Jul 7, 2013 at 4:42 AM, Robby Findler <ro...@eecs.northwestern.edu> wrote: > Raco test runs the file if there is not test sub module. Fwiw. > > > On Saturday, July 6, 2013, Eli Barzilay wrote: >> >> It doesn't make much sense to use a test submodule in files that are >> intended to be (only) tests. >> >> >> Earlier today, Robby Findler wrote: >> > That's the plan. Thanks for fixing them. >> > >> > Robby >> > >> > >> > On Sat, Jul 6, 2013 at 8:47 AM, Sam Tobin-Hochstadt <sa...@ccs.neu.edu> >> > wrote: >> > >> > I recently discovered that DrDr runs `raco test` by default on >> > files, >> > and has since February. However, the DrDr documentation says that it >> > runs `racket -t`. I discovered this because the change caused many >> > of >> > the `net` tests to no longer actually run; see >> > >> > >> > http://drdr.racket-lang.org/27042/pkgs/racket-pkgs/racket-test/tests/ >> > net/url.rkt >> > >> > which appears to be successful, even though running `racket` on that >> > file would have shown the error that I fixed in >> > https://github.com/plt/racket/commit/a4e529a816 . These tests are a >> > particularly bad case because they run all their tests in the `main` >> > submodule, and thus are *not* executed by `raco test`. If the plan >> > is >> > for everything to run with `raco test`, I'll fix these tests. But >> > what is the plan? >> >> -- >> ((lambda (x) (x x)) (lambda (x) (x x))) Eli Barzilay: >> http://barzilay.org/ Maze is Life! _________________________ Racket Developers list: http://lists.racket-lang.org/dev
