facebookarchive / nuclide

An open IDE for web and native mobile development, built on top of Atom
https://nuclide.io
Other
7.79k stars 682 forks source link

"Nuclide->Working Sets->Find in active" is not working. #1616

Open ezh opened 5 years ago

ezh commented 5 years ago

Issue and Steps to Reproduce

"Nuclide->Working Sets->Find in active" is not working. Screenshot of issue This is remote folder.

Expected Behavior

Nuclide should list files with appropriate content in "Project Find Results" tab.

Actual Behavior

There is always empty result.

Versions

Additional Details

P.S. I didn't find any unit tests related to this function. Should we write them?

pelmers commented 5 years ago

actually there is some tests for this function: https://github.com/facebook/nuclide/blob/master/pkg/nuclide-remote-projects/__atom_tests__/RemoteDirectorySearcher-test.js#L43-L52

But I think the issue here is a windows specific bug -- the path expansion is happening locally (so it's getting a Windows path) but the search is remote (so expects / separators)