Refactor progress

Author: brunorijsman

common/tests/test_allocation.py

@@ -203,13 +203,17 @@ def test_from_enc_str_bad_fragment():
     Attempt to create an Allocation from a bad APIAllocation: one of the fragments has an invalid
     block UUID.
     """
+    # pylint: disable=protected-access
     pool, blocks = create_test_pool_and_blocks([10])
     # Bad block UUID
     with pytest.raises(InvalidBlockUUIDError):
-        _allocation = Allocation.from_enc_str(f"not-a-uuid:0:5", pool)
+        _allocation = Allocation.from_enc_str("not-a-uuid:0:5", pool)
     # Bad start index
     with pytest.raises(InvalidPSRDIndex):
         _allocation = Allocation.from_enc_str(f"{blocks[0].uuid}:20:5", pool)
     # Bad size
     with pytest.raises(InvalidPSRDIndex):
         _allocation = Allocation.from_enc_str(f"{blocks[0].uuid}:3:99999", pool)
+    # No data taken from pool
+    assert pool.nr_used_bytes == 0
+    assert blocks[0]._data == bytes.fromhex("00010203040506070809")