Added two new tests for Links and Multilinks in HTMLItems:

An Item, when dereferencing a Link or Multilink should return the
correct items from the database.

Now if we name the linked items with numeric names (in the Link
example we have a status with id='1' and name='2' and vice-versa, in
the Multilink example we have the same for keyword) and the property
'name' of these items is also the key, it still works for Links but
not for Multilinks: The multilink in this case returns the keyword
with the *name* '1', not the one with the *id* '1'. I consider this a
bug and have implemented the test before looking deeper into it...

Author: Ralf Schlatterbeck

test/test_cgi.py

@@ -8,12 +8,13 @@
 # but WITHOUT ANY WARRANTY; without even the implied warranty of
 # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
 #
-# $Id: test_cgi.py,v 1.29 2006-12-11 23:36:15 richard Exp $
+# $Id: test_cgi.py,v 1.30 2007-07-05 19:21:57 schlatterbeck Exp $
 
 import unittest, os, shutil, errno, sys, difflib, cgi, re
 
 from roundup.cgi import client
 from roundup.cgi.exceptions import FormError
+from roundup.cgi.templating import HTMLItem
 from roundup.cgi.form_parser import FormParser
 from roundup import init, instance, password, hyperdb, date
 
@@ -191,6 +192,42 @@ def testEmptyStringSet(self):
         self.assertEqual(self.parseForm({'title': ' '}, 'issue', nodeid),
             ({('issue', nodeid): {'title': None}}, []))
 
+    def testStringLinkId(self):
+        self.db.status.set('1', name='2')
+        self.db.status.set('2', name='1')
+        issue = self.db.issue.create(title='i1-status1', status='1')
+        self.assertEqual(self.db.issue.get(issue,'status'),'1')
+        self.assertEqual(self.db.status.lookup('1'),'2')
+        self.assertEqual(self.db.status.lookup('2'),'1')
+        form = cgi.FieldStorage()
+        cl = client.Client(self.instance, None, {'PATH_INFO':'/'}, form)
+        cl.classname = 'issue'
+        cl.nodeid = issue
+        cl.db = self.db
+        item = HTMLItem(cl, 'issue', issue)
+        self.assertEqual(item.status.id, '1')
+        self.assertEqual(item.status.name, '2')
+
+    def testStringMultilinkId(self):
+        id = self.db.keyword.create(name='2')
+        self.assertEqual(id,'1')
+        id = self.db.keyword.create(name='1')
+        self.assertEqual(id,'2')
+        issue = self.db.issue.create(title='i1-status1', topic=['1'])
+        self.assertEqual(self.db.issue.get(issue,'topic'),['1'])
+        self.assertEqual(self.db.keyword.lookup('1'),'2')
+        self.assertEqual(self.db.keyword.lookup('2'),'1')
+        form = cgi.FieldStorage()
+        cl = client.Client(self.instance, None, {'PATH_INFO':'/'}, form)
+        cl.classname = 'issue'
+        cl.nodeid = issue
+        cl.db = self.db
+        cl.userid = '1'
+        item = HTMLItem(cl, 'issue', issue)
+        for topic in item.topic:
+            self.assertEqual(topic.id, '1')
+            self.assertEqual(topic.name, '2')
+
     def testFileUpload(self):
         file = FileUpload('foo', 'foo.txt')
         self.assertEqual(self.parseForm({'content': file}, 'file'),