Roundup Tracker - Issues

Message7079

Author rouilj
Recipients cmeerw, rouilj, schlatterbeck, tttech-klonner
Date 2021-02-12.17:42:46
Message-id <20210212174241.4304F6A01BD@pe15.cs.umb.edu>
In-reply-to <1613143109.77.0.35134070202.issue2551112@roundup.psfhosted.org>
In message <1613143109.77.0.35134070202.issue2551112@roundup.psfhosted.org>,
Christof Meerwald writes:
>Yes, moving it to a "preload" method would definitely be an improvement.

I am fine with that. Ralf any ideas on how to add this file to the
test suite? We have 0 lines of coverage there, so anything would be
good.

>Another option could be to have a base class you can derive from that 
>doesn't have the "preload" functionality. Either option would work for 
>me.

I don't think cluttering the code with a new RequestDispatcherBase
class and then creating:

  RequestDispatcher(RequestDispatcherBase):

     def __init__(...):

     def preload(...):

with just these two (or just __init__) methods is warranted.
History
Date User Action Args
2021-02-12 17:42:46rouiljsetrecipients: + schlatterbeck, cmeerw, tttech-klonner
2021-02-12 17:42:46rouiljlinkissue2551112 messages
2021-02-12 17:42:46rouiljcreate