linux-kernel - Re: [PATCH v1] kunit: tool: add missing test data file content

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Mon, 23 Mar 2020 12:15:54 -0700
From:   Brendan Higgins <brendanhiggins@...gle.com>
To:     shuah <shuah@...nel.org>
Cc:     David Gow <davidgow@...gle.com>,
        Heidi Fahim <heidifahim@...gle.com>,
        "open list:KERNEL SELFTEST FRAMEWORK" 
        <linux-kselftest@...r.kernel.org>,
        KUnit Development <kunit-dev@...glegroups.com>,
        Linux Kernel Mailing List <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH v1] kunit: tool: add missing test data file content

On Mon, Mar 23, 2020 at 11:44 AM shuah <shuah@...nel.org> wrote:
>
> On 3/23/20 11:36 AM, Brendan Higgins wrote:
> > A test data file for one of the kunit_tool unit tests was missing; add
> > it in so that unit tests can run properly.
> >
> > Signed-off-by: Brendan Higgins <brendanhiggins@...gle.com>
> > ---
> > Shuah, this is a fix for a broken test. Can you apply this for 5.7?
>
> Can you please add more details on what kind of data this file supplies
> and which test fails if it doesn't exist.

Sure,

This adds a raw dmesg test log to test the kunit_tool's dmesg parser.
test_prefix_poundsign and test_output_with_prefix_isolated_correctly
fail without this test log.

I will send a new revision with this information in the commit message.

> > ---
> >   .../testing/kunit/test_data/test_pound_sign.log  | Bin 0 -> 1656 bytes
> >   1 file changed, 0 insertions(+), 0 deletions(-)
> >
> > diff --git a/tools/testing/kunit/test_data/test_pound_sign.log b/tools/testing/kunit/test_data/test_pound_sign.log
> > index e69de29bb2d1d6434b8b29ae775ad8c2e48c5391..28ffa5ba03bfa81ea02ea9d38e7de7acf3dd9e5d 100644
> > GIT binary patch
> > literal 1656
> > zcmah}U2EGg6n$=g#f7|Vtj^>lPBOzDM#o@...9+Kgd${FPEBlGBgsqs?{^h<Y3h$o
> > zFBaGLocpP>13GNVmW<8=R3_K%5Q9W*u~4upWe`4q(jqBTdcAw?ZG=v-jA5@...^*5
> > zoU$NALGF+lEFssq<A{~zdHR7p&7+U(X~E!_yGM{l@...Q%(~pazHH!+GB!sI;iCKZ
> > zY69Cjp-?V{LrnyMQ5I_>Rp5<1_i#FmdPY1z2tkYI|M1-7PdS}Ub_h8eK~m)?&(I;{
> > zd<2<NV1vyl7BWOggn_Hc5ba`wRu)R=x;oPiRuheYD}$9XJTpphG^wKX*mr|pw(;#T
> > zTvPxWb#R&-VC|~9KeFD0ooNCooO~P|@...L)n#t&WQm2JSh%gFRMuv7!M#yqYailx
> > z8TM&AUN~yqVM$ThVIE55OcVU4mW$eHC#dHEeUvCiE1wT#?U%cS^A_HAK$VqiIBG75
> > z($V`G!unJPuo@k2@...y7$WV7g73Ls@...giPqc=`3mz^u|IR`05hOx29+=__XXIv
> > z%Xf#6<%P11b*dyatBVv$thED!=x%_Ts?sj1OY%b*t$Y}rODc$J2is^#<A~w+w`~mf
> > zCs_oC7u=9pAjzurLE}*e38}&1-KX^pd*7wM-Q35(VDtTJOgeOnB`K*rii#c_+)*qi
> > zNQ+5Dqv>MG0wcp<uf!}(SCXw)kqXk>xCSP@...Rs58c`TmTbn>%WO@TFp;w*gB6RU
> > km;Ljlo8cvfMVVCc>`K4bOfE$-fO&T9$C>+RbV7Fh7t6}$ApigX
> >
> > literal 0
> > HcmV?d00001
> >
> >
> > base-commit: 021ed9f551da33449a5238e45e849913422671d7
> >
>
> thanks,
> -- Shuah

Cheers