Reorder middleware to allow route model binding to be delayed

This commit is contained in:
David Bomba 2021-05-13 23:01:23 +10:00
parent 777c2dd24a
commit 6e967035d8
5 changed files with 26 additions and 14 deletions

View File

@ -163,20 +163,23 @@ class Kernel extends HttpKernel
protected $middlewarePriority = [ protected $middlewarePriority = [
ContactTokenAuth::class,
ContactSetDb::class,
SetInviteDb::class,
ContactRegister::class,
ShopTokenAuth::class,
PhantomSecret::class,
ContactKeyLogin::class,
CheckClientExistence::class,
ClientPortalEnabled::class,
UrlSetDb::class,
SetWebDb::class,
SetDb::class, SetDb::class,
SetWebDb::class,
UrlSetDb::class,
ContactSetDb::class,
SetEmailDb::class,
SetInviteDb::class,
SetDbByCompanyKey::class, SetDbByCompanyKey::class,
TokenAuth::class, TokenAuth::class,
ContactTokenAuth::class,
ShopTokenAuth::class,
ContactKeyLogin::class,
ContactRegister::class,
PhantomSecret::class,
CheckClientExistence::class,
ClientPortalEnabled::class,
PasswordProtection::class,
Locale::class,
SubstituteBindings::class, SubstituteBindings::class,
]; ];
} }

View File

@ -96,8 +96,8 @@ class CSVImport implements ShouldQueue {
MultiDB::setDb( $this->company->db ); MultiDB::setDb( $this->company->db );
Auth::login( $this->company->owner(), true ); Auth::login( $this->company->owner(), true );
$this->company->owner()->setCompany( $this->company ); auth()->user()->setCompany($this->company);
$this->buildMaps(); $this->buildMaps();

View File

@ -42,6 +42,7 @@ class ImportCsvTest extends TestCase
$this->withoutMiddleware( $this->withoutMiddleware(
ThrottleRequests::class ThrottleRequests::class
); );
config(['database.default' => config('ninja.db.default')]);
// $this->faker = \Faker\Factory::create(); // $this->faker = \Faker\Factory::create();

View File

@ -19,6 +19,7 @@ use App\Models\CompanyUser;
use App\Models\User; use App\Models\User;
use Illuminate\Support\Facades\DB; use Illuminate\Support\Facades\DB;
use Illuminate\Support\Facades\Hash; use Illuminate\Support\Facades\Hash;
use Illuminate\Validation\ValidationException;
use Tests\TestCase; use Tests\TestCase;
/** /**
@ -33,6 +34,8 @@ class MultiDBUserTest extends TestCase
{ {
parent::setUp(); parent::setUp();
$this->withoutExceptionHandling();
if (! config('ninja.db.multi_db_enabled')) { if (! config('ninja.db.multi_db_enabled')) {
$this->markTestSkipped('Multi DB not enabled - skipping'); $this->markTestSkipped('Multi DB not enabled - skipping');
} }
@ -200,10 +203,11 @@ class MultiDBUserTest extends TestCase
} catch (ValidationException $e) { } catch (ValidationException $e) {
$message = json_decode($e->validator->getMessageBag(), 1); $message = json_decode($e->validator->getMessageBag(), 1);
$this->assertNotNull($message); $this->assertNotNull($message);
} }
if ($response) { if ($response) {
$response->assertStatus(302); $response->assertStatus(403);
} }
} }
@ -242,5 +246,8 @@ class MultiDBUserTest extends TestCase
{ {
DB::connection('db-ninja-01')->table('users')->delete(); DB::connection('db-ninja-01')->table('users')->delete();
DB::connection('db-ninja-02')->table('users')->delete(); DB::connection('db-ninja-02')->table('users')->delete();
config(['database.default' => config('ninja.db.default')]);
} }
} }

View File

@ -128,6 +128,7 @@ trait MockAccountData
*/ */
public function makeTestData() public function makeTestData()
{ {
config(['database.default' => config('ninja.db.default')]);
/* Warm up the cache !*/ /* Warm up the cache !*/
$cached_tables = config('ninja.cached_tables'); $cached_tables = config('ninja.cached_tables');