what is that "testing" doing? give a more descriptive name, and indent inner block