Callbacks not working in unit tests?

Methods that are not global and have the (:test) annotation are supposed to be static. This is mentioned in the Programmer's Guide..

https://developer.garmin.com/connect...test#unittests

Unit tests are written mostly like any other class, module, or function in Monkey C, but have the following requirements:

• Tests methods must be marked with the :test annotation
  
• Test methods must take a Logger object
  
• Tests methods that are not global (part of a test class or custom test module) must be static methods
  

Most of the time test code does not try to access members or methods of the containing class or module directly, so this requirement doesn't matter. But since you're trying to reference self (implicitly via method(:add) and explicitly via self.method(:add)), it becomes a problem.

If you make the test function static, and you create an object instance to test, it seems to work just fine.

module Tests {
class SimpleTest {
function add(a, b) {
return a + b;
}
(:test) static function test(logger) {

var obj = new SimpleTest();

logger.debug("Createing callback...");
var m = obj.method(:add);
logger.debug("Created callback.");

logger.debug("Calling callback...");
var res = m.invoke(1, 2);
logger.debug("Callback result: " + res);

return true;
}
}
}


I think we could do better here by reporting an error if a class or module function is found to have a test annotation and is not explicitly declared static. We could also go one step further and just tell the compiler that any function with a test annotation is static.

Travis

Travis.ConnectIQ thanks! That hint about static test methods solved another problem as well - the "instanceof" is not working without it, claiming it cannot resolve class name (cannot find symbol). Weird error message - I would have never figured it out myself.