Fix git diff detection code

[joyceerhl]

For https://github.com/microsoft/vscode-python/issues/13569

I regressed this as part of the filesystem refactor. In the following snippet, editor.document.uri has scheme file, but the git diff editor has scheme git, so doing a full URI object comparison via arePathsSame always returned false. I believe this is an exceptional case where we should be using fsPath instead of the URI object when comparing URIs.

        const gitSchemeEditor = this.documentManager.visibleTextEditors.find(
            (editorUri) =>
                editorUri.document &&
                editorUri.document.uri.scheme === 'git' &&
                this.fs.arePathsSame(editorUri.document.uri, editor.document.uri)
        );

• Pull request represents a single change (i.e. not fixing disparate/unrelated things in a single PR).
• Title summarizes what is changing.
• Has a news entry file (remember to thank yourself!).
• Appropriate comments and documentation strings in the code.
• Has sufficient logging.
• Has telemetry for enhancements.
• Unit tests & system/integration tests are added/updated.
• Test plan is updated as appropriate.
• package-lock.json has been regenerated by running npm install (if dependencies have changed).
• The wiki is updated with any design decisions/details.

[sonarqubecloud]

Kudos, SonarCloud Quality Gate passed!

0 Bugs
0 Vulnerabilities (and 0 Security Hotspots to review)
0 Code Smells

No Coverage information
No Duplication information

[codecov-commenter]

Codecov Report

Merging #13851 into main will decrease coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main   #13851      +/-   ##
==========================================
- Coverage   59.98%   59.95%   -0.03%     
==========================================
  Files         685      685              
  Lines       38006    38006              
  Branches     5474     5474              
==========================================
- Hits        22797    22788       -9     
- Misses      14034    14040       +6     
- Partials     1175     1178       +3     

Impacted Files	Coverage Δ
src/client/datascience/crossProcessLock.ts	79.41% <0.00%> (-11.77%)	⬇️
src/client/common/utils/platform.ts	56.00% <0.00%> (-8.00%)	⬇️
src/client/linters/pydocstyle.ts	86.66% <0.00%> (-2.23%)	⬇️
src/client/datascience/debugLocationTracker.ts	76.56% <0.00%> (-1.57%)	⬇️
src/client/common/process/proc.ts	14.49% <0.00%> (-0.73%)	⬇️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6741162...4a5769f. Read the comment docs.

[DonJayamanne]

Suggested change

	editorUri.document.uri.fsPath === editor.document.uri.fsPath
	editorUri.document.uri.fsPath === editor.document.uri.fsPath

Adding a comment would be useful & probably adding a test to ensure this works with different resource schemes (I'm assuming that's the case here).

[DonJayamanne]

I.e. we need to ensure we do not regress again, comment + test would be useful.

[joyceerhl]

Oh, sorry, merged this before this comment showed up! I'll put up a new PR @DonJayamanne