-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 On 03/02/2011 12:02 PM, Jeff Ortel wrote: > > > On 03/01/2011 01:28 PM, Jay Dobies wrote: > On 03/01/2011 02:26 PM, Jay Dobies wrote: >>>> We have them in BaseAPI to delete all collections the API is tied to. >>>> Anyone mind if I start to move those to the model objects themselves? >>>> Conceptually it makes more sense in a unit test to say "Consumer - >>>> delete all of those between tests." > > Changed my mind. Turns out the clean() method is as simple as > collection.remove(). I had seen elsewhere to iterate over each > collection calling remove and that's the functionality I wanted to > migrate. But if it's that simple, no need to clog up the model with the > simple chain down to the collection method. > >> The clean() method is overridden in RepoApi essentially to delete the >> repo content on the filesystem. We may have other cases like this.
Ohhhh, interesting. It's still just intended for unit tests right? Or do you know of any places in the app that we're using it? > > > >> _______________________________________________ Pulp-list mailing list [email protected] https://www.redhat.com/mailman/listinfo/pulp-list > _______________________________________________ > Pulp-list mailing list > [email protected] > https://www.redhat.com/mailman/listinfo/pulp-list - -- Jay Dobies RHCE# 805008743336126 Freenode: jdob http://pulpproject.org -----BEGIN PGP SIGNATURE----- Version: GnuPG v2.0.14 (GNU/Linux) Comment: Using GnuPG with Fedora - http://enigmail.mozdev.org/ iQEcBAEBAgAGBQJNbnulAAoJEOMmcTqOSQHCRaoIAIfygMXx05ZJkylOXOUwC1V5 g2W8Lzc5u/NXCfMsSoj4DthKXruJxq+cMotRyVfsQlCu0hJxTj9atah7Ef9yKwPZ xTBxdB6GkM5GGVLJr+vahfm2H7mA0i/J+pykEI0TLlA0nY1UUZ2nobTNStgGKZ1W mLPI9lrphayIv9DyZdyqpZuEcYg5hawVDoXHqo0woYPCNW2MoASbq3s81krIERTE mnaUAS1QBxLj6hI0sl9bAWhljxT3NOEy2Iji0cChAHIZi8sPeBtEHvO6Z9UCsUxw 5WR3ZB89T3ffnaKAVIKrrHnjQrsRkbdDmFXbaZlszDN2MmwC7YG6c9sTeNiIV5k= =WVAt -----END PGP SIGNATURE----- _______________________________________________ Pulp-list mailing list [email protected] https://www.redhat.com/mailman/listinfo/pulp-list
