unofficial mirror of meta@public-inbox.org
 help / color / mirror / Atom feed
From: "Thomas Weißschuh" <thomas@t-8ch.de>
To: Eric Wong <e@80x24.org>
Cc: meta@public-inbox.org
Subject: Re: [PATCH] t/index-git-times: support non-master default branch
Date: Tue, 26 Oct 2021 07:05:27 +0200	[thread overview]
Message-ID: <ebf92218-1470-4602-b534-6dae59639dc6@t-8ch.de> (raw)
In-Reply-To: <20211025225822.GA28875@dcvr>

[-- Attachment #1: Type: text/plain, Size: 653 bytes --]

Hi,

On 2021-10-25 16:58-0600, Eric Wong wrote:
> Thanks, pushed as 9f3fb207.   It looks like there's several
> other tests broken with non-master being the default.  Can we
> expect fixes for those, too?  Thanks again.

Which tests are failing for you?

The only failures I still see are in t/lei-q-remote-import.t which is unrelated
to the git branch and some more that fail only during parallel test execution.

In t/lei-q-remote-import.t it seems the search results are not memoized to the
local store. (See attachment)

If you are able to reproduce those issues I'd be happy to have them fixed by
you. If not I will try to figure them out.

Thomas

[-- Attachment #2: lei-q-remote-import.txt --]
[-- Type: text/plain, Size: 6074 bytes --]

# lei_err=# /tmp/pi-lei-q-remote-import-286963-YR1S/lei-daemon/.config/lei/config created
# # /tmp/pi-lei-q-remote-import-286963-YR1S/lei-daemon/.local/share/lei/store 0/0
# # 0 written to /tmp/pi-lei-q-remote-import-286963-YR1S/lei-daemon/o.mboxrd (0 matches)
ok 1 - lei q -o mboxrd:/tmp/pi-lei-q-remote-import-286963-YR1S/lei-daemon/o.mboxrd m:qp@example.com
ok 2 - output exists but is empty
# lei_err=# /usr/bin/curl -Sf -s -d '' http://[::1]:38791/t2/?x=m&q=m%3Aqp%40example.com
# # /tmp/pi-lei-q-remote-import-286963-YR1S/lei-daemon/.local/share/lei/store 0/0
# # http://[::1]:38791/t2/ 1/1
# # 1 written to /tmp/pi-lei-q-remote-import-286963-YR1S/lei-daemon/o.mboxrd (1 matches)
ok 3 - lei q -o mboxrd:/tmp/pi-lei-q-remote-import-286963-YR1S/lei-daemon/o.mboxrd m:qp@example.com -I http://[::1]:38791/t2/
ok 4 - got results after remote search
# lei_err=# /tmp/pi-lei-q-remote-import-286963-YR1S/lei-daemon/.local/share/lei/store 0/0
# # 0 written to /tmp/pi-lei-q-remote-import-286963-YR1S/lei-daemon/o.mboxrd (0 matches)
ok 5 - lei q -o mboxrd:/tmp/pi-lei-q-remote-import-286963-YR1S/lei-daemon/o.mboxrd m:qp@example.com
not ok 6 - output exists after import but is not empty
#   Failed test 'output exists after import but is not empty'
#   at t/lei-q-remote-import.t line 39.
# # /tmp/pi-lei-q-remote-import-286963-YR1S/lei-daemon/.local/share/lei/store 0/0
# # 0 written to /tmp/pi-lei-q-remote-import-286963-YR1S/lei-daemon/o.mboxrd (0 matches)
not ok 7 - got results w/o remote search
#   Failed test 'got results w/o remote search'
#   at t/lei-q-remote-import.t line 41.
#     Structures begin differing at:
#          $got->[0] = Does not exist
#     $expected->[0] = PublicInbox::Eml=HASH(0x55fea9904d90)
# lei_err=# /usr/bin/curl -Sf -s -d '' http://[::1]:38791/t2/?x=m&q=m%3A199707281508.AAA24167%40hoyogw.example
# # /tmp/pi-lei-q-remote-import-286963-YR1S/lei-daemon/.local/share/lei/store 0/0
# # http://[::1]:38791/t2/ 1/1
# # 1 written to /tmp/pi-lei-q-remote-import-286963-YR1S/lei-daemon/o.mboxrd (1 matches)
ok 8 - lei q -o mboxrd:/tmp/pi-lei-q-remote-import-286963-YR1S/lei-daemon/o.mboxrd m:199707281508.AAA24167@hoyogw.example -I http://[::1]:38791/t2/ --no-import-remote
ok 9 - got another after remote search
# lei_err=# /tmp/pi-lei-q-remote-import-286963-YR1S/lei-daemon/.local/share/lei/store 0/0
# # 0 written to /tmp/pi-lei-q-remote-import-286963-YR1S/lei-daemon/o.mboxrd (0 matches)
ok 10 - lei q -o mboxrd:/tmp/pi-lei-q-remote-import-286963-YR1S/lei-daemon/o.mboxrd m:199707281508.AAA24167@hoyogw.example
ok 11 - --no-import-remote did not memoize
# lei_err=# /tmp/pi-lei-q-remote-import-286963-YR1S/lei-daemon/.local/share/lei/store 0/0
# # 0 written to /tmp/pi-lei-q-remote-import-286963-YR1S/lei-daemon/o.mboxrd (0 matches)
ok 12 - lei q -o mboxrd:/tmp/pi-lei-q-remote-import-286963-YR1S/lei-daemon/o.mboxrd m:qp@example.com --lock=none
not ok 13 - --lock=none respected
#   Failed test '--lock=none respected'
#   at t/lei-q-remote-import.t line 55.
# # /tmp/pi-lei-q-remote-import-286963-YR1S/lei-daemon/.local/share/lei/store 0/0
# # 0 written to /tmp/pi-lei-q-remote-import-286963-YR1S/lei-daemon/o.mboxrd (0 matches)
# lei_err=dotlock timeout /tmp/pi-lei-q-remote-import-286963-YR1S/lei-daemon/o.mboxrd.lock
ok 14 - dotlock fails
ok 15 - timeout noted
ok 16 - nothing output on lock failure
# lei_err=# /tmp/pi-lei-q-remote-import-286963-YR1S/lei-daemon/.local/share/lei/store 0/0
# # 0 written to /tmp/pi-lei-q-remote-import-286963-YR1S/lei-daemon/o.mboxrd (0 matches)
ok 17 - lei q -o mboxrd:/tmp/pi-lei-q-remote-import-286963-YR1S/lei-daemon/o.mboxrd m:qp@example.com --lock=dotlock,timeout=0.000001 (succeeds after lock removal)
ok 18 - lei add-external -q .../public-inbox.org/t/data-gen/lei-q-remote-import.local-external
ok 19 - lei q -q -o mboxrd:/tmp/pi-lei-q-remote-import-286963-YR1S/lei-daemon/o.mboxrd --only http://[::1]:38791/t2/ m:testmessage@example.com
ok 20 - no warnings or errors
ok 21 - got result from remote external
ok 22 - got expected result
ok 23 - lei q --no-external -o mboxrd:/dev/stdout m:testmessage@example.com
ok 24 - message not imported when in local external
# lei_err=# /tmp/pi-lei-q-remote-import-286963-YR1S/lei-daemon/.local/share/lei/store 0/0
# # .../public-inbox.org/t/data-gen/lei-q-remote-import.local-external 1/1
# # 1 written to /tmp/pi-lei-q-remote-import-286963-YR1S/lei-daemon/o.mboxrd (1 matches)
ok 25 - lei q -o mboxrd:/tmp/pi-lei-q-remote-import-286963-YR1S/lei-daemon/o.mboxrd m:testmessage@example.com
ok 26 - got expected result after clobber
ok 27 - lei q -o mboxrd:/dev/stdout m:never-before-seen@example.com
not ok 28 - --import-before imported totally unseen message
#   Failed test '--import-before imported totally unseen message'
#   at t/lei-q-remote-import.t line 94.
#                   ''
#     doesn't match '(?^ms:seen\@example\.com>\nStatus: RO\n\nwhatever)'
# lei_err=# /usr/bin/curl -Sf -s -d '' http://[::1]:38791/t2/?x=m&q=z%3A0..
# # http://[::1]:38791/t2/ 15/15
# # 15 written to /tmp/pi-lei-q-remote-import-286963-YR1S/lei-daemon/md/ (15 matches)
ok 29 - lei q --save z:0.. -o /tmp/pi-lei-q-remote-import-286963-YR1S/lei-daemon/md --only http://[::1]:38791/t2/
# lei_err=# http://[::1]:38791/t2/ limiting to 2021-10-24  7:03 +0200 and newer
# # /usr/bin/curl -Sf -s -d '' http://[::1]:38791/t2/?x=m&q=(z%3A0..)+AND+dt%3A20211024050357..
# # http://[::1]:38791/t2/ 11/11
# # 0 written to /tmp/pi-lei-q-remote-import-286963-YR1S/lei-daemon/md/ (11 matches)
ok 30 - lei up /tmp/pi-lei-q-remote-import-286963-YR1S/lei-daemon/md
ok 31 - lei up remote dedupe works on maildir
ok 32 - lei edit-search /tmp/pi-lei-q-remote-import-286963-YR1S/lei-daemon/md
ok 33 - lastresult set
ok 34 - lei daemon-pid (daemon-pid after t/lei-q-remote-import.t:106)
ok 35 - daemon running after t/lei-q-remote-import.t:106
ok 36 - lei daemon-kill (daemon-kill after t/lei-q-remote-import.t:106)
ok 37 - t/lei-q-remote-import.t:106 daemon stopped
ok 38 - t/lei-q-remote-import.t:106 daemon XDG_RUNTIME_DIR/lei/errors.log empty
1..38
# Looks like you failed 4 tests of 38.

  reply	other threads:[~2021-10-26  5:05 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-10-25 22:24 [PATCH] t/index-git-times: support non-master default branch Thomas Weißschuh
2021-10-25 22:58 ` Eric Wong
2021-10-26  5:05   ` Thomas Weißschuh [this message]
2021-10-26  5:28     ` Eric Wong
2021-10-26 18:43       ` Thomas Weißschuh
2021-10-27 21:15         ` lei-q-remote-import failures [was: [PATCH] t/index-git-times: support non-master default branch] Eric Wong
2021-10-27 21:24           ` Thomas Weißschuh
2021-10-27 23:48             ` Eric Wong
2021-10-28 14:03               ` Thomas Weißschuh
2021-10-28 19:16                 ` [PATCH] test_common: clear XDG_CACHE_HOME before lei tests Eric Wong
2021-10-28 19:22                   ` Thomas Weißschuh
2021-10-26 21:24       ` [PATCH] t/index-git-times: support non-master default branch Thomas Weißschuh
2021-10-27  4:07         ` [PATCH] test_common: key test inboxes to init.defaultBranch Eric Wong
2021-10-27  5:04           ` Thomas Weißschuh

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://public-inbox.org/README

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=ebf92218-1470-4602-b534-6dae59639dc6@t-8ch.de \
    --to=thomas@t-8ch.de \
    --cc=e@80x24.org \
    --cc=meta@public-inbox.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).