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

mute some stderr outputs in tests #1538

Merged
merged 1 commit into from
Aug 16, 2020
Merged

mute some stderr outputs in tests #1538

merged 1 commit into from
Aug 16, 2020

Conversation

kmuto
Copy link
Owner

@kmuto kmuto commented Aug 16, 2020

テストで標準エラーにWARNを出してしまうものがあり、取り込む or 捨てる で対処しました。

@kmuto kmuto merged commit 59d6267 into master Aug 16, 2020
@kmuto kmuto deleted the mute_stderr branch August 16, 2020 07:00
@takahashim
Copy link
Collaborator

この修正について、コードが間違っててWARNが出てしまうケースの場合は出力してくれた方がありがたそうなんですが、それも抑制したりしないでしょうか?(test/test_latexbuilder.rbの修正は適切そうです)

@takahashim
Copy link
Collaborator

takahashim commented Aug 16, 2020

それと、 warning: duplicate ID: inlineicon のwarningが出るのはテストケースの問題というよりも本質的な仕様の問題(@<icon>{foo}が複数出現する場合はどういう扱いにするか)なので、別途issueを立てておきます

@takahashim
Copy link
Collaborator

↑の件は #1539 に書いておきました

@kmuto
Copy link
Owner Author

kmuto commented Aug 16, 2020

makerのほうはrevertしましょうか。iconのほうはなんとかしたい。

@kmuto
Copy link
Owner Author

kmuto commented Aug 16, 2020

というよりは、iconのほうをまず直して、STDERRがあったらassertエラーを起こさせたほうがよさそうかな。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants