Bug 1458742 - Correct marionette.log.level docs for case insensitivity. r?maja_zf draft
authorAndreas Tolfsen <ato@sny.no>
Wed, 02 May 2018 17:18:51 -0700
changeset 791711 5a3379ca89b9eb6dc571bd8d3416579b40bfdbce
parent 791710 5207b1392b11db534550a5eb801302e6dbb58f95
child 791712 81ec168d2cdd7693c4dec6e2a609bdbd9cde1d1f
push id108881
push userbmo:ato@sny.no
push dateFri, 04 May 2018 22:50:30 +0000
reviewersmaja_zf
bugs1458742
milestone61.0a1
Bug 1458742 - Correct marionette.log.level docs for case insensitivity. r?maja_zf The marionette.log.level value is case insensitive and this commit points that out in the documentation associated with the preference. MozReview-Commit-ID: KZGFE9feFRl
testing/geckodriver/README.md
testing/marionette/prefs/marionette.js
--- a/testing/geckodriver/README.md
+++ b/testing/geckodriver/README.md
@@ -338,16 +338,17 @@ removed once the interactability checks 
   <td><code>level</code>
   <td>string
   <td>Set the level of verbosity of geckodriver and Firefox.
    Available levels are <code>trace</code>,
    <code>debug</code>, <code>config</code>,
    <code>info</code>, <code>warn</code>,
    <code>error</code>, and <code>fatal</code>.
    If left undefined the default is <code>info</code>.
+   The value is treated case-insensitively.
  </tr>
 </table>
 
 
 `prefs` object
 --------------
 
 <table>
--- a/testing/marionette/prefs/marionette.js
+++ b/testing/marionette/prefs/marionette.js
@@ -9,18 +9,21 @@
 
 // Starts and stops the Marionette server.
 pref("marionette.enabled", false);
 
 // Delay server startup until a modal dialogue has been clicked to
 // allow time for user to set breakpoints in the Browser Toolbox.
 pref("marionette.debugging.clicktostart", false);
 
-// Marionette logging verbosity.  Allowed values are "fatal", "error",
-// "warn", "info", "config", "debug", and "trace".
+// Verbosity of Marionette logger repository.
+//
+// Available levels are, in descending order of severity,
+// "trace", "debug", "config", "info", "warn", "error", and "fatal".
+// The value is treated case-insensitively.
 pref("marionette.log.level", "info");
 
 // Port to start Marionette server on.
 pref("marionette.port", 2828);
 
 // Sets recommended automation preferences when Marionette is started.
 pref("marionette.prefs.recommended", true);