2020-03-23 17:38:12

by Brendan Higgins

[permalink] [raw]
Subject: [PATCH v1] kunit: tool: add missing test data file content

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 <[email protected]>
---
Shuah, this is a fix for a broken test. Can you apply this for 5.7?
---
.../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@mlt9+Kgd${FPEBlGBgsqs?{^h<Y3h$o
zFBaGLocpP>13GNVmW<8=R3_K%5Q9W*u~4upWe`4q(jqBTdcAw?ZG=v-jA5@FZ|^*5
zoU$NALGF+lEFssq<A{~zdHR7p&7+U(X~E!_yGM{l@40vQ%(~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|@vNKL)n#t&WQm2JSh%gFRMuv7!M#yqYailx
z8TM&AUN~yqVM$ThVIE55OcVU4mW$eHC#dHEeUvCiE1wT#?U%cS^A_HAK$VqiIBG75
z($V`G!unJPuo@k2@gj4y7$WV7g73Ls@d32giPqc=`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@rQbRs58c`TmTbn>%WO@TFp;w*gB6RU
km;Ljlo8cvfMVVCc>`K4bOfE$-fO&T9$C>+RbV7Fh7t6}$ApigX

literal 0
HcmV?d00001


base-commit: 021ed9f551da33449a5238e45e849913422671d7
--
2.25.1.696.g5e7596f4ac-goog


2020-03-23 18:45:41

by Shuah Khan

[permalink] [raw]
Subject: Re: [PATCH v1] kunit: tool: add missing test data file content

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 <[email protected]>
> ---
> 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.

> ---
> .../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@mlt9+Kgd${FPEBlGBgsqs?{^h<Y3h$o
> zFBaGLocpP>13GNVmW<8=R3_K%5Q9W*u~4upWe`4q(jqBTdcAw?ZG=v-jA5@FZ|^*5
> zoU$NALGF+lEFssq<A{~zdHR7p&7+U(X~E!_yGM{l@40vQ%(~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|@vNKL)n#t&WQm2JSh%gFRMuv7!M#yqYailx
> z8TM&AUN~yqVM$ThVIE55OcVU4mW$eHC#dHEeUvCiE1wT#?U%cS^A_HAK$VqiIBG75
> z($V`G!unJPuo@k2@gj4y7$WV7g73Ls@d32giPqc=`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@rQbRs58c`TmTbn>%WO@TFp;w*gB6RU
> km;Ljlo8cvfMVVCc>`K4bOfE$-fO&T9$C>+RbV7Fh7t6}$ApigX
>
> literal 0
> HcmV?d00001
>
>
> base-commit: 021ed9f551da33449a5238e45e849913422671d7
>

thanks,
-- Shuah

2020-03-23 19:17:58

by Brendan Higgins

[permalink] [raw]
Subject: Re: [PATCH v1] kunit: tool: add missing test data file content

On Mon, Mar 23, 2020 at 11:44 AM shuah <[email protected]> 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 <[email protected]>
> > ---
> > 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@mlt9+Kgd${FPEBlGBgsqs?{^h<Y3h$o
> > zFBaGLocpP>13GNVmW<8=R3_K%5Q9W*u~4upWe`4q(jqBTdcAw?ZG=v-jA5@FZ|^*5
> > zoU$NALGF+lEFssq<A{~zdHR7p&7+U(X~E!_yGM{l@40vQ%(~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|@vNKL)n#t&WQm2JSh%gFRMuv7!M#yqYailx
> > z8TM&AUN~yqVM$ThVIE55OcVU4mW$eHC#dHEeUvCiE1wT#?U%cS^A_HAK$VqiIBG75
> > z($V`G!unJPuo@k2@gj4y7$WV7g73Ls@d32giPqc=`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@rQbRs58c`TmTbn>%WO@TFp;w*gB6RU
> > km;Ljlo8cvfMVVCc>`K4bOfE$-fO&T9$C>+RbV7Fh7t6}$ApigX
> >
> > literal 0
> > HcmV?d00001
> >
> >
> > base-commit: 021ed9f551da33449a5238e45e849913422671d7
> >
>
> thanks,
> -- Shuah

Cheers

2020-03-23 19:25:59

by Brendan Higgins

[permalink] [raw]
Subject: Re: [PATCH v1] kunit: tool: add missing test data file content

On Mon, Mar 23, 2020 at 12:15 PM Brendan Higgins
<[email protected]> wrote:
>
> On Mon, Mar 23, 2020 at 11:44 AM shuah <[email protected]> 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 <[email protected]>
> > > ---
> > > 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.

I just sent a new revision with an updated message.