From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on dcvr.yhbt.net X-Spam-Level: X-Spam-ASN: X-Spam-Status: No, score=-4.0 required=3.0 tests=ALL_TRUSTED,BAYES_00 shortcircuit=no autolearn=ham autolearn_force=no version=3.4.2 Received: from localhost (dcvr.yhbt.net [127.0.0.1]) by dcvr.yhbt.net (Postfix) with ESMTP id 7B8B11F463 for ; Sun, 24 Nov 2019 00:22:37 +0000 (UTC) From: Eric Wong To: meta@public-inbox.org Subject: [PATCH 00/17] test fixes and cleanups Date: Sun, 24 Nov 2019 00:22:20 +0000 Message-Id: <20191124002237.15713-1-e@80x24.org> In-Reply-To: <20191116114333.GA19173@dcvr> References: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit List-Id: There's some fixes for race conditions around daemon startup and shutdown and resurrects start_script for slightly improved test performance. And slowly eliminating all END{} block usages Eric Wong (17): tests: disable daemon workers in a few more places tests: use strict everywhere t/v1-add-remove-add: quiet down "git init" t/xcpdb-reshard: test xcpdb --compact t/httpd-corner: wait for worker process death t/nntpd-tls: sometimes SSL_connect succeeds quickly .gitignore: ignore local prove(1) files daemon: use sigprocmask to block signals at startup daemon: use sigprocmask when respawning workers daemon: avoid race when quitting workers t/common: start_script replaces spawn_listener t/nntpd-validate: get rid of threads dependency xapcmd: replace Xtmpdirs with File::Temp->newdir tests: use File::Temp->newdir instead of tempdir() tests: quiet down commit graph t/perf-*.t: use $ENV{GIANT_INBOX_DIR} consistently tests: move giant inbox/git dependent tests to xt/ .gitignore | 2 + MANIFEST | 11 ++- lib/PublicInbox/Daemon.pm | 35 +++++-- lib/PublicInbox/Xapcmd.pm | 73 +++++--------- t/.gitconfig | 4 + t/admin.t | 4 +- t/altid.t | 4 +- t/altid_v2.t | 3 +- t/cgi.t | 3 +- t/common.perl | 184 ++++++++++++++++++++++++++--------- t/config.t | 4 +- t/convert-compact.t | 3 +- t/edit.t | 3 +- t/emergency.t | 4 +- t/feed.t | 3 +- t/filter_rubylang.t | 5 +- t/git.t | 6 +- t/html_index.t | 4 +- t/httpd-corner.psgi | 2 +- t/httpd-corner.t | 70 +++++++------ t/httpd-https.t | 31 ++---- t/httpd-unix.t | 51 +++++----- t/httpd.t | 16 ++- t/import.t | 4 +- t/indexlevels-mirror.t | 3 +- t/init.t | 3 +- t/mda.t | 3 +- t/mda_filter_rubylang.t | 3 +- t/mid.t | 1 + t/msgmap.t | 4 +- t/nntpd-tls.t | 42 +++----- t/nntpd.t | 19 ++-- t/nulsubject.t | 4 +- t/over.t | 4 +- t/plack.t | 4 +- t/psgi_attach.t | 4 +- t/psgi_bad_mids.t | 4 +- t/psgi_mount.t | 4 +- t/psgi_multipart_not.t | 4 +- t/psgi_scan_all.t | 4 +- t/psgi_search.t | 4 +- t/psgi_text.t | 4 +- t/psgi_v2.t | 3 +- t/purge.t | 4 +- t/qspawn.t | 1 + t/replace.t | 6 +- t/search-thr-index.t | 4 +- t/search.t | 4 +- t/solver_git.t | 3 +- t/spamcheck_spamc.t | 4 +- t/v1-add-remove-add.t | 6 +- t/v1reindex.t | 3 +- t/v2-add-remove-add.t | 3 +- t/v2mda.t | 3 +- t/v2mirror.t | 26 ++--- t/v2reindex.t | 3 +- t/v2writable.t | 13 ++- t/watch_filter_rubylang.t | 3 +- t/watch_maildir.t | 5 +- t/watch_maildir_v2.t | 10 +- t/www_listing.t | 11 +-- t/xcpdb-reshard.t | 7 +- {t => xt}/git-http-backend.t | 19 ++-- {t => xt}/nntpd-validate.t | 57 +++++------ {t => xt}/perf-msgview.t | 6 +- {t => xt}/perf-nntpd.t | 34 +++---- {t => xt}/perf-threading.t | 8 +- 67 files changed, 462 insertions(+), 431 deletions(-) create mode 100644 t/.gitconfig rename {t => xt}/git-http-backend.t (87%) rename {t => xt}/nntpd-validate.t (85%) rename {t => xt}/perf-msgview.t (85%) rename {t => xt}/perf-nntpd.t (79%) rename {t => xt}/perf-threading.t (72%)