delta/openstack/taskflow.git/taskflow/tests/unit/persistence/base.py, branch master

Use assertIs(Not)None to check for None

2016-11-07T08:50:39+00:00

[H203] Use assertIs(Not)None to check for None (off by default) Unit
test assertions tend to give better messages for more specific
assertions. As a result, assertIsNone(...) is preferred over
assertEqual(None, ...) and assertIs(None, ...), and assertIsNotNone(...)
is preferred over assertNotEqual(None, ...) and assertIsNot(None,
...). Off by default.

Trivial fix

Change-Id: I5b6ac7d99f0689843eb98cb3e9b9b10531322640

Fix order of assertEqual for unit.persistence

2015-10-17T08:29:23+00:00

First parameter should be the expected value.

Change-Id: If255cc908cf013b6c58f5bf666ccaa50f04f7759
Partial-Bug: #1357117

Fix how the dir persistence backend was not listing logbooks

2015-09-04T18:20:24+00:00

Due to the usage of the os.path.islink check this means that no
logbooks would be returned when get_logbooks was called, which is
not the behavior we want.

Closes-Bug: #1492403

Change-Id: Ife6a5bec777c9e2d820391914ce2c6fbbadf4f79

Rename logbook module -> models module

2015-07-09T00:04:35+00:00

Since this module contains more than the logbook
class and really is a our generic models that are used
to hold the runtime structure it is more appropriate to
place it under a models module and deprecate the usage
of the old module by placing a warning there (so that
when it is imported that warning is triggered).

Change-Id: I79def5ee08f560d38f2c9dcefd0b33becc2a4d36

lazy loading for logbooks and flowdetails

2015-03-11T20:57:07+00:00

You can now specify a "lazy" argument when asking a
persistence backend for a logbook for flowdetail.
This will return a logbook with no flowdetails in it,
or a flowdetail with no atomdetails in it, respectively.

Change-Id: Ie81a19a2107b44e2c35c2a92507bec03eee55c50

Switch to using 'oslo_utils' vs 'oslo.utils'

2015-01-15T07:56:11+00:00

Prefer the non-deprecated 'oslo_utils' instead of
the namespaced 'oslo.utils' wherever it was previously
used.

Change-Id: I9a78150ef5266e1ff22147278162fe3cfe1b2e3f

Move over to using oslo.utils [reflection, uuidutils]

2014-12-18T21:55:41+00:00

The reflection module is now part of oslo.utils so we should
remove our local version and use that version instead; this
also goes for the uuidutils module which is now part of
oslo.utils as well so we no longer need our local version
copied from the incubator...

Note that one reflection method `find_subclasses` which was to
specific to taskflow is now moved to the misc utility module
instead of its prior home in the reflection module.

Change-Id: I069881c80b0b2916cc0c414992b80171f7eeb79f

Ensure that the zookeeper backend creates missing atoms

2014-11-24T18:06:02+00:00

When 'create_missing' is true the atom should be created
instead of raising an exception; this is used when a flow
detail is updated with a new detail and then saved.

This also adds test cases that verify this happens so that
we verify this on an ongoing basis.

Fixes bug 1395812

Change-Id: I4851a08ff1ab4101dbec4a6656177908095c3c52

Remove direct usage of the deprecated failure location

2014-10-21T22:37:44+00:00

Internally we should be using the new location and not the
deprecated location wherever possible. This avoids emitting
warnings messages on our own code, which is a dirty habit.

Change-Id: Idac5a772eca7529d92542ada3be1cea092880e25

Revert "Move taskflow.utils.misc.Failure to its own module"

2014-03-31T12:59:31+00:00

This reverts commit 42ca240e8157b840c298d14fbf478ae570376633 which
was a breaking change in a library consumed by other OpenStack
projects with no deprecation or backwards compatibility
considerations. It was able to merge because openstack/taskflow is
apparently not yet part of the integrated gate via the proposed
I202f4809afd689155e2cc4a00fc704fd772a0e92 change.

Change-Id: I96cf36dc317499df91e43502efc85221f8177395
Closes-Bug: #1300161