You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@couchdb.apache.org by "janl (via GitHub)" <gi...@apache.org> on 2025/12/20 10:01:57 UTC
[PR] feat: concat eunit test logs and remove test tmp dirs in Makefile [couchdb]
janl opened a new pull request, #5830:
URL: https://github.com/apache/couchdb/pull/5830
This allows debugging of test runs in a unified log.
It also retains individual test run state dirs if
`make eunit` fails. The next successful run will clean it up
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@couchdb.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] feat: concat eunit test logs and remove test tmp dirs in Makefile [couchdb]
Posted by "nickva (via GitHub)" <gi...@apache.org>.
nickva commented on code in PR #5830:
URL: https://github.com/apache/couchdb/pull/5830#discussion_r2638942396
##########
Makefile:
##########
@@ -193,6 +193,8 @@ eunit: export ERL_AFLAGS = -config $(CURDIR)/rel/files/eunit.config
eunit: export COUCHDB_QUERY_SERVER_JAVASCRIPT = $(CURDIR)/bin/couchjs $(CURDIR)/share/server/main.js
eunit: export COUCHDB_TEST_ADMIN_PARTY_OVERRIDE=1
eunit: ${SUBDIRS}
+ @cat tmp/couchdb-tests/*/log/couch.log > tmp/couch.log
Review Comment:
In case of an eunit failure we won't run the commands here so we won't get all the logs. We only get them on success. But we may want the opposite case - in case of success ignore them, but in case of failure inspect them.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@couchdb.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] feat: concat eunit test logs and remove test tmp dirs in Makefile [couchdb]
Posted by "janl (via GitHub)" <gi...@apache.org>.
janl merged PR #5830:
URL: https://github.com/apache/couchdb/pull/5830
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@couchdb.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org