Closed Bug 675103 Opened 13 years ago Closed 13 years ago

get rid of TestEnv in tests/support.py

Categories

(Cloud Services :: Server: Core, defect)

Product:
Cloud Services
Component:
Server: Core
Platform:
x86
macOS
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED INVALID

People

(Reporter: telliott, Assigned: rmiller)

References

Details

This class started out too sync specific and has had bits like auth pulled out. At this point, it just tries to find your basedir and instantiate a config file in bad ways. It becomes doubly pointless if we're ditching .ini files.

Any way we can pass an ENV parameter to make test that lets us define a config location? Then tests should have an internal array that's the default conf, or get a config object from the passed-in filename. That would save us from these traversals.

I believe account-portal is the main product using this class.
Assignee: nobody → rmiller
Blocks: 676423
Superseded by Bug 675106 I believe -- Rob ended up refactoring it
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.