Signed-off-by: NAmador Pahim <apahim@redhat.com>

There's a problem with assuming that all existing plugins will
be setup at API docs creation time, initially on RPM builds but
could also happen on systems that lack dependencies for some of
the plugins.

Let's only build docs for those that have been properly setup
and are available to be imported.
Signed-off-by: NCleber Rosa <crosa@redhat.com>

Including additions of words to ignore, and substitution of words
we better keep off the ignore list.
Signed-off-by: NCleber Rosa <crosa@redhat.com>

With two different versions of pylint, style warnings were reported:

 avocado/core/loader.py:662:20: W503 line break before binary operator
 avocado/core/loader.py:668:20: W503 line break before binary operator
 avocado/core/loader.py:676:20: W503 line break before binary operator

Let's fix by breaking that into two statements.
Signed-off-by: NCleber Rosa <crosa@redhat.com>

Signed-off-by: NAmador Pahim <apahim@redhat.com>

Currently we just pickle the varianter object and hope we'll be able to
load it with other Avocado version. It usually works, but is not really
scalable. This patch adds a way to serialize Varianter into a list of
dicts and serializes it using json, turning the unsafe objects to
strings. Then it adds a way to load Varianter status using this
serialized state.

This procedure is lossy as it always assumes we are in non-debug mode
and it replaces the variants with safe representations. As a benefit one
does not need the original plugins to be able to reuse/replay the
variants as Avocado loads the representation using fake classes.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

Currently we dump args using pickle, which tries to pickle all objects.
This could fail between versions or even between instances (eg. with
different set of plugins). Let's just use safe json serialization
replacing unsafe objects with None to avoid unnecessary crashes while
still being able to make the safe information available.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

The variants file is called multiplex even though we changed the
terminology long ago. Let's support both names for now and slowly move
to the correct name.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

We had already split the mux into mutliple files.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

Signed-off-by: NAmador Pahim <apahim@redhat.com>

Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

* https://github.com/avocado-framework/avocado:
  Added support for POWER9 cpu arch identification

Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

* https://github.com/avocado-framework/avocado:
  loader: recursive test discovery
  loader: add '__init__.py' to path when path is a dir

This patch adds to the loader the ability to recursively discover tests
when the docstring `🥑 recursive` is specified in the test class.

This recursive discovery will walk all the way up in the recursion
chain, adding the parents test methods to the child class information.

Reference: https://trello.com/c/GtFlhcOjSigned-off-by: NAmador Pahim <apahim@redhat.com>

In `_find_avocado_tests()`, `path` is a python source code file which
will be processed by `ast.parse()`. If it's a directory, `ast.parse()`
will fall in exception.

This patch makes the `path` to point to `path/__init__.py` when `path`
is a directory.
Signed-off-by: NAmador Pahim <apahim@redhat.com>

Currently when the target file does not exists the FileLoader reports:
"Does not look like an INSTRUMENTED test, nor is it executable" which
can be a bit misleading. Let's change it to: "File not found
($locations)".
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

Added support for POWER9 cpu arch identification
Signed-off-by: NSatheesh Rajendran <sathnaga@linux.vnet.ibm.com>

Signed-off-by: NCleber Rosa <crosa@redhat.com>

Signed-off-by: NCleber Rosa <crosa@redhat.com>

We rely on garbage collector to run __del__s before we check all tmpdirs
were cleaned. Let's force-run gc.collect() and (just to be sure) run
sync.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

* https://github.com/avocado-framework/avocado:
  VM Runner: use default on option parser for port and username
  VM Runner: do not keep parser as attribute
  Remote Runner: use default on option parser for port and username
  Remote Runner: do not keep parser as attribute
  Remote and VM runner: remove configured attribute assignment

The suite order means the order in which the variants will be applied to
tests. Right now we run all variants of first test, then the same for
the second, third, ... tests. The new option "--execution-order" allows
to alternatively specify "tests-per-variant" mode where we run first
variant of all tests, then second variant, third, ...
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

Signed-off-by: NCleber Rosa <crosa@redhat.com>

We need optional plugins in ReadTheDocs, but they are not installed by
default. Let's use `make develop` there as it should do and install
normally when outside ReadTheDocs environment.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

Currently the yaml_to_mux plugin is a core plugin without properly
defined requirements. Let's separate it to optional_plugins, define
dependencies and also build it separately in RPM.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

With more and more optional plugins it could be useful to build their
API documentation in order to be able to refer to it from the main
documentation.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

Let's report full result on doc build failure to be able to see what is
wrong.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

Signed-off-by: NCleber Rosa <crosa@redhat.com>

Signed-off-by: NCleber Rosa <crosa@redhat.com>

Instead of current, let's stick to the default of using the word
default, and also the option value from optparse itself.
Signed-off-by: NCleber Rosa <crosa@redhat.com>

Signed-off-by: NCleber Rosa <crosa@redhat.com>

Signed-off-by: NCleber Rosa <crosa@redhat.com>

Signed-off-by: NCleber Rosa <crosa@redhat.com>

Signed-off-by: NCleber Rosa <crosa@redhat.com>

This patch adds the missing definition of `MODE_0644` in
`LoaderTestFunctional` class.
Signed-off-by: NAmador Pahim <apahim@redhat.com>

Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

* https://github.com/avocado-framework/avocado:
  selftests: remove redundant test
  test: always execute tearDown()

Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

* https://github.com/avocado-framework/avocado:
  avocado list command: also show tag statistics
  avocado list command: also list test tags

Signed-off-by: NAmador Pahim <apahim@redhat.com>

Signed-off-by: NAmador Pahim <apahim@redhat.com>

Signed-off-by: NAmador Pahim <apahim@redhat.com>

The format of the docstring directives has not been formally described
so far.  Because at least two different features are built on that,
and possibly more will come, it's a good idea to define a format.

This introduces a stricter format, basically adding a number of
requirements or rules.  To make the following description more
consistent let's call the "🥑" part of the docstring directive
"the marker" and what follows it the "content".

1) One or more spaces are required between the "🥑" marker and
   the content.  The following is a valid docstring directive:

   "🥑 enable"

   While this is invalid:

   ":avocado:enable"

2) A more limited set of characters are now allowed in the content,
   namely one from the following set: [a-zA-Z0-9_:,=].  The following
   is a valid docstring directive:

   "🥑 foo=bar,foz=baz:extra"

   While the following is invalid:

   "🥑 foo=bar!"

3) The first character of the content must be alphanumeric, so the
   following is a valid docstring directive:

   "🥑 foo=bar"

   While the following is invalid:

   "🥑 =bar"

4) An end of string (or end of line) is now required after the content
   (which itself may not contain white spaces).  The following is a
   valid docstring directive:

   "🥑 enable"

   While the following is invalid:

   "🥑 enable FOO"
Signed-off-by: NCleber Rosa <crosa@redhat.com>