fix: remove leading slash on Windows

[Ascenio]

Status

READY

Description

Removes leading / for file paths on Windows during license checking, by leveraging toFilePath instead of path.

Fixes #1227.

Uri
Before:

Before:

Logs
Before:

After:

Type of Change

• ✨ New feature (non-breaking change which adds functionality)
• 🛠️ Bug fix (non-breaking change which fixes an issue)
• ❌ Breaking change (fix or feature that would cause existing functionality to change)
• 🧹 Code refactor
• ✅ Build configuration change
• 📝 Documentation
• 🗑️ Chore

[alestiago]

Hi @Ascenio!

Thanks for raising the PR. Could you make sure to add some tests to the change?

[Ascenio]

Hi @Ascenio!

Thanks for raising the PR. Could you make sure to add some tests to the change?

I think the tests kind of are already there because the only way to test this would be to validate the whole packages check licenses command, for which there are already tests.

Just enabling the tests to run for windows would be enough.

Considering only the licenses command:
Before:

After:

Considering the whole test suite:
Before:

After:

[Ascenio]

The problem is we can't enable all of them at once or else the pipeline will break. It has to be incremental, by adding @TestOn('vm') for all files at first, and then removing it but also adding testOn: 'vm||windows for each test that was fixed to support both platforms.