mirror of
https://github.com/tahoe-lafs/tahoe-lafs.git
synced 2025-04-19 08:16:19 +00:00
Remove status.RateAndTimeMixin
Both UploadResultsRenderer and DownloadResultsRenderer do not use RateAndTimeMixin anymore: safe to remove it now. Tests for methods formerly in RateAndTimeMixin have been moved to test.web.test_util: specifically test_abbreviate_rate() and test_abbreviate_time().
This commit is contained in:
parent
4a47e8311f
commit
e6eb387707
@ -1035,29 +1035,6 @@ class Web(WebMixin, WebErrorMixin, testutil.StallMixin, testutil.ReallyEqualMixi
|
||||
|
||||
return d
|
||||
|
||||
def test_status_numbers(self):
|
||||
urrm = status.UploadResultsRendererMixin()
|
||||
self.failUnlessReallyEqual(urrm.render_time(None, None), "")
|
||||
self.failUnlessReallyEqual(urrm.render_time(None, 2.5), "2.50s")
|
||||
self.failUnlessReallyEqual(urrm.render_time(None, 0.25), "250ms")
|
||||
self.failUnlessReallyEqual(urrm.render_time(None, 0.0021), "2.1ms")
|
||||
self.failUnlessReallyEqual(urrm.render_time(None, 0.000123), "123us")
|
||||
self.failUnlessReallyEqual(urrm.render_rate(None, None), "")
|
||||
self.failUnlessReallyEqual(urrm.render_rate(None, 2500000), "2.50MBps")
|
||||
self.failUnlessReallyEqual(urrm.render_rate(None, 30100), "30.1kBps")
|
||||
self.failUnlessReallyEqual(urrm.render_rate(None, 123), "123Bps")
|
||||
|
||||
drrm = status.DownloadResultsRendererMixin()
|
||||
self.failUnlessReallyEqual(drrm.render_time(None, None), "")
|
||||
self.failUnlessReallyEqual(drrm.render_time(None, 2.5), "2.50s")
|
||||
self.failUnlessReallyEqual(drrm.render_time(None, 0.25), "250ms")
|
||||
self.failUnlessReallyEqual(drrm.render_time(None, 0.0021), "2.1ms")
|
||||
self.failUnlessReallyEqual(drrm.render_time(None, 0.000123), "123us")
|
||||
self.failUnlessReallyEqual(drrm.render_rate(None, None), "")
|
||||
self.failUnlessReallyEqual(drrm.render_rate(None, 2500000), "2.50MBps")
|
||||
self.failUnlessReallyEqual(drrm.render_rate(None, 30100), "30.1kBps")
|
||||
self.failUnlessReallyEqual(drrm.render_rate(None, 123), "123Bps")
|
||||
|
||||
def test_GET_FILEURL(self):
|
||||
d = self.GET(self.public_url + "/foo/bar.txt")
|
||||
d.addCallback(self.failUnlessIsBarDotTxt)
|
||||
|
@ -27,14 +27,6 @@ from allmydata.web.common import (
|
||||
from allmydata.interfaces import IUploadStatus, IDownloadStatus, \
|
||||
IPublishStatus, IRetrieveStatus, IServermapUpdaterStatus
|
||||
|
||||
class RateAndTimeMixin(object):
|
||||
|
||||
def render_time(self, ctx, data):
|
||||
return abbreviate_time(data)
|
||||
|
||||
def render_rate(self, ctx, data):
|
||||
return abbreviate_rate(data)
|
||||
|
||||
|
||||
class UploadResultsRendererMixin(Element):
|
||||
# this requires a method named 'upload_results'
|
||||
|
Loading…
x
Reference in New Issue
Block a user