refactor(scale_pattern): ♻️ use proper Set argument for exclude

Signed-off-by: Albert Mañosa <26429103+albertms10@users.noreply.github.com>
albertms10 · Jan 19, 2025 · d558df6 · d558df6
1 parent cdc3c9d
commit d558df6
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 4 deletions.
diff --git a/lib/src/scale/scale_pattern.dart b/lib/src/scale/scale_pattern.dart
@@ -429,10 +429,9 @@ final class ScalePattern {
   ///
   /// Example:
   /// ```dart
-  /// ScalePattern.major.exclude(const [Interval.m2])
-  ///   == ScalePattern.majorPentatonic
+  /// ScalePattern.major.exclude({Interval.m2}) == ScalePattern.majorPentatonic
   /// ```
-  ScalePattern exclude(List<Interval> intervals) {
+  ScalePattern exclude(Set<Interval> intervals) {
     final steps = <Interval>[];
     for (var i = 0; i < _intervalSteps.length; i++) {
       final interval = _intervalSteps[i];

diff --git a/test/src/scale/scale_pattern_test.dart b/test/src/scale/scale_pattern_test.dart
@@ -571,7 +571,7 @@ void main() {
     group('.exclude()', () {
       test('returns a new ScalePattern excluding intervals', () {
         expect(
-          ScalePattern.major.exclude(const [Interval.m2]),
+          ScalePattern.major.exclude({Interval.m2}),
           ScalePattern.majorPentatonic,
         );
       });