Add validation for existing setup path

[radurentea]

Description

Added validation for esp-idf path, when user selects "Find ESP-IDF in your system" while configuring the extension.

Jira: VSC-1174

Type of change

• Enhancement

Steps to test this pull request

• Configure extension:
  
• Choose "Select ESP-IDF version:

1. Try to do it with a valid esp-idf folder (it should work)
2. Try to do it with an invalid esp-idf folder (error should appear and install button should be disabled)
   

How has this been tested?

As described above

Test Configuration:

• ESP-IDF Version: 5.2.2
• OS (Windows,Linux and macOS): Windows 11

Checklist

• PR Self Reviewed
• Applied Code formatting
• Added Documentation
• Added Unit Test
• Verified on all platforms - Windows,Linux and macOS

[github-actions]

Download the artifacts for this pull request:

• esp-idf-extension.vsix.zip

[brianignacio5]

Few comments but otherwise LGTM

[radurentea]

This should not be tested yet. New improvements were requested by Kondal and I'm currently working on implementing them.

[radurentea]

All changes are done, can be tested

[AndriiFilippov]

@radurentea hi !

Tested under:
OS - Windows 10

tested with multiple esp-idf folders, do see validation ✔️

But there is a small issue with validation:
The ESP-IDF path always stores the valid path. If you want to reconfigure your extension and during reconfiguration select the wrong path - there is a validation ✔️ BUT -> click "Home" and return back. It will still store the correct (previous) path, but the invalid path selection will remain.

[radurentea]

@AndriiFilippov Thank you, I've fixed the issue. When you have time please test it again

[radurentea]

Fixed and I've also stylized a bit also the following bug found by Andrii on the configuration page, where we were displaying the text wrongly.

[AndriiFilippov]

@radurentea hi !

good 👍

validation should works only for existing ESP-IDF folder.
Download ESP-IDF case:

[AndriiFilippov]

@radurentea hi !

everything works ✔️
LGTM 👍
but it is still uncomfortable to get validation just on unfocus. Are there any chances to somehow maneuver around this situation?

[radurentea]

Hi Andrii, we've discussed this before the implementation and we agreed on this. We can't make it real time because there will be to many calls

[radurentea]

Hi @brianignacio5, PTAL again

[brianignacio5]

A few code duplication changes required.

On MacOS, I see the following error after pressing Express install option and the UI is empty so can't continue:

[radurentea]

A few code duplication changes required.

On MacOS, I see the following error after pressing Express install option and the UI is empty so can't continue:

Hi @brianignacio5, please take another look with the new updates. I've tested on Mac and I couldn't not reproduce the error.

[brianignacio5]

Tested on MacOS and the previous issue doesn't happen anymore.

LGTM

[radurentea]

Hi @AndriiFilippov, please test when you have a moment.

[AndriiFilippov]

@radurentea hi !

Tested under:
OS - Windows 11 / MacOS

LGTM 👍