Included tests for internal/collector/github/factory

[nathannaveen]

• Included tests for internal/collector/github/factory
• Included mocks for query
• Added interface for query

Signed-off-by: nathannaveen 42319948+nathannaveen@users.noreply.github.com

[nathannaveen]

I removed:

criticality_score/internal/collector/github/repo.go

Lines 45 to 48 in 95942e0

	if r.BasicData != nil {
	// Already finished. Don't init() more than once.
	return nil
	}

I removed this because r.BasicData is never going to be nil. The New function in the factory.go file is the only place init is called. r.BasicData is never set when the code calls r.init()because r is initialized without BasicData:

r := &repo{
    client:  f.client,
    origURL: u,
    logger:  f.logger.With(zap.String("url", u.String())),
}

[nathannaveen]

@calebbrown A friendly ping!