]> granicus.if.org Git - php/commitdiff
Fix brittle test
authorChristoph M. Becker <cmbecker69@gmx.de>
Wed, 22 Jul 2020 18:45:55 +0000 (20:45 +0200)
committerChristoph M. Becker <cmbecker69@gmx.de>
Wed, 22 Jul 2020 18:45:56 +0000 (20:45 +0200)
This test fails occasionally due to timing issues, because the session
file may have been unlinked by the first `session_start()`'s GC.  We
adapt the test expectation to this reality.

ext/session/tests/session_module_name_variation4.phpt

index 00e6975045b12d3934afd9a4bd6889d93e196ca3..3ac731835f41378080a65b9e79fec6b760f05666 100644 (file)
@@ -29,10 +29,12 @@ $_SESSION["Blah"] = "Hello World!";
 $_SESSION["Foo"] = FALSE;
 $_SESSION["Guff"] = 1234567890;
 var_dump($_SESSION);
+$oldsession = $_SESSION;
 
 var_dump(session_write_close());
 session_start();
-var_dump($_SESSION);
+// the session may have been GC'd or not; we accept either outcome
+var_dump($_SESSION === $oldsession || $_SESSION === []);
 var_dump(session_destroy());
 session_start();
 var_dump($_SESSION);
@@ -51,14 +53,7 @@ array(3) {
   int(1234567890)
 }
 bool(true)
-array(3) {
-  ["Blah"]=>
-  string(12) "Hello World!"
-  ["Foo"]=>
-  bool(false)
-  ["Guff"]=>
-  int(1234567890)
-}
+bool(true)
 bool(true)
 array(0) {
 }