Remove direct references to TopLevelAllocator from unit tests

The RootAllocatorFactory should be used throughout the code to allow us to change allocators via configuration or other software choices. Some unit tests still reference TopLevelAllocator directly. We also need to do a better job of handling exceptions that can be handled by close()ing an allocator that isn't in the proper state (remaining open child allocators, outstanding buffers, etc).