Improve explain on db_logging (#1898)

Author: ildyria

Makefile

@@ -109,7 +109,16 @@ release_major: gen_major
 TESTS_PHP := $(shell find tests/Feature -name "*Test.php" -printf "%f\n")
 TEST_DONE := $(addprefix build/,$(TESTS_PHP:.php=.done))
 
-build/%.done: tests/Feature/%.php
-	XDEBUG_MODE=coverage vendor/bin/phpunit --filter $* && touch build/$*.done
+build:
+	mkdir build
+
+build/Base%.done:
+	touch build/Base$*.done
+
+build/%UnitTest.done:
+	touch build/$*UnitTest.done
+
+build/%.done: tests/Feature/%.php build
+	vendor/bin/phpunit --no-coverage --filter $* && touch build/$*.done
 
 all_tests: $(TEST_DONE)

app/Providers/AppServiceProvider.php

@@ -38,6 +38,22 @@
 
 class AppServiceProvider extends ServiceProvider
 {
+	/**
+	 * Defines which queries to ignore when doing explain.
+	 *
+	 * @var array<int,string>
+	 */
+	private array $ignore_log_SQL =
+	[
+		'information_schema', // Not interesting
+
+		// We do not want infinite loops
+		'EXPLAIN',
+
+		// Way too noisy
+		'configs',
+	];
+
 	public array $singletons =
 	[
 		SymLinkFunctions::class => SymLinkFunctions::class,
@@ -151,8 +167,7 @@ private function logSQL(QueryExecuted $query): void
 	{
 		// Quick exit
 		if (
-			Str::contains(request()->getRequestUri(), 'logs', true) ||
-			Str::contains($query->sql, ['information_schema', 'EXPLAIN', 'configs'])
+			Str::contains(request()->getRequestUri(), 'logs', true)
 		) {
 			return;
 		}
@@ -161,7 +176,11 @@ private function logSQL(QueryExecuted $query): void
 		$msg = '(' . $query->time . 'ms) ' . $query->sql . ' [' . implode(', ', $query->bindings) . ']';
 
 		// For pgsql and sqlite we log the query and exit early
-		if (config('database.default', 'mysql') !== 'mysql' || config('database.explain', false) === false) {
+		if (config('database.default', 'mysql') !== 'mysql' ||
+			config('database.explain', false) === false ||
+			!Str::contains($query->sql, 'select') ||
+			Str::contains($query->sql, $this->ignore_log_SQL)
+		) {
 			Log::debug($msg);
 
 			return;

database/migrations/2018_08_15_102039_move_albums.php

@@ -38,10 +38,10 @@ public function up(): void
 					]);
 				}
 			} else {
-				Log::notice(__FUNCTION__ . ':' . __LINE__ . ' ' . env('DB_OLD_LYCHEE_PREFIX', '') . 'lychee_albums does not exist!');
+				Log::notice(__METHOD__ . ':' . __LINE__ . ' ' . env('DB_OLD_LYCHEE_PREFIX', '') . 'lychee_albums does not exist!');
 			}
 		} else {
-			Log::notice(__FUNCTION__ . ':' . __LINE__ . ' albums is not empty.');
+			Log::notice(__METHOD__ . ':' . __LINE__ . ' albums is not empty.');
 		}
 	}
 

phpunit.xml

@@ -28,6 +28,7 @@
   <php>
     <env name="APP_ENV" value="testing"/>
     <env name="DB_LOG_SQL" value="true"/>
+    <env name="DB_LOG_SQL_EXPLAIN" value="true"/>
     <env name="BCRYPT_ROUNDS" value="4"/>
     <env name="CACHE_DRIVER" value="array"/>
     <env name="SESSION_DRIVER" value="array"/>

tests/Feature/WebAuthTest.php

@@ -30,6 +30,7 @@ public function setUp(): void
 	{
 		parent::setUp();
 		$this->setUpRequiresEmptyWebAuthnCredentials();
+		config(['app.url' => 'https://localhost']);
 	}
 
 	public function tearDown(): void