Merge branch 'master' into feature/gb/gitlab-domains-source

* master:
  Check presence of GitLab API secret when building a domains source
  Make GitLab API Secret a supported parameter
  Improve error reporting in the main package
  Check if GitLab API secret has been provided too
  Avoid using `testify/assert` in favor of `require`
  Test domains source not fully configured
  Improve error handing when creating new domains config source
  Fix formatting in internal/source/gitlab/client/client_test.go

Conflicts:
	acceptance_test.go
	internal/source/gitlab/client/client_test.go

1 files changed, 16 insertions, 3 deletions

diff --git a/internal/source/domains.go b/internal/source/domains.go
index f47884ef..85d06152 100644
--- a/internal/source/domains.go
+++ b/internal/source/domains.go
@@ -36,11 +36,20 @@ type Domains struct {
 // NewDomains is a factory method for domains initializing a mutex. It should
 // not initialize `dm` as we later check the readiness by comparing it with a
 // nil value.
-func NewDomains(config Config) *Domains {
+func NewDomains(config Config) (*Domains, error) {
+	if len(config.GitlabServerURL()) == 0 || len(config.GitlabAPISecret()) == 0 {
+		return &Domains{disk: disk.New()}, nil
+	}
+
+	gitlab, err := gitlab.New(config)
+	if err != nil {
+		return nil, err
+	}
+
 	return &Domains{
+		gitlab: gitlab,
 		disk:   disk.New(),
-		gitlab: gitlab.New(config),
-	}
+	}, nil
 }
 
 // GetDomain retrieves a domain information from a source. We are using two
@@ -69,6 +78,10 @@ func (d *Domains) IsReady() bool {
 }
 
 func (d *Domains) source(domain string) Source {
+	if d.gitlab == nil {
+		return d.disk
+	}
+
 	for _, name := range newSourceDomains {
 		if domain == name {
 			return d.gitlab