diff --git a/src/allmydata/frontends/magic_folder.py b/src/allmydata/frontends/magic_folder.py index c06bb8d93..b8ed0ae3f 100644 --- a/src/allmydata/frontends/magic_folder.py +++ b/src/allmydata/frontends/magic_folder.py @@ -619,7 +619,7 @@ class Uploader(QueueMixin): fp = self._get_filepath(relpath_u) pathinfo = get_pathinfo(unicode_from_filepath(fp)) progress = PercentProgress() - self._log("add pending size: {}: {}".format(relpath_u, pathinfo.size)) + self._log(u"add pending size: {}: {}".format(relpath_u, pathinfo.size)) item = UploadItem(relpath_u, progress, pathinfo.size) item.set_status('queued', self._clock.seconds()) self._deque.append(item) @@ -655,7 +655,8 @@ class Uploader(QueueMixin): try: return self._real_notify(opaque, path, events_mask) except Exception as e: - self._log("error calling _real_notify: {}".format(e)) + self._log(u"error calling _real_notify: {}".format(e)) + return Failure() def _real_notify(self, opaque, path, events_mask): self._log("inotify event %r, %r, %r\n" % (opaque, path, ', '.join(self._inotify.humanReadableMask(events_mask)))) diff --git a/src/allmydata/test/web/test_web.py b/src/allmydata/test/web/test_web.py index 8d809bbd2..34d0d78b1 100644 --- a/src/allmydata/test/web/test_web.py +++ b/src/allmydata/test/web/test_web.py @@ -127,6 +127,7 @@ class FakeStatusItem(object): import mock self.progress = mock.Mock() self.progress.progress = 100.0 + self.size = 1234 def status_history(self): return self.history @@ -1017,7 +1018,14 @@ class Web(WebMixin, WebErrorMixin, testutil.StallMixin, testutil.ReallyEqualMixi self.assertEqual( data, [ - {"status": "done", "path": "rel/path", "kind": "upload", "percent_done": 100.0, "done_at": 12345}, + { + "status": "done", + "path": "rel/path", + "kind": "upload", + "percent_done": 100.0, + "done_at": 12345, + "size": 1234, + }, ] )