* https://github.com/avocado-framework/avocado:
  Fix human output for replay jobs

* https://github.com/avocado-framework/avocado:
  fix unicode handling in journal plugin

Commit 2e6ecce6 made the human results a real plugin. But the change
broke the human results for replay jobs. This is a fix that makes the
human results to show the SRC JOB ID properly again.
Signed-off-by: NAmador Pahim <apahim@redhat.com>

Instead of recording in the database the test name converted to str(),
which is expected to fail for non-ascii characters, let's record the
repr() of the test name so we don't have to handle the encoding here.

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

* https://github.com/avocado-framework/avocado:
  Plugin listing: add result events based plugin to the list
  avocado/plugins/journal.py: add missing name and description
  avocado/plugins/run.py: simple typo (white space missing) fix

With the introduction of the "result events" interface, they should
also be listed by the `avocado plugins` commands.
Signed-off-by: NCleber Rosa <crosa@redhat.com>

All plugins need this basic information.  Recently intoduced
JournalResult is missing it, so let's add them.
Signed-off-by: NCleber Rosa <crosa@redhat.com>

The run plugin description is missing a white space.  Let's fix that.
Signed-off-by: NCleber Rosa <crosa@redhat.com>

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

1. Added explicit build dependencies upon kmod to prevent test_lv_utils check
   failures.
2. Modified test_build_docs to explicitly ignore bogus warnings due to a bug
   in older versions of python-sphinx package causing checks to fail.
3. Enabled EL7 checks during package build.
Signed-off-by: NMerlin Mathesius <mmathesi@redhat.com>

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

Avocado assigns a name based on the job-id. Let's reflect this in our
documentation.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

The default block-comment uses python or no-highlight. Let's manually
set the diff/yaml/... code-blocks whenever it could be useful to
increase readability of our documentation.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

I think the setUp section is a better place for the "SKIP working only
in setUp" than the "Running third party test suites" section. Let's move
it there.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

We emphasize the `whiteboard` extensively, but we should not forget to
mention the `outputdir` which is reserved for custom test-produced files
to be attached.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

The html output plugin is optional and not installed by default. Let's
increase the visibility of the ways to make it working.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

* https://github.com/avocado-framework/avocado:
  Fixup: format strings for tap output

Before this patch tap output to a file fails with below error

 File "/usr/lib/python2.7/site-packages/avocado_framework-42.0-py2.7.egg/avocado/core/result.py", line 178, in check_test
    self.end_test(state)

  File "/usr/lib/python2.7/site-packages/avocado_framework-42.0-py2.7.egg/avocado/plugins/tap.py", line 69, in end_test
    self.__write("#   " + line)

  File "/usr/lib/python2.7/site-packages/avocado_framework-42.0-py2.7.egg/avocado/plugins/tap.py", line 35, in writeln
    return self.output.write(msg % args + "\n")

TypeError: not enough arguments for format string

This patch addresses above issue.
Signed-off-by: NSatheesh Rajendran <sathnaga@linux.vnet.ibm.com>
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

Occasionally the 2s wait is not enough, let's wait 2.5s to be sure we
don't hit the 2s ignore window.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

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

* https://github.com/avocado-framework/avocado:
  Pylint fixes

Recent pylint update is more pedantic about module level spacing. There
are no changes to code, only couple of extra spaces to make it happy.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

When running tests remotely, html plugin might be already present. Let's
remove the previously existing files to avoid failure and to always get
the latest results.

NOTE: We might want to re-consider whether to skip the html update, or
whether to always update results, but the current status is not optimal.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

The joblogdir was added several years ago but it was not defined in
remote.test. Let's add it and use the local_job_logdir because this
variable is used during start_tests/start_test/end_test/... which could
make the local avocado to process files which does not exists or files
which does not belong to this job.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

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

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

Besides the port of the TAP plugin, this makes the Result only
consider a test to have run after it finishes

The way the Result class has been accounting tests run is a
consequence of the implementation detail.  Before any tests are
actually run, that is, in `start_tests()`, the counter is incremented.
This is obviously wrong.

The counter should only be incremented at `end_test()`, which is
already done.  AFAICT, the current approach was working because the
subclasses of Result, such as HumanResult, would print the current
status on their own `end_test()`, and then only after that they would
call the parent (Result) `end_test()`.  This, together with the
reconcile method, seems to have kept the counters accurate to end
users, even when they were not accurate internally.
Signed-off-by: NCleber Rosa <crosa@redhat.com>

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

Even though Avocado already publishes an interface for result plugins
(in the avocado.core.plugin_interfaces module), that one is suitable
for plugins that will generate one output after the job as a whole
finishes.

There is another type of use case, already present in Avocado, but
still not implemented using the newer plugin mechanism.  The use
case is result plugins that must respond to events, some that come
from the job and some that come from tests.

The EventResult interface is a collection of Job interfaces that
would happen before tests start running (pre_tests and post_tests)
and events that are closer to the test themselves (start_test,
test_progress and end_test).

The new dispatcher calls plugins that implement the ResultEvents
interface.
Signed-off-by: NCleber Rosa <crosa@redhat.com>

This is the right place to criticize the test suite.  Even though I
don't feel it's right to abort the application from within the job
class, this prevents the current user experience to break.

In the near future, I'd like to see removed, and avocado runs without
a test suite to run and give these exact results: no tests were
available to be run, so none were executed.
Signed-off-by: NCleber Rosa <crosa@redhat.com>

The html plugin can only cope with finished results. Currently this is
not problem, because `render` is not called, but following patches adds
the possibility to postprocess results in such cases so let's prepare
for it.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

A result has a 1:1 relation to a Job. To match one to the other the
job `unique_id` is saved to the result as `job_unique_id`.

The same way that a job alwayas was a unique_id, the result should
never miss the job_unique_id.  So, let's not try to hide or postpone
errors from such as scenario.
Signed-off-by: NCleber Rosa <crosa@redhat.com>

This change introduces what is going to be the only one result
instance to a job.  Instead of keeping track of the same information
on multiple plugins, plugins will refer to the unique job result
instance.

This changes does not remove the result proxy just yet, because by
doing so, the plugins that depend on it would cease to run.  Once
plugins are ported to use the unique result instance, the result proxy
will be removed.
Signed-off-by: NCleber Rosa <crosa@redhat.com>

Some tests might finish before producing any output, then the runner
should reflect that instead of IOError followed by hang because of
missing test_state.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

* https://github.com/avocado-framework/avocado:
  update pydot version
  call read_dot() from the corresponding module