Using "@Medium" group to enforce time limit.

cyrilverloop · cyrilverloop · commit c5908ef92dfd · 2022-02-21T10:36:27.000+01:00
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -4,6 +4,10 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [Unreleased]
+### Changed
+- "@medium" group to enforce time limit.
+
 ## [3.3.0] - 2022-02-20
 ### Added
 - Time limit.
diff --git a/ci/phpunit.dist.xml b/ci/phpunit.dist.xml
@@ -7,6 +7,8 @@
         beStrictAboutTodoAnnotatedTests="true"
         verbose="true"
         colors="true"
+        enforceTimeLimit="true"
+        timeoutForMediumTests="6"
 >
     <coverage processUncoveredFiles="true">
         <include>
diff --git a/composer.json b/composer.json
@@ -20,7 +20,6 @@
         }
     ],
     "require": {
-        "php": "7.3.*",
-        "phpunit/php-invoker": "^3.1"
+        "php": "7.3.*"
     }
 }
diff --git a/composer.lock b/composer.lock
diff --git a/tests/Community/Training/Easy/ContainerTerminal/CGTest.php b/tests/Community/Training/Easy/ContainerTerminal/CGTest.php
@@ -12,6 +12,7 @@
  *
  * @covers \CyrilVerloop\Codingame\Community\Training\Easy\ContainerTerminal\ContainerTerminal
  * @group containerTerminal
+ * @medium
  */
 final class CGTest extends PuzzleTest
 {
diff --git a/tests/Community/Training/Easy/RectanglePartition/CGTest.php b/tests/Community/Training/Easy/RectanglePartition/CGTest.php
@@ -12,6 +12,7 @@
  *
  * @covers \CyrilVerloop\Codingame\Community\Training\Easy\RectanglePartition\RectanglePartition
  * @group rectanglePartition
+ * @medium
  */
 final class CGTest extends PuzzleTest
 {
diff --git a/tests/Community/Training/Easy/RobotShow/CGTest.php b/tests/Community/Training/Easy/RobotShow/CGTest.php
@@ -12,6 +12,7 @@
  *
  * @covers \CyrilVerloop\Codingame\Community\Training\Easy\RobotShow\RobotShow
  * @group robotShow
+ * @medium
  */
 final class CGTest extends PuzzleTest
 {
diff --git a/tests/Community/Training/Easy/RockPaperScissorsLizardSpock/CGTest.php b/tests/Community/Training/Easy/RockPaperScissorsLizardSpock/CGTest.php
@@ -12,6 +12,7 @@
  *
  * @covers \CyrilVerloop\Codingame\Community\Training\Easy\RockPaperScissorsLizardSpock\RockPaperScissorsLizardSpock
  * @group rockPaperScissorsLizardSpock
+ * @medium
  */
 final class CGTest extends PuzzleTest
 {
diff --git a/tests/PuzzleTest.php b/tests/PuzzleTest.php
@@ -5,21 +5,12 @@
 namespace CyrilVerloop\Codingame\Tests;
 
 use PHPUnit\Framework\TestCase;
-use SebastianBergmann\Invoker\Invoker;
 
 /**
  * Base class for testing puzzles.
  */
 abstract class PuzzleTest extends TestCase
 {
-    // Constants :
-
-    /**
-     * @var int the time limit in seconds.
-     */
-    const TIME_LIMIT = 6;
-
-
     // Properties :
 
     /** @var \CyrilVerloop\Codingame\Puzzle the puzzle. */
@@ -38,8 +29,7 @@ protected function expectExecuteOutputAnswer(string $filename, string $expectedA
         $stdin = fopen($filename, 'r');
 
         $this->expectOutputString($expectedAnswer);
-
-        $invoker = new Invoker();
-        $invoker->invoke([$this->puzzle, "execute"], [$stdin], self::TIME_LIMIT);
+        
+        $this->puzzle->execute($stdin);
     }
 }
diff --git a/tests/Training/Easy/ASCIIArt/CGTest.php b/tests/Training/Easy/ASCIIArt/CGTest.php
@@ -12,6 +12,7 @@
  *
  * @covers \CyrilVerloop\Codingame\Training\Easy\ASCIIArt\ASCIIArt
  * @group ASCIIArt
+ * @medium
  */
 final class CGTest extends PuzzleTest
 {
diff --git a/tests/Training/Easy/Defibrillators/CGTest.php b/tests/Training/Easy/Defibrillators/CGTest.php
@@ -12,6 +12,7 @@
  *
  * @covers \CyrilVerloop\Codingame\Training\Easy\Defibrillators\Defibrillators
  * @group defibrillators
+ * @medium
  */
 final class CGTest extends PuzzleTest
 {
diff --git a/tests/Training/Easy/HorseRacingDuals/CGTest.php b/tests/Training/Easy/HorseRacingDuals/CGTest.php
@@ -12,6 +12,7 @@
  *
  * @covers \CyrilVerloop\Codingame\Training\Easy\HorseRacingDuals\HorseRacingDuals
  * @group horseRacingDuals
+ * @medium
  */
 final class CGTest extends PuzzleTest
 {
diff --git a/tests/Training/Easy/MIMEType/CGTest.php b/tests/Training/Easy/MIMEType/CGTest.php
@@ -12,6 +12,7 @@
  *
  * @covers \CyrilVerloop\Codingame\Training\Easy\MIMEType\MIMEType
  * @group MIMEType
+ * @medium
  */
 final class CGTest extends PuzzleTest
 {
diff --git a/tests/Training/Easy/Temperatures/CGTest.php b/tests/Training/Easy/Temperatures/CGTest.php
@@ -12,6 +12,7 @@
  *
  * @covers \CyrilVerloop\Codingame\Training\Easy\Temperatures\Temperatures
  * @group temperatures
+ * @medium
  */
 final class CGTest extends PuzzleTest
 {
diff --git a/tests/Training/Easy/Unary/CGTest.php b/tests/Training/Easy/Unary/CGTest.php
@@ -12,6 +12,7 @@
  *
  * @covers \CyrilVerloop\Codingame\Training\Easy\Unary\Unary
  * @group unary
+ * @medium
  */
 final class CGTest extends PuzzleTest
 {
diff --git a/tests/Training/Expert/MusicScores/CGTest.php b/tests/Training/Expert/MusicScores/CGTest.php
@@ -12,6 +12,7 @@
  *
  * @covers \CyrilVerloop\Codingame\Training\Expert\MusicScores\MusicScores
  * @group musicScores
+ * @medium
  */
 final class CGTest extends PuzzleTest
 {
diff --git a/tests/Training/Expert/TheResistance/CGTest.php b/tests/Training/Expert/TheResistance/CGTest.php
@@ -12,6 +12,7 @@
  *
  * @covers \CyrilVerloop\Codingame\Training\Expert\TheResistance\TheResistance
  * @group theResistance
+ * @medium
  */
 final class CGTest extends PuzzleTest
 {
diff --git a/tests/Training/Hard/BlunderEpisode2/CGTest.php b/tests/Training/Hard/BlunderEpisode2/CGTest.php
@@ -12,6 +12,7 @@
  *
  * @covers \CyrilVerloop\Codingame\Training\Hard\BlunderEpisode2\BlunderEpisode2
  * @group blunderEpisode2
+ * @medium
  */
 final class CGTest extends PuzzleTest
 {
diff --git a/tests/Training/Hard/BlunderEpisode3/CGTest.php b/tests/Training/Hard/BlunderEpisode3/CGTest.php
@@ -12,6 +12,7 @@
  *
  * @covers \CyrilVerloop\Codingame\Training\Hard\BlunderEpisode3\BlunderEpisode3
  * @group blunderEpisode3
+ * @medium
  */
 final class CGTest extends PuzzleTest
 {
diff --git a/tests/Training/Hard/CGXFormatter/CGTest.php b/tests/Training/Hard/CGXFormatter/CGTest.php
@@ -12,6 +12,7 @@
  *
  * @covers \CyrilVerloop\Codingame\Training\Hard\CGXFormatter\CGXFormatter
  * @group CGXFormatter
+ * @medium
  */
 final class CGTest extends PuzzleTest
 {
diff --git a/tests/Training/Hard/GenomeSequencing/CGTest.php b/tests/Training/Hard/GenomeSequencing/CGTest.php
@@ -12,6 +12,7 @@
  *
  * @covers \CyrilVerloop\Codingame\Training\Hard\GenomeSequencing\GenomeSequencing
  * @group genomeSequencing
+ * @medium
  */
 final class CGTest extends PuzzleTest
 {
diff --git a/tests/Training/Hard/RollerCoaster/CGTest.php b/tests/Training/Hard/RollerCoaster/CGTest.php
@@ -12,6 +12,7 @@
  *
  * @covers \CyrilVerloop\Codingame\Training\Hard\RollerCoaster\RollerCoaster
  * @group rollerCoaster
+ * @medium
  */
 final class CGTest extends PuzzleTest
 {
diff --git a/tests/Training/Hard/SuperComputer/CGTest.php b/tests/Training/Hard/SuperComputer/CGTest.php
@@ -12,6 +12,7 @@
  *
  * @covers \CyrilVerloop\Codingame\Training\Hard\SuperComputer\SuperComputer
  * @group superComputer
+ * @medium
  */
 final class CGTest extends PuzzleTest
 {
diff --git a/tests/Training/Hard/Surface/CGTest.php b/tests/Training/Hard/Surface/CGTest.php
@@ -12,6 +12,7 @@
  *
  * @covers \CyrilVerloop\Codingame\Training\Hard\Surface\Surface
  * @group surface
+ * @medium
  */
 final class CGTest extends PuzzleTest
 {
diff --git a/tests/Training/Hard/TANNetwork/CGTest.php b/tests/Training/Hard/TANNetwork/CGTest.php
@@ -12,6 +12,7 @@
  *
  * @covers \CyrilVerloop\Codingame\Training\Hard\TANNetwork\TANNetwork
  * @group TANNetwork
+ * @medium
  */
 final class CGTest extends PuzzleTest
 {
diff --git a/tests/Training/Hard/Winamax/CGTest.php b/tests/Training/Hard/Winamax/CGTest.php
@@ -12,6 +12,7 @@
  *
  * @covers \CyrilVerloop\Codingame\Training\Hard\Winamax\Winamax
  * @group winamax
+ * @medium
  */
 final class CGTest extends PuzzleTest
 {
diff --git a/tests/Training/Medium/BlunderEpisode1/CGTest.php b/tests/Training/Medium/BlunderEpisode1/CGTest.php
@@ -12,6 +12,7 @@
  *
  * @covers \CyrilVerloop\Codingame\Training\Medium\BlunderEpisode1\BlunderEpisode1
  * @group blunderEpisode1
+ * @medium
  */
 final class CGTest extends PuzzleTest
 {
diff --git a/tests/Training/Medium/ConwaySequence/CGTest.php b/tests/Training/Medium/ConwaySequence/CGTest.php
@@ -12,6 +12,7 @@
  *
  * @covers \CyrilVerloop\Codingame\Training\Medium\ConwaySequence\ConwaySequence
  * @group conwaySequence
+ * @medium
  */
 final class CGTest extends PuzzleTest
 {
diff --git a/tests/Training/Medium/DwarfsStandingOnTheShouldersOfGiants/CGTest.php b/tests/Training/Medium/DwarfsStandingOnTheShouldersOfGiants/CGTest.php
@@ -12,6 +12,7 @@
  *
  * @covers \CyrilVerloop\Codingame\Training\Medium\DwarfsStandingOnTheShouldersOfGiants\DwarfsStandingOnTheShouldersOfGiants
  * @group dwarfsStandingOnTheShouldersOfGiants
+ * @medium
  */
 final class CGTest extends PuzzleTest
 {
diff --git a/tests/Training/Medium/MayanCalculation/CGTest.php b/tests/Training/Medium/MayanCalculation/CGTest.php
@@ -12,6 +12,7 @@
  *
  * @covers \CyrilVerloop\Codingame\Training\Medium\MayanCalculation\MayanCalculation
  * @group mayanCalculation
+ * @medium
  */
 final class CGTest extends PuzzleTest
 {
diff --git a/tests/Training/Medium/NetworkCabling/CGTest.php b/tests/Training/Medium/NetworkCabling/CGTest.php
@@ -12,6 +12,7 @@
  *
  * @covers \CyrilVerloop\Codingame\Training\Medium\NetworkCabling\NetworkCabling
  * @group networkCabling
+ * @medium
  */
 final class CGTest extends PuzzleTest
 {
diff --git a/tests/Training/Medium/Scrabble/CGTest.php b/tests/Training/Medium/Scrabble/CGTest.php
@@ -12,6 +12,7 @@
  *
  * @covers \CyrilVerloop\Codingame\Training\Medium\Scrabble\Scrabble
  * @group scrabble
+ * @medium
  */
 final class CGTest extends PuzzleTest
 {
diff --git a/tests/Training/Medium/StockExchangeLosses/CGTest.php b/tests/Training/Medium/StockExchangeLosses/CGTest.php
@@ -12,6 +12,7 @@
  *
  * @covers \CyrilVerloop\Codingame\Training\Medium\StockExchangeLosses\StockExchangeLosses
  * @group stockExchangeLosses
+ * @medium
  */
 final class CGTest extends PuzzleTest
 {
diff --git a/tests/Training/Medium/TelephoneNumbers/CGTest.php b/tests/Training/Medium/TelephoneNumbers/CGTest.php
@@ -12,6 +12,7 @@
  *
  * @covers \CyrilVerloop\Codingame\Training\Medium\TelephoneNumbers\TelephoneNumbers
  * @group telephoneNumbers
+ * @medium
  */
 final class CGTest extends PuzzleTest
 {
diff --git a/tests/Training/Medium/TheGift/CGTest.php b/tests/Training/Medium/TheGift/CGTest.php
@@ -12,6 +12,7 @@
  *
  * @covers \CyrilVerloop\Codingame\Training\Medium\TheGift\TheGift
  * @group theGift
+ * @medium
  */
 final class CGTest extends PuzzleTest
 {
diff --git a/tests/Training/Medium/War/CGTest.php b/tests/Training/Medium/War/CGTest.php
@@ -12,6 +12,7 @@
  *
  * @covers \CyrilVerloop\Codingame\Training\Medium\War\War
  * @group war
+ * @medium
  */
 final class CGTest extends PuzzleTest
 {

Original file line number	Diff line number	Diff line change
`@@ -7,6 +7,8 @@`
`7`	`7`	`beStrictAboutTodoAnnotatedTests="true"`
`8`	`8`	`verbose="true"`
`9`	`9`	`colors="true"`
	`10`	`+ enforceTimeLimit="true"`
	`11`	`+ timeoutForMediumTests="6"`
`10`	`12`	`>`
`11`	`13`	`<coverage processUncoveredFiles="true">`
`12`	`14`	`<include>`
Original file line number	Diff line number	Diff line change
`@@ -20,7 +20,6 @@`
`20`	`20`	`}`
`21`	`21`	`],`
`22`	`22`	`"require": {`
`23`		`- "php": "7.3.*",`
`24`		`- "phpunit/php-invoker": "^3.1"`
	`23`	`+ "php": "7.3.*"`
`25`	`24`	`}`
`26`	`25`	`}`
Original file line number	Diff line number	Diff line change
`@@ -12,6 +12,7 @@`
`12`	`12`	`*`
`13`	`13`	`* @covers \CyrilVerloop\Codingame\Community\Training\Easy\ContainerTerminal\ContainerTerminal`
`14`	`14`	`* @group containerTerminal`
	`15`	`+ * @medium`
`15`	`16`	`*/`
`16`	`17`	`final class CGTest extends PuzzleTest`
`17`	`18`	`{`