fix: R encoding of pathlib.Path objects

[jhrcook]

Description

Snakemake currently throws the following error if a pathlib.Path object is passed in params (where dir is the name of the directory created with Path(“dir”)):

ValueError: Unsupported value for conversion into R: dir

I have made the following change to the REncoder class to add support explicit for conversion of pathlib.Path objects.

class REncoder:
    """Encoding Pyton data structures into R."""

    ….

    @classmethod
    def encode_value(cls, value):
        if value is None:
            return "NULL"
       ...
        elif isinstance(value, Path):  # Just added these two lines.
            return str(value)
        ...
        raise ValueError("Unsupported value for conversion into R: {}".format(value))

I have added a test that throws an error without this change in place but runs successfully with this fix.

QC

• The PR contains a test case for the changes or the changes are already covered by an existing test case.
• The documentation (docs/) is updated to reflect the changes or this is not necessary (e.g. if the change does neither modify the language nor the behavior or functionalities of Snakemake).

[johanneskoester]

Ah, this was already fixed by a different PR before. Nevertheless, the test case is very useful, so I will keep and merge that. Thanks!

[johanneskoester]

Did you commit the test case? I cannot see it in the diff.

[jhrcook]

Sorry, I didn’t realize that I needed to force add the test files to git. I have included them in the latest commit.

[sonarcloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
No Duplication information