diff options
| author | Dylan Baker <dylan@pnwbakers.com> | 2018-01-22 17:22:04 -0800 |
|---|---|---|
| committer | Fabian Bieler <fabianbieler@fastmail.fm> | 2018-02-05 19:44:16 +0100 |
| commit | dfd2a855f6230045f5201e105e99a660be5b2ee0 (patch) | |
| tree | 4cc0eb0a7e60d27f2c5de1eae8debdf894d5ad91 | |
| parent | fb928640a6e837684b35f4cfc8bd243140e20746 (diff) | |
framework: use statuses instead of relying on string conversions
This is a small cleanup, but we need to touch this code further for
subtest enumeration work, and we'll want to use statuses there too.
Signed-off-by: Fabian Bieler <fabianbieler@fastmail.fm>
Reviewed-by: Dylan Baker <dylan@pnwbakers.com>
| -rw-r--r-- | framework/test/base.py | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/framework/test/base.py b/framework/test/base.py index 0b7ebab2e..2d761efa9 100644 --- a/framework/test/base.py +++ b/framework/test/base.py @@ -235,12 +235,12 @@ class Test(object): """Convert the raw output of the test into a form piglit understands. """ if is_crash_returncode(self.result.returncode): - self.result.result = 'crash' + self.result.result = status.CRASH elif self.result.returncode != 0: - if self.result.result == 'pass': - self.result.result = 'warn' + if self.result.result == status.PASS: + self.result.result = status.WARN else: - self.result.result = 'fail' + self.result.result = status.FAIL def run(self): """ |