Writing good assertions is a difficult task. The easiest test you can think of probably involves the wall time or the memory, but assertions on these dimensions are actually very weak. Time is not a stable metric. External factors such as machine load can have significant impacts on wall time between two profiles of identical code. Volatile tests must be avoided at all costs as they make your test suite less reliable and degrade the trust your team has in any failures.
Avoid writing assertions that depend on time. When running a piece of code, time is just a consequence of what happened in the code. Look deeper. Understanding which functions were called at runtime is exactly what a profiler is good at.
Use time to identify the slow parts in your code and then write assertions on the root cause.
A typical example is a microservice architecture where the number of HTTP requests on external web services is likely to be the main performance issue. The more HTTP requests you have in the code, the slower the code is going to be. The number of external requests is a stable value and it should not change between two profiles generated from the same codebase. Thus, limiting the number of HTTP requests allowed for a project is a great way to ensure good code performance:
1 2 3 4 5
# fragile, might break from time to time - main.wall_time < 50ms # robust, the real root cause for slowness - metrics.http.requests.count < 5
We already used such an approach when we wrote assertions for GitList and phpmd:
1 2 3 4 5
# limit the number of sub-processes (using the built-in metric) - metrics.symfony.processes.count < 10 # be sure we don't hit the cache more than needed - metrics.parses.count == metrics.cache_driver.count
You can also limit the number of SQL queries:
- metrics.sql.queries.count < 10
What is the ideal number? 5? 10? It really depends on your project. As you know, the fastest code is the code that is never called. Checking that a function is never called is also a very good practice.
Websites with a lot of traffic might want no SQL queries on their homepage:
- metrics.sql.queries.count == 0
Another best practice for any website is to never send emails synchronously:
- metrics.emails.sent.count == 0
Some more examples on popular PHP libraries:
1 2 3 4 5 6 7 8 9 10
# check that the Twig C extension is installed - metrics.twig.attributes.php.get.count == 0 # limit the number of DB connections # can be 2 if you have a specific connection for the session - metrics.sql.connections.count <= 1 # same for Redis and AMQP - metrics.redis.connections.count <= 1 - metrics.amqp.connections.count <= 1
Another good practice for production servers is to generate all cached files before a deployment goes live. The following assertions check this assumption:
1 2 3 4 5 6 7 8 9 10 11 12 13 14 15
# no Twig/Smarty compilation - is_dev() or metrics.twig.compile.count == 0 - is_dev() or metrics.smarty.compile.count == 0 # no Symfony metadata checks - is_dev() or metrics.symfony.config_check.count == 0 # no Doctrine parsing - is_dev() or (metrics.doctrine.annotations.parsed.count + metrics.doctrine.annotations.read.count + metrics.doctrine.dql.parsed.count + metrics.doctrine.entities.metadata.count + metrics.doctrine.proxies.generated.count) == 0 # no YAML loaded - is_dev() or metrics.symfony.yaml.reads.count == 0 # Assetic controller must not be called (assets should be dumped) - is_dev() or metrics.assetic.controller.calls.count == 0
is_dev() function returns
false when the assertion is run in an
environment configured for production usage.
Blackfire promotes a metrics-first approach to performance. Write good assertions and they will catch problems before you ever need to analyze a call graph.
Don't be afraid to create custom metrics. This is where Blackfire shines. The ability to create custom assertions based on your team's code patterns is a powerful tool. Reusing a custom metric on an Open-Source library often? Contact us and we will consider adding it to our built-in repository of metrics.
Time makes it easy to find the root cause of a performance issue, but it is a poor metric when it comes to performance assertions.
Having the appropriate metrics makes it possible to capture a lot of information about your code, PHP, or any framework you might rely on. And the possibilities are then huge.
To give you a starting point, and let you benefit from our experience and the community's experience, we've written some default tests, which will be applied to your profiles and will warn you if we detect some bad practices in your code. That's what we will see in the next chapter.