numaru / vscode-ceedling-test-adapter

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

Windows 10: Ceedling failed to run in the configured Shell #46

Open kbsteere opened 4 years ago

kbsteere commented 4 years ago

I have attempted to include the bash.exe, powershell.exe and all paths to those as well but I still get the error message: "Ceedling failed to run in the configured shell. Please check the ceedlingExplorer.shellPath option."

numaru commented 4 years ago

Can you run the ceedling command line in those shells ?

kbsteere commented 4 years ago

Yes, I can run it with cmd and powershell without any issue.

On Fri, Mar 20, 2020, 16:01 Kin Numaru notifications@github.com wrote:

Can you run the ceedling command line in those shells ?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/numaru/vscode-ceedling-test-adapter/issues/46#issuecomment-601886862, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADKHVI4ENBJBZVVZ3RATMWTRIPDTDANCNFSM4LQSQOCA .

numaru commented 4 years ago

Is this message prompted when you clicked the error in the tree view?

image

If yes, I think the issue is that the extension require the xml_test_report plugin to parse the results. Also, a line needs to be added in the project.yml

:plugins:
  :load_paths:
    - vendor/ceedling/plugins
  :enabled:
+   - xml_tests_report
    - stdout_pretty_tests_report
    - module_generator
    - gcov
numaru commented 4 years ago

Also, what is your ceedling version?

I tried with these ones on w10:

C:\Users\kin\Documents\Laboratory\github\win32\vscode-ceedling-test-adapter [master ≡]
> ceedling version
Welcome to Ceedling!
  Ceedling::   0.28.3
  CMock::      2.4.5
  Unity::      2.4.2
  CException:: 1.3.1
C:\Users\kin\Documents\Laboratory\github\win32\vscode-ceedling-test-adapter [master ≡]
> (get-command ceedling).path
C:\Ruby24-x64\bin\ceedling.bat

with no ceedlingExplorer.shellPath defined and I don't get this issue.

kbsteere commented 4 years ago

I never see the error about xml_reports

Here are my versions VS Code : 1.43.1 Electron: 7.1.11 Node.js: 12.8.1

Welcome to Ceedling! Ceedling:: 0.29.1 CMock:: 2.5.1 Unity:: 2.5.0 CException:: 1.3.2

On Sat, Mar 21, 2020 at 5:30 AM Kin Numaru notifications@github.com wrote:

Also, what is your ceedling version?

I tried with these ones on w10:

C:\Users\kin\Documents\Laboratory\github\win32\vscode-ceedling-test-adapter [master ≡]

ceedling version

Welcome to Ceedling!

Ceedling:: 0.28.3

CMock:: 2.4.5

Unity:: 2.4.2

CException:: 1.3.1

C:\Users\kin\Documents\Laboratory\github\win32\vscode-ceedling-test-adapter [master ≡]

(get-command ceedling).path

C:\Ruby24-x64\bin\ceedling.bat

with no ceedlingExplorer.shellPath defined.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/numaru/vscode-ceedling-test-adapter/issues/46#issuecomment-602019035, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADKHVI6Z33LM4WUPN7VFBRDRISCMXANCNFSM4LQSQOCA .

-- Kyle B. Steere

numaru commented 4 years ago

I updated my Ceedling version and I'm still be able to use the extension. 🐱

To find out if the issue come from the environment or the Ceedling project, can you create a new project using the command ceedling example temp_sensor? I'm using this project to test the extension. I assume we are testing with the lastest ceedling and vscode versions, using no extension settings.

The extension shows the error message if it fails to run the command ceedling summary in the ceedlingExplorer.projectPath.

e.g.

C:\Users\kin\Documents\Laboratory\github\win32\temp_sensor
> ceedling summary ; write-host $? ; write-host $env:comspec

--------------------
OVERALL TEST SUMMARY
--------------------
TESTED:  47
PASSED:  47
FAILED:   0
IGNORED:  0

--------------------------
GCOV: OVERALL TEST SUMMARY
--------------------------

No tests executed.

NOTE: Summaries may be out of date with project sources.

True
C:\WINDOWS\system32\cmd.exe
mb81 commented 4 years ago

I can do this: ceedling

But i am not able to get the plugin running due to "Ceedling failed to run in the configured shell. Please check the ceedlingExplorer.shellPath option." Is there a way to get this running using wsl? Ceedling:: 0.29.1 Unity:: 2.5.0 CMock:: 2.5.1 CException:: 1.3.2

numaru commented 4 years ago

@mb81 Despite the fact the error message is the same, I see the problem you are describing as a separated one because this is not the same shell used. I think it deserve its own issue.

Anyway, I tried with:

and it works.

A workaround could be to use the wsl extension of vscode. see #39

mb81 commented 4 years ago

Thank you for letting me trying again the "bash" in shellPath, which i tried several times, and in different variants. It works ^^ . Also it works with the remote Wsl connection.
Nice tool :)