test: improve test output and coverage

Author: jorgsowa

ext/session/tests/session_set_cookie_params_invalid_ini.phpt

@@ -0,0 +1,22 @@
+--TEST--
+Test session.cookie_samesite with invalid INI value
+--INI--
+session.cookie_samesite=Invalid
+--EXTENSIONS--
+session
+--SKIPIF--
+<?php include('skipif.inc'); ?>
+--FILE--
+<?php
+
+ob_start();
+
+var_dump(ini_get("session.cookie_samesite"));
+
+echo "Done";
+ob_end_flush();
+?>
+--EXPECTF--
+Warning: PHP Startup: session.cookie_samesite must be "Strict", "Lax", "None", or "" in Unknown on line 0
+string(0) ""
+Done

ext/session/tests/session_set_cookie_params_variation6.phpt

@@ -11,7 +11,7 @@ session
 
 ob_start();
 
-// Valid values
+echo "-- Valid values --\n";
 var_dump(ini_get("session.cookie_samesite"));
 var_dump(session_set_cookie_params(["samesite" => "Strict"]));
 var_dump(ini_get("session.cookie_samesite"));
@@ -20,11 +20,15 @@ var_dump(ini_get("session.cookie_samesite"));
 var_dump(session_set_cookie_params(["samesite" => ""]));
 var_dump(ini_get("session.cookie_samesite"));
 
-// Invalid value
+echo "-- Invalid value via session_set_cookie_params --\n";
 var_dump(session_set_cookie_params(["samesite" => "Invalid"]));
 var_dump(ini_get("session.cookie_samesite"));
 
-// Cannot change while session is active
+echo "-- Invalid value via ini_set --\n";
+var_dump(ini_set("session.cookie_samesite", "Invalid"));
+var_dump(ini_get("session.cookie_samesite"));
+
+echo "-- Cannot change while session is active --\n";
 var_dump(session_set_cookie_params(["samesite" => "Lax"]));
 var_dump(session_start());
 var_dump(session_set_cookie_params(["samesite" => "Strict"]));
@@ -34,17 +38,25 @@ echo "Done";
 ob_end_flush();
 ?>
 --EXPECTF--
+-- Valid values --
 string(3) "Lax"
 bool(true)
 string(6) "Strict"
 bool(true)
 string(4) "None"
 bool(true)
 string(0) ""
+-- Invalid value via session_set_cookie_params --
 
 Warning: session_set_cookie_params(): session.cookie_samesite must be "Strict", "Lax", "None", or "" in %s on line %d
 bool(false)
 string(0) ""
+-- Invalid value via ini_set --
+
+Warning: ini_set(): session.cookie_samesite must be "Strict", "Lax", "None", or "" in %s on line %d
+bool(false)
+string(0) ""
+-- Cannot change while session is active --
 bool(true)
 bool(true)