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

check-match for rackunit #171

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
3 participants
@jpolitz
Contributor

jpolitz commented Nov 20, 2012

Thread

http://www.mail-archive.com/dev@racket-lang.org/msg07427.html

Files

check.rkt:
Added the actual check-match macro.

test.rkt:
Just a provide statement

check-test.rkt:
7 additional tests for check-match, and a macro to help create tests for it

check.scrbl:
Documentation and examples for check-match

Notes

  1. I tested with

raco test collects/tests/rackunit

It prints out a lot of stuff because there are existing tests hardcoded to fail in there. But, I wrote seven tests and it claims that 127 tests (rather than 120 before) succeed now, so that seems promising.

  1. I don't know how to get the use of @racket[match] in the docs to link over to the actual match page. I'm a Scribble noob.
  2. I'm also a syntaxy-macros noob, it might be possible to get the useful error information through check-match itself in a more appropriate/less crappy way.
Add check-match to rackunit
check.rkt:
  Added the actual check-match macro.

test.rkt:
  Just a provide statement

check-test.rkt:
  7 additional tests for check-match, and a macro to help create tests

check.scrbl:
  Documentation and examples for check-match
@takikawa

This comment has been minimized.

Show comment
Hide comment
@takikawa

takikawa Nov 20, 2012

Contributor

To get match to link, all you should need to do is add (require (for-label racket/match)) in the relevant scrbl file.

Contributor

takikawa commented Nov 20, 2012

To get match to link, all you should need to do is add (require (for-label racket/match)) in the relevant scrbl file.

@takikawa

This comment has been minimized.

Show comment
Hide comment
@takikawa

takikawa Nov 20, 2012

Contributor

Merged now. Thanks!

Contributor

takikawa commented Nov 20, 2012

Merged now. Thanks!

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