Implement shed_diff xunit reporting

Author: hexylena

planemo/commands/cmd_shed_diff.py

@@ -7,6 +7,7 @@
 from planemo.cli import pass_context
 from planemo import options
 from planemo import shed
+from planemo.reports import build_report
 
 
 @click.command("shed_diff")
@@ -30,6 +31,13 @@
     help="Do not attempt smart diff of XML to filter out attributes "
          "populated by the Tool Shed.",
 )
+@click.option(
+    "--report_xunit",
+    type=click.Path(file_okay=True, resolve_path=True),
+    help="Output diff as a faked XUnit report, useful when you want to "
+         "automatically diff repositories and be warned when out-of-date.",
+    default=None,
+)
 @pass_context
 def cli(ctx, paths, **kwds):
     """Produce diff between local repository and Tool Shed contents.
@@ -61,8 +69,39 @@ def cli(ctx, paths, **kwds):
     difference applies only to the file contents of files that would actually be
     uploaded to the repository.
     """
+
+    # In a little bit of cheating, we're defining this variable here to collect
+    # a "report" on our shed_diff
+    collected_data = {
+        'results': {
+            'total': 0,
+            'errors': 0,
+            'failures': 0,
+            'skips': 0,
+        },
+        'tests': [],
+    }
+
     def diff(realized_repository):
-        return shed.diff_repo(ctx, realized_repository, **kwds)
+        result = shed.diff_repo(ctx, realized_repository, **kwds)
+        # Collect data about what happened
+        collected_data['results']['total'] += 1
+        if result >= 200:
+            collected_data['results']['errors'] += 1
+        elif result > 0:
+            collected_data['results']['failures'] += 1
+        collected_data['tests'].append({
+            'classname': realized_repository.name,
+            'result': result,
+        })
+
+        return result
 
     exit_code = shed.for_each_repository(ctx, diff, paths, **kwds)
+
+    if 'report_xunit' in kwds:
+        with open(kwds['report_xunit'], 'w') as handle:
+            handle.write(build_report.template_data(
+                collected_data, template_name='diff_xunit.tpl'))
+
     sys.exit(exit_code)

planemo/reports/diff_xunit.tpl

@@ -0,0 +1,25 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<testsuite name="planemodiff"
+           tests="{{ results.total }}"
+           errors="{{ results.errors }}"
+           failures="{{ results.failures }}"
+           skip="{{ results.skips }}">
+    {% for testcase in tests %}
+    <testcase classname="{{ testcase.classname }}" name="recursive_diff" time="{{ testcase.time }}"/>
+        {% if testcase.result == 1 %}
+            <error type="planemo.Different" message="Repository is different">
+                <!-- TODO: copy of diff output -->
+            </error>
+        {% endif %}
+        {% if testcase.result == 2 %}
+            <error type="planemo.RepoDoesNotExit" message="Target repository does not exist">
+                <!-- TODO: copy of diff output -->
+            </error>
+        {% endif %}
+        {% if testcase.result > 2 %}
+            <error type="planemo.DiffError" message="Error executing diff">
+                <!-- TODO: copy of diff output -->
+            </error>
+        {% endif %}
+    {% endfor %}
+</testsuite>