[robotframework] push by pekka.klarck - rm todo and code that made no sense on 2011-12-01 14:51 GMT

[robotframework]

Revision: 2f88efe34e08
Author:   Pekka Klärck
Date:     Thu Dec  1 06:51:07 2011
Log:      rm todo and code that made no sense
http://code.google.com/p/robotframework/source/detail?r=2f88efe34e08

Modified:
 /atest/testdata/rebot/created_normal.xml
 /src/robot/result/builders.py

=======================================
--- /atest/testdata/rebot/created_normal.xml    Sat May 31 09:57:24 2008
+++ /atest/testdata/rebot/created_normal.xml    Thu Dec  1 06:51:07 2011
@@ -4,20 +4,20 @@
   <suite name="Root">
     <suite name="Sub 1">
       <test name="Test 1.1">
-        <status status="pass"/>
+        <status status="PASS"/>
       </test>
          <test name="Test 1.2">
            <doc>FAIL Error message</doc>
-        <status status='FAIL'>Error message</status>
+        <status status="FAIL">Error message</status>
       </test>
-      <status status="fail"/>
+      <status status="FAIL"/>
     </suite>
     <suite name="Sub 2">
       <test name="Test 2.1">
-        <status status="pass"/>
+        <status status="PASS"/>
       </test>
-      <status status='pass'/>
+      <status status="PASS"/>
     </suite>
-    <status status='fail'/>
+    <status status="FAIL"/>
   </suite>
 </robot>
=======================================
--- /src/robot/result/builders.py       Thu Dec  1 04:43:29 2011
+++ /src/robot/result/builders.py       Thu Dec  1 06:51:07 2011
@@ -173,10 +173,8 @@
 class _StatusElement(_Element):
     tag = 'status'

-    # TODO: Could elements handle default values themselves?
-
     def _set_status(self, elem, result):
-        result.status = elem.get('status', 'FAIL').upper()
+        result.status = elem.get('status', 'FAIL')

     def _set_message(self, elem, result):
         result.message = elem.text or ''