Warning: Can't synchronize with repository "(default)" (Unsupported version control system "svn": No module named svn). Look in the Trac log for more information.

Ticket #332 (closed defect: fixed)

Opened 14 years ago

Last modified 12 years ago

[PATCH] Some tests have an incorrect uri for the database

Reported by: alberto@… Owned by: anonymous
Priority: low Milestone: 0.9
Component: TurboGears Version:
Severity: minor Keywords: tests
Cc:

Description

test_i18n and test_database have incorrect URIs for the database. They try to create the test DB at /, which in my linux box needs root privileges (which I deem unneccesary). I've fixed the URIs in this patch so it creates them in /tmp and :memory: respectively.

Attachments

tests.patch Download (1.2 KB) - added by alberto 14 years ago.
the patch
test_i18n-temp.diff Download (659 bytes) - added by bbockelm@… 14 years ago.
Platform independent temp files for test_i18n.py

Change History

Changed 14 years ago by alberto

the patch

comment:1 Changed 14 years ago by kevin

  • Status changed from new to closed
  • Resolution set to fixed

/tmp is actually not ideal either. That will fail on Windows systems. testutil should probably determine a directory to put test data into.

Still... this is better than what's there now. committed in [450]. Thanks!

comment:2 Changed 14 years ago by bbockelm@…

  • Status changed from closed to reopened
  • Resolution fixed deleted

Try this instead:

import tempfile tempfile.gettempdir()

This is platform independent.

Changed 14 years ago by bbockelm@…

Platform independent temp files for test_i18n.py

comment:3 Changed 14 years ago by kevin

  • Status changed from reopened to closed
  • Resolution set to fixed

committed in [457]. Thanks!

Note: See TracTickets for help on using tickets.