diff options
| author | Lukas Reschke <lukas@owncloud.com> | 2015-04-16 10:48:25 +0200 |
|---|---|---|
| committer | Lukas Reschke <lukas@owncloud.com> | 2015-04-20 13:00:02 +0200 |
| commit | 9bc48451b939df58b7e7e90300c9c716c348027f (patch) | |
| tree | ca5e8858f60e8f479a4bb299b26d867bbfda7ab6 /tests/lib/security | |
| parent | 89353695648eb686e6fffd49f86a6d014121f551 (diff) | |
| download | nextcloud-server-9bc48451b939df58b7e7e90300c9c716c348027f.tar.gz nextcloud-server-9bc48451b939df58b7e7e90300c9c716c348027f.zip | |
Adjust tests and statuscode
Diffstat (limited to 'tests/lib/security')
| -rw-r--r-- | tests/lib/security/certificatemanager.php | 20 |
1 files changed, 15 insertions, 5 deletions
diff --git a/tests/lib/security/certificatemanager.php b/tests/lib/security/certificatemanager.php index aea1fb71bc5..fab1c208443 100644 --- a/tests/lib/security/certificatemanager.php +++ b/tests/lib/security/certificatemanager.php @@ -14,8 +14,6 @@ class CertificateManagerTest extends \Test\TestCase { private $certificateManager; /** @var String */ private $username; - /** @var \OC\User\User */ - private $user; protected function setUp() { parent::setUp(); @@ -68,12 +66,24 @@ class CertificateManagerTest extends \Test\TestCase { } /** + * @return array + */ + public function dangerousFileProvider() { + return [ + ['.htaccess'], + ['../../foo.txt'], + ['..\..\foo.txt'], + ]; + } + + /** * @expectedException \Exception * @expectedExceptionMessage Filename is not valid + * @dataProvider dangerousFileProvider + * @param string $filename */ - function testAddDangerousFile() { - $this->certificateManager->addCertificate(file_get_contents(__DIR__.'/../../data/certificates/expiredCertificate.crt'), '.htaccess'); - $this->certificateManager->addCertificate(file_get_contents(__DIR__.'/../../data/certificates/expiredCertificate.crt'), '../../foo.txt'); + function testAddDangerousFile($filename) { + $this->certificateManager->addCertificate(file_get_contents(__DIR__.'/../../data/certificates/expiredCertificate.crt'), $filename); } function testRemoveDangerousFile() { |