Remove unnecessary properties from TacTestSetup class.
Bug #1307 reported by
Celso Providelo
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Launchpad itself |
Invalid
|
Low
|
Unassigned | ||
Bug Description
As jamesh has pointed, the current set of properties included in the base class to build "harness" tests for daemons based on twisted .tac files, are unnecessary since they store mostly static information, simple attributes should work.
IMO, we also could offer a kind of default name for functional files derived from the own daemon name, like: pidfile = '%s.pid' % daemon_name ...
| Changed in launchpad: | |
| assignee: | nobody → name101 |
| status: | New → Accepted |
| Changed in launchpad: | |
| assignee: | cprov → nobody |
| status: | Accepted → New |
| Changed in launchpad: | |
| status: | Unconfirmed → Confirmed |
| tags: |
added: build-infrastructure removed: infrastructure test-system |
| Changed in launchpad-foundations: | |
| importance: | Medium → Low |
| visibility: | private → public |
| Changed in launchpad-foundations: | |
| status: | Triaged → Invalid |
To post a comment you must log in.
Closing - this is something someone should just do when looking at the code, not worth having a bug open about it.