Merge pull request #13 from tstromberg/main

add more test coverage

Author: tstromberg

cmd/server/interfaces.go

@@ -0,0 +1,21 @@
+package main
+
+import (
+	"context"
+
+	"github.com/codeGROOVE-dev/discordian/internal/discord"
+	"github.com/codeGROOVE-dev/discordian/internal/github"
+)
+
+// GitHubManager defines GitHub operations needed by the server.
+type GitHubManager interface {
+	RefreshInstallations(ctx context.Context) error
+	AllOrgs() []string
+	ClientForOrg(org string) (*github.OrgClient, bool)
+	AppClient() *github.AppClient
+}
+
+// DiscordGuildManager defines Discord guild management operations.
+type DiscordGuildManager interface {
+	RegisterClient(guildID string, client *discord.Client)
+}

cmd/server/main.go

@@ -174,9 +174,9 @@ func run(ctx context.Context, cancel context.CancelFunc) int {
 	return 0
 }
 
-// configAdapter adapts config.Manager to usermapping.ReverseConfigLookup.
+// configAdapter adapts bot.ConfigManager to usermapping.ReverseConfigLookup.
 type configAdapter struct {
-	mgr *config.Manager
+	mgr bot.ConfigManager
 }
 
 func (ca *configAdapter) Config(org string) (usermapping.OrgConfig, bool) {
@@ -194,11 +194,11 @@ type coordinatorManager struct {
 	notifyMgr      *notify.Manager
 	reverseMapper  *usermapping.ReverseMapper
 	active         map[string]context.CancelFunc
-	guildManager   *discord.GuildManager
+	guildManager   DiscordGuildManager
 	failed         map[string]time.Time
 	coordinators   map[string]*bot.Coordinator
-	configManager  *config.Manager
-	githubManager  *github.Manager
+	configManager  bot.ConfigManager
+	githubManager  GitHubManager
 	cfg            config.ServerConfig
 	dmsSent        int64
 	dailyReports   int64