test(pipenv): Rewrite test mocks

[zharinov]

Changes

Prepare the codebase for usage of external detect-tools package: mock fs-extra instead of internal fs utils.

This should make our extracted code to act as drop-in replacement with no other changes required. The alternative could be mocking of the detect-tools itself, but in this case we are losing test coverage (while pretending we have it).

Context

• Ref: Move tool detection to renovatebot/detect-tools #29000

Documentation (please check one with an [x])

• I have updated the documentation, or
• No documentation update is required

How I've tested my work (please select one)

I have verified these changes via:

• Code inspection only, or
• Newly added/modified unit tests, or
• No unit tests but ran on a real repository, or
• Both unit tests + ran on a real repository

[viceice]

🤔 looks like a step back from our tooling best practices. we explicitly want to not mock fs-extra to allow partial real fa reads.

[zharinov]

pipenv <- fs <- fs-extra
          ^^
          Good: mock our own code which we sure works fine

pipenv <- detect-tools <- fs-extra
          ^^^^^^^^^^^^
          Bad: mock the code we never run on production
               and we not sure if it's correct

pipenv <- fs <- fs-extra
                ^^^^^^^^
                This PR: mock the underlying level to make sure
                new external tool behaves similar

pipenv <- detect-tool <- fs-extra
          ^^^^^^^^^^^    ^^^^^^^^
          Next PR: the implementation is replaced,
                   but tests are still working because
                   we target the underlying level

[rarkins]

@zharinov failures on windows:

    - Expected  - 3
    + Received  + 3

      Array [
        Array [
    -     "/tmp/renovate/cache/others/pipenv",
    +     "D:/tmp/renovate/cache/others/pipenv",
        ],
        Array [
    -     "/tmp/renovate/cache/others/pip",
    +     "D:/tmp/renovate/cache/others/pip",
        ],
        Array [
    -     "/tmp/renovate/cache/others/virtualenvs",
    +     "D:/tmp/renovate/cache/others/virtualenvs",
        ],
      ]

      910 |     ]);
      911 |
    > 912 |     expect(fsExtra.ensureDir.mock.calls).toEqual([
          |                                          ^
      913 |       ['/tmp/renovate/cache/others/pipenv'],
      914 |       ['/tmp/renovate/cache/others/pip'],
      915 |       ['/tmp/renovate/cache/others/virtualenvs'],

[renovate-release]

🎉 This issue has been resolved in version 37.421.8 🎉

The release is available on:

• GitHub release
• npm package (@latest dist-tag)
• 37.421.8

Your semantic-release bot 📦🚀