Integrated Dynamics

Integrated Dynamics

63M Downloads

Arithmetic Cast Precedence Tests

met4000 opened this issue · 2 comments

commented

Issue type:

  • ❓ Question

Question:

In #1285 (implementing #1284), I forgot to add any tests for that ordering. In particular checking Integer -> Long, Integer ->
Double, and Long -> Double, but maybe also Integer -> Integer etc. (that final type could probably be done by iterating over the list of all the types). The tests would flag if this ordering gets inadvertently changed, and the final test type would also catch if something breaks in e.g. getLowestType and suddenly all arithmetic operators cast to only one output type.

Would you accept a PR adding these tests - and if so, would any of those specific tests be redundant?

commented

Someone will answer your question soon. In the meantime, you might be able to get help more quickly on our Discord server.

commented

Adding tests for that definitely makes sense, a PR is definitely welcome!