Make query recorders more consistent

When running tests, the cache is cleared a bit inconsistently and it can lead to some funny behaviour like !85399 (comment 927714587).

Also, some extra License checks occasionally occur due to how the cache is cleared and some interesting interactions with some other aspects of our testing suite, such as throttling user updates (e.g. last sign in).

This MR makes the cache clearing behaviour in tests more consistent and ignores License SQL calls in query recorders. It may unearth some N+1 problems. (it didn't! hooray!)

Related to #360896

lib/gitlab/testing/clear_process_memory_cache_middleware.rb

@@ -10,9 +10,9 @@ def initialize(app)
       def call(env)
         Gitlab::ProcessMemoryCache.cache_backend.clear
 
-        @app.call(env).tap do
-          Gitlab::ProcessMemoryCache.cache_backend.clear
-        end
+        @app.call(env)
+      ensure
+        Gitlab::ProcessMemoryCache.cache_backend.clear
       end
     end
   end