Add more helpful error message for improper config

[alexdebrie]

This adds a more helpful error message for those that don't change the project value from the template in serverless.yml. It now says:

  Error --------------------------------------------------

  Error: Incorrect configuration. Please change the 'project' key in the 'provider' block in your Serverless config file.

Why is this helpful?

The serverless.yml template looks like this:

provider:
  name: google
  runtime: nodejs
  project: my-project
  # the path to the credentials file needs to be absolute
  credentials: ~/.gcloud/keyfile.json

If you don't change the project value to the ID for your project, you'll get this error:

  Error --------------------------------------------------

  Error: Access Not Configured. Google Cloud Deployment Manager API has not been used in project 1043443670838 before or it is disabled. Enable it by visiting https://console.developers.google.com/apis/api/deploymentmanager.googleapis.com/overview?project=1043443670838 then retry. If you enabled this API recently, wait a few minutes for the action to propagate to our systems and retry.

This happened to me, and two other users have reported it in the last few days. It's very confusing because I had enabled those APIs, but the real problem was that I was using the wrong project. Clicking the link in that error message wasn't helpful because that project doesn't exist in my GCP account.

[pmuens]

Thank you very much for this PR @alexdebrie 👍

Just looked through the code and added one comment.

AFAIK googleProvider.js is not 100% covered by tests. Can you confirm that you've tested it thoroughly?

Thanks again for working on this fix!

[pmuens]

Is 1043443644444 a Google Cloud Functions error code?

[alexdebrie]

1043443644444 is the Id number of my-project.

From what I understand, a project name is unique across all GCP, and each project had an Id number. We put my-project in the serverless.yml for the GCF template. If you don't change that, it will spit an error about project 1043443644444 not being configured correctly.

[alexdebrie]

Thanks, @pmuens.

Yep, I tested thoroughly. Also, it's pretty safe / low impact -- does some very careful checking of the error object and only throws a different error in a pretty specific circumstance.

[pmuens]

Thanks for clarifying and fixing that @alexdebrie 👍

Merging...