From 28bb4724e7ba212979ac25c16d0132fbfb50a901 Mon Sep 17 00:00:00 2001
From: Ned Batchelder <ned@nedbatchelder.com>
Date: Thu, 8 Nov 2012 11:09:57 -0500
Subject: [PATCH] Put the coverage data files in the reports dir, where they
 will be treated like the disposable things they are.

---
 cms/.coveragerc                | 2 +-
 common/lib/capa/.coveragerc    | 2 +-
 common/lib/xmodule/.coveragerc | 2 +-
 lms/.coveragerc                | 2 +-
 rakefile                       | 6 +++---
 5 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/cms/.coveragerc b/cms/.coveragerc
index 268193f94a..5a2c6e0e93 100644
--- a/cms/.coveragerc
+++ b/cms/.coveragerc
@@ -1,6 +1,6 @@
 # .coveragerc for cms
 [run]
-data_file = cms/.coverage
+data_file = reports/cms/.coverage
 
 [report]
 ignore_errors = True
diff --git a/common/lib/capa/.coveragerc b/common/lib/capa/.coveragerc
index 955a76294b..3818cb480a 100644
--- a/common/lib/capa/.coveragerc
+++ b/common/lib/capa/.coveragerc
@@ -1,6 +1,6 @@
 # .coveragerc for common/lib/capa
 [run]
-data_file = common/lib/capa/.coverage
+data_file = reports/common/lib/capa/.coverage
 
 [report]
 ignore_errors = True
diff --git a/common/lib/xmodule/.coveragerc b/common/lib/xmodule/.coveragerc
index c31a858749..9f8c72ba37 100644
--- a/common/lib/xmodule/.coveragerc
+++ b/common/lib/xmodule/.coveragerc
@@ -1,6 +1,6 @@
 # .coveragerc for common/lib/xmodule
 [run]
-data_file = common/lib/xmodule/.coverage
+data_file = reports/common/lib/xmodule/.coverage
 
 [report]
 ignore_errors = True
diff --git a/lms/.coveragerc b/lms/.coveragerc
index f41b79da68..13c5f9801e 100644
--- a/lms/.coveragerc
+++ b/lms/.coveragerc
@@ -1,6 +1,6 @@
 # .coveragerc for lms
 [run]
-data_file = lms/.coverage
+data_file = reports/lms/.coverage
 
 [report]
 ignore_errors = True
diff --git a/rakefile b/rakefile
index 0239614e57..a1e98e9b6b 100644
--- a/rakefile
+++ b/rakefile
@@ -209,10 +209,10 @@ namespace :coverage do
             sh("coverage xml -o reports/#{dir}/coverage.xml --rcfile=#{dir}/.coveragerc")
         end
     end
-end
 
-TEST_TASK_DIRS.each do |dir|
-    CLOBBER.include("#{dir}/.coverage")
+    TEST_TASK_DIRS.each do |dir|
+        CLOBBER.include("#{dir}/.coverage")
+    end
 end
 
 task :runserver => :lms