mirror of
https://github.com/silverstripe/silverstripe-framework
synced 2024-10-22 14:05:37 +02:00
3873e4ba00
See https://github.com/silverstripe/silverstripe-framework/pull/7037 and https://github.com/silverstripe/silverstripe-framework/issues/6681 Squashed commit of the following: commit8f65e56532
Author: Ingo Schommer <me@chillu.com> Date: Thu Jun 22 22:25:50 2017 +1200 Fixed upgrade guide spelling commit76f95944fa
Author: Damian Mooyman <damian@silverstripe.com> Date: Thu Jun 22 16:38:34 2017 +1200 BUG Fix non-test class manifest including sapphiretest / functionaltest commit9379834cb4
Author: Damian Mooyman <damian@silverstripe.com> Date: Thu Jun 22 15:50:47 2017 +1200 BUG Fix nesting bug in Kernel commit188ce35d82
Author: Damian Mooyman <damian@silverstripe.com> Date: Thu Jun 22 15:14:51 2017 +1200 BUG fix db bootstrapping issues commit7ed4660e7a
Author: Damian Mooyman <damian@silverstripe.com> Date: Thu Jun 22 14:49:07 2017 +1200 BUG Fix issue in DetailedErrorFormatter commit738f50c497
Author: Damian Mooyman <damian@silverstripe.com> Date: Thu Jun 22 11:49:19 2017 +1200 Upgrading notes on mysite/_config.php commit6279d28e5e
Author: Damian Mooyman <damian@silverstripe.com> Date: Thu Jun 22 11:43:28 2017 +1200 Update developer documentation commit5c90d53a84
Author: Damian Mooyman <damian@silverstripe.com> Date: Thu Jun 22 10:48:44 2017 +1200 Update installer to not use global databaseConfig commitf9b2ba4755
Author: Damian Mooyman <damian@silverstripe.com> Date: Wed Jun 21 21:04:39 2017 +1200 Fix behat issues commit5b59a912b6
Author: Damian Mooyman <damian@silverstripe.com> Date: Wed Jun 21 17:07:11 2017 +1200 Move HTTPApplication to SilverStripe\Control namespace commite2c4a18f63
Author: Damian Mooyman <damian@silverstripe.com> Date: Wed Jun 21 16:29:03 2017 +1200 More documentation Fix up remaining tests Refactor temp DB into TempDatabase class so it’s available outside of unit tests. commit5d235e64f3
Author: Damian Mooyman <damian@silverstripe.com> Date: Wed Jun 21 12:13:15 2017 +1200 API HTTPRequestBuilder::createFromEnvironment() now cleans up live globals BUG Fix issue with SSViewer Fix Security / View tests commitd88d4ed4e4
Author: Damian Mooyman <damian@silverstripe.com> Date: Tue Jun 20 16:39:43 2017 +1200 API Refactor AppKernel into CoreKernel commitf7946aec33
Author: Damian Mooyman <damian@silverstripe.com> Date: Tue Jun 20 16:00:40 2017 +1200 Docs and minor cleanup commit12bd31f936
Author: Damian Mooyman <damian@silverstripe.com> Date: Tue Jun 20 15:34:34 2017 +1200 API Remove OutputMiddleware API Move environment / global / ini management into Environment class API Move getTempFolder into TempFolder class API Implement HTTPRequestBuilder / CLIRequestBuilder BUG Restore SS_ALLOWED_HOSTS check in original location API CoreKernel now requires $basePath to be passed in API Refactor installer.php to use application to bootstrap API move memstring conversion globals to Convert BUG Fix error in CoreKernel nesting not un-nesting itself properly. commitbba9791146
Author: Damian Mooyman <damian@silverstripe.com> Date: Mon Jun 19 18:07:53 2017 +1200 API Create HTTPMiddleware and standardise middleware for request handling commit2a10c2397b
Author: Damian Mooyman <damian@silverstripe.com> Date: Mon Jun 19 17:42:42 2017 +1200 Fixed ORM tests commitd75a8d1d93
Author: Damian Mooyman <damian@silverstripe.com> Date: Mon Jun 19 17:15:07 2017 +1200 FIx i18n tests commit06364af3c3
Author: Damian Mooyman <damian@silverstripe.com> Date: Mon Jun 19 16:59:34 2017 +1200 Fix controller namespace Move states to sub namespace commit2a278e2953
Author: Damian Mooyman <damian@silverstripe.com> Date: Mon Jun 19 12:49:45 2017 +1200 Fix forms namespace commitb65c21241b
Author: Damian Mooyman <damian@silverstripe.com> Date: Thu Jun 15 18:56:48 2017 +1200 Update API usages commitd1d4375c95
Author: Damian Mooyman <damian@silverstripe.com> Date: Thu Jun 15 18:41:44 2017 +1200 API Refactor $flush into HTPPApplication API Enforce health check in Controller::pushCurrent() API Better global backup / restore Updated Director::test() to use new API commitb220534f06
Author: Damian Mooyman <damian@silverstripe.com> Date: Tue Jun 13 22:05:57 2017 +1200 Move app nesting to a test state helper commit603704165c
Author: Damian Mooyman <damian@silverstripe.com> Date: Tue Jun 13 21:46:04 2017 +1200 Restore kernel stack to fix multi-level nesting commit2f6336a15b
Author: Damian Mooyman <damian@silverstripe.com> Date: Tue Jun 13 17:23:21 2017 +1200 API Implement kernel nesting commitfc7188da7d
Author: Damian Mooyman <damian@silverstripe.com> Date: Tue Jun 13 15:43:13 2017 +1200 Fix core tests commita0ae723514
Author: Damian Mooyman <damian@silverstripe.com> Date: Tue Jun 13 15:23:52 2017 +1200 Fix manifest tests commitca03395251
Author: Damian Mooyman <damian@silverstripe.com> Date: Tue Jun 13 15:00:00 2017 +1200 API Move extension management into test state commitc66d433977
Author: Damian Mooyman <damian@silverstripe.com> Date: Tue Jun 13 14:10:59 2017 +1200 API Refactor SapphireTest state management into SapphireTestState API Remove Injector::unregisterAllObjects() API Remove FakeController commitf26ae75c6e
Author: Damian Mooyman <damian@silverstripe.com> Date: Mon Jun 12 18:04:34 2017 +1200 Implement basic CLI application object commit001d559662
Author: Damian Mooyman <damian@silverstripe.com> Date: Mon Jun 12 17:39:38 2017 +1200 Remove references to SapphireTest::is_running_test() Upgrade various code commitde079c041d
Author: Damian Mooyman <damian@silverstripe.com> Date: Wed Jun 7 18:07:33 2017 +1200 API Implement APP object API Refactor of Session
462 lines
15 KiB
PHP
462 lines
15 KiB
PHP
<?php
|
|
|
|
namespace SilverStripe\View\Tests;
|
|
|
|
use SilverStripe\Core\Injector\Injector;
|
|
use SilverStripe\Core\TempFolder;
|
|
use SilverStripe\Versioned\Versioned;
|
|
use Psr\SimpleCache\CacheInterface;
|
|
use SilverStripe\Dev\SapphireTest;
|
|
use SilverStripe\Control\Director;
|
|
use SilverStripe\View\SSViewer;
|
|
use Symfony\Component\Cache\Simple\FilesystemCache;
|
|
use Symfony\Component\Cache\Simple\NullCache;
|
|
|
|
// Not actually a data object, we just want a ViewableData object that's just for us
|
|
|
|
class SSViewerCacheBlockTest extends SapphireTest
|
|
{
|
|
protected static $extra_dataobjects = array(
|
|
SSViewerCacheBlockTest\TestModel::class
|
|
);
|
|
|
|
public static function getExtraDataObjects()
|
|
{
|
|
$classes = parent::getExtraDataObjects();
|
|
|
|
// Add extra classes if versioning is enabled
|
|
if (class_exists(Versioned::class)) {
|
|
$classes[] = SSViewerCacheBlockTest\VersionedModel::class;
|
|
}
|
|
return $classes;
|
|
}
|
|
|
|
/**
|
|
* @var SSViewerCacheBlockTest\TestModel
|
|
*/
|
|
protected $data = null;
|
|
|
|
protected function _reset($cacheOn = true)
|
|
{
|
|
$this->data = new SSViewerCacheBlockTest\TestModel();
|
|
|
|
$cache = null;
|
|
if ($cacheOn) {
|
|
$cache = new FilesystemCache('cacheblock', 0, TempFolder::getTempFolder(BASE_PATH)); // cache indefinitely
|
|
} else {
|
|
$cache = new NullCache();
|
|
}
|
|
|
|
Injector::inst()->registerService($cache, CacheInterface::class . '.cacheblock');
|
|
Injector::inst()->get(CacheInterface::class . '.cacheblock')->clear();
|
|
}
|
|
|
|
protected function _runtemplate($template, $data = null)
|
|
{
|
|
if ($data === null) {
|
|
$data = $this->data;
|
|
}
|
|
if (is_array($data)) {
|
|
$data = $this->data->customise($data);
|
|
}
|
|
|
|
return SSViewer::execute_string($template, $data);
|
|
}
|
|
|
|
public function testParsing()
|
|
{
|
|
|
|
// ** Trivial checks **
|
|
|
|
// Make sure an empty cached block parses
|
|
$this->_reset();
|
|
$this->assertEquals($this->_runtemplate('<% cached %><% end_cached %>'), '');
|
|
|
|
// Make sure an empty cacheblock block parses
|
|
$this->_reset();
|
|
$this->assertEquals($this->_runtemplate('<% cacheblock %><% end_cacheblock %>'), '');
|
|
|
|
// Make sure an empty uncached block parses
|
|
$this->_reset();
|
|
$this->assertEquals($this->_runtemplate('<% uncached %><% end_uncached %>'), '');
|
|
|
|
// ** Argument checks **
|
|
|
|
// Make sure a simple cacheblock parses
|
|
$this->_reset();
|
|
$this->assertEquals($this->_runtemplate('<% cached %>Yay<% end_cached %>'), 'Yay');
|
|
|
|
// Make sure a moderately complicated cacheblock parses
|
|
$this->_reset();
|
|
$this->assertEquals($this->_runtemplate('<% cached \'block\', Foo, "jumping" %>Yay<% end_cached %>'), 'Yay');
|
|
|
|
// Make sure a complicated cacheblock parses
|
|
$this->_reset();
|
|
$this->assertEquals(
|
|
$this->_runtemplate(
|
|
'<% cached \'block\', Foo, Test.Test(4).Test(jumping).Foo %>Yay<% end_cached %>'
|
|
),
|
|
'Yay'
|
|
);
|
|
|
|
// ** Conditional Checks **
|
|
|
|
// Make sure a cacheblock with a simple conditional parses
|
|
$this->_reset();
|
|
$this->assertEquals($this->_runtemplate('<% cached if true %>Yay<% end_cached %>'), 'Yay');
|
|
|
|
// Make sure a cacheblock with a complex conditional parses
|
|
$this->_reset();
|
|
$this->assertEquals($this->_runtemplate('<% cached if Test.Test(yank).Foo %>Yay<% end_cached %>'), 'Yay');
|
|
|
|
// Make sure a cacheblock with a complex conditional and arguments parses
|
|
$this->_reset();
|
|
$this->assertEquals(
|
|
$this->_runtemplate(
|
|
'<% cached Foo, Test.Test(4).Test(jumping).Foo if Test.Test(yank).Foo %>Yay<% end_cached %>'
|
|
),
|
|
'Yay'
|
|
);
|
|
}
|
|
|
|
/**
|
|
* Test that cacheblocks actually cache
|
|
*/
|
|
public function testBlocksCache()
|
|
{
|
|
// First, run twice without caching, to prove we get two different values
|
|
$this->_reset(false);
|
|
|
|
$this->assertEquals($this->_runtemplate('<% cached %>$Foo<% end_cached %>', array('Foo' => 1)), '1');
|
|
$this->assertEquals($this->_runtemplate('<% cached %>$Foo<% end_cached %>', array('Foo' => 2)), '2');
|
|
|
|
// Then twice with caching, should get same result each time
|
|
$this->_reset(true);
|
|
|
|
$this->assertEquals($this->_runtemplate('<% cached %>$Foo<% end_cached %>', array('Foo' => 1)), '1');
|
|
$this->assertEquals($this->_runtemplate('<% cached %>$Foo<% end_cached %>', array('Foo' => 2)), '1');
|
|
}
|
|
|
|
/**
|
|
* Test that the cacheblocks invalidate when a flush occurs.
|
|
*/
|
|
public function testBlocksInvalidateOnFlush()
|
|
{
|
|
Director::test('/?flush=1');
|
|
$this->_reset(true);
|
|
|
|
// Generate cached value for foo = 1
|
|
$this->assertEquals($this->_runtemplate('<% cached %>$Foo<% end_cached %>', array('Foo' => 1)), '1');
|
|
|
|
// Test without flush
|
|
Director::test('/');
|
|
$this->assertEquals($this->_runtemplate('<% cached %>$Foo<% end_cached %>', array('Foo' => 3)), '1');
|
|
|
|
// Test with flush
|
|
Director::test('/?flush=1');
|
|
$this->assertEquals($this->_runtemplate('<% cached %>$Foo<% end_cached %>', array('Foo' => 2)), '2');
|
|
}
|
|
|
|
public function testVersionedCache()
|
|
{
|
|
if (!class_exists(Versioned::class)) {
|
|
$this->markTestSkipped('testVersionedCache requires Versioned extension');
|
|
}
|
|
$origReadingMode = Versioned::get_reading_mode();
|
|
|
|
// Run without caching in stage to prove data is uncached
|
|
$this->_reset(false);
|
|
Versioned::set_stage(Versioned::DRAFT);
|
|
$data = new SSViewerCacheBlockTest\VersionedModel();
|
|
$data->setEntropy('default');
|
|
$this->assertEquals(
|
|
'default Stage.Stage',
|
|
SSViewer::execute_string('<% cached %>$Inspect<% end_cached %>', $data)
|
|
);
|
|
$data = new SSViewerCacheBlockTest\VersionedModel();
|
|
$data->setEntropy('first');
|
|
$this->assertEquals(
|
|
'first Stage.Stage',
|
|
SSViewer::execute_string('<% cached %>$Inspect<% end_cached %>', $data)
|
|
);
|
|
|
|
// Run without caching in live to prove data is uncached
|
|
$this->_reset(false);
|
|
Versioned::set_stage(Versioned::LIVE);
|
|
$data = new SSViewerCacheBlockTest\VersionedModel();
|
|
$data->setEntropy('default');
|
|
$this->assertEquals(
|
|
'default Stage.Live',
|
|
$this->_runtemplate('<% cached %>$Inspect<% end_cached %>', $data)
|
|
);
|
|
$data = new SSViewerCacheBlockTest\VersionedModel();
|
|
$data->setEntropy('first');
|
|
$this->assertEquals(
|
|
'first Stage.Live',
|
|
$this->_runtemplate('<% cached %>$Inspect<% end_cached %>', $data)
|
|
);
|
|
|
|
// Then with caching, initially in draft, and then in live, to prove that
|
|
// changing the versioned reading mode doesn't cache between modes, but it does
|
|
// within them
|
|
$this->_reset(true);
|
|
Versioned::set_stage(Versioned::DRAFT);
|
|
$data = new SSViewerCacheBlockTest\VersionedModel();
|
|
$data->setEntropy('default');
|
|
$this->assertEquals(
|
|
'default Stage.Stage',
|
|
$this->_runtemplate('<% cached %>$Inspect<% end_cached %>', $data)
|
|
);
|
|
$data = new SSViewerCacheBlockTest\VersionedModel();
|
|
$data->setEntropy('first');
|
|
$this->assertEquals(
|
|
'default Stage.Stage', // entropy should be ignored due to caching
|
|
$this->_runtemplate('<% cached %>$Inspect<% end_cached %>', $data)
|
|
);
|
|
|
|
Versioned::set_stage(Versioned::LIVE);
|
|
$data = new SSViewerCacheBlockTest\VersionedModel();
|
|
$data->setEntropy('first');
|
|
$this->assertEquals(
|
|
'first Stage.Live', // First hit in live, so display current entropy
|
|
$this->_runtemplate('<% cached %>$Inspect<% end_cached %>', $data)
|
|
);
|
|
$data = new SSViewerCacheBlockTest\VersionedModel();
|
|
$data->setEntropy('second');
|
|
$this->assertEquals(
|
|
'first Stage.Live', // entropy should be ignored due to caching
|
|
$this->_runtemplate('<% cached %>$Inspect<% end_cached %>', $data)
|
|
);
|
|
|
|
Versioned::set_reading_mode($origReadingMode);
|
|
}
|
|
|
|
/**
|
|
* Test that cacheblocks conditionally cache with if
|
|
*/
|
|
public function testBlocksConditionallyCacheWithIf()
|
|
{
|
|
// First, run twice with caching
|
|
$this->_reset(true);
|
|
|
|
$this->assertEquals($this->_runtemplate('<% cached if True %>$Foo<% end_cached %>', array('Foo' => 1)), '1');
|
|
$this->assertEquals($this->_runtemplate('<% cached if True %>$Foo<% end_cached %>', array('Foo' => 2)), '1');
|
|
|
|
// Then twice without caching
|
|
$this->_reset(true);
|
|
|
|
$this->assertEquals($this->_runtemplate('<% cached if False %>$Foo<% end_cached %>', array('Foo' => 1)), '1');
|
|
$this->assertEquals($this->_runtemplate('<% cached if False %>$Foo<% end_cached %>', array('Foo' => 2)), '2');
|
|
|
|
// Then once cached, once not (and the opposite)
|
|
$this->_reset(true);
|
|
|
|
$this->assertEquals(
|
|
$this->_runtemplate(
|
|
'<% cached if Cache %>$Foo<% end_cached %>',
|
|
array('Foo' => 1, 'Cache' => true )
|
|
),
|
|
'1'
|
|
);
|
|
$this->assertEquals(
|
|
$this->_runtemplate(
|
|
'<% cached if Cache %>$Foo<% end_cached %>',
|
|
array('Foo' => 2, 'Cache' => false)
|
|
),
|
|
'2'
|
|
);
|
|
|
|
$this->_reset(true);
|
|
|
|
$this->assertEquals(
|
|
$this->_runtemplate(
|
|
'<% cached if Cache %>$Foo<% end_cached %>',
|
|
array('Foo' => 1, 'Cache' => false)
|
|
),
|
|
'1'
|
|
);
|
|
$this->assertEquals(
|
|
$this->_runtemplate(
|
|
'<% cached if Cache %>$Foo<% end_cached %>',
|
|
array('Foo' => 2, 'Cache' => true )
|
|
),
|
|
'2'
|
|
);
|
|
}
|
|
|
|
/**
|
|
* Test that cacheblocks conditionally cache with unless
|
|
*/
|
|
public function testBlocksConditionallyCacheWithUnless()
|
|
{
|
|
// First, run twice with caching
|
|
$this->_reset(true);
|
|
|
|
$this->assertEquals(
|
|
$this->_runtemplate(
|
|
'<% cached unless False %>$Foo<% end_cached %>',
|
|
array('Foo' => 1)
|
|
),
|
|
'1'
|
|
);
|
|
$this->assertEquals(
|
|
$this->_runtemplate(
|
|
'<% cached unless False %>$Foo<% end_cached %>',
|
|
array('Foo' => 2)
|
|
),
|
|
'1'
|
|
);
|
|
|
|
// Then twice without caching
|
|
$this->_reset(true);
|
|
|
|
$this->assertEquals(
|
|
$this->_runtemplate(
|
|
'<% cached unless True %>$Foo<% end_cached %>',
|
|
array('Foo' => 1)
|
|
),
|
|
'1'
|
|
);
|
|
$this->assertEquals(
|
|
$this->_runtemplate(
|
|
'<% cached unless True %>$Foo<% end_cached %>',
|
|
array('Foo' => 2)
|
|
),
|
|
'2'
|
|
);
|
|
}
|
|
|
|
/**
|
|
* Test that nested uncached blocks work
|
|
*/
|
|
public function testNestedUncachedBlocks()
|
|
{
|
|
// First, run twice with caching, to prove we get the same result back normally
|
|
$this->_reset(true);
|
|
|
|
$this->assertEquals(
|
|
$this->_runtemplate(
|
|
'<% cached %> A $Foo B <% end_cached %>',
|
|
array('Foo' => 1)
|
|
),
|
|
' A 1 B '
|
|
);
|
|
$this->assertEquals(
|
|
$this->_runtemplate(
|
|
'<% cached %> A $Foo B <% end_cached %>',
|
|
array('Foo' => 2)
|
|
),
|
|
' A 1 B '
|
|
);
|
|
|
|
// Then add uncached to the nested block
|
|
$this->_reset(true);
|
|
|
|
$this->assertEquals(
|
|
$this->_runtemplate(
|
|
'<% cached %> A <% uncached %>$Foo<% end_uncached %> B <% end_cached %>',
|
|
array('Foo' => 1)
|
|
),
|
|
' A 1 B '
|
|
);
|
|
$this->assertEquals(
|
|
$this->_runtemplate(
|
|
'<% cached %> A <% uncached %>$Foo<% end_uncached %> B <% end_cached %>',
|
|
array('Foo' => 2)
|
|
),
|
|
' A 2 B '
|
|
);
|
|
}
|
|
|
|
/**
|
|
* Test that nested blocks with different keys works
|
|
*/
|
|
public function testNestedBlocks()
|
|
{
|
|
$this->_reset(true);
|
|
|
|
$template = '<% cached Foo %> $Fooa <% cached Bar %>$Bara<% end_cached %> $Foob <% end_cached %>';
|
|
|
|
// Do it the first time to load the cache
|
|
$this->assertEquals(
|
|
$this->_runtemplate(
|
|
$template,
|
|
array('Foo' => 1, 'Fooa' => 1, 'Foob' => 3, 'Bar' => 1, 'Bara' => 2)
|
|
),
|
|
' 1 2 3 '
|
|
);
|
|
|
|
// Do it again, the input values are ignored as the cache is hit for both elements
|
|
$this->assertEquals(
|
|
$this->_runtemplate(
|
|
$template,
|
|
array('Foo' => 1, 'Fooa' => 9, 'Foob' => 9, 'Bar' => 1, 'Bara' => 9)
|
|
),
|
|
' 1 2 3 '
|
|
);
|
|
|
|
// Do it again with a new key for Bar, Bara is picked up, Fooa and Foob are not
|
|
$this->assertEquals(
|
|
$this->_runtemplate(
|
|
$template,
|
|
array('Foo' => 1, 'Fooa' => 9, 'Foob' => 9, 'Bar' => 2, 'Bara' => 9)
|
|
),
|
|
' 1 9 3 '
|
|
);
|
|
|
|
// Do it again with a new key for Foo, Fooa and Foob are picked up, Bara are not
|
|
$this->assertEquals(
|
|
$this->_runtemplate(
|
|
$template,
|
|
array('Foo' => 2, 'Fooa' => 9, 'Foob' => 9, 'Bar' => 2, 'Bara' => 1)
|
|
),
|
|
' 9 9 9 '
|
|
);
|
|
}
|
|
|
|
public function testNoErrorMessageForControlWithinCached()
|
|
{
|
|
$this->_reset(true);
|
|
$this->assertNotNull($this->_runtemplate('<% cached %><% with Foo %>$Bar<% end_with %><% end_cached %>'));
|
|
}
|
|
|
|
/**
|
|
* @expectedException \SilverStripe\View\SSTemplateParseException
|
|
*/
|
|
public function testErrorMessageForCachedWithinControlWithinCached()
|
|
{
|
|
$this->_reset(true);
|
|
$this->_runtemplate(
|
|
'<% cached %><% with Foo %><% cached %>$Bar<% end_cached %><% end_with %><% end_cached %>'
|
|
);
|
|
}
|
|
|
|
public function testNoErrorMessageForCachedWithinControlWithinUncached()
|
|
{
|
|
$this->_reset(true);
|
|
$this->assertNotNull(
|
|
$this->_runtemplate(
|
|
'<% uncached %><% with Foo %><% cached %>$Bar<% end_cached %><% end_with %><% end_uncached %>'
|
|
)
|
|
);
|
|
}
|
|
|
|
/**
|
|
* @expectedException \SilverStripe\View\SSTemplateParseException
|
|
*/
|
|
public function testErrorMessageForCachedWithinIf()
|
|
{
|
|
$this->_reset(true);
|
|
$this->_runtemplate('<% cached %><% if Foo %><% cached %>$Bar<% end_cached %><% end_if %><% end_cached %>');
|
|
}
|
|
|
|
/**
|
|
* @expectedException \SilverStripe\View\SSTemplateParseException
|
|
*/
|
|
public function testErrorMessageForInvalidConditional()
|
|
{
|
|
$this->_reset(true);
|
|
$this->_runtemplate('<% cached Foo if %>$Bar<% end_cached %>');
|
|
}
|
|
}
|