fix: fix rule-specific excludes not working on Windows (#518)

* fix: fix rule-specific excludes not working on Windows

* docs: update changelog

* test: make tests platform-specific

* chore: update skipped test descriptions

Author: Roman Petrov

CHANGELOG.md

@@ -1,5 +1,9 @@
 # Changelog
 
+## Unreleased
+
+* fix: rule-specific excludes not working on Windows.
+
 ## 4.5.0-dev.3
 
 * fix: make check-unused-l10n report class fields.

lib/src/config_builder/models/analysis_options.dart

@@ -20,7 +20,11 @@ class AnalysisOptions {
 
   const AnalysisOptions(this._path, this.options);
 
-  String? get folderPath => _path?.split('/$_analysisOptionsFileName').first;
+  String? get folderPath {
+    final finalPath = _path;
+
+    return finalPath == null ? null : p.dirname(finalPath);
+  }
 
   Iterable<String> readIterableOfString(Iterable<String> pathSegments) {
     Object? data = options;

test/config_builder/models/analysis_options_test.dart

@@ -187,5 +187,23 @@ void main() {
 
       expect(options.readMapOfMap(['dart_code_metrics', 'rules4']), isEmpty);
     });
+
+    test('returns correct "folderPath" on posix platforms', () async {
+      const options =
+          AnalysisOptions('./unix/folder/analysis_options.yaml', {});
+
+      expect(options.folderPath, './unix/folder');
+    }, onPlatform: <String, dynamic>{
+      'windows': const Skip('Test should work only on posix platforms.')
+    });
+
+    test('returns correct "folderPath" on windows platforms', () async {
+      const options =
+          AnalysisOptions(r'C:\windows\folder\analysis_options.yaml', {});
+
+      expect(options.folderPath, r'C:\windows\folder');
+    }, onPlatform: <String, dynamic>{
+      'posix': const Skip('Test should work only on windows platforms.')
+    });
   });
 }