spip / SPIP

ecrire/tests/Squelettes/Balise/data/squelettes/formulaires/test_dyn_lang.php

Indentation +3 added lines, -3 removed lines

@@ -3,7 +3,7 @@
 declare(strict_types=1);
 
 function formulaires_test_dyn_lang_charger_dist($lang_skel) {
-	return [
-		'message_ok' => $lang_skel === $GLOBALS['spip_lang'] ? 'OK' : "NOK : La langue dans le squelette appelant est {$lang_skel} mais la langue dans charger() est " . $GLOBALS['spip_lang'],
-	];
+    return [
+        'message_ok' => $lang_skel === $GLOBALS['spip_lang'] ? 'OK' : "NOK : La langue dans le squelette appelant est {$lang_skel} mais la langue dans charger() est " . $GLOBALS['spip_lang'],
+    ];
 }

Spacing +1 added lines, -1 removed lines

@@ -4,6 +4,6 @@
 
 function formulaires_test_dyn_lang_charger_dist($lang_skel) {
 	return [
-		'message_ok' => $lang_skel === $GLOBALS['spip_lang'] ? 'OK' : "NOK : La langue dans le squelette appelant est {$lang_skel} mais la langue dans charger() est " . $GLOBALS['spip_lang'],
+		'message_ok' => $lang_skel === $GLOBALS['spip_lang'] ? 'OK' : "NOK : La langue dans le squelette appelant est {$lang_skel} mais la langue dans charger() est ".$GLOBALS['spip_lang'],
 	];
 }

ecrire/tests/Squelettes/Balise/data/squelettes/balise/formulaire_test_phraseur.php

Indentation +20 added lines, -20 removed lines

@@ -13,36 +13,36 @@
 \***************************************************************************/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 #securite
 
 function balise_FORMULAIRE_TEST_PHRASEUR($p) {
-	return calculer_balise_dynamique($p, 'FORMULAIRE_TEST_PHRASEUR', ['id_rubrique']);
+    return calculer_balise_dynamique($p, 'FORMULAIRE_TEST_PHRASEUR', ['id_rubrique']);
 }
 
 function balise_FORMULAIRE_TEST_PHRASEUR_stat($args, $context_compil) {
-	// le denier arg peut contenir l'url sur lequel faire le retour
-	// exemple dans un squelette article.html : [(#FORMULAIRE_FORUM{#SELF})]
+    // le denier arg peut contenir l'url sur lequel faire le retour
+    // exemple dans un squelette article.html : [(#FORMULAIRE_FORUM{#SELF})]
 
-	// recuperer les donnees du forum auquel on repond.
-	[$idr, $url] = $args;
+    // recuperer les donnees du forum auquel on repond.
+    [$idr, $url] = $args;
 
-	return [$idr, $url];
+    return [$idr, $url];
 }
 
 function balise_FORMULAIRE_TEST_PHRASEUR_dyn($id_rubrique, $url) {
-	$res = 'OK';
-
-	if (!preg_match('#^\d+$#', $id_rubrique)) {
-		$res = 'Erreur id_rubrique non numerique : ' . var_export($id_rubrique, true);
-	}
-
-	return [
-		'formulaires/test_phraseur',
-		0,
-		[
-			'result' => $res,
-		],
-	];
+    $res = 'OK';
+
+    if (!preg_match('#^\d+$#', $id_rubrique)) {
+        $res = 'Erreur id_rubrique non numerique : ' . var_export($id_rubrique, true);
+    }
+
+    return [
+        'formulaires/test_phraseur',
+        0,
+        [
+            'result' => $res,
+        ],
+    ];
 }

Spacing +1 added lines, -1 removed lines

@@ -35,7 +35,7 @@
 	$res = 'OK';
 
 	if (!preg_match('#^\d+$#', $id_rubrique)) {
-		$res = 'Erreur id_rubrique non numerique : ' . var_export($id_rubrique, true);
+		$res = 'Erreur id_rubrique non numerique : '.var_export($id_rubrique, true);
 	}
 
 	return [

ecrire/tests/Squelettes/Balise/BaliseDetacheeTest.php

Indentation +26 added lines, -26 removed lines

@@ -10,37 +10,37 @@
 
 class BaliseDetacheeTest extends SquelettesTestCase
 {
-	public function testNecessiteNomSite(): void {
-		$this->assertNotEmptyCode('<BOUCLE_meta(spip_meta){nom=nom_site}>#VALEUR</BOUCLE_meta>');
-	}
-
-	#[Depends('testNecessiteNomSite')]
-	public function testBaliseDetacheeInterne(): void {
-		$templating = Templating::fromString();
-		$expected = $templating->render('<BOUCLE_meta(spip_meta){nom=nom_site}>#VALEUR</BOUCLE_meta>');
-		$actual = $templating->render(
-			'<BOUCLE_meta(spip_meta){nom=nom_site}>
+    public function testNecessiteNomSite(): void {
+        $this->assertNotEmptyCode('<BOUCLE_meta(spip_meta){nom=nom_site}>#VALEUR</BOUCLE_meta>');
+    }
+
+    #[Depends('testNecessiteNomSite')]
+    public function testBaliseDetacheeInterne(): void {
+        $templating = Templating::fromString();
+        $expected = $templating->render('<BOUCLE_meta(spip_meta){nom=nom_site}>#VALEUR</BOUCLE_meta>');
+        $actual = $templating->render(
+            '<BOUCLE_meta(spip_meta){nom=nom_site}>
 				<BOUCLE_meta2(spip_meta){nom=adresse_site}>
 					#_meta:VALEUR
 				</BOUCLE_meta2>
 			</BOUCLE_meta>'
-		);
+        );
 
-		$this->assertEquals($expected, trim($actual));
-	}
+        $this->assertEquals($expected, trim($actual));
+    }
 
-	public function testBaliseDetacheeHorsBoucle(): void {
-		// en dehors de sa boucle, une balise detachee n'est pas reconnue
-		$this->assertEmptyCode(
-			'<BOUCLE_meta(spip_meta){nom=nom_site}></BOUCLE_meta>
+    public function testBaliseDetacheeHorsBoucle(): void {
+        // en dehors de sa boucle, une balise detachee n'est pas reconnue
+        $this->assertEmptyCode(
+            '<BOUCLE_meta(spip_meta){nom=nom_site}></BOUCLE_meta>
 			<BOUCLE_meta2(spip_meta){nom=version_base}>#_meta:VALEUR</BOUCLE_meta2>'
-		);
-	}
-
-	/**
-	 * @depends testBaliseDetacheeInterne
-	 */
-	public function testBaliseDetacheeComplexe(): void {
-		$this->assertOkSquelette(__DIR__ . '/data/balise_detachee.html');
-	}
+        );
+    }
+
+    /**
+     * @depends testBaliseDetacheeInterne
+     */
+    public function testBaliseDetacheeComplexe(): void {
+        $this->assertOkSquelette(__DIR__ . '/data/balise_detachee.html');
+    }
 }

Spacing +1 added lines, -1 removed lines

@@ -41,6 +41,6 @@
 	 * @depends testBaliseDetacheeInterne
 	 */
 	public function testBaliseDetacheeComplexe(): void {
-		$this->assertOkSquelette(__DIR__ . '/data/balise_detachee.html');
+		$this->assertOkSquelette(__DIR__.'/data/balise_detachee.html');
 	}
 }

ecrire/tests/Squelettes/Balise/InclureTest.php

Indentation +48 added lines, -48 removed lines

@@ -9,65 +9,65 @@
 
 class InclureTest extends SquelettesTestCase
 {
-	public function testInclureNormal(): void {
-		$dir = $this->relativePath(__DIR__);
-		$this->assertEqualsCode('Hello World', '<INCLURE{fond=' . $dir . '/data/texte_hello_world}>');
-		$this->assertEqualsCode('Hello World', '<INCLURE{fond=' . $dir . '/data/texte_hello_world}/>');
-	}
+    public function testInclureNormal(): void {
+        $dir = $this->relativePath(__DIR__);
+        $this->assertEqualsCode('Hello World', '<INCLURE{fond=' . $dir . '/data/texte_hello_world}>');
+        $this->assertEqualsCode('Hello World', '<INCLURE{fond=' . $dir . '/data/texte_hello_world}/>');
+    }
 
-	public function testInclureDouble(): void {
-		$dir = $this->relativePath(__DIR__);
-		$this->assertEqualsCode(
-			'Hello WorldHello World',
-			'<INCLURE{fond=' . $dir . '/data/texte_hello_world}>'
-			. '<INCLURE{fond=' . $dir . '/data/texte_hello_world}>'
-		);
-		$this->assertEqualsCode(
-			'Hello WorldHello World',
-			'
+    public function testInclureDouble(): void {
+        $dir = $this->relativePath(__DIR__);
+        $this->assertEqualsCode(
+            'Hello WorldHello World',
+            '<INCLURE{fond=' . $dir . '/data/texte_hello_world}>'
+            . '<INCLURE{fond=' . $dir . '/data/texte_hello_world}>'
+        );
+        $this->assertEqualsCode(
+            'Hello WorldHello World',
+            '
 			 <INCLURE{fond=' . $dir . '/data/texte_hello_world}>'
-			. '<INCLURE{fond=' . $dir . '/data/texte_hello_world}>'
-		);
-	}
+            . '<INCLURE{fond=' . $dir . '/data/texte_hello_world}>'
+        );
+    }
 
-	public function testInclureArray(): void {
-		$dir = $this->relativePath(__DIR__);
-		$array = '#LISTE{
+    public function testInclureArray(): void {
+        $dir = $this->relativePath(__DIR__);
+        $array = '#LISTE{
 			' . $dir . '/data/texte_hello_world,
 			' . $dir . '/data/texte_hello_world,
 			' . $dir . '/data/texte_hello_world}';
-		$this->assertEqualsCode('Hello WorldHello WorldHello World', "<INCLURE{fond={$array}}>");
-	}
+        $this->assertEqualsCode('Hello WorldHello WorldHello World', "<INCLURE{fond={$array}}>");
+    }
 
-	public function testInclureOldParam(): void {
-		$dir = $this->relativePath(__DIR__);
-		$this->assertEqualsCode('Kitty', '<INCLURE{fond=' . $dir . '/data/balise_env_test}{test=Kitty}>');
-		$this->assertEqualsCode('Kitty', '<INCLURE{fond=' . $dir . '/data/balise_env_test}{test=Kitty}/>');
-	}
+    public function testInclureOldParam(): void {
+        $dir = $this->relativePath(__DIR__);
+        $this->assertEqualsCode('Kitty', '<INCLURE{fond=' . $dir . '/data/balise_env_test}{test=Kitty}>');
+        $this->assertEqualsCode('Kitty', '<INCLURE{fond=' . $dir . '/data/balise_env_test}{test=Kitty}/>');
+    }
 
-	public function testInclureNormalParam(): void {
-		$dir = $this->relativePath(__DIR__);
-		$this->assertEqualsCode('Kitty', '<INCLURE{fond=' . $dir . '/data/balise_env_test, test=Kitty}>');
-		$this->assertEqualsCode('Kitty', '<INCLURE{fond=' . $dir . '/data/balise_env_test, test=Kitty}/>');
-	}
+    public function testInclureNormalParam(): void {
+        $dir = $this->relativePath(__DIR__);
+        $this->assertEqualsCode('Kitty', '<INCLURE{fond=' . $dir . '/data/balise_env_test, test=Kitty}>');
+        $this->assertEqualsCode('Kitty', '<INCLURE{fond=' . $dir . '/data/balise_env_test, test=Kitty}/>');
+    }
 
-	public function testInclureArrayParam(): void {
-		$dir = $this->relativePath(__DIR__);
-		$array = '#LISTE{
+    public function testInclureArrayParam(): void {
+        $dir = $this->relativePath(__DIR__);
+        $array = '#LISTE{
 			' . $dir . '/data/balise_env_test,
 			' . $dir . '/data/texte_hello_world,
 			' . $dir . '/data/balise_env_test}';
-		$this->assertEqualsCode('KittyHello WorldKitty', "<INCLURE{fond={$array}, test=Kitty}>");
-		$this->assertEqualsCode('KittyHello WorldKitty', "<INCLURE{fond={$array}, test=Kitty}/>");
-	}
+        $this->assertEqualsCode('KittyHello WorldKitty', "<INCLURE{fond={$array}, test=Kitty}>");
+        $this->assertEqualsCode('KittyHello WorldKitty', "<INCLURE{fond={$array}, test=Kitty}/>");
+    }
 
-	/**
-	 * Un inclure manquant doit creer une erreur de compilation pour SPIP qui ne doivent pas s'afficher dans le public si
-	 * visiteur
-	 */
-	public function testInclureManquantGenereErreurCompilation(): void {
-		$templating = Templating::fromString();
-		$infos = $templating->rawRender('<INCLURE{fond=carabistouille/de/tripoli/absente}/>ok');
-		$this->assertCount(1, $infos['erreurs']);
-	}
+    /**
+     * Un inclure manquant doit creer une erreur de compilation pour SPIP qui ne doivent pas s'afficher dans le public si
+     * visiteur
+     */
+    public function testInclureManquantGenereErreurCompilation(): void {
+        $templating = Templating::fromString();
+        $infos = $templating->rawRender('<INCLURE{fond=carabistouille/de/tripoli/absente}/>ok');
+        $this->assertCount(1, $infos['erreurs']);
+    }
 }

Spacing +16 added lines, -16 removed lines

@@ -11,52 +11,52 @@
 {
 	public function testInclureNormal(): void {
 		$dir = $this->relativePath(__DIR__);
-		$this->assertEqualsCode('Hello World', '<INCLURE{fond=' . $dir . '/data/texte_hello_world}>');
-		$this->assertEqualsCode('Hello World', '<INCLURE{fond=' . $dir . '/data/texte_hello_world}/>');
+		$this->assertEqualsCode('Hello World', '<INCLURE{fond='.$dir.'/data/texte_hello_world}>');
+		$this->assertEqualsCode('Hello World', '<INCLURE{fond='.$dir.'/data/texte_hello_world}/>');
 	}
 
 	public function testInclureDouble(): void {
 		$dir = $this->relativePath(__DIR__);
 		$this->assertEqualsCode(
 			'Hello WorldHello World',
-			'<INCLURE{fond=' . $dir . '/data/texte_hello_world}>'
-			. '<INCLURE{fond=' . $dir . '/data/texte_hello_world}>'
+			'<INCLURE{fond='.$dir.'/data/texte_hello_world}>'
+			. '<INCLURE{fond='.$dir.'/data/texte_hello_world}>'
 		);
 		$this->assertEqualsCode(
 			'Hello WorldHello World',
 			'
-			 <INCLURE{fond=' . $dir . '/data/texte_hello_world}>'
-			. '<INCLURE{fond=' . $dir . '/data/texte_hello_world}>'
+			 <INCLURE{fond=' . $dir.'/data/texte_hello_world}>'
+			. '<INCLURE{fond='.$dir.'/data/texte_hello_world}>'
 		);
 	}
 
 	public function testInclureArray(): void {
 		$dir = $this->relativePath(__DIR__);
 		$array = '#LISTE{
-			' . $dir . '/data/texte_hello_world,
-			' . $dir . '/data/texte_hello_world,
-			' . $dir . '/data/texte_hello_world}';
+			' . $dir.'/data/texte_hello_world,
+			' . $dir.'/data/texte_hello_world,
+			' . $dir.'/data/texte_hello_world}';
 		$this->assertEqualsCode('Hello WorldHello WorldHello World', "<INCLURE{fond={$array}}>");
 	}
 
 	public function testInclureOldParam(): void {
 		$dir = $this->relativePath(__DIR__);
-		$this->assertEqualsCode('Kitty', '<INCLURE{fond=' . $dir . '/data/balise_env_test}{test=Kitty}>');
-		$this->assertEqualsCode('Kitty', '<INCLURE{fond=' . $dir . '/data/balise_env_test}{test=Kitty}/>');
+		$this->assertEqualsCode('Kitty', '<INCLURE{fond='.$dir.'/data/balise_env_test}{test=Kitty}>');
+		$this->assertEqualsCode('Kitty', '<INCLURE{fond='.$dir.'/data/balise_env_test}{test=Kitty}/>');
 	}
 
 	public function testInclureNormalParam(): void {
 		$dir = $this->relativePath(__DIR__);
-		$this->assertEqualsCode('Kitty', '<INCLURE{fond=' . $dir . '/data/balise_env_test, test=Kitty}>');
-		$this->assertEqualsCode('Kitty', '<INCLURE{fond=' . $dir . '/data/balise_env_test, test=Kitty}/>');
+		$this->assertEqualsCode('Kitty', '<INCLURE{fond='.$dir.'/data/balise_env_test, test=Kitty}>');
+		$this->assertEqualsCode('Kitty', '<INCLURE{fond='.$dir.'/data/balise_env_test, test=Kitty}/>');
 	}
 
 	public function testInclureArrayParam(): void {
 		$dir = $this->relativePath(__DIR__);
 		$array = '#LISTE{
-			' . $dir . '/data/balise_env_test,
-			' . $dir . '/data/texte_hello_world,
-			' . $dir . '/data/balise_env_test}';
+			' . $dir.'/data/balise_env_test,
+			' . $dir.'/data/texte_hello_world,
+			' . $dir.'/data/balise_env_test}';
 		$this->assertEqualsCode('KittyHello WorldKitty', "<INCLURE{fond={$array}, test=Kitty}>");
 		$this->assertEqualsCode('KittyHello WorldKitty', "<INCLURE{fond={$array}, test=Kitty}/>");
 	}

ecrire/tests/Squelettes/Cache/CacheSessionTest.php

Indentation +119 added lines, -119 removed lines

@@ -10,123 +10,123 @@
 
 class CacheSessionTest extends SquelettesTestCase
 {
-	private static array $errors = [];
-	private static string $squelettes;
-
-	public static function setUpBeforeClass(): void {
-		self::$squelettes = self::relativePath(__DIR__ . '/data/squelettes');
-		$GLOBALS['dossier_squelettes'] = self::$squelettes;
-		$GLOBALS['delais'] = 3600; // See boostrap.php qui met delais = 0 (inhibe le cache)
-		include_spip('inc/invalideur');
-		purger_repertoire(_DIR_CACHE . 'calcul/', ['subdir' => true]);
-	}
-
-	public static function tearDownAfterClass(): void {
-		$GLOBALS['dossier_squelettes'] = '';
-		$GLOBALS['delais'] = 0;
-	}
-
-	protected function setUp(): void {
-		$this->resetErrors();
-	}
-
-	public function testVerifierPathMajInvalideurs(): void {
-		$this->assertEquals(self::$squelettes, $GLOBALS['dossier_squelettes'] ?? null);
-		$this->assertTrue(file_exists(__DIR__ . '/data/squelettes/inc/maj_invalideurs.php'));
-		$this->assertNotFalse(find_in_path('inc/maj_invalideurs.php'));
-		$this->assertNotFalse(include_spip('inc/maj_invalideurs'));
-		$this->assertEquals('inc_maj_invalideurs', charger_fonction('maj_invalideurs', 'inc', true));
-	}
-
-	/** Vérifier qu’on sait attraper les données de cache */
-	#[Depends('testVerifierPathMajInvalideurs')]
-	#[DataProvider('providerVerifierCaptureMajInvalideurs')]
-	public function testVerifierCaptureMajInvalideurs(int $expectedCountErrors, string $squelette, bool $session_attendue): void {
-		$this->runWithSquelette($squelette, $session_attendue);
-		$this->assertCount($expectedCountErrors, $this->getErrors(), $this->showErrors());
-	}
-
-	public static function providerVerifierCaptureMajInvalideurs(): array {
-		return [
-			[0, 'inclure/A_session_wo', false],
-			[2, 'inclure/A_session_wo', true],
-			[2, 'inclure/A_session_w', false],
-			[0, 'inclure/A_session_w', true],
-		];
-	}
-
-	#[Depends('testVerifierCaptureMajInvalideurs')]
-	#[DataProvider('providerCachesSessionnes')]
-	public function testCachesSessionnes(int $expectedCountErrors, string $squelette, bool $session_attendue): void {
-		$this->runWithSquelette($squelette, $session_attendue);
-		$this->assertCount($expectedCountErrors, $this->getErrors(), $this->showErrors());
-	}
-
-	public static function providerCachesSessionnes(): array {
-		return [
-			[0, 'cache_session_wo_1', false],
-			[0, 'cache_session_wo_2', false],
-			[0, 'cache_session_wo_3', false],
-			[0, 'cache_session_wo_4', false],
-			[0, 'cache_session_wo_5', false],
-			[0, 'cache_session_wo_6', false],
-			[0, 'cache_session_wo_7', false],
-			[0, 'cache_session_w_1', true],
-			[0, 'cache_session_w_2', true],
-			[0, 'cache_session_w_3', true],
-		];
-	}
-
-	private function runWithSquelette(string $fond, bool $session_attendue) {
-		unset($GLOBALS['cache_utilise_session']);
-		recuperer_fond($fond, [
-			'assert_session' => $session_attendue,
-			'caller' => 'none',
-			'salt' => $this->saltContext(),
-		]);
-		unset($GLOBALS['cache_utilise_session']);
-		recuperer_fond('root', [
-			'sousfond' => $fond,
-			'inc_assert_session' => $session_attendue,
-			'salt' => $this->saltContext(),
-		]);
-	}
-
-	private function saltContext(): string {
-		return (string) time() . ':' . uniqid();
-	}
-
-	private function getErrors(): array {
-		return self::$errors;
-	}
-
-	private function showErrors(): string {
-		return "Errors:\n" . implode("\n", array_map(fn ($e) => "- $e", self::getErrors()));
-	}
-
-	private function resetErrors(): void {
-		self::$errors = [];
-	}
-
-	public static function addError(string $msg, array $page): void {
-		self::$errors[] = sprintf(
-			'%s pour %s: %s',
-			$msg,
-			$page['source'],
-			self::trace_contexte($page['contexte'])
-		);
-	}
-
-	private static function trace_contexte(array $contexte): string {
-		foreach ($contexte as $k => $v) {
-			if (str_starts_with($k, 'date_') || $k === 'salt') {
-				unset($contexte[$k]);
-			}
-		}
-		if (isset($contexte['caller']) && str_starts_with($contexte['caller'], 'tests/squelettes/')) {
-			$contexte['caller'] = substr($contexte['caller'], 17);
-		}
-
-		return json_encode($contexte, JSON_THROW_ON_ERROR);
-	}
+    private static array $errors = [];
+    private static string $squelettes;
+
+    public static function setUpBeforeClass(): void {
+        self::$squelettes = self::relativePath(__DIR__ . '/data/squelettes');
+        $GLOBALS['dossier_squelettes'] = self::$squelettes;
+        $GLOBALS['delais'] = 3600; // See boostrap.php qui met delais = 0 (inhibe le cache)
+        include_spip('inc/invalideur');
+        purger_repertoire(_DIR_CACHE . 'calcul/', ['subdir' => true]);
+    }
+
+    public static function tearDownAfterClass(): void {
+        $GLOBALS['dossier_squelettes'] = '';
+        $GLOBALS['delais'] = 0;
+    }
+
+    protected function setUp(): void {
+        $this->resetErrors();
+    }
+
+    public function testVerifierPathMajInvalideurs(): void {
+        $this->assertEquals(self::$squelettes, $GLOBALS['dossier_squelettes'] ?? null);
+        $this->assertTrue(file_exists(__DIR__ . '/data/squelettes/inc/maj_invalideurs.php'));
+        $this->assertNotFalse(find_in_path('inc/maj_invalideurs.php'));
+        $this->assertNotFalse(include_spip('inc/maj_invalideurs'));
+        $this->assertEquals('inc_maj_invalideurs', charger_fonction('maj_invalideurs', 'inc', true));
+    }
+
+    /** Vérifier qu’on sait attraper les données de cache */
+    #[Depends('testVerifierPathMajInvalideurs')]
+    #[DataProvider('providerVerifierCaptureMajInvalideurs')]
+    public function testVerifierCaptureMajInvalideurs(int $expectedCountErrors, string $squelette, bool $session_attendue): void {
+        $this->runWithSquelette($squelette, $session_attendue);
+        $this->assertCount($expectedCountErrors, $this->getErrors(), $this->showErrors());
+    }
+
+    public static function providerVerifierCaptureMajInvalideurs(): array {
+        return [
+            [0, 'inclure/A_session_wo', false],
+            [2, 'inclure/A_session_wo', true],
+            [2, 'inclure/A_session_w', false],
+            [0, 'inclure/A_session_w', true],
+        ];
+    }
+
+    #[Depends('testVerifierCaptureMajInvalideurs')]
+    #[DataProvider('providerCachesSessionnes')]
+    public function testCachesSessionnes(int $expectedCountErrors, string $squelette, bool $session_attendue): void {
+        $this->runWithSquelette($squelette, $session_attendue);
+        $this->assertCount($expectedCountErrors, $this->getErrors(), $this->showErrors());
+    }
+
+    public static function providerCachesSessionnes(): array {
+        return [
+            [0, 'cache_session_wo_1', false],
+            [0, 'cache_session_wo_2', false],
+            [0, 'cache_session_wo_3', false],
+            [0, 'cache_session_wo_4', false],
+            [0, 'cache_session_wo_5', false],
+            [0, 'cache_session_wo_6', false],
+            [0, 'cache_session_wo_7', false],
+            [0, 'cache_session_w_1', true],
+            [0, 'cache_session_w_2', true],
+            [0, 'cache_session_w_3', true],
+        ];
+    }
+
+    private function runWithSquelette(string $fond, bool $session_attendue) {
+        unset($GLOBALS['cache_utilise_session']);
+        recuperer_fond($fond, [
+            'assert_session' => $session_attendue,
+            'caller' => 'none',
+            'salt' => $this->saltContext(),
+        ]);
+        unset($GLOBALS['cache_utilise_session']);
+        recuperer_fond('root', [
+            'sousfond' => $fond,
+            'inc_assert_session' => $session_attendue,
+            'salt' => $this->saltContext(),
+        ]);
+    }
+
+    private function saltContext(): string {
+        return (string) time() . ':' . uniqid();
+    }
+
+    private function getErrors(): array {
+        return self::$errors;
+    }
+
+    private function showErrors(): string {
+        return "Errors:\n" . implode("\n", array_map(fn ($e) => "- $e", self::getErrors()));
+    }
+
+    private function resetErrors(): void {
+        self::$errors = [];
+    }
+
+    public static function addError(string $msg, array $page): void {
+        self::$errors[] = sprintf(
+            '%s pour %s: %s',
+            $msg,
+            $page['source'],
+            self::trace_contexte($page['contexte'])
+        );
+    }
+
+    private static function trace_contexte(array $contexte): string {
+        foreach ($contexte as $k => $v) {
+            if (str_starts_with($k, 'date_') || $k === 'salt') {
+                unset($contexte[$k]);
+            }
+        }
+        if (isset($contexte['caller']) && str_starts_with($contexte['caller'], 'tests/squelettes/')) {
+            $contexte['caller'] = substr($contexte['caller'], 17);
+        }
+
+        return json_encode($contexte, JSON_THROW_ON_ERROR);
+    }
 }

Spacing +5 added lines, -5 removed lines

@@ -14,11 +14,11 @@ 
 	private static string $squelettes;
 
 	public static function setUpBeforeClass(): void {
-		self::$squelettes = self::relativePath(__DIR__ . '/data/squelettes');
+		self::$squelettes = self::relativePath(__DIR__.'/data/squelettes');
 		$GLOBALS['dossier_squelettes'] = self::$squelettes;
 		$GLOBALS['delais'] = 3600; // See boostrap.php qui met delais = 0 (inhibe le cache)
 		include_spip('inc/invalideur');
-		purger_repertoire(_DIR_CACHE . 'calcul/', ['subdir' => true]);
+		purger_repertoire(_DIR_CACHE.'calcul/', ['subdir' => true]);
 	}
 
 	public static function tearDownAfterClass(): void {
@@ -32,7 +32,7 @@ 
 
 	public function testVerifierPathMajInvalideurs(): void {
 		$this->assertEquals(self::$squelettes, $GLOBALS['dossier_squelettes'] ?? null);
-		$this->assertTrue(file_exists(__DIR__ . '/data/squelettes/inc/maj_invalideurs.php'));
+		$this->assertTrue(file_exists(__DIR__.'/data/squelettes/inc/maj_invalideurs.php'));
 		$this->assertNotFalse(find_in_path('inc/maj_invalideurs.php'));
 		$this->assertNotFalse(include_spip('inc/maj_invalideurs'));
 		$this->assertEquals('inc_maj_invalideurs', charger_fonction('maj_invalideurs', 'inc', true));
@@ -93,7 +93,7 @@ 
 	}
 
 	private function saltContext(): string {
-		return (string) time() . ':' . uniqid();
+		return (string) time().':'.uniqid();
 	}
 
 	private function getErrors(): array {
@@ -101,7 +101,7 @@ 
 	}
 
 	private function showErrors(): string {
-		return "Errors:\n" . implode("\n", array_map(fn ($e) => "- $e", self::getErrors()));
+		return "Errors:\n".implode("\n", array_map(fn ($e) => "- $e", self::getErrors()));
 	}
 
 	private function resetErrors(): void {

ecrire/tests/Squelettes/Modeles/DocumentsTest.php

Indentation +30 added lines, -30 removed lines

@@ -11,45 +11,45 @@
 
 class DocumentsTest extends SquelettesTestCase
 {
-	/**
-	 * D'abord une image uploadee en vignette, et sans titre
-	 * On teste le rendu de son src, width, height
-	 */
-	#[DataProvider('providerDocumentSansTitreModeles')]
-	public function testDocumentSansTitreModeles(string $modele): void {
-		$id_document = $this->getIdDocumentImageSansTitreNiDescriptif();
-		$modele = sprintf($modele, $id_document);
-		$this->assertOkCode(<<<SPIP
+    /**
+     * D'abord une image uploadee en vignette, et sans titre
+     * On teste le rendu de son src, width, height
+     */
+    #[DataProvider('providerDocumentSansTitreModeles')]
+    public function testDocumentSansTitreModeles(string $modele): void {
+        $id_document = $this->getIdDocumentImageSansTitreNiDescriptif();
+        $modele = sprintf($modele, $id_document);
+        $this->assertOkCode(<<<SPIP
 			[(#SET{modele,[(#ENV*{modele}|propre)]})]
 			[(#SET{src,#GET{modele}|extraire_balise{img}|extraire_attribut{src}})]
 			[(#GET{src}|quote_amp|=={#URL_DOCUMENT}|oui)OK ]
 			\<img#ID_DOCUMENT\> src pas bon: "#URL_DOCUMENT" != "[(#GET{src})]"
 			SPIP,
-			[
-				'id_document' => $id_document,
-				'modele' => $modele,
-			]
-		);
-	}
+            [
+                'id_document' => $id_document,
+                'modele' => $modele,
+            ]
+        );
+    }
 
-	public static function providerDocumentSansTitreModeles() {
-		return [
-			'img' => ['<img%s>'],
-			'doc' => ['<doc%s>'],
-			'emb' => ['<emb%s>'],
-		];
-	}
+    public static function providerDocumentSansTitreModeles() {
+        return [
+            'img' => ['<img%s>'],
+            'doc' => ['<doc%s>'],
+            'emb' => ['<emb%s>'],
+        ];
+    }
 
-	private function getIdDocumentImageSansTitreNiDescriptif(): int {
-		$templating = Templating::fromString();
-		$result = $templating->render(<<<SPIP
+    private function getIdDocumentImageSansTitreNiDescriptif(): int {
+        $templating = Templating::fromString();
+        $result = $templating->render(<<<SPIP
 		<BOUCLE_d(DOCUMENTS){mode=image}{titre=''}{descriptif=''}{0,1}>#ID_DOCUMENT</BOUCLE_d>
 		NA Ce test exige une image chargee en mode "image" et n'ayant ni titre ni descriptif
 		<//B_d>
 		SPIP);
-		if ($this->isNa($result)) {
-			$this->markTestSkipped($result);
-		}
-		return (int) $result;
-	}
+        if ($this->isNa($result)) {
+            $this->markTestSkipped($result);
+        }
+        return (int) $result;
+    }
 }

ecrire/tests/Squelettes/Boucle/BoucleJointuresTest.php

Indentation +72 added lines, -72 removed lines

@@ -9,84 +9,84 @@
 
 class BoucleJointuresTest extends SquelettesTestCase
 {
-	public function testJointureArticleIdmot1(): void {
-		// S'assurer que la seconde boucle est bien optimisée en l'absence de id_mot dans le env
-		$code1 = '<BOUCLE(ARTICLES){id_rubrique?}{par id_article}{0,3}>:#ID_ARTICLE:</BOUCLE>';
-		$code2 = '<BOUCLE(ARTICLES){id_rubrique?}{id_mot?}{par id_article}{0,3}>:#ID_ARTICLE:</BOUCLE>';
+    public function testJointureArticleIdmot1(): void {
+        // S'assurer que la seconde boucle est bien optimisée en l'absence de id_mot dans le env
+        $code1 = '<BOUCLE(ARTICLES){id_rubrique?}{par id_article}{0,3}>:#ID_ARTICLE:</BOUCLE>';
+        $code2 = '<BOUCLE(ARTICLES){id_rubrique?}{id_mot?}{par id_article}{0,3}>:#ID_ARTICLE:</BOUCLE>';
 
-		$templating = Templating::fromString();
-		$result1 = $templating->render($code1, ['id_mot' => 1]);
-		if (empty($result1)) {
-			$this->markTestSkipped("Pas d'articles dans la base pour tester la jointure id_mot");
-		}
-		$result2 = $templating->render($code2, ['id_mot' => 1]);
-		$this->assertNotEquals(
-			$result1,
-			$result2,
-			"La jointure conditionnelle {id_mot?} n'a pas d'effet si un id_mot est dans le contexte"
-		);
+        $templating = Templating::fromString();
+        $result1 = $templating->render($code1, ['id_mot' => 1]);
+        if (empty($result1)) {
+            $this->markTestSkipped("Pas d'articles dans la base pour tester la jointure id_mot");
+        }
+        $result2 = $templating->render($code2, ['id_mot' => 1]);
+        $this->assertNotEquals(
+            $result1,
+            $result2,
+            "La jointure conditionnelle {id_mot?} n'a pas d'effet si un id_mot est dans le contexte"
+        );
 
-		$templating = Templating::fromString();
-		$result1 = $templating->render($code1, []);
-		$result2 = $templating->render($code2, []);
-		$this->assertEquals(
-			$result1,
-			$result2,
-			"La jointure conditionnelle {id_mot?} n'est pas neutre en l'absence de id_mot dans le contexte"
-		);
-	}
+        $templating = Templating::fromString();
+        $result1 = $templating->render($code1, []);
+        $result2 = $templating->render($code2, []);
+        $this->assertEquals(
+            $result1,
+            $result2,
+            "La jointure conditionnelle {id_mot?} n'est pas neutre en l'absence de id_mot dans le contexte"
+        );
+    }
 
-	public function testJointureArticleIdmot2(): void {
-		// S'assurer que la seconde boucle est bien optimisée en l'absence de id_mot dans le env
-		$code1 = '<BOUCLE(ARTICLES){id_rubrique?}{par id_article}{0,3}>:#ID_ARTICLE+#ENV{id_mot}:</BOUCLE>';
-		$code2 = '<BOUCLE(ARTICLES){id_rubrique?}{id_mot?}{par id_article}{0,3}>:#ID_ARTICLE+#ENV{id_mot}:</BOUCLE>';
+    public function testJointureArticleIdmot2(): void {
+        // S'assurer que la seconde boucle est bien optimisée en l'absence de id_mot dans le env
+        $code1 = '<BOUCLE(ARTICLES){id_rubrique?}{par id_article}{0,3}>:#ID_ARTICLE+#ENV{id_mot}:</BOUCLE>';
+        $code2 = '<BOUCLE(ARTICLES){id_rubrique?}{id_mot?}{par id_article}{0,3}>:#ID_ARTICLE+#ENV{id_mot}:</BOUCLE>';
 
-		$templating = Templating::fromString();
-		$result1 = $templating->render($code1, ['id_mot' => 1]);
-		if (empty($result1)) {
-			$this->markTestSkipped("Pas d'articles dans la base pour tester la jointure id_mot");
-		}
-		$result2 = $templating->render($code2, ['id_mot' => 1]);
-		$this->assertNotEquals(
-			$result1,
-			$result2,
-			"La jointure conditionnelle {id_mot?} n'a pas d'effet si un id_mot est dans le contexte"
-		);
+        $templating = Templating::fromString();
+        $result1 = $templating->render($code1, ['id_mot' => 1]);
+        if (empty($result1)) {
+            $this->markTestSkipped("Pas d'articles dans la base pour tester la jointure id_mot");
+        }
+        $result2 = $templating->render($code2, ['id_mot' => 1]);
+        $this->assertNotEquals(
+            $result1,
+            $result2,
+            "La jointure conditionnelle {id_mot?} n'a pas d'effet si un id_mot est dans le contexte"
+        );
 
-		$templating = Templating::fromString();
-		$result1 = $templating->render($code1, []);
-		$result2 = $templating->render($code2, []);
-		$this->assertEquals(
-			$result1,
-			$result2,
-			"La jointure conditionnelle {id_mot?} n'est pas neutre en l'absence de id_mot dans le contexte"
-		);
-	}
+        $templating = Templating::fromString();
+        $result1 = $templating->render($code1, []);
+        $result2 = $templating->render($code2, []);
+        $this->assertEquals(
+            $result1,
+            $result2,
+            "La jointure conditionnelle {id_mot?} n'est pas neutre en l'absence de id_mot dans le contexte"
+        );
+    }
 
-	public function testJointureArticleIdmot3(): void {
-		// S'assurer que la seconde boucle conserve sa jointure sur id_mot du fait de l'utilisation de #ID_MOT dans la boucle
-		$code1 = '<BOUCLE(ARTICLES){id_rubrique?}{par id_article}{0,3}>:#ID_ARTICLE+#ID_MOT:</BOUCLE>';
-		$code2 = '<BOUCLE(ARTICLES){id_rubrique?}{id_mot?}{par id_article}{0,3}>:#ID_ARTICLE+#ID_MOT:</BOUCLE>';
+    public function testJointureArticleIdmot3(): void {
+        // S'assurer que la seconde boucle conserve sa jointure sur id_mot du fait de l'utilisation de #ID_MOT dans la boucle
+        $code1 = '<BOUCLE(ARTICLES){id_rubrique?}{par id_article}{0,3}>:#ID_ARTICLE+#ID_MOT:</BOUCLE>';
+        $code2 = '<BOUCLE(ARTICLES){id_rubrique?}{id_mot?}{par id_article}{0,3}>:#ID_ARTICLE+#ID_MOT:</BOUCLE>';
 
-		$templating = Templating::fromString();
-		$result1 = $templating->render($code1, ['id_mot' => 1]);
-		if (empty($result1)) {
-			$this->markTestSkipped("Pas d'articles dans la base pour tester la jointure id_mot");
-		}
-		$result2 = $templating->render($code2, ['id_mot' => 1]);
-		$this->assertNotEquals(
-			$result1,
-			$result2,
-			"La jointure conditionnelle {id_mot?} n'a pas d'effet si un id_mot est dans le contexte"
-		);
+        $templating = Templating::fromString();
+        $result1 = $templating->render($code1, ['id_mot' => 1]);
+        if (empty($result1)) {
+            $this->markTestSkipped("Pas d'articles dans la base pour tester la jointure id_mot");
+        }
+        $result2 = $templating->render($code2, ['id_mot' => 1]);
+        $this->assertNotEquals(
+            $result1,
+            $result2,
+            "La jointure conditionnelle {id_mot?} n'a pas d'effet si un id_mot est dans le contexte"
+        );
 
-		$templating = Templating::fromString();
-		$result1 = $templating->render($code1, []);
-		$result2 = $templating->render($code2, []);
-		$this->assertNotEquals(
-			$result1,
-			$result2,
-			"La jointure conditionnelle {id_mot?} n'a pas été conservée en l'absence de id_mot dans le contexte"
-		);
-	}
+        $templating = Templating::fromString();
+        $result1 = $templating->render($code1, []);
+        $result2 = $templating->render($code2, []);
+        $this->assertNotEquals(
+            $result1,
+            $result2,
+            "La jointure conditionnelle {id_mot?} n'a pas été conservée en l'absence de id_mot dans le contexte"
+        );
+    }
 }

ecrire/tests/Squelettes/Boucle/BoucleGeneriqueTest.php

Indentation +38 added lines, -38 removed lines

@@ -8,42 +8,42 @@
 
 class BoucleGeneriqueTest extends SquelettesTestCase
 {
-	public function testBoucleMetaSimple(): void {
-		$this->assertNotEmptyCode('<BOUCLE_meta(spip_meta)>#NOM</BOUCLE_meta>');
-		$this->assertOkCode('ok<BOUCLE_meta(spip_meta)> </BOUCLE_meta>');
-		$this->assertOkCode('<BOUCLE_meta(spip_meta)> </BOUCLE_meta>ok');
-	}
-
-	public function testBoucleMetaSimpleRaccourcisFinBoucle(): void {
-		$this->assertOkCode('<BOUCLE_meta(spip_meta) />ok');
-	}
-
-	public function testBoucleMetaSimpleAvantApres(): void {
-		$this->assertOkCode('<B_meta>ok<BOUCLE_meta(spip_meta)> </BOUCLE_meta>');
-		$this->assertOkCode('<BOUCLE_meta(spip_meta)> </BOUCLE_meta>ok</B_meta>');
-	}
-
-	public function testBoucleMetaSimpleSinon(): void {
-		$this->assertNotOkCode('<BOUCLE_meta(spip_meta)> </BOUCLE_meta>ok<//B_meta>');
-		$this->assertOkCode('<BOUCLE_meta(spip_meta)> </BOUCLE_meta>ok</B_meta>non<//B_meta>');
-
-		$this->assertOkCode('<BOUCLE_meta(spip_meta)></BOUCLE_meta>ok<//B_meta>');
-		$this->assertOkCode('<BOUCLE_meta(spip_meta)></BOUCLE_meta>non</B_meta>ok<//B_meta>');
-
-		$this->assertOkCode('<BOUCLE_meta(spip_meta) />ok<//B_meta>');
-		$this->assertOkCode('<BOUCLE_meta(spip_meta) />non</B_meta>ok<//B_meta>');
-	}
-
-	public function testBoucleMetaSimpleCritere(): void {
-		$this->assertEqualsCode($GLOBALS['meta']['nom_site'], '<BOUCLE_meta(spip_meta){nom=nom_site}>#VALEUR</BOUCLE_meta>');
-		$this->assertEmptyCode('<BOUCLE_meta(spip_meta){nom=gristinapolitainsic}>#VALEUR</BOUCLE_meta>');
-		$this->assertOkCode('<BOUCLE_meta(spip_meta){nom=gristinapolitainsic}>#VALEUR</BOUCLE_meta>ok<//B_meta>');
-	}
-
-	/**
-	 * @link http://trac.rezo.net/trac/spip/ticket/1931
-	 */
-	public function testBoucleVide(): void {
-		$this->assertOkSquelette(__DIR__ . '/data/boucle_vide.html');
-	}
+    public function testBoucleMetaSimple(): void {
+        $this->assertNotEmptyCode('<BOUCLE_meta(spip_meta)>#NOM</BOUCLE_meta>');
+        $this->assertOkCode('ok<BOUCLE_meta(spip_meta)> </BOUCLE_meta>');
+        $this->assertOkCode('<BOUCLE_meta(spip_meta)> </BOUCLE_meta>ok');
+    }
+
+    public function testBoucleMetaSimpleRaccourcisFinBoucle(): void {
+        $this->assertOkCode('<BOUCLE_meta(spip_meta) />ok');
+    }
+
+    public function testBoucleMetaSimpleAvantApres(): void {
+        $this->assertOkCode('<B_meta>ok<BOUCLE_meta(spip_meta)> </BOUCLE_meta>');
+        $this->assertOkCode('<BOUCLE_meta(spip_meta)> </BOUCLE_meta>ok</B_meta>');
+    }
+
+    public function testBoucleMetaSimpleSinon(): void {
+        $this->assertNotOkCode('<BOUCLE_meta(spip_meta)> </BOUCLE_meta>ok<//B_meta>');
+        $this->assertOkCode('<BOUCLE_meta(spip_meta)> </BOUCLE_meta>ok</B_meta>non<//B_meta>');
+
+        $this->assertOkCode('<BOUCLE_meta(spip_meta)></BOUCLE_meta>ok<//B_meta>');
+        $this->assertOkCode('<BOUCLE_meta(spip_meta)></BOUCLE_meta>non</B_meta>ok<//B_meta>');
+
+        $this->assertOkCode('<BOUCLE_meta(spip_meta) />ok<//B_meta>');
+        $this->assertOkCode('<BOUCLE_meta(spip_meta) />non</B_meta>ok<//B_meta>');
+    }
+
+    public function testBoucleMetaSimpleCritere(): void {
+        $this->assertEqualsCode($GLOBALS['meta']['nom_site'], '<BOUCLE_meta(spip_meta){nom=nom_site}>#VALEUR</BOUCLE_meta>');
+        $this->assertEmptyCode('<BOUCLE_meta(spip_meta){nom=gristinapolitainsic}>#VALEUR</BOUCLE_meta>');
+        $this->assertOkCode('<BOUCLE_meta(spip_meta){nom=gristinapolitainsic}>#VALEUR</BOUCLE_meta>ok<//B_meta>');
+    }
+
+    /**
+     * @link http://trac.rezo.net/trac/spip/ticket/1931
+     */
+    public function testBoucleVide(): void {
+        $this->assertOkSquelette(__DIR__ . '/data/boucle_vide.html');
+    }
 }

Spacing +1 added lines, -1 removed lines

@@ -44,6 +44,6 @@
 	 * @link http://trac.rezo.net/trac/spip/ticket/1931
 	 */
 	public function testBoucleVide(): void {
-		$this->assertOkSquelette(__DIR__ . '/data/boucle_vide.html');
+		$this->assertOkSquelette(__DIR__.'/data/boucle_vide.html');
 	}
 }

ecrire/tests/Squelettes/Boucle/BoucleRecursiveTest.php

Indentation +6 added lines, -6 removed lines

@@ -8,10 +8,10 @@
 
 class BoucleRecursiveTest extends SquelettesTestCase
 {
-	/**
-	 * @link http://trac.rezo.net/trac/spip/ticket/764
-	 */
-	public function testBoucleRecursiveSet(): void {
-		$this->assertOkSquelette(__DIR__ . '/data/bug764.html');
-	}
+    /**
+     * @link http://trac.rezo.net/trac/spip/ticket/764
+     */
+    public function testBoucleRecursiveSet(): void {
+        $this->assertOkSquelette(__DIR__ . '/data/bug764.html');
+    }
 }

Spacing +1 added lines, -1 removed lines

@@ -12,6 +12,6 @@
 	 * @link http://trac.rezo.net/trac/spip/ticket/764
 	 */
 	public function testBoucleRecursiveSet(): void {
-		$this->assertOkSquelette(__DIR__ . '/data/bug764.html');
+		$this->assertOkSquelette(__DIR__.'/data/bug764.html');
 	}
 }