Skip to content

clawrtc config path test assumes Unix separators on Windows #5378

@hungle123-dev

Description

@hungle123-dev

Summary

miners/clawrtc/test_config.py::TestGetConfigPath::test_default_path assumes Unix path separators by checking that str(path).endswith('.clawrtc/config.json'). On Windows, get_config_path() correctly returns C:\Users\...\.clawrtc\config.json, so the assertion fails even though the returned path is correct.

Reproduction

On Windows from a fresh checkout:

python -m pytest miners\clawrtc\test_config.py -q

Actual result

Before fix:

1 failed, 24 passed

Failure:

AssertionError: assert False
'C:\\Users\\acer\\.clawrtc\\config.json'.endswith('.clawrtc/config.json')

Expected result

The test should assert the Path structure (.clawrtc parent and config.json file name) instead of hard-coding / as the separator.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions