numaru / vscode-ceedling-test-adapter

Ceedling Test Adapter for the VS Code Test Explorer
MIT License
37 stars 13 forks source link

UNRESPONSIVE extension host: 'numaru.vscode-ceedling-test-adapter' when using ceedlingExplorer.testCaseMacroAliases #107

Open olowo726 opened 2 years ago

olowo726 commented 2 years ago

Hi,

For version 1.10.1 I experience

[warning] UNRESPONSIVE extension host: 'numaru.vscode-ceedling-test-adapter' took 97% of 5184.417ms, saved PROFILE here: 'file:///tmp/exthost-f33d44.cpuprofile' [{"id":"numaru.vscode-ceedling-test-adapter","total":5005607,"percentage":97},{"id":"program","total":178571,"percentage":3}]

when

"ceedlingExplorer.testCaseMacroAliases": [
        "TEST_VALUE"
      ]

It doesn't get stuck for any other value for testCaseMacroAliases but then the only alias used in our code is TEST_VALUE

olowo726 commented 2 years ago

This would be very useful for me so just let me know if I can help in any way.

numaru commented 2 years ago

Hello, I tried to reproduce without success. I suspect the function regex but I would like to reproduce to be able to fix it. With 100 files which contains 32 tests each, I'm still below 150ms to parse the functions.

Is your project open source? If not, could you make a "minimal project" which has the issue and share it?

olowo726 commented 2 years ago

Hi,

Unfortunately the project isn't open source but I managed to reduce it to quite a small project that I can share with the issue, see attached namura.zip .

numaru commented 2 years ago

Is there the issue without any .c files? I have a "Ceedling: Error" node in the test view but not the described unresponsive message

olowo726 commented 2 years ago

Yea, it's there even when I remove all source files.

On Wed, Jun 22, 2022, 21:47 Kin Numaru @.***> wrote:

Is there the issue without any .c files? I have a "Ceedling: Error" node in the test view but not the described unresponsive message

— Reply to this email directly, view it on GitHub https://github.com/numaru/vscode-ceedling-test-adapter/issues/107#issuecomment-1163535077, or unsubscribe https://github.com/notifications/unsubscribe-auth/AGK2BOHUIAGY3SLW3PYIH7DVQNUVRANCNFSM5YJTRLCQ . You are receiving this because you authored the thread.Message ID: @.***>

numaru commented 2 years ago

Arg, it's gonna be a tough one :smirk_cat: I tried to install the other extensions used in settings.json but still no message. Should I expect a pop up or a message in the terminal? Does the issue only happen with "TEST_VALUE" when no source is here also?

olowo726 commented 2 years ago

For where the message appears see attached screenshot. Screenshot from 2022-06-22 22-31-05

The issue is gone if I remove the testCaseMacroAlias from the settings.json in the zip attached earlier in this thread.

numaru commented 2 years ago

Can you share the generated .cpuprofile file?

olowo726 commented 2 years ago

Sure, attached (renamed to .txt go get through github filter) exthost-7546ff.cpuprofile.txt