xini / silverstripe-section-io

tests/SectionIOTest/CustomSectionIO.php

Indentation +14 added lines, -14 removed lines

@@ -6,18 +6,18 @@
 
 class CustomSectionIO extends SectionIO
 {
-    protected static function performFlush($banExpression)
-    {
-        $result = array();
-        $urls = static::getUrls();
-        if (count($urls) > 0) {
-            foreach ($urls as $url) {
-                $data = array();
-                $data['url'] = $url;
-                $data['banExpression'] = $banExpression;
-                $result[] = $data;
-            }
-        }
-        return $result;
-    }
+	protected static function performFlush($banExpression)
+	{
+		$result = array();
+		$urls = static::getUrls();
+		if (count($urls) > 0) {
+			foreach ($urls as $url) {
+				$data = array();
+				$data['url'] = $url;
+				$data['banExpression'] = $banExpression;
+				$result[] = $data;
+			}
+		}
+		return $result;
+	}
 }

tests/SectionIOTest.php

Indentation +227 added lines, -227 removed lines

@@ -16,236 +16,236 @@
 
 class SectionIOTest extends SapphireTest
 {
-    protected static $fixture_file = 'SectionIOTest.yml';
+	protected static $fixture_file = 'SectionIOTest.yml';
 
-    public static function setUpBeforeClass()
-    {
-        parent::setUpBeforeClass();
+	public static function setUpBeforeClass()
+	{
+		parent::setUpBeforeClass();
         
-        // add config values
-        Config::modify()->set(SectionIO::class, 'flush_on_dev_build', true);
-        Config::modify()->set(SectionIO::class, 'api_url', 'https://example.com');
-        Config::modify()->set(SectionIO::class, 'account_id', '123456');
-        Config::modify()->set(SectionIO::class, 'application_id', '987654');
-        Config::modify()->set(SectionIO::class, 'environment_name', 'Production');
-        Config::modify()->set(SectionIO::class, 'proxy_name', 'myproxy');
-        Config::modify()->set(SectionIO::class, 'username', 'someuser');
-        Config::modify()->set(SectionIO::class, 'password', 'MySafePassword');
-        Config::modify()->set(SectionIO::class, 'verify_ssl', false);
-
-        // remove extensions otherwise the fixtures will break the tests (by calling the live flush)
-        File::remove_extension(SectionIOFileExtension::class);
-        SiteTree::remove_extension(SectionIOSiteTreeExtension::class);
-    }
-
-    public function setUp()
-    {
-        parent::setUp();
+		// add config values
+		Config::modify()->set(SectionIO::class, 'flush_on_dev_build', true);
+		Config::modify()->set(SectionIO::class, 'api_url', 'https://example.com');
+		Config::modify()->set(SectionIO::class, 'account_id', '123456');
+		Config::modify()->set(SectionIO::class, 'application_id', '987654');
+		Config::modify()->set(SectionIO::class, 'environment_name', 'Production');
+		Config::modify()->set(SectionIO::class, 'proxy_name', 'myproxy');
+		Config::modify()->set(SectionIO::class, 'username', 'someuser');
+		Config::modify()->set(SectionIO::class, 'password', 'MySafePassword');
+		Config::modify()->set(SectionIO::class, 'verify_ssl', false);
+
+		// remove extensions otherwise the fixtures will break the tests (by calling the live flush)
+		File::remove_extension(SectionIOFileExtension::class);
+		SiteTree::remove_extension(SectionIOSiteTreeExtension::class);
+	}
+
+	public function setUp()
+	{
+		parent::setUp();
         
-        // Create a test files for each of the fixture references
-        $fileIDs = array_merge(
-            $this->allFixtureIDs(File::class),
-            $this->allFixtureIDs(Image::class)
-        );
-        foreach ($fileIDs as $fileID) {
-            /** @var File $file */
-            $file = DataObject::get_by_id(File::class, $fileID);
-            $file->setFromString(str_repeat('x', 1000000), $file->getFilename());
-        }
-    }
-
-    public static function tearDownAfterClass()
-    {
-        // re-add extensions
-        File::add_extension(SectionIOFileExtension::class);
-        SiteTree::add_extension(SectionIOSiteTreeExtension::class);
+		// Create a test files for each of the fixture references
+		$fileIDs = array_merge(
+			$this->allFixtureIDs(File::class),
+			$this->allFixtureIDs(Image::class)
+		);
+		foreach ($fileIDs as $fileID) {
+			/** @var File $file */
+			$file = DataObject::get_by_id(File::class, $fileID);
+			$file->setFromString(str_repeat('x', 1000000), $file->getFilename());
+		}
+	}
+
+	public static function tearDownAfterClass()
+	{
+		// re-add extensions
+		File::add_extension(SectionIOFileExtension::class);
+		SiteTree::add_extension(SectionIOSiteTreeExtension::class);
         
-        parent::tearDownAfterClass();
-    }
-
-    public function testFlushAll()
-    {
-        $result = CustomSectionIO::flushAll();
-
-        $this->assertCount(
-            1,
-            $result,
-            'one url returned for one application id'
-        );
-
-        // url
-        $this->assertEquals(
-            'https://example.com/account/123456/application/987654/environment/Production/proxy/myproxy/state',
-            $result[0]['url'],
-            'URL is concatenated correctly'
-        );
-
-        // ban expression
-        $this->assertEquals(
-            'obj.http.x-url ~ /',
-            $result[0]['banExpression'],
-            'ban expression is correct'
-        );
-    }
-
-    public function testFlush()
-    {
-        $result = CustomSectionIO::flush();
-
-        $this->assertCount(
-            1,
-            $result,
-            'one url returned for one application id'
-        );
-
-        // url
-        $this->assertEquals(
-            'https://example.com/account/123456/application/987654/environment/Production/proxy/myproxy/state',
-            $result[0]['url'],
-            'URL is concatenated correctly'
-        );
-
-        // ban expression
-        $this->assertEquals(
-            'obj.http.x-url ~ /',
-            $result[0]['banExpression'],
-            'ban expression is correct'
-        );
-
-        // test deactivated flush on build
-        Config::modify()->set(SectionIO::class, 'flush_on_dev_build', false);
-        $result = CustomSectionIO::flush();
-        $this->assertNull(
-            $result,
-            'null returned if flush on build deactivated'
-        );
-    }
-
-    public function testMultipleApplicationIDs()
-    {
-        // add second application to config
-        Config::modify()->set(SectionIO::class, 'application_id', '2546987,856954');
-
-        $result = CustomSectionIO::flushAll();
-
-        $this->assertCount(
-            2,
-            $result,
-            'two urls returned for two application id'
-        );
-
-        // url
-        $this->assertEquals(
-            'https://example.com/account/123456/application/2546987/environment/Production/proxy/myproxy/state',
-            $result[0]['url'],
-            'URL is concatenated correctly for app 1'
-        );
-        $this->assertEquals(
-            'https://example.com/account/123456/application/856954/environment/Production/proxy/myproxy/state',
-            $result[1]['url'],
-            'URL is concatenated correctly for app 2'
-        );
-
-        // add second application to config with spaces in csv
-        Config::modify()->set(SectionIO::class, 'application_id', '741852, 369258');
-
-        $result = CustomSectionIO::flushAll();
-
-        $this->assertCount(
-            2,
-            $result,
-            'two urls returned for two application id'
-        );
-
-        // url
-        $this->assertEquals(
-            'https://example.com/account/123456/application/741852/environment/Production/proxy/myproxy/state',
-            $result[0]['url'],
-            'URL is concatenated correctly for app 1'
-        );
-        $this->assertEquals(
-            'https://example.com/account/123456/application/369258/environment/Production/proxy/myproxy/state',
-            $result[1]['url'],
-            'URL is concatenated correctly for app 2'
-        );
-    }
-
-    public function testFlushImage()
-    {
-        $imageId = $this->idFromFixture(Image::class, 'testImage');
-
-        $result = CustomSectionIO::flushImage($imageId);
+		parent::tearDownAfterClass();
+	}
+
+	public function testFlushAll()
+	{
+		$result = CustomSectionIO::flushAll();
+
+		$this->assertCount(
+			1,
+			$result,
+			'one url returned for one application id'
+		);
+
+		// url
+		$this->assertEquals(
+			'https://example.com/account/123456/application/987654/environment/Production/proxy/myproxy/state',
+			$result[0]['url'],
+			'URL is concatenated correctly'
+		);
+
+		// ban expression
+		$this->assertEquals(
+			'obj.http.x-url ~ /',
+			$result[0]['banExpression'],
+			'ban expression is correct'
+		);
+	}
+
+	public function testFlush()
+	{
+		$result = CustomSectionIO::flush();
+
+		$this->assertCount(
+			1,
+			$result,
+			'one url returned for one application id'
+		);
+
+		// url
+		$this->assertEquals(
+			'https://example.com/account/123456/application/987654/environment/Production/proxy/myproxy/state',
+			$result[0]['url'],
+			'URL is concatenated correctly'
+		);
+
+		// ban expression
+		$this->assertEquals(
+			'obj.http.x-url ~ /',
+			$result[0]['banExpression'],
+			'ban expression is correct'
+		);
+
+		// test deactivated flush on build
+		Config::modify()->set(SectionIO::class, 'flush_on_dev_build', false);
+		$result = CustomSectionIO::flush();
+		$this->assertNull(
+			$result,
+			'null returned if flush on build deactivated'
+		);
+	}
+
+	public function testMultipleApplicationIDs()
+	{
+		// add second application to config
+		Config::modify()->set(SectionIO::class, 'application_id', '2546987,856954');
+
+		$result = CustomSectionIO::flushAll();
+
+		$this->assertCount(
+			2,
+			$result,
+			'two urls returned for two application id'
+		);
+
+		// url
+		$this->assertEquals(
+			'https://example.com/account/123456/application/2546987/environment/Production/proxy/myproxy/state',
+			$result[0]['url'],
+			'URL is concatenated correctly for app 1'
+		);
+		$this->assertEquals(
+			'https://example.com/account/123456/application/856954/environment/Production/proxy/myproxy/state',
+			$result[1]['url'],
+			'URL is concatenated correctly for app 2'
+		);
+
+		// add second application to config with spaces in csv
+		Config::modify()->set(SectionIO::class, 'application_id', '741852, 369258');
+
+		$result = CustomSectionIO::flushAll();
+
+		$this->assertCount(
+			2,
+			$result,
+			'two urls returned for two application id'
+		);
+
+		// url
+		$this->assertEquals(
+			'https://example.com/account/123456/application/741852/environment/Production/proxy/myproxy/state',
+			$result[0]['url'],
+			'URL is concatenated correctly for app 1'
+		);
+		$this->assertEquals(
+			'https://example.com/account/123456/application/369258/environment/Production/proxy/myproxy/state',
+			$result[1]['url'],
+			'URL is concatenated correctly for app 2'
+		);
+	}
+
+	public function testFlushImage()
+	{
+		$imageId = $this->idFromFixture(Image::class, 'testImage');
+
+		$result = CustomSectionIO::flushImage($imageId);
         
-        // ban expression
-        $this->assertThat(
-            $result[0]['banExpression'],
-            $this->logicalOr(
-                'obj.http.x-url ~ "^/assets/SectionTest/test_image\.png$"'
-                    .' || obj.http.x-url ~ "^/assets/SectionTest/test_image__[a-zA-Z0-9_]*\.png$"',
-                'obj.http.x-url ~ "^/assets/SectionTest/55b443b601/test_image\.png$"'
-                    .' || obj.http.x-url ~ "^/assets/SectionTest/55b443b601/test_image__[a-zA-Z0-9_]*\.png$"'
-            ),
-            'ban expression is correct'
-        );
-    }
-
-    public function testFlushFile()
-    {
-        $fileId = $this->idFromFixture(File::class, 'testFile');
-
-        $result = CustomSectionIO::flushFile($fileId);
-
-        // ban expression
-        $this->assertThat(
-            $result[0]['banExpression'],
-            $this->logicalOr(
-                'obj.http.x-url ~ "^/assets/SectionTest/test_document\.pdf$"',
-                'obj.http.x-url ~ "^/assets/SectionTest/55b443b601/test_document\.pdf$"'
-            ),
-            'ban expression is correct'
-        );
-    }
-
-    public function testFlushSiteTree()
-    {
-        $pageId = $this->idFromFixture(Page::class, 'ceo');
-
-        // test single page flush
-        Config::modify()->set(SectionIO::class, 'sitetree_flush_strategy', 'single');
-        $result = CustomSectionIO::flushSiteTree($pageId);
-        $this->assertEquals(
-            'obj.http.content-type ~ "text/html"'
-            .' && obj.http.x-url ~ "^/about\-us/my\-staff/ceo/$"',
-            $result[0]['banExpression'],
-            'ban expression is correct'
-        );
-
-        // test parents flush
-        Config::modify()->set(SectionIO::class, 'sitetree_flush_strategy', 'parents');
-        $result = CustomSectionIO::flushSiteTree($pageId);
-        $this->assertEquals(
-            'obj.http.content-type ~ "text/html"'
-            .' && (obj.http.x-url ~ "^/about\-us/my\-staff/ceo/$" || obj.http.x-url ~ "^/about\-us/my\-staff/$" || obj.http.x-url ~ "^/about\-us/$")',
-            $result[0]['banExpression'],
-            'ban expression is correct'
-        );
-
-        // test all pages flush
-        Config::modify()->set(SectionIO::class, 'sitetree_flush_strategy', 'all');
-        $result = CustomSectionIO::flushSiteTree($pageId);
-        $this->assertEquals(
-            'obj.http.content-type ~ "text/html"',
-            $result[0]['banExpression'],
-            'ban expression is correct'
-        );
-
-        // test whole site flush
-        Config::modify()->set(SectionIO::class, 'sitetree_flush_strategy', 'everything');
-        $result = CustomSectionIO::flushSiteTree($pageId);
-        $this->assertEquals(
-            'obj.http.x-url ~ /',
-            $result[0]['banExpression'],
-            'ban expression is correct'
-        );
-    }
+		// ban expression
+		$this->assertThat(
+			$result[0]['banExpression'],
+			$this->logicalOr(
+				'obj.http.x-url ~ "^/assets/SectionTest/test_image\.png$"'
+					.' || obj.http.x-url ~ "^/assets/SectionTest/test_image__[a-zA-Z0-9_]*\.png$"',
+				'obj.http.x-url ~ "^/assets/SectionTest/55b443b601/test_image\.png$"'
+					.' || obj.http.x-url ~ "^/assets/SectionTest/55b443b601/test_image__[a-zA-Z0-9_]*\.png$"'
+			),
+			'ban expression is correct'
+		);
+	}
+
+	public function testFlushFile()
+	{
+		$fileId = $this->idFromFixture(File::class, 'testFile');
+
+		$result = CustomSectionIO::flushFile($fileId);
+
+		// ban expression
+		$this->assertThat(
+			$result[0]['banExpression'],
+			$this->logicalOr(
+				'obj.http.x-url ~ "^/assets/SectionTest/test_document\.pdf$"',
+				'obj.http.x-url ~ "^/assets/SectionTest/55b443b601/test_document\.pdf$"'
+			),
+			'ban expression is correct'
+		);
+	}
+
+	public function testFlushSiteTree()
+	{
+		$pageId = $this->idFromFixture(Page::class, 'ceo');
+
+		// test single page flush
+		Config::modify()->set(SectionIO::class, 'sitetree_flush_strategy', 'single');
+		$result = CustomSectionIO::flushSiteTree($pageId);
+		$this->assertEquals(
+			'obj.http.content-type ~ "text/html"'
+			.' && obj.http.x-url ~ "^/about\-us/my\-staff/ceo/$"',
+			$result[0]['banExpression'],
+			'ban expression is correct'
+		);
+
+		// test parents flush
+		Config::modify()->set(SectionIO::class, 'sitetree_flush_strategy', 'parents');
+		$result = CustomSectionIO::flushSiteTree($pageId);
+		$this->assertEquals(
+			'obj.http.content-type ~ "text/html"'
+			.' && (obj.http.x-url ~ "^/about\-us/my\-staff/ceo/$" || obj.http.x-url ~ "^/about\-us/my\-staff/$" || obj.http.x-url ~ "^/about\-us/$")',
+			$result[0]['banExpression'],
+			'ban expression is correct'
+		);
+
+		// test all pages flush
+		Config::modify()->set(SectionIO::class, 'sitetree_flush_strategy', 'all');
+		$result = CustomSectionIO::flushSiteTree($pageId);
+		$this->assertEquals(
+			'obj.http.content-type ~ "text/html"',
+			$result[0]['banExpression'],
+			'ban expression is correct'
+		);
+
+		// test whole site flush
+		Config::modify()->set(SectionIO::class, 'sitetree_flush_strategy', 'everything');
+		$result = CustomSectionIO::flushSiteTree($pageId);
+		$this->assertEquals(
+			'obj.http.x-url ~ /',
+			$result[0]['banExpression'],
+			'ban expression is correct'
+		);
+	}
 }