search

loopier / animatron

Animatron for Godot 4.x <
15 stars 1 forks source link

Tester #18

Open loopier opened 8 months ago

loopier commented 8 months ago

I'm going to add a /test command to batch-test all commands without having to run them every time.

GUT doesn't seem to really do what it's testing. I tried UT commands, and they pass, but I can't see the result and that may be misleading. For example, I ran a test on /scale that passed the test (I use Status to check if it fails), but It wasn't actually working properly. I could only detect the error by evaluating the command from the editor.

So I thought a /test command would be useful. I'll try to automate it in a GDScript function, otherwise I'll manually write it in a def file.

Do you think it's a good idea?

totalgee commented 8 months ago

BTW, I just made some fixes (2d47153) to the test_main unit tests, now they don't crash when running, and even better they pass (even if there are orphans...haven't looked at that yet). It was previously crashing when you tried to run a command that expected one argument, but you gave it more than one (callv was returning null ... also, in another place your lambda function needed to return a Status object rather than nothing). You were also relying on a "bla" animation being there, so I changed it to "default", which we both have.