_artifactcache modules: Adhere to policy on private symbolstristan/private-refactor-2

Additionally:

  o This shares more code by creating ArtifactCache.get_artifact_fullname(),
    which is used for the extract directory relative path in both backends,
    and for the ostree "ref" in the ostree backend.

  o Further, this removes some redundant documentation in derived abstract
    methods, and clarifies in both backends which methods are abstract methods,
    we should only be documenting abstract method semantics in one place, where
    they are defined.

This is a part of issue #285

1 files changed, 3 insertions, 3 deletions

diff --git a/tests/artifactcache/tar.py b/tests/artifactcache/tar.py
index 4a2f5658a..ef39be31c 100644
--- a/tests/artifactcache/tar.py
+++ b/tests/artifactcache/tar.py
@@ -5,7 +5,7 @@ from contextlib import ExitStack
 
 import pytest
 
-from buildstream._artifactcache.tarcache import Tar
+from buildstream._artifactcache.tarcache import _Tar
 from buildstream import utils, ProgramNotFoundError
 
 
@@ -19,7 +19,7 @@ def test_archive_default():
         test_file = stack.enter_context(open(os.path.join(src, 'test'), 'a'))
         test_file.write('Test')
 
-        Tar.archive(os.path.join(tar_dir, 'test.tar'), '.', src)
+        _Tar.archive(os.path.join(tar_dir, 'test.tar'), '.', src)
 
         with tarfile.open(os.path.join(tar_dir, 'test.tar')) as tar:
             tar.extractall(path=scratch)
@@ -58,7 +58,7 @@ def test_extract_default():
         with tarfile.open(os.path.join(tar_dir, 'test.tar'), 'a:') as tar:
             tar.add(src, 'contents')
 
-        Tar.extract(os.path.join(tar_dir, 'test.tar'), scratch)
+        _Tar.extract(os.path.join(tar_dir, 'test.tar'), scratch)
 
         assert os.listdir(os.path.join(scratch, 'contents')) == os.listdir(src)