Skip to content
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

Updates to testing system #322

Closed
wants to merge 3 commits into from
Closed

Updates to testing system #322

wants to merge 3 commits into from

Conversation

equwal
Copy link
Contributor

@equwal equwal commented Jul 26, 2018

I did three things:

  1. I added a macro called tests which reduces a lot of boilerplate. Essentially:

(test predicate1)
(test predicate2)
(test predicate3)

Can now be written

(tests predicate1 predicate2 predicate3)

Macros are fun.

  1. I decided to go through and collect broken tests that are commented out into their own file, at which point I realized I could run them and see if any worked now.
  2. Several of them did indeed work, so I put them back in.

All this because I realized I would have to write test twice instead of once.
#309

@equwal equwal closed this Jul 26, 2018
@equwal equwal reopened this Jul 26, 2018
@equwal
Copy link
Contributor Author

equwal commented Jul 26, 2018

Accidentally removed some tests, will be back.

@equwal equwal closed this Jul 26, 2018
@equwal
Copy link
Contributor Author

equwal commented Jul 26, 2018

#323

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant