break statement not required
I guess,test failing due to connection error? We just fixed the other dataset on master. Could you rebase from master and push to rerun the CI ? If I'm not wrong this function returns None if no main class was found.
I think it makes things less clear not to have a return at the end of the function.
I guess we can have one return in the for loop instead of the break statement, AND one return at the end to explicitly return None.
What do you think ?