diff options
| author | Andy Chase <asperous2@gmail.com> | 2013-06-26 12:43:37 -0700 |
|---|---|---|
| committer | Andy Chase <asperous2@gmail.com> | 2013-07-05 14:09:44 -0700 |
| commit | 1d9d22cc7676fac14bfe4a5b67537ccfb4f1849d (patch) | |
| tree | 857c39e94435f2c902b6aafdf92d6bf45d67ca0c /tests | |
| parent | 4aef3d0ab9f08b1e5ad5dc776461fc425368c8db (diff) | |
| download | forums-1d9d22cc7676fac14bfe4a5b67537ccfb4f1849d.tar forums-1d9d22cc7676fac14bfe4a5b67537ccfb4f1849d.tar.gz forums-1d9d22cc7676fac14bfe4a5b67537ccfb4f1849d.tar.bz2 forums-1d9d22cc7676fac14bfe4a5b67537ccfb4f1849d.tar.xz forums-1d9d22cc7676fac14bfe4a5b67537ccfb4f1849d.zip | |
[ticket/11620] Add testable facade for sessions.php
Since many functions in session.php have global variables inside the function,
this exposes those functions through a testable facade that uses testable_factory's
mock global variables to modify global variables used in the functions.
This is using the facade pattern to provide a testable "front" to the functions in sessions.php.
PHPBB3-11620
Diffstat (limited to 'tests')
| -rw-r--r-- | tests/session/testable_facade.php | 49 |
1 files changed, 49 insertions, 0 deletions
diff --git a/tests/session/testable_facade.php b/tests/session/testable_facade.php new file mode 100644 index 0000000000..d4c03ec8d5 --- /dev/null +++ b/tests/session/testable_facade.php @@ -0,0 +1,49 @@ +<?php +/** + * + * @package testing + * @copyright (c) 2013 phpBB Group + * @license http://opensource.org/licenses/gpl-2.0.php GNU General Public License v2 + * + */ + +require_once dirname(__FILE__) . '/testable_factory.php'; +require_once dirname(__FILE__) . '/../../phpBB/includes/session.php'; + +/** + * This class exists to expose session.php's functions in a more testable way. + * + * Since many functions in session.php have global variables inside the function, + * this exposes those functions through a testable facade that uses testable_factory's + * mock global variables to modify global variables used in the functions. + * + * This is using the facade pattern to provide a testable "front" to the functions in sessions.php. + * + */ +class phpbb_session_testable_facade +{ + public static function extract_current_page($db, $session_factory, $root_path, $php_self, $query_string, $request_uri) { + $session_factory->get_session($db); + global $request; + $request->overwrite('PHP_SELF', $php_self, phpbb_request_interface::SERVER); + $request->overwrite('QUERY_STRING', $query_string, phpbb_request_interface::SERVER); + $request->overwrite('REQUEST_URI', $request_uri, phpbb_request_interface::SERVER); + return phpbb_session::extract_current_page($root_path, phpbb_request_interface::SERVER); + } + + // [To be completed] + // public static function extract_current_hostname() {} + // public static function session_begin($update_session_page = true) {} + // public static function session_create($user_id = false, $set_admin = false, $persist_login = false, $viewonline = true) {} + // public static function session_kill($new_session = true) {} + // public static function session_gc() {} + // public static function set_cookie($name, $cookiedata, $cookietime) {} + // public static function check_ban($user_id = false, $user_ips = false, $user_email = false, $return = false) {} + // public static function check_dnsbl($mode, $ip = false) {} + // public static function set_login_key($user_id = false, $key = false, $user_ip = false) {} + // public static function reset_login_keys($user_id = false) {} + // public static function validate_referer($check_script_path = false) {} + // public static function update_session($session_data, $session_id = null) {} + // public static function unset_admin() {} +} + |