All of lore.kernel.org
 help / color / mirror / Atom feed
From: brian avery <avery.brian@gmail.com>
To: bitbake-devel@lists.openembedded.org
Subject: [PATCH 0/5] toaster: fix mysql migrations and add django test fixture
Date: Thu, 18 Feb 2016 21:21:43 -0800	[thread overview]
Message-ID: <cover.1455859137.git.avery.brian@gmail.com> (raw)

This series contains a clean up of the database migrations to fix the issue that was preventing Toaster from starting up when using mysql backend.
See https://bugzilla.yoctoproject.org/show_bug.cgi?id=9116

After fixing these migrations and getting Toaster to start up properly using mysql backend a number of unittests failed due to differences in testing on sqlite.
To remove these differences I've changed the data that was in the setUp to be more agnostic and added .lower() where needed to make sure the orderby re-ordering in the same way. I also took this opportunitiy to convert the toastergui unit tests to use a django fixture making it easier to load the database state for testing.

Bugs fixed discovered by fixing the unit tests are:
 - toaster: SoftwareRecipesTable apply default order_by
 - toaster: toastergui Fix invalid char test and implementation
and one already reported
 - toaster: PackagesTable show only installed packages

The following changes since commit eb71785922100bf0f52c3429e60568524d488fd3:

  toaster: update the meta-yocto toaster configuration file (2016-02-16 17:37:47 -0800)

are available in the git repository at:

  git://git.yoctoproject.org/poky-contrib bavery/test/michaelw/mysql-migration
  http://git.yoctoproject.org/cgit.cgi/poky-contrib/log/?h=bavery/test/michaelw/mysql-migration

Michael Wood (5):
  toaster: orm migrations Sort out migrations mess
  toaster: SoftwareRecipesTable apply default order_by
  toaster: toastergui unit tests convert to use fixtures
  toaster: PackagesTable show only installed packages
  toaster: toastergui Fix invalid char test and implementation

 lib/toaster/orm/migrations/0001_initial.py         |  26 --
 .../orm/migrations/0002_auto_20151210_1209.py      |  41 --
 .../orm/migrations/0002_customimagerecipe.py       |  24 ++
 .../orm/migrations/0003_customimagepackage.py      |   2 +-
 lib/toaster/orm/migrations/0004_merge.py           |  15 -
 ...0002_auto_20151223_1528.py => 0004_provides.py} |   2 +-
 .../orm/migrations/0005_auto_20160118_1055.py      |  19 -
 .../0006_customimagerecipe_last_updated.py         |  19 -
 .../fixtures/toastergui-unittest-data.xml          | 446 +++++++++++++++++++++
 lib/toaster/toastergui/tables.py                   |   5 +-
 lib/toaster/toastergui/tests.py                    | 257 ++----------
 lib/toaster/toastergui/views.py                    |   7 +-
 12 files changed, 524 insertions(+), 339 deletions(-)
 delete mode 100644 bitbake/lib/toaster/orm/migrations/0002_auto_20151210_1209.py
 create mode 100644 bitbake/lib/toaster/orm/migrations/0002_customimagerecipe.py
 delete mode 100644 bitbake/lib/toaster/orm/migrations/0004_merge.py
 rename bitbake/lib/toaster/orm/migrations/{0002_auto_20151223_1528.py => 0004_provides.py} (94%)
 delete mode 100644 bitbake/lib/toaster/orm/migrations/0005_auto_20160118_1055.py
 delete mode 100644 bitbake/lib/toaster/orm/migrations/0006_customimagerecipe_last_updated.py
 create mode 100644 bitbake/lib/toaster/toastergui/fixtures/toastergui-unittest-data.xml

--
1.9.1


             reply	other threads:[~2016-02-19  5:15 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-02-19  5:21 brian avery [this message]
2016-02-19  5:21 ` [PATCH 1/5] toaster: orm migrations Sort out migrations mess brian avery
2016-02-19  5:21 ` [PATCH 2/5] toaster: SoftwareRecipesTable apply default order_by brian avery
2016-02-19  5:21 ` [PATCH 3/5] toaster: toastergui unit tests convert to use fixtures brian avery
2016-02-19  5:21 ` [PATCH 4/5] toaster: PackagesTable show only installed packages brian avery
2016-02-19  5:21 ` [PATCH 5/5] toaster: toastergui Fix invalid char test and implementation brian avery

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.1455859137.git.avery.brian@gmail.com \
    --to=avery.brian@gmail.com \
    --cc=bitbake-devel@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.