pkg/linter/test/rules/prefer_spread_collections_test.dart

// Copyright (c) 2021, the Dart project authors. Please see the AUTHORS file
// for details. All rights reserved. Use of this source code is governed by a
// BSD-style license that can be found in the LICENSE file.

import 'package:test_reflective_loader/test_reflective_loader.dart';

import '../rule_test_support.dart';

main() {
  defineReflectiveSuite(() {
    defineReflectiveTests(PreferSpreadCollectionsTest);
  });
}

@reflectiveTest
class PreferSpreadCollectionsTest extends LintRuleTest {
  // TODO(srawlins): These tests mostly use a `dynamic` variable, but I don't
  // think the lint rule is specific to `dynamic` values; it seems odd to
  // specifically rely on them.

  @override
  String get lintRule => LintNames.prefer_spread_collections;

  test_constInitializedWithNonConstantValue() async {
    await assertDiagnostics(r'''
const thangs = [];
const cc = []..addAll(thangs);
''', [
      // No lint
      error(CompileTimeErrorCode.CONST_INITIALIZED_WITH_NON_CONSTANT_VALUE, 30,
          18),
    ]);
  }

  test_listLiteralTarget_conditional() async {
    await assertDiagnostics(r'''
dynamic x;
var y = ['a']..addAll(1 == 2 ? x : []);
''', [
      lint(26, 6),
    ]);
  }

  test_listLiteralTarget_conditional_constList() async {
    await assertDiagnostics(r'''
dynamic x;
var y = ['a']..addAll(1 == 2 ? x : const []);
''', [
      lint(26, 6),
    ]);
  }

  test_listLiteralTarget_identifier() async {
    await assertDiagnostics(r'''
dynamic x;
var y = []..addAll(x);
''', [
      lint(23, 6),
    ]);
  }

  test_listLiteralTarget_ifNull() async {
    await assertDiagnostics(r'''
dynamic x;
var y = ['a']..addAll(x ?? []);
''', [
      lint(26, 6),
    ]);
  }

  test_listLiteralTarget_ifNull_constList() async {
    await assertDiagnostics(r'''
dynamic x;
var y = ['a']..addAll(x ?? const []);
''', [
      lint(26, 6),
    ]);
  }

  test_listLiteralTarget_listLiteral() async {
    // This is reported as `prefer_inlined_adds`.
    await assertNoDiagnostics(r'''
var l = ['a']..addAll(['b']);
''');
  }

  test_listLiteralTarget_multipleCascades() async {
    await assertDiagnostics(r'''
void f(List<int> p) {
  ['a']..addAll(p.map((i) => i.toString()))..addAll(['c']);
}
''', [
      lint(31, 6),
    ]);
  }

  test_nonCollection() async {
    await assertNoDiagnostics(r'''
class A {
  void addAll(Iterable iterable) {}
}

void f() {
  A()..addAll(['a']);
}
''');
  }

  test_otherListTarget_listLiteral() async {
    await assertNoDiagnostics(r'''
var l1 = [];
var l2 = l1..addAll(['b']);
''');
  }
}