On Tue, May 08, 2018 at 10:32:26AM -0700, Dylan Baker wrote:
> test_list is an iterator, you can't walk an iterator more than once. To
> solve this we use itertools.tee, which creates a shared buffer for the
> iterators to draw from. This fixes errors like:
>
> [000/480]
> Traceback (most recent c
s the call chain so that we call .itertests() in the
place we actually consume the generator, not earlier.
Reported-by: Stanislav Lisovskiy
Signed-off-by: Arkadiusz Hiler
---
framework/profile.py | 28 +---
1 file changed, 13 insertions(+), 15 deletions(-)
diff --git a
will help making
the execution less prone to automated list generation errors (which has
already bitten us) as well as human errors.
Signed-off-by: Arkadiusz Hiler
---
framework/programs/run.py | 4
1 file changed, 4 insertions(+)
diff --git a/framework/programs/run.py b/framework/programs/r
On Mon, Aug 14, 2017 at 10:21:13AM -0700, Dylan Baker wrote:
> Quoting Arkadiusz Hiler (2017-08-14 05:09:01)
> > Currently, if a test from provided testlist fails to be discovered by
> > the framework, piglit blows up with an exception.
> >
> > This is both good - f
freedesktop.org/show_bug.cgi?id=99649
Cc: Dylan Baker
Cc: Tomi Sarvela
Cc: Martin Peres
Signed-off-by: Arkadiusz Hiler
---
framework/profile.py | 10 +++---
framework/programs/run.py | 12
framework/test/base.py| 12
3 files changed, 31 insertions(+)
?id=99649
Cc: Dylan Baker
Cc: Tomi Sarvela
Cc: Martin Peres
Tested-by: Petri Latvala
Signed-off-by: Arkadiusz Hiler
---
framework/profile.py | 10 +++---
framework/programs/run.py | 6 ++
framework/test/base.py| 12
3 files changed, 25 insertions(+), 3 deletions(-)
On Thu, Jul 27, 2017 at 09:54:13AM -0700, Dylan Baker wrote:
> Quoting Arkadiusz Hiler (2017-07-27 02:24:33)
> > > >
> > > > __all__ = [
> > > > 'RegexFilter',
> > > > @@ -314,6 +316,9 @@ class TestProfile(object):
> &g
On Wed, Jul 26, 2017 at 11:56:47AM -0700, Dylan Baker wrote:
> Quoting Arkadiusz Hiler (2017-07-26 00:46:21)
> > Currently, if a test from provided testlist fails to be discovered by
> > the framework, piglit blows up with an exception.
>
> Thank you for keeping the d
, but allow some flexibility with the
new option that skips the missing tests with a warning.
bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=99649
Cc: Tomi Sarvela
Cc: Petri Latvala
Reviewed-by: Martin Peres
Signed-off-by: Arkadiusz Hiler
---
framework/profile.py | 5 +
fram
ssing tests as notrun, instead of ignoring them with
a warning
2) add another possible result state - missing, so it would be more obvious
But those would require much more work and code juggling, so I would like to
know your opinions first.
Any thoughts?
Arkadiusz Hiler (1):
framework
10 matches
Mail list logo