Add github_team data source

[raphink]

This PR adds a data source for Github teams.

[stack72]

Hi @raphink

This is looking good - unfortunately, before merging, would you be able to write some documentation for it?

Thanks

Paul

[raphink]

@stack72 sorry, I thought I had. I will certainly do that.

[raphink]

@stack72 added doc

[stack72]

Hi @raphink

Thanks for adding the docs - these look great. On last check, the test is failing:

% make testacc TEST=./builtin/providers/github TESTARGS='-run=TestAccGithubTeamDataSource_noMatchReturnsError'                               ✭
==> Checking that code complies with gofmt requirements...
go generate $(go list ./... | grep -v /terraform/vendor/)
2017/05/21 00:45:36 Generated command/internal_plugin_list.go
TF_ACC=1 go test ./builtin/providers/github -v -run=TestAccGithubTeamDataSource_noMatchReturnsError -timeout 120m
=== RUN   TestAccGithubTeamDataSource_noMatchReturnsError
--- FAIL: TestAccGithubTeamDataSource_noMatchReturnsError (1.25s)
	testing.go:274: Step 0, expected error:

		Error refreshing: 1 error(s) occurred:

		* data.github_team.test: 1 error(s) occurred:

		* data.github_team.test: data.github_team.test: Could not find team with slug: non-existing

		To match:

		Not Found


FAIL
exit status 1
FAIL	github.com/hashicorp/terraform/builtin/providers/github	1.260s
make: *** [testacc] Error 1

[raphink]

Ah, my bad @stack72. It should be fine now.

[stack72]

LGTM now sir :)

% make testacc TEST=./builtin/providers/github TESTARGS='-run=TestAccGithubTeamDataSource'                                           
==> Checking that code complies with gofmt requirements...
go generate $(go list ./... | grep -v /terraform/vendor/)
2017/05/22 15:27:19 Generated command/internal_plugin_list.go
TF_ACC=1 go test ./builtin/providers/github -v -run=TestAccGithubTeamDataSource -timeout 120m
=== RUN   TestAccGithubTeamDataSource_noMatchReturnsError
--- PASS: TestAccGithubTeamDataSource_noMatchReturnsError (0.73s)
PASS
ok  	github.com/hashicorp/terraform/builtin/providers/github	0.746s

[raphink]

Great thanks!

[ghost]

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.

If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.