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

Fleshed out complex.jl tests #9615

Merged
merged 1 commit into from
Jan 5, 2015
Merged

Conversation

kshyatt
Copy link
Contributor

@kshyatt kshyatt commented Jan 5, 2015

exp2, exp10, and cis had no tests. I added some for ^ and inv which had uncovered branches.

@ViralBShah ViralBShah added the test This change adds or pertains to unit tests label Jan 5, 2015
jakebolewski added a commit that referenced this pull request Jan 5, 2015
@jakebolewski jakebolewski merged commit 7d4c3a5 into JuliaLang:master Jan 5, 2015
@johnmyleswhite
Copy link
Member

Thank you so much for all the tests you've been adding, @kshyatt.

@kshyatt
Copy link
Contributor Author

kshyatt commented Jan 5, 2015

I ❤️ testing!

@ViralBShah
Copy link
Member

I ❤️ people who ❤️ testing! :-)

@ViralBShah
Copy link
Member

@kshyatt I just saw that you are at UCSB studying physics. Seems like we are getting some contributions from there. I am pretty sure I have seen a couple of commits from one other person in UCSB physics. Brings back fond memories - @StefanKarpinski and I both spent many years there!

@kshyatt
Copy link
Contributor Author

kshyatt commented Jan 5, 2015

You are probably thinking of @garrison, who's in the same research group as me. Always nice to see some UCSB people doing awesome things. I ❤️ people who ❤️ those who ❤️ testing - it's ♥️ all the way down.

tkelman pushed a commit that referenced this pull request Jan 6, 2015
@tkelman
Copy link
Contributor

tkelman commented Jan 6, 2015

backported the new tests in 72974fb, good times

@kshyatt kshyatt deleted the complextests branch May 5, 2015 00:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
test This change adds or pertains to unit tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants