fix: making unique region tags for helloerror samples for functions (#3030)

* fix: making unique region tags for helloerror samples for functions

* refactor: separating tests

* chore: lint fix

* chore: rename region tag

Author: pattishin

functions/helloworld/helloError/index.js

@@ -30,7 +30,6 @@ exports.helloError = (event, context, callback) => {
   // [START functions_helloworld_error]
   // These WILL be reported to Error Reporting
   throw new Error('I failed you'); // Will cause a cold start if not caught
-
   // [END functions_helloworld_error]
 };
 
@@ -42,12 +41,12 @@ exports.helloError = (event, context, callback) => {
  * @param {function} callback The callback function.
  */
 exports.helloError2 = (event, context, callback) => {
-  // [START functions_helloworld_error]
+  // [START functions_helloworld_error_2]
   // These WILL be reported to Error Reporting
   console.error(new Error('I failed you')); // Logging an Error object
   console.error('I failed you'); // Logging something other than an Error object
   throw 1; // Throwing something other than an Error object
-  // [END functions_helloworld_error]
+  // [END functions_helloworld_error_2]
 };
 
 /**
@@ -58,16 +57,16 @@ exports.helloError2 = (event, context, callback) => {
  * @param {function} callback The callback function.
  */
 exports.helloError3 = (event, context, callback) => {
+  // [START functions_helloworld_error_3]
   // This will NOT be reported to Error Reporting
-  // [START functions_helloworld_error]
   callback('I failed you');
-  // [END functions_helloworld_error]
+  // [END functions_helloworld_error_3]
 };
 
 // HTTP Cloud Function that returns an error.
 functions.http('helloError4', (req, res) => {
+  // [START functions_helloworld_error_4]
   // This will NOT be reported to Error Reporting
-  // [START functions_helloworld_error]
   res.status(500).send('I failed you');
-  // [END functions_helloworld_error]
+  // [END functions_helloworld_error_4]
 });

functions/helloworld/helloError/test/index.test.js

@@ -23,11 +23,19 @@ describe('index.test.js', () => {
       it('helloError: should throw an error', () => {
         assert.throws(program.helloError, 'Error: I failed you');
       });
+    });
+  });
 
+  describe('functions_helloworld_error_2', () => {
+    describe('Error handling (unit tests)', () => {
       it('helloError2: should throw a value', () => {
         assert.throws(program.helloError2, '1');
       });
+    });
+  });
 
+  describe('functions_helloworld_error_3', () => {
+    describe('Error handling (unit tests)', () => {
       it('helloError3: callback should return an errback value', () => {
         const cb = sinon.stub();
         program.helloError3(null, null, cb);