Add shared_runners_enabled argument to gitlab_project resource

[iAnomaly]

No description provided.

[iAnomaly]

@jorcau @roidelapluie, this replaces #104 with the requested missing tests. Please note that tests have been failing on master since @roidelapluie merged #132 6 days ago. This PR does not add any new failures.

Thanks!

[roidelapluie]

@iAnomaly this is breaking tests

[iAnomaly]

Hey @roidelapluie. As I pointed out in my previous comment, these tests have been failing since you merged #132; 8 days before you merged my code. Specifically, commit hashicorp@c95dfb0 is when the build stopped passing.

You can compare the test results between the two and see it is the same failure:

1. Build of hashicorp@c95dfb0 from merge Cleans up "project" resource schema and testing framework #132: https://travis-ci.org/terraform-providers/terraform-provider-gitlab/jobs/530438424
2. Build of hashicorp@82886a6 from merge of my pull request Add shared_runners_enabled argument to gitlab_project resource #134: https://travis-ci.org/terraform-providers/terraform-provider-gitlab/jobs/533726463

Specifically, the failure is from line 539 of the build log:

--- FAIL: TestAccGitlabProject_basic (6.03s)
testing.go:568: Step 1 error: Check failed: Check 2/2 error: 1 error occurred:
* Check 19/22 error: attribute tags expected "&{F:0xf04970 m:map[4151227546:tag1 1852302624:tag2] once:{m:{state:0 sema:0} done:1}}" received "&{F:0xf04970 m:map[1852302624:tag2 4151227546:tag1] once:{m:{state:0 sema:0} done:1}}"

Does that make sense? Let me know if I am misunderstanding you.

P.S. Sorry for not squashing my commits before you merged. I thought GitHub would allow you to do that automatically at merge time. Alas, I've become too comfortable with GitLab.

[roidelapluie]

Github allows me to do it but I dont like it.

I changed the value to "computed".

[roidelapluie]

(there was a failing test because of this .. in addition to the other issue)