Return value of getTenantIdByDomain() must be of the type string, null returned

[jonagoldman]

Describe the bug

Calling Tenancy::findByDomain($domain) throws an exception if tenant not in cache and does't exist.

Expected behavior

TenantCouldNotBeIdentifiedException should be thrown instead.

Your setup

• Laravel version: 7.5.2
• stancl/tenancy version: 2.3.3
• Storage driver: MySQL

Stacktrace

Return value of
Stancl\\Tenancy\\StorageDrivers\\Database\\CachedTenantResolver::getTenantIdByDomain()
must be of the type string, null returned at
/xxx/vendor/stancl/tenancy/src/StorageDrivers/Database/CachedTenantResolver.php:33)

# 0 /xxx/vendor/stancl/tenancy/src/StorageDrivers/Database/DatabaseStorageDriver.php(72):
Stancl\\Tenancy\\StorageDrivers\\Database\\CachedTenantResolver->getTenantIdByDomain()

# 1 /xxx/vendor/stancl/tenancy/src/TenantManager.php(220):
Stancl\\Tenancy\\StorageDrivers\\Database\\DatabaseStorageDriver->findByDomain()

# 2 /xxx/vendor/laravel/framework/src/Illuminate/Support/Facades/Facade.php(261):
Stancl\\Tenancy\\TenantManager->findByDomain()