spip / SPIP

ecrire/tests/Testing/TemplateTest.php

Indentation +9 added lines, -9 removed lines

@@ -9,14 +9,14 @@
 
 class TemplateTest extends SquelettesTestCase
 {
-	public function testNativeRecupererFond(): void {
-		$dir = $this->relativePath(__DIR__);
-		$this->assertEquals('Hello World', recuperer_fond($dir . '/data/texte_hello_world'));
-	}
+    public function testNativeRecupererFond(): void {
+        $dir = $this->relativePath(__DIR__);
+        $this->assertEquals('Hello World', recuperer_fond($dir . '/data/texte_hello_world'));
+    }
 
-	public function testRenderer(): void {
-		$dir = $this->relativePath(__DIR__);
-		$template = new Template($dir . '/data/texte_hello_world');
-		$this->assertEquals('Hello World', $template->render());
-	}
+    public function testRenderer(): void {
+        $dir = $this->relativePath(__DIR__);
+        $template = new Template($dir . '/data/texte_hello_world');
+        $this->assertEquals('Hello World', $template->render());
+    }
 }

Spacing +2 added lines, -2 removed lines

@@ -11,12 +11,12 @@
 {
 	public function testNativeRecupererFond(): void {
 		$dir = $this->relativePath(__DIR__);
-		$this->assertEquals('Hello World', recuperer_fond($dir . '/data/texte_hello_world'));
+		$this->assertEquals('Hello World', recuperer_fond($dir.'/data/texte_hello_world'));
 	}
 
 	public function testRenderer(): void {
 		$dir = $this->relativePath(__DIR__);
-		$template = new Template($dir . '/data/texte_hello_world');
+		$template = new Template($dir.'/data/texte_hello_world');
 		$this->assertEquals('Hello World', $template->render());
 	}
 }

ecrire/tests/Testing/TemplatingTest.php

Indentation +120 added lines, -120 removed lines

@@ -13,137 +13,137 @@
 
 class TemplatingTest extends SquelettesTestCase
 {
-	public function testFileLoader(): void {
-		$loader = new FileLoader();
-		$templating = new Templating($loader);
-		$this->assertInstanceOf(FileLoader::class, $templating->getLoader());
-		$this->assertEquals($loader, $templating->getLoader());
-
-		$file = __DIR__ . '/data/texte_hello_world.html';
-		$expected = trim(file_get_contents($file));
-
-		// Indirect render
-		$template = $templating->load($file);
-		$actual = $template->render();
-		$this->assertEquals($expected, $actual);
-
-		// Quick render
-		$actual = $templating->render($file);
-		$this->assertEquals($expected, $actual);
-
-		// Assert render
-		$this->assertEqualsTemplate($expected, $templating, $file);
-	}
-
-	public function testFileLoaderException(): void {
-		$templating = new Templating(new FileLoader());
-		$file = __DIR__ . '/data/inexistant_file.html';
-		$this->expectException(TemplateNotFoundException::class);
-		$templating->render($file);
-	}
-
-	public function testStringLoader(): void {
-		$templating = Templating::fromString();
-		$expected = 'Hello World';
-		$actual = $templating->render($expected);
-		$this->assertEquals($expected, $actual);
-	}
-
-	public function testChainLoader(): void {
-		$template = new Templating(new ChainLoader([new FileLoader(), new StringLoader()]));
-
-		$file = __DIR__ . '/data/texte_hello_world.html';
-		$expected = trim(file_get_contents($file));
-		$actual = $template->render($file);
-		$this->assertEquals($expected, $actual);
-
-		$string = 'Not a file';
-		$actual = $template->render($string);
-		$this->assertEquals($string, $actual);
-	}
-
-	public function testCodeRenderAvecFonctionEtApresCode(): void {
-		$loader = new StringLoader([
-			'fonctions' => "
+    public function testFileLoader(): void {
+        $loader = new FileLoader();
+        $templating = new Templating($loader);
+        $this->assertInstanceOf(FileLoader::class, $templating->getLoader());
+        $this->assertEquals($loader, $templating->getLoader());
+
+        $file = __DIR__ . '/data/texte_hello_world.html';
+        $expected = trim(file_get_contents($file));
+
+        // Indirect render
+        $template = $templating->load($file);
+        $actual = $template->render();
+        $this->assertEquals($expected, $actual);
+
+        // Quick render
+        $actual = $templating->render($file);
+        $this->assertEquals($expected, $actual);
+
+        // Assert render
+        $this->assertEqualsTemplate($expected, $templating, $file);
+    }
+
+    public function testFileLoaderException(): void {
+        $templating = new Templating(new FileLoader());
+        $file = __DIR__ . '/data/inexistant_file.html';
+        $this->expectException(TemplateNotFoundException::class);
+        $templating->render($file);
+    }
+
+    public function testStringLoader(): void {
+        $templating = Templating::fromString();
+        $expected = 'Hello World';
+        $actual = $templating->render($expected);
+        $this->assertEquals($expected, $actual);
+    }
+
+    public function testChainLoader(): void {
+        $template = new Templating(new ChainLoader([new FileLoader(), new StringLoader()]));
+
+        $file = __DIR__ . '/data/texte_hello_world.html';
+        $expected = trim(file_get_contents($file));
+        $actual = $template->render($file);
+        $this->assertEquals($expected, $actual);
+
+        $string = 'Not a file';
+        $actual = $template->render($string);
+        $this->assertEquals($string, $actual);
+    }
+
+    public function testCodeRenderAvecFonctionEtApresCode(): void {
+        $loader = new StringLoader([
+            'fonctions' => "
 				function so_smile(): string {
 					return ' So Smile';
 				}
 			",
-			'apres_code' => '[(#VAL|so_smile)]',
-		]);
-		$templating = new Templating($loader);
-		$this->assertEquals('Hello World So Smile', $templating->render('Hello World'));
+            'apres_code' => '[(#VAL|so_smile)]',
+        ]);
+        $templating = new Templating($loader);
+        $this->assertEquals('Hello World So Smile', $templating->render('Hello World'));
 
-		$templating = Templating::fromString([
-			'fonctions' => "
+        $templating = Templating::fromString([
+            'fonctions' => "
 				function so_smile(): string {
 					return ' So Smile';
 				}
 			",
-			'apres_code' => '[(#VAL|so_smile)]',
-		]);
-		$this->assertEquals('Hello World So Smile', $templating->render('Hello World'));
-	}
-
-	public function testCodeRenderAvecFonctionPrecedenteNonPresente(): void {
-		$template = Templating::fromString();
-		$this->assertNotEquals('Hello World So Smile', $template->render('Hello World'));
-		$this->assertEquals('Hello World', $template->render('Hello World'));
-		$this->assertNotEquals('Hello Kitty So Smile', $template->render('Hello Kitty'));
-		$this->assertEquals('Hello Kitty', $template->render('Hello Kitty'));
-	}
-
-	public function testCodeRender(): void {
-		$this->assertEqualsCode('Hello World', 'Hello World');
-	}
-
-	public function testCodeRenderAvecFonctionVide(): void {
-		// pas de fichier de fonctions
-		$this->assertOkCode("[(#SQUELETTE|replace{'.html','_fonctions.php'}|find_in_path|non)ok]");
-
-		// fichier de fonction
-		$templating = Templating::fromString([
-			'fonctions' => "
+            'apres_code' => '[(#VAL|so_smile)]',
+        ]);
+        $this->assertEquals('Hello World So Smile', $templating->render('Hello World'));
+    }
+
+    public function testCodeRenderAvecFonctionPrecedenteNonPresente(): void {
+        $template = Templating::fromString();
+        $this->assertNotEquals('Hello World So Smile', $template->render('Hello World'));
+        $this->assertEquals('Hello World', $template->render('Hello World'));
+        $this->assertNotEquals('Hello Kitty So Smile', $template->render('Hello Kitty'));
+        $this->assertEquals('Hello Kitty', $template->render('Hello Kitty'));
+    }
+
+    public function testCodeRender(): void {
+        $this->assertEqualsCode('Hello World', 'Hello World');
+    }
+
+    public function testCodeRenderAvecFonctionVide(): void {
+        // pas de fichier de fonctions
+        $this->assertOkCode("[(#SQUELETTE|replace{'.html','_fonctions.php'}|find_in_path|non)ok]");
+
+        // fichier de fonction
+        $templating = Templating::fromString([
+            'fonctions' => "
 				function so_smile(): string {
 					return ' So Smile';
 				}
 			",
-		]);
-		$this->assertOk($templating->render("[(#SQUELETTE|replace{'.html','_fonctions.php'}|find_in_path|oui)ok]"));
-
-		// pas de fichier de fonctions
-		$templating = Templating::fromString();
-		$this->assertNotOk($templating->render("[(#SQUELETTE|replace{'.html','_fonctions.php'}|find_in_path|oui)ok]"));
-	}
-
-	public function testCodeRenderAvantApres(): void {
-		$templating = Templating::fromString([
-			'avant_code' => 'Nice ',
-			'apres_code' => ' So Beautiful',
-		]);
-		$this->assertEquals('Nice Hello World So Beautiful', $templating->render('Hello World'));
-	}
-
-	public function testCodeRawRenderInfos(): void {
-		$templating = Templating::fromString();
-		$infos = $templating->rawRender('#SELF');
-		$this->assertTrue(is_array($infos));
-		$this->assertTrue(isset($infos['squelette']));
-		$this->assertTrue(isset($infos['fond']));
-	}
-
-	public function testCodeRawRenderInfosErreurCompilationFiltreAbsent(): void {
-		$templating = Templating::fromString();
-
-		$infos = $templating->rawRender('#CACHE{0}[(#SELF|ce_filtre_nexiste_pas)]');
-		$this->assertTrue(is_array($infos['erreurs']));
-		$this->assertCount(1, $infos['erreurs']);
-	}
-
-	public function testCodeRawRenderInfosErreurCompilationAbsentsDansNouvelleDemandeCorrecte(): void {
-		$templating = Templating::fromString();
-
-		$infos = $templating->rawRender('#CACHE{0}Aucun Probleme ici');
-		$this->assertCount(0, $infos['erreurs']);
-	}
+        ]);
+        $this->assertOk($templating->render("[(#SQUELETTE|replace{'.html','_fonctions.php'}|find_in_path|oui)ok]"));
+
+        // pas de fichier de fonctions
+        $templating = Templating::fromString();
+        $this->assertNotOk($templating->render("[(#SQUELETTE|replace{'.html','_fonctions.php'}|find_in_path|oui)ok]"));
+    }
+
+    public function testCodeRenderAvantApres(): void {
+        $templating = Templating::fromString([
+            'avant_code' => 'Nice ',
+            'apres_code' => ' So Beautiful',
+        ]);
+        $this->assertEquals('Nice Hello World So Beautiful', $templating->render('Hello World'));
+    }
+
+    public function testCodeRawRenderInfos(): void {
+        $templating = Templating::fromString();
+        $infos = $templating->rawRender('#SELF');
+        $this->assertTrue(is_array($infos));
+        $this->assertTrue(isset($infos['squelette']));
+        $this->assertTrue(isset($infos['fond']));
+    }
+
+    public function testCodeRawRenderInfosErreurCompilationFiltreAbsent(): void {
+        $templating = Templating::fromString();
+
+        $infos = $templating->rawRender('#CACHE{0}[(#SELF|ce_filtre_nexiste_pas)]');
+        $this->assertTrue(is_array($infos['erreurs']));
+        $this->assertCount(1, $infos['erreurs']);
+    }
+
+    public function testCodeRawRenderInfosErreurCompilationAbsentsDansNouvelleDemandeCorrecte(): void {
+        $templating = Templating::fromString();
+
+        $infos = $templating->rawRender('#CACHE{0}Aucun Probleme ici');
+        $this->assertCount(0, $infos['erreurs']);
+    }
 }

Spacing +3 added lines, -3 removed lines

@@ -19,7 +19,7 @@ 
 		$this->assertInstanceOf(FileLoader::class, $templating->getLoader());
 		$this->assertEquals($loader, $templating->getLoader());
 
-		$file = __DIR__ . '/data/texte_hello_world.html';
+		$file = __DIR__.'/data/texte_hello_world.html';
 		$expected = trim(file_get_contents($file));
 
 		// Indirect render
@@ -37,7 +37,7 @@ 
 
 	public function testFileLoaderException(): void {
 		$templating = new Templating(new FileLoader());
-		$file = __DIR__ . '/data/inexistant_file.html';
+		$file = __DIR__.'/data/inexistant_file.html';
 		$this->expectException(TemplateNotFoundException::class);
 		$templating->render($file);
 	}
@@ -52,7 +52,7 @@ 
 	public function testChainLoader(): void {
 		$template = new Templating(new ChainLoader([new FileLoader(), new StringLoader()]));
 
-		$file = __DIR__ . '/data/texte_hello_world.html';
+		$file = __DIR__.'/data/texte_hello_world.html';
 		$expected = trim(file_get_contents($file));
 		$actual = $template->render($file);
 		$this->assertEquals($expected, $actual);

ecrire/tests/Squelettes/Critere/OrigineTraductionTest.php

Indentation +18 added lines, -18 removed lines

@@ -11,27 +11,27 @@ 
 class OrigineTraductionTest extends SquelettesTestCase
 {
 
-	/** Un article non traduit est bien {origine_traduction} */
-	public function testArticleTraduitEstOrigineTraduction(): void {
-		$templating = Templating::fromString();
-		$result = $templating->render(<<<SPIP
+    /** Un article non traduit est bien {origine_traduction} */
+    public function testArticleTraduitEstOrigineTraduction(): void {
+        $templating = Templating::fromString();
+        $result = $templating->render(<<<SPIP
 		<BOUCLE_t(ARTICLES)/>[(#TOTAL_BOUCLE|<{2}|?{NA il faut des articles})]<//B_t>
 		<BOUCLE_ori(ARTICLES){origine_traduction}{id_trad=0}{0,1}> </BOUCLE_ori>
 		Erreur boucle origine
 		<//B_ori>
 		ok
 		SPIP);
-		if ($this->isNa($result)) {
-			$this->markTestSkipped($result);
-		}
+        if ($this->isNa($result)) {
+            $this->markTestSkipped($result);
+        }
 
-		$this->assertOk($result);
-	}
+        $this->assertOk($result);
+    }
 
-	/** Un article traduit n'a qu'une traduction qui est {origine_traduction} */
-	public function testArticleTraduitAUneSeuleTraductionOrigineTraduction(): void {
-		$templating = Templating::fromString();
-		$result = $templating->render(<<<SPIP
+    /** Un article traduit n'a qu'une traduction qui est {origine_traduction} */
+    public function testArticleTraduitAUneSeuleTraductionOrigineTraduction(): void {
+        $templating = Templating::fromString();
+        $result = $templating->render(<<<SPIP
 		<BOUCLE_ori2(ARTICLES){id_trad>0}{origine_traduction}>
 		<BOUCLE_casse(ARTICLES){traduction}{origine_traduction}{!id_article}>
 			Boum ! #ID_ARTICLE ne devrait pas etre origine
@@ -41,10 +41,10 @@ 
 		<//B_ori2>
 		OK
 		SPIP);
-		if ($this->isNa($result)) {
-			$this->markTestSkipped($result);
-		}
+        if ($this->isNa($result)) {
+            $this->markTestSkipped($result);
+        }
 
-		$this->assertOk($result);
-	}
+        $this->assertOk($result);
+    }
 }

ecrire/tests/Squelettes/Critere/DoublonsTest.php

Indentation +25 added lines, -25 removed lines

@@ -10,58 +10,58 @@
 
 class DoublonsTest extends SquelettesTestCase
 {
-	public function testCritereDoublons(): void {
-		$this->assertOkCode(
-			'
+    public function testCritereDoublons(): void {
+        $this->assertOkCode(
+            '
 			<BOUCLE_a(ARTICLES){statut=.*}{doublons}></BOUCLE_a>
 			<BOUCLE_b(ARTICLES){statut=.*}{tout}{doublons}{0,1}>Erreur doublons articles</BOUCLE_b>
 			ok<//B_b>
 			'
-		);
-		$this->assertOkCode(
-			'
+        );
+        $this->assertOkCode(
+            '
 			<BOUCLE_a(ARTICLES){doublons}></BOUCLE_a>
 			<BOUCLE_b(ARTICLES){doublons}{0,1}>Erreur doublons articles</BOUCLE_b>
 			ok<//B_b>
 			'
-		);
-	}
+        );
+    }
 
-	public function testCritereDoublonsMultiplesBoucles(): void {
-		$this->assertOkSquelette(__DIR__ . '/data/doublons.html');
-	}
+    public function testCritereDoublonsMultiplesBoucles(): void {
+        $this->assertOkSquelette(__DIR__ . '/data/doublons.html');
+    }
 
-	public function testDoublonsAuteurs(): void {
-		$templating = Templating::fromString();
-		$n = $templating->render('<BOUCLE_a(AUTEURS)>#COMPTEUR_BOUCLE</BOUCLE_a>');
+    public function testDoublonsAuteurs(): void {
+        $templating = Templating::fromString();
+        $n = $templating->render('<BOUCLE_a(AUTEURS)>#COMPTEUR_BOUCLE</BOUCLE_a>');
 
-		if ($n <= 1) {
-			$this->markTestSkipped('Il faut au moins 2 auteurs ayant publie sur le site !');
-		}
+        if ($n <= 1) {
+            $this->markTestSkipped('Il faut au moins 2 auteurs ayant publie sur le site !');
+        }
 
-		$this->assertOkCode('
+        $this->assertOkCode('
 			<BOUCLE_a(AUTEURS){doublons}></BOUCLE_a>
 			<BOUCLE_b(AUTEURS){doublons}{0,1}>Erreur doublons Auteurs</BOUCLE_b>OK<//B_b>
 		');
-	}
+    }
 
-	#[Depends('testDoublonsAuteurs')]
-	public function testDoublonsNommesAuteurs(): void {
-		$this->assertOkCode('
+    #[Depends('testDoublonsAuteurs')]
+    public function testDoublonsNommesAuteurs(): void {
+        $this->assertOkCode('
 			<BOUCLE_a(AUTEURS){doublons polisson}></BOUCLE_a>
 			<BOUCLE_b(AUTEURS){doublons polisson}{0,1}>Erreur doublons Auteurs</BOUCLE_b>OK<//B_b>
 		');
-		$this->assertOkCode('
+        $this->assertOkCode('
 			<BOUCLE_a(AUTEURS){1/2}{doublons kakis}></BOUCLE_a>
 			<BOUCLE_b(AUTEURS){2/2}{doublons kokos}></BOUCLE_b>
 			<BOUCLE_c(AUTEURS){doublons kakis}{0,1}>ok</BOUCLE_c>Erreur doubles doublons Auteurs<//B_c>
 		');
-		$this->assertOkCode('
+        $this->assertOkCode('
 			<BOUCLE_a(AUTEURS){1/2}{doublons kakis}></BOUCLE_a>
 			<BOUCLE_b(AUTEURS){2/2}{doublons kokos}></BOUCLE_b>
 			<BOUCLE_c(AUTEURS){doublons kakis}{0,1}> </BOUCLE_c>Erreur doubles doublons Auteurs<//B_c>
 			<BOUCLE_d(AUTEURS){doublons kokos}{0,1}> </BOUCLE_d>Erreur doubles doublons Auteurs<//B_d>
 			<BOUCLE_e(AUTEURS){doublons kakis}{doublons kokos}{0,1}>Erreur doubles doublons Auteurs</BOUCLE_e>OK<//B_e>
 		');
-	}
+    }
 }

Spacing +1 added lines, -1 removed lines

@@ -28,7 +28,7 @@
 	}
 
 	public function testCritereDoublonsMultiplesBoucles(): void {
-		$this->assertOkSquelette(__DIR__ . '/data/doublons.html');
+		$this->assertOkSquelette(__DIR__.'/data/doublons.html');
 	}
 
 	public function testDoublonsAuteurs(): void {

ecrire/tests/Squelettes/Critere/TraductionTest.php

Indentation +18 added lines, -18 removed lines

@@ -11,10 +11,10 @@ 
 class TraductionTest extends SquelettesTestCase
 {
 
-	/** Un article sans trad */
-	public function testArticleSansTraduction(): void {
-		$templating = Templating::fromString();
-		$result = $templating->render(<<<SPIP
+    /** Un article sans trad */
+    public function testArticleSansTraduction(): void {
+        $templating = Templating::fromString();
+        $result = $templating->render(<<<SPIP
 		<BOUCLE_principale(ARTICLES){id_trad=0}{0,1}>
 		<BOUCLE_check(ARTICLES){traduction}> </BOUCLE_check>
 			boucle check: le critere {traduction} a echoue
@@ -25,17 +25,17 @@ 
 		<//B_principale>
 		ok
 		SPIP);
-		if ($this->isNa($result)) {
-			$this->markTestSkipped($result);
-		}
+        if ($this->isNa($result)) {
+            $this->markTestSkipped($result);
+        }
 
-		$this->assertOk($result);
-	}
+        $this->assertOk($result);
+    }
 
-	/** un article et ses traductions */
-	public function testArticleAvecTraductions(): void {
-		$templating = Templating::fromString();
-		$result = $templating->render(<<<SPIP
+    /** un article et ses traductions */
+    public function testArticleAvecTraductions(): void {
+        $templating = Templating::fromString();
+        $result = $templating->render(<<<SPIP
 		<BOUCLE_s(ARTICLES){id_trad>0}{0,1}>
 		<BOUCLE_t(ARTICLES){traduction}> </BOUCLE_t>
 		</BOUCLE_s>
@@ -43,10 +43,10 @@ 
 		<//B_s>
 		ok
 		SPIP);
-		if ($this->isNa($result)) {
-			$this->markTestSkipped($result);
-		}
+        if ($this->isNa($result)) {
+            $this->markTestSkipped($result);
+        }
 
-		$this->assertOk($result);
-	}
+        $this->assertOk($result);
+    }
 }

ecrire/tests/Squelettes/Critere/BrancheTest.php

Indentation +16 added lines, -16 removed lines

@@ -10,18 +10,18 @@ 
 
 class BrancheTest extends SquelettesTestCase
 {
-	/**
-	 * 	Un test pour le critere {branche}
-	 *
-	 * 	verifie :
-	 * 	- une rubrique est dans sa branche
-	 * 	- sa fille est dans sa branche
-	 * 	- elle n'est pas dans la branche de sa fille
-	 *  - que la boucle documents compile sans erreur
-	 * 	- que la boucle articles compile sans erreur
-	 */
-	public function testCritereBranche(): void {
-		$this->assertOkCode(<<<SPIP
+    /**
+     * 	Un test pour le critere {branche}
+     *
+     * 	verifie :
+     * 	- une rubrique est dans sa branche
+     * 	- sa fille est dans sa branche
+     * 	- elle n'est pas dans la branche de sa fille
+     *  - que la boucle documents compile sans erreur
+     * 	- que la boucle articles compile sans erreur
+     */
+    public function testCritereBranche(): void {
+        $this->assertOkCode(<<<SPIP
 		<BOUCLE_a(RUBRIQUES){id_parent>0}>
 			<BOUCLE_b(RUBRIQUES){id_rubrique=#ID_PARENT}>
 				<BOUCLE_d(RUBRIQUES){branche}{id_rubrique}>
@@ -40,13 +40,13 @@ 
 		</BOUCLE_a>
 		ok
 		SPIP);
-	}
+    }
 
-	public function testCompileBoucle(): void {
-		$this->assertOkCode(<<<SPIP
+    public function testCompileBoucle(): void {
+        $this->assertOkCode(<<<SPIP
 		<BOUCLE_art(ARTICLES){branche}{0,1}> </BOUCLE_art>
 		<BOUCLE_docs(DOCUMENTS){branche}{0,1}> </BOUCLE_docs>
 		OK
 		SPIP);
-	}
+    }
 }

ecrire/tests/Squelettes/Critere/DoublonsNotesTest.php

Indentation +47 added lines, -47 removed lines

@@ -10,19 +10,19 @@ 
 
 class DoublonsNotesTest extends SquelettesTestCase
 {
-	public static function tearDownAfterClass(): void {
-		sql_delete('spip_articles', 'id_article = -1');
-	}
+    public static function tearDownAfterClass(): void {
+        sql_delete('spip_articles', 'id_article = -1');
+    }
 
-	/**
-	 * On cherche un article avec un document en note dans le texte,
-	 * et on veut qu'il soit pris par {doublons}
-	 * cf. https://git.spip.net/spip/spip/issues/779
-	 */
-	public function testCritereDoublonsNotes(): void {
-		$id_document = $this->creer_article_a_doublons_notes();
+    /**
+     * On cherche un article avec un document en note dans le texte,
+     * et on veut qu'il soit pris par {doublons}
+     * cf. https://git.spip.net/spip/spip/issues/779
+     */
+    public function testCritereDoublonsNotes(): void {
+        $id_document = $this->creer_article_a_doublons_notes();
 
-		$this->assertOkCode(<<<SPIP
+        $this->assertOkCode(<<<SPIP
 			<BOUCLE_d(DOCUMENTS){id_document=#ENV{id_document}}{statut==.*}>
 				<BOUCLE_a(ARTICLES){id_article=-1}{statut==.*}>[(#TEXTE|?)]</BOUCLE_a>
 				<BOUCLE_test(DOCUMENTS){id_document}{doublons}>
@@ -34,41 +34,41 @@ 
 				erreur, pas de document
 			<//B_d>
 			SPIP,
-			[
-				'id_article' => -1,
-				'id_document' => $id_document,
-			]
-		);
-	}
+            [
+                'id_article' => -1,
+                'id_document' => $id_document,
+            ]
+        );
+    }
 
-	/**
-	 * Creation article de test pour doublons_notes.html
-	 * On cherche un document, on le met dans la note d'un texte,
-	 * @return int id_document
-	 */
-	private function creer_article_a_doublons_notes(): int {
-		$id_document = sql_getfetsel(
-			'id_document',
-			'spip_documents',
-			sql_in('mode', ['logoon','logooff','vignette'], 'not'),
-			orderby: 'rand()',
-			limit: '0,1'
-		);
-		if (!$id_document) {
-			$this->markTestSkipped('Il faut un document');
-		}
-		$data = [
-			'id_article' => -1,
-			'titre' => 'test pour doublons_notes.html',
-			'statut' => 'prepa',
-			'texte' => 'hello [[ xx <doc' . $id_document . '> ]].'
-		];
-		$id_article = sql_getfetsel('id_article', 'spip_articles', 'id_article = -1');
-		if ($id_article === null) {
-			sql_insertq('spip_articles', $data);
-		} else {
-			sql_updateq('spip_articles', $data, ['id_article = -1']);
-		}
-		return $id_document;
-	}
+    /**
+     * Creation article de test pour doublons_notes.html
+     * On cherche un document, on le met dans la note d'un texte,
+     * @return int id_document
+     */
+    private function creer_article_a_doublons_notes(): int {
+        $id_document = sql_getfetsel(
+            'id_document',
+            'spip_documents',
+            sql_in('mode', ['logoon','logooff','vignette'], 'not'),
+            orderby: 'rand()',
+            limit: '0,1'
+        );
+        if (!$id_document) {
+            $this->markTestSkipped('Il faut un document');
+        }
+        $data = [
+            'id_article' => -1,
+            'titre' => 'test pour doublons_notes.html',
+            'statut' => 'prepa',
+            'texte' => 'hello [[ xx <doc' . $id_document . '> ]].'
+        ];
+        $id_article = sql_getfetsel('id_article', 'spip_articles', 'id_article = -1');
+        if ($id_article === null) {
+            sql_insertq('spip_articles', $data);
+        } else {
+            sql_updateq('spip_articles', $data, ['id_article = -1']);
+        }
+        return $id_document;
+    }
 }

Spacing +2 added lines, -2 removed lines

@@ -50,7 +50,7 @@ 
 		$id_document = sql_getfetsel(
 			'id_document',
 			'spip_documents',
-			sql_in('mode', ['logoon','logooff','vignette'], 'not'),
+			sql_in('mode', ['logoon', 'logooff', 'vignette'], 'not'),
 			orderby: 'rand()',
 			limit: '0,1'
 		);
@@ -61,7 +61,7 @@ 
 			'id_article' => -1,
 			'titre' => 'test pour doublons_notes.html',
 			'statut' => 'prepa',
-			'texte' => 'hello [[ xx <doc' . $id_document . '> ]].'
+			'texte' => 'hello [[ xx <doc'.$id_document.'> ]].'
 		];
 		$id_article = sql_getfetsel('id_article', 'spip_articles', 'id_article = -1');
 		if ($id_article === null) {

ecrire/tests/Squelettes/Critere/OperatorRegexpLikeTest.php

Indentation +50 added lines, -50 removed lines

@@ -10,83 +10,83 @@
 
 class OperatorRegexpLikeTest extends SquelettesTestCase
 {
-	private function getArticle(): string {
-		$templating = Templating::fromString();
-		return $templating->render(<<<SPIP
+    private function getArticle(): string {
+        $templating = Templating::fromString();
+        return $templating->render(<<<SPIP
 		<BOUCLE_a(ARTICLES){titre>=A}{titre<=Z}{0,1}>#ID_ARTICLE:[(#TITRE|substr{0,1})]</BOUCLE_a>
 		NA Ce test exige un article ayant un titre qui commence par une lettre A-Z
 		<//B_a>
 		SPIP);
-	}
+    }
 
-	/** @return array{id_article: int, starts_with: string} */
-	private function getArticleIdTitle(): array {
-		$result = $this->getArticle();
-		[$id_article, $starts_with] = explode(':', trim($result));
-		return [
-			'id_article' => (int) $id_article,
-			'starts_with' => $starts_with
-		];
-	}
+    /** @return array{id_article: int, starts_with: string} */
+    private function getArticleIdTitle(): array {
+        $result = $this->getArticle();
+        [$id_article, $starts_with] = explode(':', trim($result));
+        return [
+            'id_article' => (int) $id_article,
+            'starts_with' => $starts_with
+        ];
+    }
 
-	public function testHasArticle(): void {
-		$result = $this->getArticle();
-		if ($this->isNa($result)) {
-			$this->markTestSkipped($result);
-		}
-		[$id_article, $starts_with] = explode(':', trim($result));
-		$id_article = (int) $id_article;
+    public function testHasArticle(): void {
+        $result = $this->getArticle();
+        if ($this->isNa($result)) {
+            $this->markTestSkipped($result);
+        }
+        [$id_article, $starts_with] = explode(':', trim($result));
+        $id_article = (int) $id_article;
 
-		$this->assertGreaterThan(0, $id_article);
-	}
+        $this->assertGreaterThan(0, $id_article);
+    }
 
-	#[Depends('testHasArticle')]
-	public function testLike(): void {
-		$art = $this->getArticleIdTitle();
-		$contexte = [
-			'id_article' => $art['id_article'],
-			'like' => $art['starts_with'] . '%',
-		];
-		$this->assertOkCode(<<<SPIP
+    #[Depends('testHasArticle')]
+    public function testLike(): void {
+        $art = $this->getArticleIdTitle();
+        $contexte = [
+            'id_article' => $art['id_article'],
+            'like' => $art['starts_with'] . '%',
+        ];
+        $this->assertOkCode(<<<SPIP
 			<BOUCLE_b(ARTICLES){titre like #ENV{like}}{id_article}>ok</BOUCLE_b>
 			Echec de {titre like #ENV{like}}
 			<//B_b>
 			SPIP,
-			$contexte
-		);
-		$this->assertOkCode(<<<SPIP
+            $contexte
+        );
+        $this->assertOkCode(<<<SPIP
 			<BOUCLE_c(ARTICLES){titre !like #ENV{like}}{id_article}> </BOUCLE_c>
 			Echec de {titre !like #ENV{like}}
 			</B_c>
 			ok
 			<//B_c>
 			SPIP,
-			$contexte
-		);
-	}
+            $contexte
+        );
+    }
 
-	#[Depends('testHasArticle')]
-	public function testRegexp(): void {
-		$art = $this->getArticleIdTitle();
-		$contexte = [
-			'id_article' => $art['id_article'],
-			'regexp' => '^' . $art['starts_with'],
-		];
-		$this->assertOkCode(<<<SPIP
+    #[Depends('testHasArticle')]
+    public function testRegexp(): void {
+        $art = $this->getArticleIdTitle();
+        $contexte = [
+            'id_article' => $art['id_article'],
+            'regexp' => '^' . $art['starts_with'],
+        ];
+        $this->assertOkCode(<<<SPIP
 			<BOUCLE_b(ARTICLES){titre == #ENV{regexp}}{id_article}>ok</BOUCLE_b>
 			Echec de {titre == #ENV{regexp}}
 			<//B_b>
 			SPIP,
-			$contexte
-		);
-		$this->assertOkCode(<<<SPIP
+            $contexte
+        );
+        $this->assertOkCode(<<<SPIP
 			<BOUCLE_c(ARTICLES){titre !== #ENV{regexp}}{id_article}> </BOUCLE_c>
 			Echec de {titre !== #GET{regexp}}
 			</B_c>
 			ok
 			<//B_c>
 			SPIP,
-			$contexte
-		);
-	}
+            $contexte
+        );
+    }
 }

Spacing +2 added lines, -2 removed lines

@@ -45,7 +45,7 @@ 
 		$art = $this->getArticleIdTitle();
 		$contexte = [
 			'id_article' => $art['id_article'],
-			'like' => $art['starts_with'] . '%',
+			'like' => $art['starts_with'].'%',
 		];
 		$this->assertOkCode(<<<SPIP
 			<BOUCLE_b(ARTICLES){titre like #ENV{like}}{id_article}>ok</BOUCLE_b>
@@ -70,7 +70,7 @@ 
 		$art = $this->getArticleIdTitle();
 		$contexte = [
 			'id_article' => $art['id_article'],
-			'regexp' => '^' . $art['starts_with'],
+			'regexp' => '^'.$art['starts_with'],
 		];
 		$this->assertOkCode(<<<SPIP
 			<BOUCLE_b(ARTICLES){titre == #ENV{regexp}}{id_article}>ok</BOUCLE_b>

ecrire/tests/Squelettes/Filtre/IntroductionTest.php

Indentation +54 added lines, -54 removed lines

@@ -8,65 +8,65 @@
 
 class IntroductionTest extends SquelettesTestCase
 {
-	public static function setUpBeforeClass(): void {
-		include_spip('inc/filtres');
-		include_spip('public/composer');
-	}
+    public static function setUpBeforeClass(): void {
+        include_spip('inc/filtres');
+        include_spip('public/composer');
+    }
 
-	public function testPresenceFiltre(): void {
-		$introduction = $this->getFilterIntroduction();
-		if ($introduction !== 'filtre_introduction_dist') {
-			$this->markTestIncomplete(sprintf("Careful: filter %s is not 'filtre_introduction_dist'", $introduction));
-		}
+    public function testPresenceFiltre(): void {
+        $introduction = $this->getFilterIntroduction();
+        if ($introduction !== 'filtre_introduction_dist') {
+            $this->markTestIncomplete(sprintf("Careful: filter %s is not 'filtre_introduction_dist'", $introduction));
+        }
 
-		$this->assertEquals('<p>ok</p>', propre('ok'));
-		$this->assertEquals('<p>ok</p>', $introduction('ok', '', 100, ''));
-	}
+        $this->assertEquals('<p>ok</p>', propre('ok'));
+        $this->assertEquals('<p>ok</p>', $introduction('ok', '', 100, ''));
+    }
 
-	/**
-	 * la description seule ressort avec propre() sans passer par couper() or couper() enleve les balises <p> et consoeur,
-	 * il faut en tenir compte dans la coupe du texte, meme si le texte est plus petit
-	 */
-	public function testDescriptifRetourneSiPresent(): void {
-		$introduction = $this->getFilterIntroduction();
-		$this->assertEquals(
-			propre('description petite'),
-			$introduction('description petite', 'description plus longue', 100, '')
-		);
-	}
+    /**
+     * la description seule ressort avec propre() sans passer par couper() or couper() enleve les balises <p> et consoeur,
+     * il faut en tenir compte dans la coupe du texte, meme si le texte est plus petit
+     */
+    public function testDescriptifRetourneSiPresent(): void {
+        $introduction = $this->getFilterIntroduction();
+        $this->assertEquals(
+            propre('description petite'),
+            $introduction('description petite', 'description plus longue', 100, '')
+        );
+    }
 
-	/**
-	 * couper en plus...
-	 */
-	public function testTexteNonCoupeSiPetit(): void {
-		$introduction = $this->getFilterIntroduction();
-		$this->assertEquals(
-			paragrapher(couper(propre('description plus longue'), 100), true),
-			$introduction('', 'description plus longue', 100, '')
-		);
-	}
+    /**
+     * couper en plus...
+     */
+    public function testTexteNonCoupeSiPetit(): void {
+        $introduction = $this->getFilterIntroduction();
+        $this->assertEquals(
+            paragrapher(couper(propre('description plus longue'), 100), true),
+            $introduction('', 'description plus longue', 100, '')
+        );
+    }
 
-	public function testTexteCoupe(): void {
-		$introduction = $this->getFilterIntroduction();
-		$this->assertEquals(
-			paragrapher(couper(propre('description plus longue'), 10), true),
-			$introduction('', 'description plus longue', 10, '')
-		);
-		$this->assertNotEquals(
-			paragrapher(couper(propre('description plus longue'), 20), true),
-			$introduction('', 'description plus longue', 10, '')
-		);
-	}
+    public function testTexteCoupe(): void {
+        $introduction = $this->getFilterIntroduction();
+        $this->assertEquals(
+            paragrapher(couper(propre('description plus longue'), 10), true),
+            $introduction('', 'description plus longue', 10, '')
+        );
+        $this->assertNotEquals(
+            paragrapher(couper(propre('description plus longue'), 20), true),
+            $introduction('', 'description plus longue', 10, '')
+        );
+    }
 
-	public function testTexteAvecBaliseIntro(): void {
-		$introduction = $this->getFilterIntroduction();
-		$this->assertEquals(
-			paragrapher(couper(propre('plus'), 100), true),
-			$introduction('', 'description <intro>plus</intro> longue', 100, '')
-		);
-	}
+    public function testTexteAvecBaliseIntro(): void {
+        $introduction = $this->getFilterIntroduction();
+        $this->assertEquals(
+            paragrapher(couper(propre('plus'), 100), true),
+            $introduction('', 'description <intro>plus</intro> longue', 100, '')
+        );
+    }
 
-	private function getFilterIntroduction(): string {
-		return chercher_filtre('introduction');
-	}
+    private function getFilterIntroduction(): string {
+        return chercher_filtre('introduction');
+    }
 }