Commit 8a6c6f58 authored by Michal Majewski's avatar Michal Majewski Committed by Commit Bot

Reland "Reland "[test] Creating command before execution phase.""

This is a reland of 3b065110
Original change's description:
> Reland "[test] Creating command before execution phase."
>
> This is a reland of 98cc9e86
> Original change's description:
> > [test] Creating command before execution phase.
> >
> > Immutable command class with shell, flags and
> > environment.
> >
> > Command creation moved from worker to the main
> > process. Because of that there is no need to send
> > test cases beyond process boundaries and load test
> > suites in worker processes.
> >
> > Bug: v8:6917
> > Change-Id: Ib6a44278095b4f7141eb9b96802fe3e8117678a6
> > Reviewed-on: https://chromium-review.googlesource.com/791710
> > Commit-Queue: Michał Majewski <majeski@google.com>
> > Reviewed-by: Michael Achenbach <machenbach@chromium.org>
> > Cr-Commit-Position: refs/heads/master@{#49746}
>
> Bug: v8:6917
> Change-Id: I49c29a8db813c47909f2cc45070ac7721a447c7a
> Reviewed-on: https://chromium-review.googlesource.com/800370
> Reviewed-by: Michael Achenbach <machenbach@chromium.org>
> Commit-Queue: Michał Majewski <majeski@google.com>
> Cr-Commit-Position: refs/heads/master@{#49756}

Bug: v8:6917
Change-Id: I981994224e493bee4c9435cb80772b6e2ad8fbb1
Reviewed-on: https://chromium-review.googlesource.com/805336Reviewed-by: 's avatarMichael Achenbach <machenbach@chromium.org>
Commit-Queue: Michał Majewski <majeski@google.com>
Cr-Commit-Position: refs/heads/master@{#49827}
parent 2e4b4db0
...@@ -315,6 +315,7 @@ class DeoptFuzzer(base_runner.BaseTestRunner): ...@@ -315,6 +315,7 @@ class DeoptFuzzer(base_runner.BaseTestRunner):
num_tests += len(s.tests) num_tests += len(s.tests)
for t in s.tests: for t in s.tests:
t.id = test_id t.id = test_id
t.cmd = s.GetCommand(t, ctx)
test_id += 1 test_id += 1
if num_tests == 0: if num_tests == 0:
...@@ -363,6 +364,7 @@ class DeoptFuzzer(base_runner.BaseTestRunner): ...@@ -363,6 +364,7 @@ class DeoptFuzzer(base_runner.BaseTestRunner):
num_tests += len(s.tests) num_tests += len(s.tests)
for t in s.tests: for t in s.tests:
t.id = test_id t.id = test_id
t.cmd = s.GetCommand(t, ctx)
test_id += 1 test_id += 1
if num_tests == 0: if num_tests == 0:
......
...@@ -201,6 +201,8 @@ class GCFuzzer(base_runner.BaseTestRunner): ...@@ -201,6 +201,8 @@ class GCFuzzer(base_runner.BaseTestRunner):
] ]
s.tests = map(lambda t: t.CopyAddingFlags(t.variant, analysis_flags), s.tests = map(lambda t: t.CopyAddingFlags(t.variant, analysis_flags),
s.tests) s.tests)
for t in s.tests:
t.cmd = s.GetCommand(t, ctx)
progress_indicator = progress.PROGRESS_INDICATORS[options.progress]() progress_indicator = progress.PROGRESS_INDICATORS[options.progress]()
runner = execution.Runner(suites, progress_indicator, ctx) runner = execution.Runner(suites, progress_indicator, ctx)
...@@ -246,6 +248,9 @@ class GCFuzzer(base_runner.BaseTestRunner): ...@@ -246,6 +248,9 @@ class GCFuzzer(base_runner.BaseTestRunner):
if options.stress_compaction: if options.stress_compaction:
fuzzing_flags.append('--stress_compaction_random') fuzzing_flags.append('--stress_compaction_random')
s.tests.append(t.CopyAddingFlags(t.variant, fuzzing_flags)) s.tests.append(t.CopyAddingFlags(t.variant, fuzzing_flags))
for t in s.tests:
t.cmd = s.GetCommand(t, ctx)
num_tests += len(s.tests) num_tests += len(s.tests)
if num_tests == 0: if num_tests == 0:
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment