Update paths to run on windows

[liza-mae]

Replace backslash with forward slash for windows, to fix unit test failures with getRelativePath(), getRelativeParentDirs() and getInstalledPackages()

Failures:

1. dev/File #getRelativePath() returns the path relative to the repo root:

      Error: expected 'src\\dev\\__tests__\\file.js' to sort of equal 'src/dev/__tests__/file.js'
      + expected - actual

      -src\dev\__tests__\file.js
      +src/dev/__tests__/file.js
      
      at Assertion.assert (node_modules\expect.js\index.js:96:13)
      at Assertion.eql (node_modules\expect.js\index.js:230:10)
      at Context.it (C:/Users/vagrant/kibana/src/dev/__tests__/file.js:41:41)

2. dev/File #getRelativeParentDirs() returns the parents of a file, stopping at the repo root, in descending order:

      Error: expected [ 'src\\dev\\__tests__', 'src\\dev', 'src' ] to sort of equal [ 'src/dev/__tests__', 'src/dev', 'src' ]
      + expected - actual

       [
      -  "src\\dev\\__tests__"
      -  "src\\dev"
      +  "src/dev/__tests__"
      +  "src/dev"
         "src"
       ]
      
      at Assertion.assert (node_modules\expect.js\index.js:96:13)
      at Assertion.eql (node_modules\expect.js\index.js:230:10)
      at Context.it (C:/Users/vagrant/kibana/src/dev/__tests__/file.js:71:47)

3. src/dev/npm/installed_packages getInstalledPackages() reads all installed packages of a module:

 Error: expected [ { name: 'dep1',
    version: '0.0.2',
    repository: 'https://github.com/mycorp/dep1',
    licenses: [ 'Apache-2.0' ],
    directory: 'C:\\Users\\vagrant\\kibana\\src\\dev\\npm\\__tests__\\fixtures\\fixture1\\node_modules\\dep1',
    relative: 'node_modules\\dep1' },
  { name: 'privatedep',
    version: '0.0.2',
    repository: 'https://github.com/mycorp/privatedep',
    licenses: [ 'Apache-2.0' ],
    directory: 'C:\\Users\\vagrant\\kibana\\src\\dev\\npm\\__tests__\\fixtures\\fixture1\\node_modules\\privatedep',
    relative: 'node_modules\\privatedep' } ] to sort of equal [ { name: 'dep1',
    version: '0.0.2',
    licenses: [ 'Apache-2.0' ],
    repository: 'https://github.com/mycorp/dep1',
    directory: 'C:\\Users\\vagrant\\kibana\\src\\dev\\npm\\__tests__\\fixtures\\fixture1\\node_modules\\dep1',
    relative: 'node_modules/dep1' },
  { name: 'privatedep',
    version: '0.0.2',
    repository: 'https://github.com/mycorp/privatedep',
    licenses: [ 'Apache-2.0' ],
    directory: 'C:\\Users\\vagrant\\kibana\\src\\dev\\npm\\__tests__\\fixtures\\fixture1\\node_modules\\privatedep',
    relative: 'node_modules/privatedep' } ]
      + expected - actual

           "licenses": [
             "Apache-2.0"
           ]
           "name": "dep1"
      -    "relative": "node_modules\\dep1"
      +    "relative": "node_modules/dep1"
           "repository": "https://github.com/mycorp/dep1"
           "version": "0.0.2"
         }
         {
           "licenses": [
             "Apache-2.0"
           ]
           "name": "privatedep"
      -    "relative": "node_modules\\privatedep"
      +    "relative": "node_modules/privatedep"
           "repository": "https://github.com/mycorp/privatedep"
           "version": "0.0.2"
         }
       ]
      
      at Assertion.assert (node_modules\expect.js\index.js:96:13)
      at Assertion.eql (node_modules\expect.js\index.js:230:10)
      at Context.it (C:/Users/vagrant/kibana/src/dev/npm/__tests__/installed_packages.js:57:35)

Part of issue: #21315

[elasticmachine]

💚 Build Succeeded

• continuous-integration/kibana-ci/pull-request

[liza-mae]

https://www.npmjs.com/package/upath

[elasticmachine]

💚 Build Succeeded

• continuous-integration/kibana-ci/pull-request

[tylersmalley]

Apologies for the delay, I was on holiday the week prior to EAH.

I don't believe the issue is in the implementation here, but rather the tests which are explicitly testing a unix path. When running on Windows the path module will assume Windows-style paths, otherwise, it will use POSIX. You can force a specific interpreter to be used through path.{posix,win32}. Can we instead update the tests to use path.posix.resolve()?

[liza-mae]

@tylersmalley I am not sure I understand the change, can you point it to me in the code where you want this done?

[liza-mae]

Closing per #24281