mirror/php-phpbb
1
0
Fork 0
mirror of https://github.com/phpbb/phpbb.git synced 2025-05-06 15:45:34 +02:00
Code Issues Releases Wiki Activity

[feature/passwords] Add tests for manager check_hash() method

Browse Source 
Tests cover all supported hashing algorithms.

PHPBB3-11610
This commit is contained in:
Marc Alexander 2013-06-15 17:56:29 +02:00
parent 13d25e6a32
commit e7b3daeb29
1 changed files with 40 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

40
tests/crypto/manager_test.php
View File

@ -11,6 +11,7 @@ require_once dirname(__FILE__) . '/../mock/container_builder.php';
require_once dirname(__FILE__) . '/../../phpBB/includes/crypto/driver/bcrypt.php'; require_once dirname(__FILE__) . '/../../phpBB/includes/crypto/driver/bcrypt.php';
require_once dirname(__FILE__) . '/../../phpBB/includes/crypto/driver/bcrypt_2y.php'; require_once dirname(__FILE__) . '/../../phpBB/includes/crypto/driver/bcrypt_2y.php';
require_once dirname(__FILE__) . '/../../phpBB/includes/crypto/driver/salted_md5.php'; require_once dirname(__FILE__) . '/../../phpBB/includes/crypto/driver/salted_md5.php';
require_once dirname(__FILE__) . '/../../phpBB/includes/crypto/driver/phpass.php';
require_once dirname(__FILE__) . '/../../phpBB/includes/crypto/driver/helper.php'; require_once dirname(__FILE__) . '/../../phpBB/includes/crypto/driver/helper.php';
class phpbb_crypto_manager_test extends PHPUnit_Framework_TestCase class phpbb_crypto_manager_test extends PHPUnit_Framework_TestCase
@ -29,6 +30,7 @@ class phpbb_crypto_manager_test extends PHPUnit_Framework_TestCase
'crypto.driver.bcrypt' => new phpbb_crypto_driver_bcrypt($config), 'crypto.driver.bcrypt' => new phpbb_crypto_driver_bcrypt($config),
'crypto.driver.bcrypt_2y' => new phpbb_crypto_driver_bcrypt_2y($config), 'crypto.driver.bcrypt_2y' => new phpbb_crypto_driver_bcrypt_2y($config),
'crypto.driver.salted_md5' => new phpbb_crypto_driver_salted_md5($config), 'crypto.driver.salted_md5' => new phpbb_crypto_driver_salted_md5($config),
'crypto.driver.phpass' => new phpbb_crypto_driver_phpass($config),
); );
foreach ($crypto_drivers as $key => $driver) foreach ($crypto_drivers as $key => $driver)
@ -75,4 +77,42 @@ class phpbb_crypto_manager_test extends PHPUnit_Framework_TestCase
$this->assertEquals($prefix, $match[1]); $this->assertEquals($prefix, $match[1]);
$this->assertEquals($length, strlen($hash)); $this->assertEquals($length, strlen($hash));
} }
public function check_password_data()
{
if (version_compare(PHP_VERSION, '5.3.7', '<'))
{
return array(
array('foobar', 'crypto.driver.bcrypt'),
array('foobar', 'crypto.driver.salted_md5'),
array('barfoo', 'crypto.driver.phpass'),
);
}
else
{
return array(
array('foobar', 'crypto.driver.bcrypt_2y'),
array('barfoo', 'crypto.driver.bcrypt'),
array('foobar', 'crypto.driver.salted_md5'),
array('barfoo', 'crypto.driver.phpass'),
);
}
}
/**
* @dataProvider check_password_data
*/
public function test_check_password($password, $hash_type)
{
$hash = $this->manager->hash_password($password, $hash_type);
$test_word = $password;
$time = microtime(true);
// Limit each test to 3 seconds
while ((microtime(true) - $time) < 3)
{
$this->assertEquals($test_word === $password, $this->manager->check_hash($test_word, $hash));
$test_word = str_shuffle($test_word);
}
}
} }