Both "stream" and "args" are present in the job. Let's only pass
the "job" object and let the Result class to take the needed objects.

Additionally allow overriding the output for json/xunit/html in order
to allow users to specify different value than the one from args.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

The logging stream can be optionally followed by ":$LOG_LEVEL" either
using number, or the symbolic python logging level (INFO).
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

When avocado crashes before the "parser.start()" is finished,
"parser.args" is None and we fail to reconfigure the logging. This patch
initializes empty Namespace in such case.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

The TermSupport detects compatible terminal to enable or disable colors.
Let's extract this function to make the initialization easier to read
and also make this function available to the whole framework.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

The --log is a bit confusing, let's use --show to emphasize we're
dealing with the console output.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

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

When avocado fails to update the link to the latest results, the whole
job is aborted. This is unfortunate and after a discussion we considered
this as a "soft" issue.

With this patch we only log an information, that the latest link was not
updated into the job-log and proceed to testing.

This commit also extends the link creation to be
concurrent-execution-safe by creating `latest-$PID` symlink first and
then renames the symlink to `latest`. The rename is atomic operation and
does not suffer the delete->create problem, where other process creates
the file before this one.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

Safe Loader: a second, more abstract and reusable implementation [v2]

Find for files in multiple locations, caching it when successfully fetched.
Available as fetch_asset() method in avocado.Test().

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

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

Python 2.6 does not support --user in `python setup.py develop`. Let's
use system-wide develop when link/unlink is executed with python 2.6.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

Test namespace cleanup [4th round]

The get_data_path utility API of the Test class is a convenience,
it simply joins the name of a given file to the datadir location.

IMHO the value is low when compared to the pollution of the namespace
and yet another compatibility/stability promise.
Signed-off-by: NCleber Rosa <crosa@redhat.com>

The supported interface for getting the test tagged name is the
attribute `tagged_name`, so there's no need to make this utility
method visible on the API.
Signed-off-by: NCleber Rosa <crosa@redhat.com>

These two attributes are simply utility logger instances that test
writers can use to write "directly" to the test process' STDOUT and
STDERR.

I agree that having those ready to use inside a test is a nice to
have, but I don't think it justifies the polution to the test class
namespace.

Let's remove those and document how the user can use Python's standard
logging API to retrieve the logger instances that Avocado provides to
tests. This will continue to exist and be supported.
Signed-off-by: NCleber Rosa <crosa@redhat.com>

The default tag in the class constructor already is None, so no
there's no need for the conditional assignment here.
Signed-off-by: NCleber Rosa <crosa@redhat.com>

The Avocado Test class inherits from unittest.TestCase, and for that
matter it already contains implementations of setUp and tearDown.

It could be argued that the point is to better document those, but
then the same logic should be applied to all other unittest.TestCase
methods.

So, given that argument, let's remove those.
Signed-off-by: NCleber Rosa <crosa@redhat.com>

IMO it makes sense for the filename property of the Test class to
check if the detected source code file actually exists.
Signed-off-by: NCleber Rosa <crosa@redhat.com>

The current approach using "string.rstrip()" is buggy and can
remove unintended characters.
Signed-off-by: NCleber Rosa <crosa@redhat.com>

avocado.core.test: Refactor *SkipTests and support extra params [v2]

All *SkipTests are now inherited from "SkipTest" baseclass. This
baseclass contains improved __init__ which removes any unsupported
parameters from the Test.__init__ super call.

This is because those classes are used to substitute other avocado test
classes, for example "avocado-vt" tests which contain "vt_params"
argument unsupported by Test.__init__.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

Python 3 partial support [v2]

As string.letters is gone on Python 3.
Signed-off-by: NCleber Rosa <crosa@redhat.com>

Python 3 is more strict about I/O to files not openend in text
mode. Let's encode the message in utf-8 and have bytes written
to the traceback log file.
Signed-off-by: NCleber Rosa <crosa@redhat.com>

A regex match does not return its length by default, let's use len()
on it.
Signed-off-by: NCleber Rosa <crosa@redhat.com>

The tree/multiplex modules have clear conditionals when the system
is not multiplex capable, namely because of the lack of YAML libraries.

Let's reflect that on their tests.
Signed-off-by: NCleber Rosa <crosa@redhat.com>

The StringIO module now lives under io as io.StringIO on Python 3.
Let's deal with both names.
Signed-off-by: NCleber Rosa <crosa@redhat.com>

The urlparse module now lives under urllib as urllib.parse on
Python 3.  Let's deal with both names.
Signed-off-by: NCleber Rosa <crosa@redhat.com>

The ConfigParser module is named configparser on Python 3.  Let's deal
with both names.
Signed-off-by: NCleber Rosa <crosa@redhat.com>

This changes the print statements to print functions.
Signed-off-by: NCleber Rosa <crosa@redhat.com>

This commit changes the idiom on (hopefully all) except statements,
from:
   except foo, details:

to:
   except foo as details:

Which is compatible with both Python 2 and 3.
Signed-off-by: NCleber Rosa <crosa@redhat.com>

This is a simple compatibility fix that uses hashlib API that is
available on both Python 2 and 3.
Signed-off-by: NCleber Rosa <crosa@redhat.com>

This change introduces symbolic names for various filesystem
permission modes, instead of the literal octals with which
Python 3 shows no compassion.
Signed-off-by: NCleber Rosa <crosa@redhat.com>

This commit ports the, arguably old style code (even enough old
to "modern" Python 2 standards) found in the "external" spark
and gdbmi parsers. It includes changes such as:

* print style (using print "function")
* lambda parameter style
* dict "has_key" changed to "key in dict" idiom
* use of string instances instead of the external string module. This
  change is intended to make the same code compatible with Python 2
  and 3. That is, even when using 2to3, this doesn't get
  "fixed". Instead of writing a "fixer", let's just make this code be
  more Pythonic and compatible.
Signed-off-by: NCleber Rosa <crosa@redhat.com>

Avocado's setup.py used to not be based on setuptools, but plain
distutils. Then, it wasn't possible to use 2to3 when to help with
Python 3 automatic adjustments.

Since 333e439b it's now based on setuptools so we can enable 2to3.
Signed-off-by: NCleber Rosa <crosa@redhat.com>

Those two are interconnected, let's keep them in one file instead of
local imports.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>