From 888d078d522213b8d256c8a9b637cc283f0b1f65 Mon Sep 17 00:00:00 2001
From: Wolfgang Sourdeau <wsourdeau@inverse.ca>
Date: Thu, 6 Aug 2009 19:27:40 +0000
Subject: [PATCH] Monotone-Parent: f473906a9eeabc3d7969b78d83ccdeee34b2fb7a
 Monotone-Revision: f14da511e49cf36d5c6f6a7d9fe1aed0eeda1c79

Monotone-Author: wsourdeau@inverse.ca
Monotone-Date: 2009-08-06T19:27:40
Monotone-Branch: ca.inverse.sogo
---
 Tests/README | 35 +++++++++++++++++++++++++++++++++--
 1 file changed, 33 insertions(+), 2 deletions(-)

diff --git a/Tests/README b/Tests/README
index fefa3f773..96e95c28b 100644
--- a/Tests/README
+++ b/Tests/README
@@ -12,8 +12,39 @@ runnable scripts
 
 webdav-sync.py
 
-other
------
+other scripts
+-------------
 
 propfind.py - a sample implementation of a PROPFIND request using webdavlib
 
+* developers
+------------
+
+- Test methods are always prefixed with "test". Sometimes, it's easier to
+track down a problem by enabling only one test at a time. One possible method
+is to replace "def test" with "def xtest" and replace it back when the
+problems are solved.
+
+- Test failures start with "FAIL:". Those are the ones that indicate possible
+  bugs in the application, if the test is well written.
+  For example like this:
+
+======================================================================
+FAIL: 'modify' PUBLIC, 'view all' PRIVATE, 'view d&t' confidential
+----------------------------------------------------------------------
+Traceback (most recent call last):
+  File "./davacl.py", line 75, in testModifyPublicViewAllPrivateViewDConfidential
+    self._testRights({ "pu": "m", "pr": "v", "co": "d" })
+  File "./davacl.py", line 119, in _testRights
+    self._testCreate(rights)
+  File "./davacl.py", line 165, in _testCreate
+    exp_code)
+  File "./davacl.py", line 107, in _putEvent
+    % (exp_status, put.response["status"]))
+AssertionError: event creation/modification: expected status code '403' (received '201')
+
+- Test errors start with "ERRORS" and most likely indicate a bug in the test
+  code itself.
+
+- Always set a doc string on the test methods, especially for complex test
+  cases