Missing refactoring on NascentUpload-Cataclysm

Bug #106084 reported by Celso Providelo
2
Affects Status Importance Assigned to Milestone
Launchpad itself
Invalid
Low
Unassigned

Bug Description

NascentUpload-Cataclysm branch has grown to around 7 K lines, that's why we decided to left behind some wanted but not mandatory changes. They are:

 * Use standard logging module logger lookup instead of passing logger as an argument for NascentUpload classes

 * Write missing tests for SourceUploadFile, DebBinaryUploadFile, UDebBinaryUploadFile and CustomUploadFile in doc/nascentuploadfile.txt .

Those implementations are eligible for a FiF (reviewer flacoste).

Celso Providelo (cprov)
Changed in soyuz:
importance: Undecided → Medium
status: Unconfirmed → Confirmed
Revision history for this message
Celso Providelo (cprov) wrote :

The first proposed task is related with bug 117166, therefore they should be fixed together in 1.1.7 if we have time

Revision history for this message
Celso Providelo (cprov) wrote :

not 1.1.7

Celso Providelo (cprov)
Changed in soyuz:
assignee: nobody → cprov
Celso Providelo (cprov)
Changed in soyuz:
milestone: 1.1.12 → 1.2.1
Revision history for this message
Celso Providelo (cprov) wrote :

The initial tests are implemented in doc/nascentuploadfile.txt.

Another missing test-section is the DSC file lookup, specially across archives (see bug #139619).

Changed in soyuz:
milestone: 1.2.1 → 1.2.3
Celso Providelo (cprov)
Changed in soyuz:
milestone: 1.2.3 → none
Curtis Hovey (sinzui)
Changed in soyuz:
assignee: Celso Providelo (cprov) → nobody
tags: added: tech-debt
Changed in launchpad:
importance: Medium → Low
Revision history for this message
Julian Edwards (julian-edwards) wrote :

On reflection this bug is ill-defined and I'm not sure we'll ever get to it. Might as well invalidate.

Changed in launchpad:
status: Triaged → Invalid
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.