Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test: console output is polluted with output from lightspeed server #1481

Open
1 of 4 tasks
ssbarnea opened this issue Aug 1, 2024 · 0 comments
Open
1 of 4 tasks
Labels
bug Something isn't working qa Quality Assurance: CI, testing, coverage, linting.

Comments

@ssbarnea
Copy link
Member

ssbarnea commented Aug 1, 2024

Our console logs when runnings tests is polluted with output from others tools and services that run in parallel, making it harder to debug. We need to address these by ensuring that they end-up in specific log files that are collected.

output pollution with: test unit

Please note that this pollution happens with passing tests! If this would be visible on screen on failed tests, it would have not being a problem.

Only mocha output should be visible for passing tests.

✔ suggestion has no matches
{ contentmatches: [ { contentmatch: [Array] } ] }
YAMLParseError: Unexpected scalar at node end at line 2, column 1:

- name: foo
I
^

    at Composer.onError (/home/runner/work/vscode-ansible/vscode-ansible/node_modules/yaml/dist/compose/composer.js:69:34)
    at Object.resolveEnd (/home/runner/work/vscode-ansible/vscode-ansible/node_modules/yaml/dist/compose/resolve-end.js:31:21)
    at Object.composeDoc (/home/runner/work/vscode-ansible/vscode-ansible/node_modules/yaml/dist/compose/compose-doc.js:37:27)
...    ✔ suggest has invalid YAML

lightspeed server pollution with: test ui

{"level":"info","message":"{\n  \"playbookGenerationAction\": {\n    \"wizardId\": \"d406325e-dcf8-4741-afe5-0181e6571209\",\n    \"action\": 0,\n    \"toPage\": 1\n  },\n  \"metadata\": {\n    \"ansibleExtensionVersion\": \"0.0.0\"\n  }\n}"}

pollution with: test e2e

Spamming messages such below must go in specific log file, not test console:

Ansible found in the workspace
[file-inspection] language set by file inspection
[inline-suggestions] ...
[ansible-lightspeed] ...

Remediation

Tasks

@ssbarnea ssbarnea added qa Quality Assurance: CI, testing, coverage, linting. bug Something isn't working labels Aug 1, 2024
ssbarnea added a commit that referenced this issue Aug 2, 2024
ssbarnea added a commit that referenced this issue Aug 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working qa Quality Assurance: CI, testing, coverage, linting.
Projects
Status: No status
Development

No branches or pull requests

1 participant