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

Ses-ava console virtualization: report to t.log instead of console #891

Closed
kriskowal opened this issue Sep 14, 2021 · 6 comments · Fixed by #2109
Closed

Ses-ava console virtualization: report to t.log instead of console #891

kriskowal opened this issue Sep 14, 2021 · 6 comments · Fixed by #2109
Assignees
Labels
kriskowal-review-2024-01 Issues that kriskowal wants to bring to the attention of the team for review as of January, 2024 project Issues submitted for attention of project management at Agoric to communicate with stake-holders

Comments

@kriskowal
Copy link
Member

SES hides error stacks from callers and reveals them to the global console. SES-ava captures errors thrown in tests and reports them to the console to reveal their stacks, regardless of whether the test passes or fails, generating a great deal of decontextualized noise.

We would like to reveal the error to the test log buffer (t.log) instead. Since SES interacts thru the console API, we would need to virtualize the console for tests.

This console virtualization might be adaptable across other purposes.

@kriskowal kriskowal self-assigned this Sep 14, 2021
@dckc
Copy link
Contributor

dckc commented Sep 14, 2021

see also #731

@dckc
Copy link
Contributor

dckc commented Sep 14, 2021

#701 is in this direction, yes?

@kriskowal
Copy link
Member Author

kriskowal commented Sep 15, 2021 via email

@kriskowal kriskowal added the kriskowal-review-2024-01 Issues that kriskowal wants to bring to the attention of the team for review as of January, 2024 label Jan 10, 2024
@kriskowal
Copy link
Member Author

To get here, we still need to tease out the compartment shim from ses.

@erights
Copy link
Contributor

erights commented Mar 3, 2024

Hi @kriskowal , I said in #2109 that it would close this, and so co-assigned myself. But I don't understand your comments about needing compartment work, which #2109 does not do. If #2109 would not close this, please let me know what I'm missing.

@kriskowal kriskowal added the project Issues submitted for attention of project management at Agoric to communicate with stake-holders label Mar 7, 2024
@kriskowal
Copy link
Member Author

I misunderstood what was necessary to accomplish this. Please disregard.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kriskowal-review-2024-01 Issues that kriskowal wants to bring to the attention of the team for review as of January, 2024 project Issues submitted for attention of project management at Agoric to communicate with stake-holders
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants