Openembedded Core Discussions
 help / color / mirror / Atom feed
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



             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