From mboxrd@z Thu Jan 1 00:00:00 1970 From: Michal Simek Date: Fri, 11 Oct 2019 08:50:15 +0200 Subject: [U-Boot] [PATCH] test/py: hush_if_test: Add tests to cover octal/hex values In-Reply-To: References: <2861e9ee042ad93a9d36f551cd90ce7cbc6030aa.1570707876.git.michal.simek@xilinx.com> Message-ID: <54db2685-3437-eed8-73fa-63d95a59b3ea@xilinx.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On 10. 10. 19 19:06, Simon Glass wrote: > Hi Michal, > > On Thu, 10 Oct 2019 at 05:44, Michal Simek wrote: >> >> Extend test suite to cover also automatic octal/hex converstions which >> haven't been implemented in past. >> >> Signed-off-by: Michal Simek >> --- >> >> Depends on https://lists.denx.de/pipermail/u-boot/2019-September/383309.html >> >> There are of course other tests which we can run but not sure if make sense >> to have there all combinations. The most interesting are mixed tests which >> are failing before patch above is applied. >> Definitely please let me know if you want to add any other test. >> --- >> test/py/tests/test_hush_if_test.py | 27 +++++++++++++++++++++++++++ >> 1 file changed, 27 insertions(+) >> > > I worry that these tests might be very slow since it requires a lot of > interaction with U-Boot over a pipe. Is it possible to put them in C > code instead, e.g. cmd_ut? I have of course running it on my HW and it is quite fast. It is just 16 more simple tests. And if this breaks gitlab/travis CI loops then we have bigger problem. I see that CONFIG_UNIT_TEST is enabled only for sandbox. Do it make sense to enable it also for others to have better coverage? Thanks, Michal