Merge pull request #130 from wp-cli/fix/strip-ansi-colors-on-string-comparison

schlessera · web-flow · commit 1374d75f9373 · 2021-07-26T08:21:00.000+01:00
Strip ANSI colors on string comparison
diff --git a/src/Context/Support.php b/src/Context/Support.php
@@ -51,6 +51,9 @@ protected function assert_not_numeric( $actual ) {
 	}
 
 	protected function check_string( $output, $expected, $action, $message = false ) {
+		// Strip ANSI color codes before comparing strings.
+		$output = preg_replace( '/\e[[][A-Za-z0-9];?[0-9]*m?/', '', $output );
+
 		switch ( $action ) {
 			case 'be':
 				$r = rtrim( $output, "\n" ) === $expected;

Original file line number	Diff line number	Diff line change
`@@ -51,6 +51,9 @@ protected function assert_not_numeric( $actual ) {`
`51`	`51`	`}`
`52`	`52`
`53`	`53`	`protected function check_string( $output, $expected, $action, $message = false ) {`
	`54`	`+ // Strip ANSI color codes before comparing strings.`
	`55`	`+ $output = preg_replace( '/\e[[][A-Za-z0-9];?[0-9]*m?/', '', $output );`
	`56`	`+`
`54`	`57`	`switch ( $action ) {`
`55`	`58`	`case 'be':`
`56`	`59`	`$r = rtrim( $output, "\n" ) === $expected;`