Add dotfile to integration test (#217)

* Add dotfile to integration test

* Make unit test look at the actual files instead of just length

* Is the test working now?

Author: sethvargo

tests/test-node-func/.dotfile

@@ -0,0 +1 @@
+I exist!

tests/test-node-func/index.js

@@ -4,7 +4,17 @@
  * @param {!express:Request} req HTTP request context.
  * @param {!express:Response} res HTTP response context.
  */
+
+let fs = require('fs');
+
 exports.helloWorld = (req, res) => {
+  // Still send a 200 so we get the response (gaxios and other libraries barf on
+  // non-200)
+  if (!fs.existsSync('.dotfile')) {
+    res.status(200).send('Dotfile does not exist!');
+    return;
+  }
+
   let message = req.query.message || req.body.message || 'Hello World!!';
   res.status(200).send(message);
 };

tests/util.test.ts

@@ -421,7 +421,7 @@ describe('Zip', function () {
     const filesInsideZip = await getFilesInZip(zf);
     const expectedFiles = getNonIgnoredFiles(testDirNoIgnore, testDirNoIgnore);
 
-    expect(await filesInsideZip.length).equal(expectedFiles.length);
+    expect(filesInsideZip).eql(expectedFiles);
     filesInsideZip.forEach((f) => expect(expectedFiles).to.include(f));
   });
 
@@ -437,7 +437,7 @@ describe('Zip', function () {
       new Set(['ignore.txt', '.gcloudignore']),
     );
 
-    expect(await filesInsideZip.length).equal(expectedFiles.length);
+    expect(filesInsideZip).eql(expectedFiles);
     filesInsideZip.forEach((f) => expect(expectedFiles).to.include(f));
   });
 
@@ -453,7 +453,7 @@ describe('Zip', function () {
       new Set(['bar/bar.txt', 'bar/baz/baz.txt']),
     );
 
-    expect(await filesInsideZip.length).equal(expectedFiles.length);
+    expect(filesInsideZip).eql(expectedFiles);
     filesInsideZip.forEach((f) => expect(expectedFiles).to.include(f));
   });
 });