git@vger.kernel.org mailing list mirror (one of many)
 help / color / mirror / code / Atom feed
From: Elijah Newren <newren@gmail.com>
To: "Mark T. Ortell" <mtortell@ra.rockwell.com>
Cc: "git@vger.kernel.org" <git@vger.kernel.org>
Subject: Re: EXTERNAL: Re: List of Known Issues for a particular release
Date: Thu, 11 Jul 2019 13:04:48 -0700	[thread overview]
Message-ID: <CABPp-BFYYP4RjgxPqvJjfctCZq347dWi=Vy6S2W3R2mQ2Ks7_g@mail.gmail.com> (raw)
In-Reply-To: <CY4PR2201MB1559CC4E242F3689124D99AF9AF30@CY4PR2201MB1559.namprd22.prod.outlook.com>

Hi Mark,

On Thu, Jul 11, 2019 at 12:02 PM Mark T. Ortell
<mtortell@ra.rockwell.com> wrote:
>
> Elijah,
>
> Thanks for the response. I am not clear whether the test_expect_failure means that the test is trying to do something that should fail and so it is a valid test case or if it is a test case that is failing, but should succeed and has only been temporarily disabled until it is fixed. I'm guessing the former. In this case, if it successfully did whatever it were testing, that would be an issue. A simple example of this would be a test to try to login with an invalid username and password. That is expected to fail and if it passed, it would be an issue. If this is the case, then it doesn't look like it provides a list of issues. Please clarify what the test_expect_failure indicates.

Please don't top-post on this list.

test_expect_failure (as well as other helper functions in the test
harness library, such as test_expect_success, test_must_fail,
test_might_fail, etc.) are explained in t/README.  By its definition,
it technically satisfies "list of known issues" as you asked for.
However, most software products that publish a list of known issues
has probably curated problems that users are likely to see or be
curious about, and which they want to inform users of both to reduce
support load and help users avoid problems.

This list is not curated in any such way.  It's just a list of issues
developers thought to document for themselves and/or other developers.
It is thus way different than what you might want:

(1) There is evidence that some have used it for "In an ideal world,
this thing should support this feature too in which case I'd expect it
to behave a certain way that it doesn't yet."  The line between
feature (what works is fine but we could make it better) and bug (it's
not really correct if it doesn't do it this way) gets really blurry at
times, and you'd pick a much different tradeoff in communication
between developers than you would in communication from developers to
users; with other developers you spend a lot more time talking about
internals and goals and direction we'd like to move the software in.

(2) Also, some of these "known breakages" could be in corner cases
that are very unlikely to be hit be users, and perhaps not only likely
to be hit by individual users, but unlikely that anyone anywhere will
ever hit that error (some of the merge recursive tests I added might
fall into that category).

(3) There may also be cases where someone once thought that optimal
behavior would be a little different and that they were planning to
implement more features, and then later changed their mind but forgot
to clean up the testcases.

(4) ...and that's just a few off the top of my head.  I'm sure the
list has several other things that make it not quite match what you
want.

As such, Brian's answer to your question elsewhere in this thread is
probably better than mine, but if by chance you are just being forced
to go through a box checking exercise and there's no reason for
needing these results other than that someone asked that they be
provided (I sometimes had to go through such exercises when I worked
at Sandia National Labs years ago), then technically the command I
gave you could be used to satisfy it.

> Below is the output from the provided command:
<snip>

Not sure why you included this.

> Regards,
> Mark


Best Wishes,
Elijah

  reply	other threads:[~2019-07-11 20:05 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-07-11 15:28 List of Known Issues for a particular release Mark T. Ortell
2019-07-11 15:59 ` Elijah Newren
2019-07-11 19:02   ` EXTERNAL: " Mark T. Ortell
2019-07-11 20:04     ` Elijah Newren [this message]
2019-07-11 16:09 ` brian m. carlson

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: http://vger.kernel.org/majordomo-info.html

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

  git send-email \
    --in-reply-to='CABPp-BFYYP4RjgxPqvJjfctCZq347dWi=Vy6S2W3R2mQ2Ks7_g@mail.gmail.com' \
    --to=newren@gmail.com \
    --cc=git@vger.kernel.org \
    --cc=mtortell@ra.rockwell.com \
    /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.
Code repositories for project(s) associated with this public inbox

	https://80x24.org/mirrors/git.git

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).