MDL-37046 behat: Changes from integration review
authorDavid Monllao <davidm@moodle.com>
Tue, 29 Jan 2013 03:43:59 +0000 (11:43 +0800)
committerDavid Monllao <davidm@moodle.com>
Tue, 29 Jan 2013 03:56:42 +0000 (11:56 +0800)
* RTL languages problem solved
* Fixed curl errno return
* Switch to behat test environment code more readable

admin/tool/behat/lang/en/tool_behat.php
lib/behat/classes/behat_command.php
lib/behat/classes/util.php
lib/setup.php

index d890976..e5693d7 100644 (file)
@@ -25,7 +25,7 @@
 $string['allavailablesteps'] = 'All the available steps definitions';
 $string['giveninfo'] = 'Given. Processes to set up the environment';
 $string['installinfo'] = 'Read {$a} for installation and tests execution info';
-$string['moreinfoin'] = 'More info in';
+$string['moreinfoin'] = 'More info in {$a}';
 $string['newstepsinfo'] = 'Read {$a} for info about how to add new steps definitions';
 $string['newtestsinfo'] = 'Read {$a} for info about how to write new tests';
 $string['nostepsdefinitions'] = 'There aren\'t steps definitions matching this filters';
index 5ef2199..39e8c25 100644 (file)
@@ -122,7 +122,7 @@ class behat_command {
             if (!CLI_SCRIPT) {
                 $docslink = html_writer::tag('a', $docslink, array('href' => $docslink, 'target' => '_blank'));
             }
-            $msg .= '. ' . get_string('moreinfoin', 'tool_behat') . ' ' . $docslink;
+            $msg .= '. ' . get_string('moreinfoin', 'tool_behat', $docslink);
             notice($msg);
         }
 
index 828f452..5fe4aa3 100644 (file)
@@ -120,7 +120,11 @@ class behat_util extends testing_util {
 
         $request = new curl();
         $request->get($CFG->behat_wwwroot);
-        return (true && !$request->get_errno());
+
+        if ($request->get_errno() === 0) {
+            return true;
+        }
+        return false;
     }
 
     /**
index 9ceabed..ef0c38a 100644 (file)
@@ -108,12 +108,17 @@ if (isset($CFG->behat_switchcompletely)) {
 // * User has previously enabled through admin/tool/behat/cli/util.php --enable.
 // Both are required to switch to test mode
 if (isset($CFG->behat_dataroot) && isset($CFG->behat_prefix) && file_exists($CFG->behat_dataroot)) {
+
     $CFG->behat_dataroot = realpath($CFG->behat_dataroot);
-    $testenvironmentrequested = (isset($CFG->behat_switchcompletely) && php_sapi_name() !== 'cli') ||
-        php_sapi_name() === 'cli-server' ||
-        defined('BEHAT_RUNNING');
+
+    $switchcompletely = isset($CFG->behat_switchcompletely) && php_sapi_name() !== 'cli';
+    $builtinserver = php_sapi_name() === 'cli-server';
+    $behatrunning = defined('BEHAT_RUNNING');
+    $testenvironmentrequested = $switchcompletely || $builtinserver || $behatrunning;
+
     // Only switch to test environment if it has been enabled.
     $testenvironmentenabled = file_exists($CFG->behat_dataroot . '/behat/test_environment_enabled.txt');
+
     if ($testenvironmentenabled && $testenvironmentrequested) {
         $CFG->wwwroot = $CFG->behat_wwwroot;
         $CFG->passwordsaltmain = 'moodle';