Make testRandomizeOrderingSeed public on Engine (#1543)
One internal reporter needs this information.
Use a consistent name for the constructor argument assign in argument list.
diff --git a/pkgs/test_core/lib/src/runner.dart b/pkgs/test_core/lib/src/runner.dart
index c6e220c..e9ab5fd 100644
--- a/pkgs/test_core/lib/src/runner.dart
+++ b/pkgs/test_core/lib/src/runner.dart
@@ -76,7 +76,7 @@
var engine = Engine(
concurrency: config.concurrency,
coverage: config.coverage,
- randomizeOrderingSeed: config.testRandomizeOrderingSeed);
+ testRandomizeOrderingSeed: config.testRandomizeOrderingSeed);
var sinks = <IOSink>[];
Reporter createFileReporter(String reporterName, String filepath) {
diff --git a/pkgs/test_core/lib/src/runner/engine.dart b/pkgs/test_core/lib/src/runner/engine.dart
index 31aa087..359374a 100644
--- a/pkgs/test_core/lib/src/runner/engine.dart
+++ b/pkgs/test_core/lib/src/runner/engine.dart
@@ -73,7 +73,7 @@
///
/// If null or zero no shuffling will occur.
/// The same seed will shuffle the tests in the same way every time.
- int? _testRandomizeOrderingSeed;
+ int? testRandomizeOrderingSeed;
/// A pool that limits the number of test suites running concurrently.
final Pool _runPool;
@@ -204,10 +204,9 @@
///
/// [concurrency] controls how many suites are loaded and ran at once, and
/// defaults to 1.
- Engine({int? concurrency, String? coverage, int? randomizeOrderingSeed})
+ Engine({int? concurrency, String? coverage, this.testRandomizeOrderingSeed})
: _runPool = Pool(concurrency ?? 1),
- _coverage = coverage,
- _testRandomizeOrderingSeed = randomizeOrderingSeed {
+ _coverage = coverage {
_group.future.then((_) {
_onTestStartedGroup.close();
_onSuiteStartedController.close();
@@ -308,9 +307,9 @@
if (!_closed && setUpAllSucceeded) {
// shuffle the group entries
var entries = group.entries.toList();
- if (_testRandomizeOrderingSeed != null &&
- _testRandomizeOrderingSeed! > 0) {
- entries.shuffle(Random(_testRandomizeOrderingSeed));
+ if (testRandomizeOrderingSeed != null &&
+ testRandomizeOrderingSeed! > 0) {
+ entries.shuffle(Random(testRandomizeOrderingSeed));
}
for (var entry in entries) {
