Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 13 Oct 2016 03:20:25 -0400
From:      Julio Merino <julio@meroh.net>
To:        Ngie Cooper <ngie@freebsd.org>
Cc:        src-committers@freebsd.org, svn-src-all@freebsd.org,  svn-src-head@freebsd.org
Subject:   Re: svn commit: r307190 - head/contrib/netbsd-tests/fs/tmpfs
Message-ID:  <CADyfeQVByd6i5JqHyesodT_a6Fe=k5M5w113d_NEPBvG8xK8OA@mail.gmail.com>
In-Reply-To: <201610130702.u9D72sgd007580@repo.freebsd.org>
References:  <201610130702.u9D72sgd007580@repo.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Thu, Oct 13, 2016 at 3:02 AM, Ngie Cooper <ngie@freebsd.org> wrote:

> Author: ngie
> Date: Thu Oct 13 07:02:54 2016
> New Revision: 307190
> URL: https://svnweb.freebsd.org/changeset/base/307190
>
> Log:
>   Skip :uchg on FreeBSD
>
>   Unfortunately removing files with uchg set always succeeds with root on
>   FreeBSD. Unfortunately running the test as an unprivileged user isn't
> doable
>   because mounting tmpfs requires root
>
>   PR:           212861
>   Sponsored by: Dell EMC Isilon
>
> Modified:
>   head/contrib/netbsd-tests/fs/tmpfs/t_remove.sh
>
> Modified: head/contrib/netbsd-tests/fs/tmpfs/t_remove.sh
> ============================================================
> ==================
> --- head/contrib/netbsd-tests/fs/tmpfs/t_remove.sh      Thu Oct 13
> 06:56:23 2016        (r307189)
> +++ head/contrib/netbsd-tests/fs/tmpfs/t_remove.sh      Thu Oct 13
> 07:02:54 2016        (r307190)
> @@ -53,6 +53,10 @@ uchg_head() {
>         atf_set "require.user" "root"
>  }
>  uchg_body() {
> +       # Begin FreeBSD
> +       atf_skip "this fails on FreeBSD with root - bug 212861"
> +       # End FreeBSD
>

This is a bad way of disabling a test due to a known bug because, when the
bug is resolved, the test won't notice. You should be using the "expected
failures" functionality if possible, and based on the change description, I
think you can do this here easily. Try:

    atf_expect_fail "FreeBSD bug 213861 blah blah"

right before the line that triggers the test failure.

-- 
Julio Merino -- http://julio.meroh.net/



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CADyfeQVByd6i5JqHyesodT_a6Fe=k5M5w113d_NEPBvG8xK8OA>