Optim-wip: Move inception test helpers to separate file

[ProGamerGov]

This change will make it easier to use these helpers for testing other models.

[aobo-y]

leading _ is a convention for private. As we are exporting these 2 functions to be used by others, we can remove this leading _

[ProGamerGov]

Oops, yeah I'm not sure how I missed removing the underscore!

[aobo-y]

passing self here is a little weird, as these functions are not class method. How about updating these 2 functions to simply return the bool and doing the assert in the tests? This also makes the 2 utils more generic to other use cases.

[ProGamerGov]

The original idea was to follow after assertTensorAlmostEqual's input variable setup, but it does appear simpler to return bool outputs instead.

[aobo-y]

If you agree with the above suggestion, we don't need another nested inner function here.

def check_layer_in_model(model, layer):
  for _, child in model._modules.items():
    if child is None:
      continue
    if isinstance(child, layer) or check_layer_in_model(child, layer):
      return True
  return False

[ProGamerGov]

Yeah, we can just use the one function with bool outputs to test for the presence and absence of certain layers.

[aobo-y]

we may not even need this layer_not_in_model function, just not check_layer_in_model(model, layer)

[ProGamerGov]

@aobo-y

In addition to using not, we can also use self.assertFalse

self.assertFalse(check_layer_in_model(model, layer))
self.assertTrue(check_layer_in_model(model, layer))

But yeah, it looks like we don't need _check_layer_not_in_model after switching to bool outputs.