firka/flutter
firka/flutter
5
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Merge the two dartdoc smoke tests (#7443)

Browse Source 
We were checking similar things in two places. Now we check everything in one
place.
This commit is contained in:
Adam Barth 2017-01-12 09:47:50 -08:00 committed by GitHub
parent 719712b74c
commit d0cd545818
2 changed files with 4 additions and 23 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

22
dev/bots/docs.sh
View File

@ -9,28 +9,6 @@ pub global activate dartdoc 0.9.9
(cd dev/tools; pub get) (cd dev/tools; pub get)
FLUTTER_ROOT=$PWD dart dev/tools/dartdoc.dart FLUTTER_ROOT=$PWD dart dev/tools/dartdoc.dart
# Smoke test the docs to make sure we have all the major kinds of things.
if [[ ! -f dev/docs/doc/flutter/widgets/Widget/Widget.html ]]; then
echo 'Failed to find documentation for Widget class. Are the docs complete?'
exit 1
fi
if [[ ! -f dev/docs/doc/flutter/dart-io/File/File.html ]]; then
echo 'Failed to find documentation for File class. Are the docs complete?'
exit 1
fi
if [[ ! -f dev/docs/doc/flutter/dart-ui/Canvas/drawRect.html ]]; then
echo 'Failed to find documentation for Canvas.drawRect. Are the docs complete?'
exit 1
fi
if [[ ! -f dev/docs/doc/flutter/flutter_test/WidgetTester/pumpWidget.html ]]; then
echo 'Failed to find documentation for WidgetTester.pumpWidget. Are the docs complete?'
exit 1
fi
# Ensure google webmaster tools can verify our site. # Ensure google webmaster tools can verify our site.
cp dev/docs/google2ed1af765c529f57.html dev/docs/doc cp dev/docs/google2ed1af765c529f57.html dev/docs/doc

5
dev/tools/dartdoc.dart
View File

@ -87,10 +87,13 @@ dependencies:
void sanityCheckDocs() { void sanityCheckDocs() {
List<String> canaries = <String>[ List<String> canaries = <String>[
'$kDocRoot/api/dart-io/File-class.html',
'$kDocRoot/api/dart-ui/Canvas-class.html',
'$kDocRoot/api/dart-ui/Canvas/drawRect.html',
'$kDocRoot/api/flutter_test/WidgetTester/pumpWidget.html',
'$kDocRoot/api/material/Material-class.html', '$kDocRoot/api/material/Material-class.html',
'$kDocRoot/api/material/Tooltip-class.html', '$kDocRoot/api/material/Tooltip-class.html',
'$kDocRoot/api/widgets/Widget-class.html', '$kDocRoot/api/widgets/Widget-class.html',
'$kDocRoot/api/dart-ui/Canvas-class.html',
]; ];
for (String canary in canaries) { for (String canary in canaries) {
if (!new File(canary).existsSync()) if (!new File(canary).existsSync())