Don't test for tidy class on hhvm
authorArlo Breault <abreault@wikimedia.org>
Mon, 13 Feb 2017 22:42:26 +0000 (14:42 -0800)
committerArlo Breault <abreault@wikimedia.org>
Mon, 13 Feb 2017 22:54:48 +0000 (14:54 -0800)
 * It doesn't support the oo interface.

 * Should make the tidy tests run in CI.

Bug: T157730
Change-Id: Ied80f70b7cafcf64d736cb0eeb1a30d52c1d7921

tests/parser/TidySupport.php

index 6b5fb48..39c9ca5 100644 (file)
@@ -58,7 +58,7 @@ class TidySupport {
                                'tidyConfigFile' => "$IP/includes/tidy/tidy.conf",
                                'tidyCommandLine' => '',
                        ];
-                       if ( extension_loaded( 'tidy' ) && class_exists( 'tidy' ) ) {
+                       if ( extension_loaded( 'tidy' ) && ( wfIsHHVM() || class_exists( 'tidy' ) ) ) {
                                $this->config['driver'] = wfIsHHVM() ? 'RaggettInternalHHVM' : 'RaggettInternalPHP';
                        } else {
                                if ( is_executable( $wgTidyBin ) ) {