search

jpype-project / jpype

JPype is cross language bridge to allow Python programs full access to Java class libraries.
http://www.jpype.org
Apache License 2.0
1.12k stars 183 forks source link

Test cleanups #1239

Closed marscher closed 4 days ago

marscher commented 4 days ago

I just noticed that we have undefined references at some places in our code base. In this case the "first" and "second" argument in the overridden method were undefined.

This should also fix the float16 test failure which occurred every once in a while.

codecov[bot] commented 4 days ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 87.19%. Comparing base (dfdeb76) to head (338c94b). Report is 4 commits behind head on master.

Additional details and impacted files [![Impacted file tree graph](https://app.codecov.io/gh/jpype-project/jpype/pull/1239/graphs/tree.svg?width=650&height=150&src=pr&token=HYbu98U2O8&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=jpype-project)](https://app.codecov.io/gh/jpype-project/jpype/pull/1239?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=jpype-project) ```diff @@ Coverage Diff @@ ## master #1239 +/- ## ======================================= Coverage 87.19% 87.19% ======================================= Files 113 113 Lines 10296 10296 Branches 4051 4051 ======================================= Hits 8978 8978 Misses 726 726 Partials 592 592 ```

🚨 Try these New Features: