Roger Ineichen wrote:
Log message for revision 40515:
  Bugfix:
  Using getLink('File') will end in a testing failure in our application
  because we use a content type '- File' which produce a link called '- File'.
Use the more concrete url attribute instead of the text attribute in the getLink method will avoid this. See also collector issue at: http://www.zope.org/Collectors/Zope3-dev/503

Modified: Zope3/trunk/src/zope/app/apidoc/codemodule/browser/introspector.txt
===================================================================
--- Zope3/trunk/src/zope/app/apidoc/codemodule/browser/introspector.txt 
2005-12-03 04:34:49 UTC (rev 40514)
+++ Zope3/trunk/src/zope/app/apidoc/codemodule/browser/introspector.txt 
2005-12-03 12:32:48 UTC (rev 40515)
@@ -134,7 +134,8 @@
 they are provided in a aseparate section. To see anything useful, we have to
 add an object to the folder first:
- >>> browser.getLink('File').click()
+    >>> link = 'http://localhost/@@+/action.html?type_name=zope.app.file.File'
+    >>> browser.getLink(url=link).click()
     >>> import cStringIO
     >>> browser.getControl('Data').value = cStringIO.StringIO('content')
     >>> browser.getControl(name='add_input_name').value = 'file.txt'

This isn't really in the "spirit" of testbrowser. Previously the test would ensure that the desired link appeared on the page, was named correctly (if a bit too generically), and that it went to the appropriate place. The new version just makes sure the hard-coded URL points to the correct page.

I prefer some variation on the previous style (with appropriate correction so it isn't so brittle). Comments/ideas?
--
Benji York
Senior Software Engineer
Zope Corporation
_______________________________________________
Zope3-dev mailing list
Zope3-dev@zope.org
Unsub: http://mail.zope.org/mailman/options/zope3-dev/archive%40mail-archive.com

Reply via email to