pele-python / pele

Python energy landscape explorer
Other
95 stars 41 forks source link

add exception tests #126

Closed kjs73 closed 9 years ago

landscape-bot commented 9 years ago

Code Health Code quality remained the same when pulling db7d6f9 on kjs73:base_potential_test into 090e2b8 on pele-python:master.

coveralls commented 9 years ago

Coverage Status

Coverage increased (+0.09%) to 90.61% when pulling db7d6f9fc9cf77a676aad0216c37d4a207991a38 on kjs73:base_potential_test into 090e2b80cf6ddc17e487bcc14a884a53d6d47048 on pele-python:master.

js850 commented 9 years ago

These can be done using "EXPECT_THROW" or "ASSERT_THROW". See

https://github.com/pele-python/pele/blob/master/cpp_tests/source/test_array.cpp#L157

kjs73 commented 9 years ago

Cool! I did not know this existed.

On Fri, 27 Feb 2015, at 11:49 AM, Jacob Stevenson wrote:

These can be done using "EXPECT_THROW" or "ASSERT_THROW". See

https://github.com/pele-python/pele/blob/master/cpp_tests/source/test_array.cpp#L157


Reply to this email directly or view it on GitHub: https://github.com/pele-python/pele/pull/126#issuecomment-76383218

js850 commented 9 years ago

there are more assertions here

https://code.google.com/p/googletest/wiki/AdvancedGuide

landscape-bot commented 9 years ago

Code Health Code quality remained the same when pulling 5b042ef on kjs73:base_potential_test into 090e2b8 on pele-python:master.

coveralls commented 9 years ago

Coverage Status

Coverage increased (+0.08%) to 90.6% when pulling 5b042ef0be090dd62f5555c4751f17015e7b1852 on kjs73:base_potential_test into 090e2b80cf6ddc17e487bcc14a884a53d6d47048 on pele-python:master.

js850 commented 9 years ago

:+1: