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

appveyor should print boost test_suite log on failure #16976

Closed
maflcko opened this issue Sep 27, 2019 · 2 comments · Fixed by #18562
Closed

appveyor should print boost test_suite log on failure #16976

maflcko opened this issue Sep 27, 2019 · 2 comments · Fixed by #18562

Comments

@maflcko
Copy link
Member

maflcko commented Sep 27, 2019

See

- cmd: src\test_bitcoin.exe -k stdout -e stdout 2> NUL

However, it should use the -l test_suite log level and show the whole output on failure, similar to

$(AM_V_at)$(TEST_BINARY) -l test_suite -t "`cat $< | grep -E "(BOOST_FIXTURE_TEST_SUITE\\(|BOOST_AUTO_TEST_SUITE\\()" | cut -d '(' -f 2 | cut -d ',' -f 1 | cut -d ')' -f 1`" > $<.log 2>&1 || (cat $<.log && false)

@GoelBiju
Copy link

GoelBiju commented Oct 8, 2019

Hi @MarcoFalke , I am quite new to this project and I wanted to help out. Should I add in the -l test_suite argument to the appveyor YAML file? Is there anything from the example mentioned in Makefile.test.include that I need to include after the -l argument, i.e. -t "`cat $< | grep -E "(BOOST_FIXTURE_TEST_SUITE\\(|BOOST_AUTO_TEST_SUITE\\()" | cut -d '(' -f 2 | cut -d ',' -f 1 | cut -d ')' -f 1`" > $<.log 2>&1 || (cat $<.log && false) ?

@maflcko
Copy link
Member Author

maflcko commented Oct 9, 2019

I think you'd have to write -l test_suite > some_log_name.foo || (cat some_log_name.foo && false) in powershell or whatever language the appveyor thing is using

ernestognw added a commit to ernestognw/bitcoin that referenced this issue Oct 11, 2019
@maflcko maflcko closed this as completed in dcef5ad Apr 9, 2020
jasonbcox pushed a commit to Bitcoin-ABC/bitcoin-abc that referenced this issue Oct 21, 2020
…_minversion

Summary:
Add instructions related to creating test data introduced in [[bitcoin/bitcoin#16976 | PR16976]] (D7053)

This is a backport of Core [[bitcoin/bitcoin#16845 | PR16845]]

Test Plan: Proof-reading

Reviewers: O1 Bitcoin ABC, #bitcoin_abc, Fabien

Reviewed By: O1 Bitcoin ABC, #bitcoin_abc, Fabien

Subscribers: Fabien

Differential Revision: https://reviews.bitcoinabc.org/D8016
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Feb 15, 2022
knst pushed a commit to knst/dash that referenced this issue May 18, 2023
fa7af33 ci: Run unit tests sequential once (MarcoFalke)
fa68a3e appveyor: Enable minimal unit test logging to aid debugging (MarcoFalke)

Pull request description:

  Fixes bitcoin#16976

Top commit has no ACKs.

Tree-SHA512: 1f1ee8776a67afa8c1c5a16ef170c9975b6a486c087c8eba12e97d23382befd1c2801622ec70ca8e4cd1fbedce1dec46be67677ceaf07f35f1d3f3bead0200f0
knst pushed a commit to knst/dash that referenced this issue May 19, 2023
fa7af33 ci: Run unit tests sequential once (MarcoFalke)
fa68a3e appveyor: Enable minimal unit test logging to aid debugging (MarcoFalke)

Pull request description:

  Fixes bitcoin#16976

Top commit has no ACKs.

Tree-SHA512: 1f1ee8776a67afa8c1c5a16ef170c9975b6a486c087c8eba12e97d23382befd1c2801622ec70ca8e4cd1fbedce1dec46be67677ceaf07f35f1d3f3bead0200f0
knst pushed a commit to knst/dash that referenced this issue May 25, 2023
fa7af33 ci: Run unit tests sequential once (MarcoFalke)
fa68a3e appveyor: Enable minimal unit test logging to aid debugging (MarcoFalke)

Pull request description:

  Fixes bitcoin#16976

Top commit has no ACKs.

Tree-SHA512: 1f1ee8776a67afa8c1c5a16ef170c9975b6a486c087c8eba12e97d23382befd1c2801622ec70ca8e4cd1fbedce1dec46be67677ceaf07f35f1d3f3bead0200f0
knst pushed a commit to knst/dash that referenced this issue May 30, 2023
fa7af33 ci: Run unit tests sequential once (MarcoFalke)
fa68a3e appveyor: Enable minimal unit test logging to aid debugging (MarcoFalke)

Pull request description:

  Fixes bitcoin#16976

Top commit has no ACKs.

Tree-SHA512: 1f1ee8776a67afa8c1c5a16ef170c9975b6a486c087c8eba12e97d23382befd1c2801622ec70ca8e4cd1fbedce1dec46be67677ceaf07f35f1d3f3bead0200f0
knst pushed a commit to knst/dash that referenced this issue Jun 1, 2023
fa7af33 ci: Run unit tests sequential once (MarcoFalke)
fa68a3e appveyor: Enable minimal unit test logging to aid debugging (MarcoFalke)

Pull request description:

  Fixes bitcoin#16976

Top commit has no ACKs.

Tree-SHA512: 1f1ee8776a67afa8c1c5a16ef170c9975b6a486c087c8eba12e97d23382befd1c2801622ec70ca8e4cd1fbedce1dec46be67677ceaf07f35f1d3f3bead0200f0
PastaPastaPasta pushed a commit to knst/dash that referenced this issue Jun 7, 2023
fa7af33 ci: Run unit tests sequential once (MarcoFalke)
fa68a3e appveyor: Enable minimal unit test logging to aid debugging (MarcoFalke)

Pull request description:

  Fixes bitcoin#16976

Top commit has no ACKs.

Tree-SHA512: 1f1ee8776a67afa8c1c5a16ef170c9975b6a486c087c8eba12e97d23382befd1c2801622ec70ca8e4cd1fbedce1dec46be67677ceaf07f35f1d3f3bead0200f0
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants