[VSC-1515] fix unit tests not being refresh

[brianignacio5]

Description

Fix unit tests not being refreshed when Refresh tests button is clicked.

Fixes #1336

Type of change

• Bug fix (non-breaking change which fixes an issue)

Steps to test this pull request

1. Open a ESP-IDF project like unity-test that contains test cases as described in https://docs.espressif.com/projects/vscode-esp-idf-extension/en/latest/additionalfeatures/unit-testing.html
2. Add a new test case in any file test_*.c file
3. Press menu View then Testing Click the Refresh Tests button in the top of the side (Looks like a loading button)
4. The new test should appear.

• Expected behaviour:
  Test cases should be added, deleted or modified when Refresh Tests button is executed and test cases have changed.

• Expected output:

How has this been tested?

Manual testing using the unity-test esp-idf example.

Test Configuration:

• ESP-IDF Version: 5.0
• OS (Windows,Linux and macOS): MacOS

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:
You can test these changes by installing this VSIX by click menu View -> Command Palette..., type Install from VSIX and then select downloaded esp-idf-extension.vsix file to install the extension.

• esp-idf-extension.vsix.zip
  Espressif documentation preview

[radurentea]

LGTM

[Fabricio-ESP]

Hi @brianignacio5 ,
Confirmed that the new fix capture new test_ files created for the components. I may not be fully familiar with the functionality but is it correct to only look for test_ files in the components folder?

Also noticed that if I rename a test file and try to update the testing tab I see a new entry with the new file name, fur the application does not remove the old entry, so it ends up showing 2 files with the old and new name. If I restart VSCode than it removes the old entry. Is there a reason we would like to keep the old filename on the testing tab?

[brianignacio5]

Hi @brianignacio5 , Confirmed that the new fix capture new test_ files created for the components. I may not be fully familiar with the functionality but is it correct to only look for test_ files in the components folder?

Yes that is correct that is looking for test_* files.

Also noticed that if I rename a test file and try to update the testing tab I see a new entry with the new file name, fur the application does not remove the old entry, so it ends up showing 2 files with the old and new name. If I restart VSCode than it removes the old entry. Is there a reason we would like to keep the old filename on the testing tab?

Updated the PR to delete all items before creating them on refresh. PTAL again @Fabricio-ESP

[Fabricio-ESP]

Confirmed working as expected.
Testing tab was able to locate test files and update the list upon clicking the refresh button.
Note that test files should start with test_ and should be inside a folder called test.
This is not properly set int he unit_test example provided by IDF and might confuse the users.