flix / flix

The Flix Programming Language
https://flix.dev/
Other
2.08k stars 150 forks source link

Add simple tests for `InvokeMethod2` #7833

Open magnus-madsen opened 3 weeks ago

magnus-madsen commented 3 weeks ago

In combination with #7769

I suggest to use multiple names spaces as outlined in parens above.

magnus-madsen commented 3 weeks ago

CC @chanattan

magnus-madsen commented 2 weeks ago

@chanattan It would be good to have some chaining tests where the return type is different.

chanattan commented 2 weeks ago

@magnus-madsen valueOf is a static method and is not supported by invokeMethod2 but will be with invokeStaticMethod2. String.replace has no overloading by arity, I moved the tests to type overload. split can not be supported yet. The rest is being worked on.

magnus-madsen commented 2 weeks ago

@magnus-madsen valueOf is a static method and is not supported by invokeMethod2 but will be with invokeStaticMethod2. String.replace has no overloading by arity, I moved the tests to type overload. split can not be supported yet. The rest is being worked on.

Right, so some of the tests may need to wait for invoke static. But you can also try find other methods that fit the bill and test with those.