From: Ed Bartosh <ed.bartosh@linux.intel.com>
To: openembedded-core@lists.openembedded.org
Subject: [PATCH 00/10] #10618: Increase oe-selftest coverage of wic codebase (first part)
Date: Wed, 22 Mar 2017 15:42:24 +0200 [thread overview]
Message-ID: <cover.1490188583.git.ed.bartosh@linux.intel.com> (raw)
Hi,
The main goal of these changes was to increase test coverage by testing rawcopy plugin,
which was not covered by the wic test suite. While working on it I found and fixed
quite nasty bug in sparse_copy API and made set of other important changes to make
the test case working.
I've also removed rootfs_pcbios_ext plugin, which was broken for a long time and
cleaned up wic test suite to keep it in a good shape.
The goal set in bugzilla issue #10618 is to increase test coverage to at least 80%.
Now it's only 74%, so more changes will have to be made. This patchset is just a
first part of the work in this area.
Note for project maintainers:
This patchset is based on Maciej's "enhancements for non-x86 MACHINE support & fixed-size tests"
patchset: http://lists.openembedded.org/pipermail/openembedded-core/2017-March/134410.html so,
please merge it in the right order.
The following changes since commit e07ca8bd2bd4a8fa1fb9a0f3a8d9ff9a8f9c46c8:
wic: selftest: add tests for --fixed-size partition flags (2017-03-21 15:17:24 +0200)
are available in the git repository at:
git://git.yoctoproject.org/poky-contrib ed/wic/wip
http://git.yoctoproject.org/cgit.cgi/poky-contrib/log/?h=ed/wic/wip
Ed Bartosh (10):
wic: remove rootfs_pcbios_ext plugin
wic: don't set creator.bootimg_dir in plugins
wic: use kernel_dir instead of bootimg_dir
wic: fix bug in sparse_copy
wic: use STAGING_DATADIR as a default for bootimg_dir
wic: remove empty plugin methods
wic: improve getting syslinux path
oe-selftest: test wic rawcopy plugin
selftest: pylinted wic test suite.
oe-selftest: add test_rawcopy_plugin test case
meta-selftest/wic/test_rawcopy_plugin.wks.in | 6 +
meta/lib/oeqa/selftest/wic.py | 84 +++++---
scripts/lib/wic/filemap.py | 1 +
scripts/lib/wic/plugins/source/bootimg-efi.py | 16 +-
.../lib/wic/plugins/source/bootimg-partition.py | 32 +---
scripts/lib/wic/plugins/source/bootimg-pcbios.py | 53 ++---
scripts/lib/wic/plugins/source/fsimage.py | 19 --
scripts/lib/wic/plugins/source/rawcopy.py | 29 +--
.../lib/wic/plugins/source/rootfs_pcbios_ext.py | 213 ---------------------
scripts/wic | 3 +-
10 files changed, 113 insertions(+), 343 deletions(-)
create mode 100644 meta-selftest/wic/test_rawcopy_plugin.wks.in
delete mode 100644 scripts/lib/wic/plugins/source/rootfs_pcbios_ext.py
--
2.1.4
next reply other threads:[~2017-03-22 13:42 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-03-22 13:42 Ed Bartosh [this message]
2017-03-22 13:42 ` [PATCH 01/10] wic: remove rootfs_pcbios_ext plugin Ed Bartosh
2017-03-22 13:42 ` [PATCH 02/10] wic: don't set creator.bootimg_dir in plugins Ed Bartosh
2017-03-22 13:42 ` [PATCH 03/10] wic: use kernel_dir instead of bootimg_dir Ed Bartosh
2017-03-22 13:42 ` [PATCH 04/10] wic: fix bug in sparse_copy Ed Bartosh
2017-03-22 13:42 ` [PATCH 05/10] wic: use STAGING_DATADIR as a default for bootimg_dir Ed Bartosh
2017-03-22 13:42 ` [PATCH 06/10] wic: remove empty plugin methods Ed Bartosh
2017-03-22 13:42 ` [PATCH 07/10] wic: improve getting syslinux path Ed Bartosh
2017-03-22 13:42 ` [PATCH 08/10] oe-selftest: test wic rawcopy plugin Ed Bartosh
2017-03-22 13:42 ` [PATCH 09/10] selftest: pylinted wic test suite Ed Bartosh
2017-03-22 13:42 ` [PATCH 10/10] oe-selftest: add test_rawcopy_plugin test case Ed Bartosh
2017-03-22 20:53 ` ✗ patchtest: failure for #10618: Increase oe-selftest coverage of wic codebase (first part) Patchwork
2017-03-23 2:57 ` Paul Eggleton
2017-03-23 16:35 ` Leonardo Sandoval
2017-03-23 16:36 ` Leonardo Sandoval
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=cover.1490188583.git.ed.bartosh@linux.intel.com \
--to=ed.bartosh@linux.intel.com \
--cc=openembedded-core@lists.openembedded.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox