From 05edea703d12d07e5a4ea53084e7608e36d77e5e Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Marta=20Marczykowska-G=C3=B3recka?=
 <marta.maria.gorecka@gmail.com>
Date: Tue, 19 Nov 2019 00:33:30 +0100
Subject: [PATCH] Fixed metadata usage being served as strings

Metadata usage should be served as ints, as it is numeric.
---
 qubesadmin/storage.py       | 7 ++++++-
 qubesadmin/tests/storage.py | 8 ++++----
 2 files changed, 10 insertions(+), 5 deletions(-)

diff --git a/qubesadmin/storage.py b/qubesadmin/storage.py
index b7c5805..5d2de68 100644
--- a/qubesadmin/storage.py
+++ b/qubesadmin/storage.py
@@ -278,7 +278,12 @@ class Pool(object):
         pool_usage_data = pool_usage_data.decode('utf-8')
         assert pool_usage_data.endswith('\n')
         pool_usage_data = pool_usage_data[:-1]
-        return dict(l.split('=', 1) for l in pool_usage_data.splitlines())
+
+        def _int_split(text):  # pylint: disable=missing-docstring
+            key, value = text.split("=", 1)
+            return key, int(value)
+
+        return dict(_int_split(l) for l in pool_usage_data.splitlines())
 
     @property
     def config(self):
diff --git a/qubesadmin/tests/storage.py b/qubesadmin/tests/storage.py
index 6123c5b..2ea54c2 100644
--- a/qubesadmin/tests/storage.py
+++ b/qubesadmin/tests/storage.py
@@ -313,10 +313,10 @@ class TestPool(qubesadmin.tests.QubesTestCase):
             b'metadata_usage=50\n'
         pool = self.app.pools['lvm']
         self.assertEqual(pool.usage_details, {
-            'data_size': '204800',
-            'data_usage': '102400',
-            'metadata_size': '1024',
-            'metadata_usage': '50',
+            'data_size': 204800,
+            'data_usage': 102400,
+            'metadata_size': 1024,
+            'metadata_usage': 50,
         })
         self.assertAllCalled()