fix: fixing name of exclusion option (#58)

CallumKerson · Aug 8, 2023 · 234acd1 · 234acd1
1 parent 458a8e7
commit 234acd1
Show file tree

Hide file tree

Showing 5 changed files with 18 additions and 18 deletions.
diff --git a/cmd/athenaeum/cmd.go b/cmd/athenaeum/cmd.go
@@ -103,12 +103,12 @@ func runServer(cfg *Config) error {
 		audiobookOpts = append(audiobookOpts,
 			audiobooksService.WithThirdPartyNotifier(overcastNotifier.New(cfg.Host, overcastNotifier.WithLogger(logger))))
 	}
-	if len(cfg.ExcludsionsFromMainFeed.Genres) > 0 {
-		genres, err := cfg.ExcludsionsFromMainFeed.GetGenres()
+	if len(cfg.ExclusionsFromMainFeed.Genres) > 0 {
+		genres, err := cfg.ExclusionsFromMainFeed.GetGenres()
 		if err != nil {
 			return err
 		}
-		audiobookOpts = append(audiobookOpts, audiobooksService.WithGenresToExludeFromAllAudiobooks(genres...))
+		audiobookOpts = append(audiobookOpts, audiobooksService.WithGenresToExcludeFromAllAudiobooks(genres...))
 	}
 	audiobookSvc := audiobooksService.New(mediaSvc, boltAudiobookStore, audiobookOpts...)
 	if errScan := audiobookSvc.UpdateAudiobooks(context.Background()); errScan != nil {

diff --git a/cmd/athenaeum/config.go b/cmd/athenaeum/config.go
@@ -19,15 +19,15 @@ const (
 )
 
 type Config struct {
-	Host                    string
-	Port                    int
-	DB                      DB
-	Media                   Media
-	Podcast                 Podcast
-	ThirdParty              ThirdParty
-	Log                     Log
-	Cache                   Cache
-	ExcludsionsFromMainFeed ExcludsionsFromMainFeed
+	Host                   string
+	Port                   int
+	DB                     DB
+	Media                  Media
+	Podcast                Podcast
+	ThirdParty             ThirdParty
+	Log                    Log
+	Cache                  Cache
+	ExclusionsFromMainFeed ExclusionsFromMainFeed
 }
 
 type Log struct {
@@ -77,11 +77,11 @@ func (c Cache) GetTTL() time.Duration {
 	return ttl
 }
 
-type ExcludsionsFromMainFeed struct {
+type ExclusionsFromMainFeed struct {
 	Genres []string
 }
 
-func (e ExcludsionsFromMainFeed) GetGenres() ([]audiobooks.Genre, error) {
+func (e ExclusionsFromMainFeed) GetGenres() ([]audiobooks.Genre, error) {
 	genres := []audiobooks.Genre{}
 	for _, genreName := range e.Genres {
 		genre, err := audiobooks.ParseGenre(genreName)

diff --git a/cmd/athenaeum/config_test.go b/cmd/athenaeum/config_test.go
@@ -41,7 +41,7 @@ func TestConfig_FromFile(t *testing.T) {
 Host: "http://localhost:8088"
 Podcast:
     Language: FR
-ExcludsionsFromMainFeed:
+ExclusionsFromMainFeed:
     Genres:
       - nonfiction
 `
@@ -59,7 +59,7 @@ ExcludsionsFromMainFeed:
 	assert.Equal(t, "http://localhost:8088", config.Host)
 	assert.Equal(t, "FR", config.Podcast.Language)
 
-	genres, err := config.ExcludsionsFromMainFeed.GetGenres()
+	genres, err := config.ExclusionsFromMainFeed.GetGenres()
 	assert.NoError(t, err)
 	assert.Equal(t, []audiobooks.Genre{audiobooks.NonFiction}, genres)
 }

diff --git a/internal/audiobooks/service/opts.go b/internal/audiobooks/service/opts.go
@@ -20,7 +20,7 @@ func WithThirdPartyNotifier(notifier ThirdPartyNotifier) Option {
 	}
 }
 
-func WithGenresToExludeFromAllAudiobooks(genres ...audiobooks.Genre) Option {
+func WithGenresToExcludeFromAllAudiobooks(genres ...audiobooks.Genre) Option {
 	return func(service *Service) {
 		if len(genres) > 0 {
 			var genreFilters []Filter

diff --git a/internal/audiobooks/service/service_test.go b/internal/audiobooks/service/service_test.go
@@ -50,7 +50,7 @@ func TestFilteringAllAudiobooks(t *testing.T) {
 		{name: "no filter options", filterOptions: []Option{}, expectedAudiobooks: testbooks.Audiobooks},
 		{
 			name:               "filter genres from all audibooks",
-			filterOptions:      []Option{WithGenresToExludeFromAllAudiobooks(audiobooks.SciFi)},
+			filterOptions:      []Option{WithGenresToExcludeFromAllAudiobooks(audiobooks.SciFi)},
 			expectedAudiobooks: []audiobooks.Audiobook{testbooks.Audiobooks[1]},
 		},
 	}