Merge pull request #460 from edx/sarina/update-diffcover

Update diff-cover to latest version

jenkins/test.sh

@@ -77,6 +77,9 @@ rake pylint > pylint.log || cat pylint.log
 # Generate coverage reports
 rake coverage
 
+# Generate quality reports
+rake quality
+
 rake autodeploy_properties
 
 github_status state:success "passed"

rakelib/quality.rake

@@ -43,3 +43,17 @@ end
     end
     task :pep8 => :"pep8:#{system}"
 end
+
+dquality_dir = File.join(REPORT_DIR, "diff_quality")
+directory dquality_dir
+
+desc "Build the html diff quality reports, and print the reports to the console."
+task :quality => dquality_dir do
+    # Generage diff-quality html report for pep8, and print to console
+    sh("diff-quality --violations=pep8 --html-report #{dquality_dir}/diff_quality_pep8.html")
+    sh("diff-quality --violations=pep8")
+
+    # Generage diff-quality html report for pylint, and print to console
+    sh("diff-quality --violations=pylint --html-report #{dquality_dir}/diff_quality_pylint.html")
+    sh("diff-quality --violations=pylint")
+end
\ No newline at end of file

rakelib/tests.rake

@@ -161,4 +161,4 @@ task :coverage => :report_dirs do
     if not found_coverage_info
         puts "No coverage info found.  Run `rake test` before running `rake coverage`."
     end
-end
+end
\ No newline at end of file

requirements/edx/github.txt

@@ -10,4 +10,4 @@
 # Our libraries:
 -e git+https://github.com/edx/XBlock.git@b697bebd45deebd0f868613fab6722a0460ca0c1#egg=XBlock
 -e git+https://github.com/edx/codejail.git@c08967fb44d1bcdb259d3ec58812e3ac592539c2#egg=codejail
--e git+https://github.com/edx/diff-cover.git@v0.1.3#egg=diff_cover
+-e git+https://github.com/edx/diff-cover.git@v0.2.0#egg=diff_cover