Skip to content

Commit 9ea0584

Browse files
gnpricechrisbobbe
authored andcommitted
content test: Skip source-file-reading test on Windows
This fails there, or at least does in the `flutter/tests` CI environment, with an error like: ::group::❌ C:/Users/RUNNER~1/AppData/Local/Temp/flutter_customer_testing.zulip.4a7c3de6/tests/test/model/content_test.dart: all content examples are tested (failed) FileSystemException: Cannot open file, path = '/C:/Users/RUNNER~1/AppData/Local/Temp/flutter_customer_testing.zulip.4a7c3de6/tests/test/model/content_test.dart' (OS Error: The filename, directory name, or volume label syntax is incorrect. , errno = 123) dart:io _File.readAsStringSync test\model\content_test.dart 1082:39 main.<fn> ::endgroup:: I'm not sure what's wrong, but don't really feel like finding out (or working out how to fix it) -- and, conveniently, there's really no need to do so. See comment.
1 parent cf5218c commit 9ea0584

File tree

1 file changed

+3
-1
lines changed

1 file changed

+3
-1
lines changed

test/model/content_test.dart

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1087,5 +1087,7 @@ void main() {
10871087
r'^\s*testParseExample\s*\(\s*ContentExample\s*\.\s*(\w+)\);',
10881088
).allMatches(source).map((m) => m.group(1));
10891089
check(testedExamples).unorderedEquals(declaredExamples);
1090-
});
1090+
}, skip: Platform.isWindows, // [intended] purely analyzes source, so
1091+
// any one platform is enough; avoid dealing with Windows file paths
1092+
);
10911093
}

0 commit comments

Comments
 (0)