fix(core): search for electron in a node_modules folder with electron in it

[vhashimotoo]

• I have read the contribution documentation for this project.
• I agree to follow the code of conduct that this project follows, as appropriate.
• The changes are appropriately documented (if applicable).
• The changes have sufficient test coverage (if applicable).
• The test suite passes successfully on my local machine (if applicable).

Summarize your changes:

This PR tries to fix the case in using with yarn workspaces where yarn also creates a node_modules folder inside the package directory with .bin files. This adds a packageName when searching for the node_modules directory to find the folder with the electron inside.

[codecov]

Codecov Report

Merging #2326 (c7fd715) into master (cd0f052) will decrease coverage by 0.13%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #2326      +/-   ##
==========================================
- Coverage   72.32%   72.19%   -0.14%     
==========================================
  Files          76       76              
  Lines        2287     2287              
  Branches      437      437              
==========================================
- Hits         1654     1651       -3     
- Misses        471      473       +2     
- Partials      162      163       +1     

Impacted Files	Coverage Δ
packages/api/core/src/util/electron-version.ts	88.88% <100.00%> (-4.77%)	⬇️

---

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 cd0f052...c7fd715. Read the comment docs.

[malept]

Thanks for the PR. This seems like a reasonable change. Can you add a test for this? I imagine you'd have to add some fixtures.

[vhashimotoo]

@malept the tests were added, not sure why coverage is still low, maybe because this start to exit early when the electron folder is not found in node_modules (on PackageNotFoundError) instead in some other place, so this now returns the different error when a folder doesn't exist (if needed I can check if this doesn't break a more things):

master:

this pr: