Update TestFairy plugin

- Update documentation
- Add valid options to max-duration, screenshot-interval, and
  video-quality
- Test max-duration, screenshot-interval, and video-quality options

Change-Id: If010e7bc79e269b766107a5b2e57b9916ca8637d
Signed-off-by: Kien Ha <kienha9922@gmail.com>

jenkins_jobs/modules/helpers.py

@@ -448,6 +448,9 @@ def append_git_revision_config(parent, config_def):
 
 def test_fairy_common(xml_element, data):
     xml_element.set('plugin', 'TestFairy')
+    valid_max_duration = ['10m', '60m', '300m', '1440m']
+    valid_interval = [1, 2, 5]
+    valid_video_quality = ['high', 'medium', 'low']
 
     mappings = [
         # General
@@ -457,13 +460,13 @@ def test_fairy_common(xml_element, data):
         ('notify-testers', 'notifyTesters', True),
         ('autoupdate', 'autoUpdate', True),
         # Session
-        ('max-duration', 'maxDuration', '10m'),
+        ('max-duration', 'maxDuration', '10m', valid_max_duration),
         ('record-on-background', 'recordOnBackground', False),
         ('data-only-wifi', 'dataOnlyWifi', False),
         # Video
         ('video-enabled', 'isVideoEnabled', True),
-        ('screenshot-interval', 'screenshotInterval', '1'),
+        ('screenshot-interval', 'screenshotInterval', 1, valid_interval),
-        ('video-quality', 'videoQuality', 'high'),
+        ('video-quality', 'videoQuality', 'high', valid_video_quality),
         # Metrics
         ('cpu', 'cpu', True),
         ('memory', 'memory', True),

jenkins_jobs/modules/publishers.py

@@ -2881,14 +2881,29 @@ def test_fairy(parser, xml_parent, data):
     :arg bool notify-testers: Send email with changelogs to testers
         (default false)
     :arg bool autoupdate: Automatic update (default false)
-
     :arg str max-duration: Duration of the session (default 10m)
+
+        :max-duration values:
+            * **10m**
+            * **60m**
+            * **300m**
+            * **1440m**
     :arg bool record-on-background: Record on background (default false)
     :arg bool data-only-wifi: Record data only in wifi (default false)
     :arg bool video-enabled: Record video (default true)
-    :arg str screenshot-interval: Time interval between screenshots
+    :arg int screenshot-interval: Time interval between screenshots
         (default 1)
+
+        :screenshot-interval values:
+            * **1**
+            * **2**
+            * **5**
     :arg str video-quality: Video quality (default high)
+
+        :video-quality values:
+            * **high**
+            * **medium**
+            * **low**
     :arg bool cpu: Enable CPU metrics (default true)
     :arg bool memory: Enable memory metrics (default true)
     :arg bool logs: Enable logs metrics (default true)

tests/publishers/fixtures/test-fairy-android002.xml

@@ -7,12 +7,12 @@
       <testersGroups/>
       <notifyTesters>false</notifyTesters>
       <autoUpdate>false</autoUpdate>
-      <maxDuration>10m</maxDuration>
+      <maxDuration>300m</maxDuration>
       <recordOnBackground>false</recordOnBackground>
       <dataOnlyWifi>false</dataOnlyWifi>
       <isVideoEnabled>true</isVideoEnabled>
-      <screenshotInterval>1</screenshotInterval>
+      <screenshotInterval>2</screenshotInterval>
-      <videoQuality>high</videoQuality>
+      <videoQuality>low</videoQuality>
       <cpu>false</cpu>
       <memory>false</memory>
       <logs>false</logs>

tests/publishers/fixtures/test-fairy-android002.yaml

@@ -6,6 +6,9 @@ publishers:
       keystorepath: /tmp/keystorefile
       notify-testers: false
       autoupdate: false
+      max-duration: 300m
+      screenshot-interval: 2
+      video-quality: low
       cpu: false
       memory: false
       logs: false

tests/publishers/fixtures/test-fairy-ios002.xml

@@ -7,12 +7,12 @@
       <testersGroups/>
       <notifyTesters>false</notifyTesters>
       <autoUpdate>false</autoUpdate>
-      <maxDuration>10m</maxDuration>
+      <maxDuration>1440m</maxDuration>
       <recordOnBackground>false</recordOnBackground>
       <dataOnlyWifi>false</dataOnlyWifi>
       <isVideoEnabled>true</isVideoEnabled>
-      <screenshotInterval>1</screenshotInterval>
+      <screenshotInterval>5</screenshotInterval>
-      <videoQuality>high</videoQuality>
+      <videoQuality>medium</videoQuality>
       <cpu>false</cpu>
       <memory>false</memory>
       <logs>false</logs>
@@ -26,4 +26,4 @@
       <mappingFile/>
     </org.jenkinsci.plugins.testfairy.TestFairyIosRecorder>
   </publishers>
-</project>
+</project>

tests/publishers/fixtures/test-fairy-ios002.yaml

@@ -5,6 +5,9 @@ publishers:
       appfile: /tmp/appfile.ipa
       notify-testers: false
       autoupdate: false
+      max-duration: 1440m
+      screenshot-interval: 5
+      video-quality: medium
       cpu: false
       memory: false
       logs: false