adding test
This commit is contained in:
parent
107293bd61
commit
466f7a0ee5
2 changed files with 34 additions and 2 deletions
|
@ -32,6 +32,16 @@ class Profiler implements ContainerInterface
|
|||
*/
|
||||
private $rendertime;
|
||||
|
||||
/**
|
||||
* True, if the Profiler should measure the whole rendertime including functions
|
||||
*
|
||||
* @return bool
|
||||
*/
|
||||
public function isRendertime()
|
||||
{
|
||||
return $this->rendertime;
|
||||
}
|
||||
|
||||
/**
|
||||
* @param bool $enabled True, if the Profiler is enabled
|
||||
* @param bool $renderTime True, if the Profiler should measure the whole rendertime including functions
|
||||
|
@ -207,8 +217,10 @@ class Profiler implements ContainerInterface
|
|||
]
|
||||
);
|
||||
|
||||
$output = $this->getRendertimeString();
|
||||
$logger->info($message . ": " . $output, ['action' => 'profiling']);
|
||||
if ($this->isRendertime()) {
|
||||
$output = $this->getRendertimeString();
|
||||
$logger->info($message . ": " . $output, ['action' => 'profiling']);
|
||||
}
|
||||
}
|
||||
|
||||
/**
|
||||
|
|
|
@ -177,5 +177,25 @@ class ProfilerTest extends MockedTest
|
|||
}
|
||||
|
||||
$profiler->saveLog($this->logger, 'test');
|
||||
|
||||
$output = $profiler->getRendertimeString();
|
||||
|
||||
foreach ($data as $perf => $items) {
|
||||
foreach ($items['functions'] as $function) {
|
||||
// assert that the output contains the functions
|
||||
$this->assertRegExp('/' . $function . ': \d+/', $output);
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
/**
|
||||
* Test if no rendertime is set
|
||||
*/
|
||||
public function testNoRenderTime()
|
||||
{
|
||||
$profiler = new Profiler(true, false);
|
||||
|
||||
$this->assertFalse($profiler->isRendertime());
|
||||
self::assertEmpty($profiler->getRendertimeString());
|
||||
}
|
||||
}
|
||||
|
|
Loading…
Reference in a new issue