chore(doclint): move doclint tests expectations into test folders (#932)

This makes for much easier work with doclint tests.

test/golden-utils.js

@@ -85,6 +85,8 @@ function compareText(actual, expectedBuffer) {
  * @return {!{pass: boolean, message: (undefined|string)}}
  */
 function compare(goldenPath, outputPath, actual, goldenName) {
+  goldenPath = path.normalize(goldenPath);
+  outputPath = path.normalize(outputPath);
   const expectedPath = path.join(goldenPath, goldenName);
   const actualPath = path.join(outputPath, goldenName);
 
@@ -110,9 +112,13 @@ function compare(goldenPath, outputPath, actual, goldenName) {
   if (!result)
     return { pass: true };
   ensureOutputDir();
-  fs.writeFileSync(actualPath, actual);
+  if (goldenPath === outputPath) {
-  // Copy expected to the output/ folder for convenience.
+    fs.writeFileSync(addSuffix(actualPath, '-actual'), actual);
-  fs.writeFileSync(addSuffix(actualPath, '-expected'), expected);
+  } else {
+    fs.writeFileSync(actualPath, actual);
+    // Copy expected to the output/ folder for convenience.
+    fs.writeFileSync(addSuffix(actualPath, '-expected'), expected);
+  }
   if (result.diff) {
     const diffPath = addSuffix(actualPath, '-diff', result.diffExtension);
     fs.writeFileSync(diffPath, result.diff);

utils/doclint/check_public_api/test/.gitignore

@@ -0,0 +1,2 @@
+result-actual.txt
+result-diff.html

utils/doclint/check_public_api/test/golden/01-class-errors.txt

@@ -1,3 +0,0 @@
-[MarkDown] Non-existing class found: Bar
-[MarkDown] Non-existing class found: Baz
-[MarkDown] Class not found: Other

utils/doclint/check_public_api/test/test.js

@@ -44,10 +44,6 @@ afterAll(SX(async function() {
   await browser.close();
 }));
 
-beforeEach(function() {
-  GoldenUtils.addMatchers(jasmine, GOLDEN_DIR, OUTPUT_DIR);
-});
-
 describe('checkPublicAPI', function() {
   it('01-class-errors', SX(test));
   it('02-method-errors', SX(test));
@@ -61,12 +57,13 @@ describe('checkPublicAPI', function() {
 
 async function test() {
   const dirPath = path.join(__dirname, specName);
+  GoldenUtils.addMatchers(jasmine, dirPath, dirPath);
   const factory = new SourceFactory();
   const mdSources = await factory.readdir(dirPath, '.md');
   const jsSources = await factory.readdir(dirPath, '.js');
   const messages = await checkPublicAPI(page, mdSources, jsSources);
   const errors = messages.map(message => message.text);
-  expect(errors.join('\n')).toBeGolden(specName + '.txt');
+  expect(errors.join('\n')).toBeGolden('result.txt');
 }
 
 // Since Jasmine doesn't like async functions, they should be wrapped