All of lore.kernel.org
 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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.