Commit fcf7fb60 authored by Marc Vef's avatar Marc Vef
Browse files

Merge branch 'rnou/224-symlink-parallelism' into 'master' 

Resolve "Symlink parallelism"

Closes #224

See merge request !147
parents d34220d9 887ab8b1

CHANGELOG.md

+1 −0
Original line number Diff line number Diff line
@@ -9,6 +9,7 @@ to [Semantic Versioning](https://semver.org/spec/v2.0.0.html). 
### New

### Changed

- Support parallelism for path resolution tests ([!145](https://storage.bsc.es/gitlab/hpc/gekkofs/-/merge_requests/145)).

- Support parallelism for symlink tests ([!147](https://storage.bsc.es/gitlab/hpc/gekkofs/-/merge_requests/147)).

### Removed

### Fixed

- Using `unlink` now fails if it is a directory unless the `AT_REMOVEDIR` flag is used (POSIX compliance) ([!139](https://storage.bsc.es/gitlab/hpc/gekkofs/-/merge_requests/139)).

tests/integration/directories/test_symlink.py

+5 −6
Original line number Diff line number Diff line
@@ -42,11 +42,10 @@ nonexisting = "nonexisting" 
#@pytest.mark.xfail(reason="invalid errno returned on success")

def test_symlink(gkfs_daemon, gkfs_client):

    """Testing different path resolution capabilities: symlinks"""



    pid = os.getpid().__str__()

    mountdir = gkfs_daemon.mountdir

    extdir = "/tmp/ext.tmp"

    ext_linkdir = "/tmp/link.tmp"

    nodir = "/tmp/notexistent"

    extdir = "/tmp/" + pid + "ext.tmp"

    ext_linkdir = "/tmp/" + pid + "link.tmp"

    intdir = mountdir / "int"



    # Just clean if it exists, due to a failed test
@@ -74,11 +73,11 @@ def test_symlink(gkfs_daemon, gkfs_client): 
    ret = gkfs_client.symlink(extdir, ext_linkdir)

    assert ret.retval == 0



    ret = gkfs_client.getcwd_validate(str(extdir) + "/../link.tmp")

    ret = gkfs_client.getcwd_validate(str(extdir) + "/../"+pid+"link.tmp")

    assert ret.path == str(extdir)

    assert ret.retval == 0



    ret = gkfs_client.getcwd_validate(str(intdir) + "/../../../../../../../../../../../../../../../../../../../.."+str(extdir) + "/../link.tmp/../link.tmp/../../../../../../../../../../../../../../../../../../../../.."  + str(intdir))

    ret = gkfs_client.getcwd_validate(str(intdir) + "/../../../../../../../../../../../../../../../../../../../.."+str(extdir) + "/../"+ pid +"link.tmp/../"+ pid +"link.tmp/../../../../../../../../../../../../../../../../../../../../.."  + str(intdir))

    assert ret.path == str(intdir)

    assert ret.retval == 0