BuildBase says it implements IBuildBase, but does not

Bug #510919 reported by Jonathan Lange
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Launchpad itself
Fix Released
Medium
Michael Nelson

Bug Description

At the moment, the following test fails.

    def test_build_base_provides_interface(self):
        # XXX: BuildBase is supposed to implement IBuildBase, but doesn't atm.
        # Since it's not the focus of the branch, we'll postpone the work.
        build_base = BuildBase()
        self.assertProvides(build_base, IBuildBase)

(from lib/lp/buildmaster/tests/test_buildbase.py)

Related branches

Revision history for this message
Michael Nelson (michael.nelson) wrote :

From the branch:

Note: this class does not implement IBuildBase as we currently duplicate the properties defined on IBuildBase on the inheriting class tables. BuildBase cannot therefore implement IBuildBase itself, as storm requires that the corresponding __storm_table__ be defined for the class. Instead, the classes using the BuildBase mixin must ensure that they implement IBuildBase.

Changed in soyuz:
assignee: nobody → Michael Nelson (michael.nelson)
milestone: none → 10.02
status: Triaged → In Progress
Revision history for this message
Diogo Matsubara (matsubara) wrote : Bug fixed by a commit
Changed in soyuz:
status: In Progress → Fix Committed
Changed in soyuz:
status: Fix Committed → Fix Released
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.