[Notes] [Git][BuildStream/buildstream][BenjaminSchubert/fix-quota-tests]

b5fba860

by Benjamin Schubert at 2018-08-23T12:23:41Z

Mock calls to artifact_cache_size computation when testing quota

This is required in order to have the tests not depending on the
environment

@@ -266,16 +266,24 @@ def test_invalid_cache_quota(cli, datafiles, tmpdir, quota, success):
+         }
      })
 -    with mock.patch(
 -            "os.statvfs",
 -            autospec=True,
 -            return_value=mock.create_autospec(
 -                "os.statvfs_result",
 -                f_blocks=1000,
 -                f_bsize=10,
 -                f_bavail=600,
 -            )
 -    ):
 +    statvfs_patch = mock.patch(
 +        "os.statvfs",
 +        autospec=True,
 +        return_value=mock.create_autospec(
 +            "os.statvfs_result",
 +            f_blocks=1000,
 +            f_bsize=10,
 +            f_bavail=600,
 +        )
 +    )
++
 +    cache_size_patch = mock.patch(
 +        "buildstream._artifactcache.artifactcache.ArtifactCache.get_approximate_cache_size",
 +        autospec=True,
 +        return_value=0
 +    )
++
 +    with statvfs_patch, cache_size_patch:
          res = cli.run(project=project, args=['workspace', 'list'])
      if success:

[Notes] [Git][BuildStream/buildstream][BenjaminSchubert/fix-quota-tests] Mock calls to artifact_cache_size computation when testing quota