From mboxrd@z Thu Jan 1 00:00:00 1970 From: Luis Chamberlain Subject: Re: [PATCH v5 13/18] kunit: tool: add Python wrappers for running KUnit tests Date: Wed, 26 Jun 2019 22:03:50 +0000 Message-ID: <20190626220350.GA19023@42.do-not-panic.com> References: <20190617082613.109131-1-brendanhiggins@google.com> <20190617082613.109131-14-brendanhiggins@google.com> <20190626000150.GT19023@42.do-not-panic.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: Sender: linux-kernel-owner@vger.kernel.org To: Brendan Higgins Cc: Frank Rowand , Greg KH , Josh Poimboeuf , Kees Cook , Kieran Bingham , Peter Zijlstra , Rob Herring , Stephen Boyd , shuah , Theodore Ts'o , Masahiro Yamada , devicetree , dri-devel , kunit-dev@googlegroups.com, "open list:DOCUMENTATION" , linux-fsdevel@vger.kernel.org, linux-kbuild , Linux Kernel Mailing List , "open list:KERNEL SELFTEST FRAMEWORK" List-Id: devicetree@vger.kernel.org On Wed, Jun 26, 2019 at 01:02:55AM -0700, Brendan Higgins wrote: > On Tue, Jun 25, 2019 at 5:01 PM Luis Chamberlain wrote: > > > > On Mon, Jun 17, 2019 at 01:26:08AM -0700, Brendan Higgins wrote: > > > create mode 100644 tools/testing/kunit/test_data/test_is_test_passed-all_passed.log > > > create mode 100644 tools/testing/kunit/test_data/test_is_test_passed-crash.log > > > create mode 100644 tools/testing/kunit/test_data/test_is_test_passed-failure.log > > > create mode 100644 tools/testing/kunit/test_data/test_is_test_passed-no_tests_run.log > > > create mode 100644 tools/testing/kunit/test_data/test_output_isolated_correctly.log > > > create mode 100644 tools/testing/kunit/test_data/test_read_from_file.kconfig > > > > Why are these being added upstream? The commit log does not explain > > this. > > Oh sorry, those are for testing purposes. I thought that was clear > from being in the test_data directory. I will reference it in the > commit log in the next revision. Still, I don't get it. They seem to be results from a prior run. Why do we need them for testing purposes? Luis