section082のkvs_test.goについて

[stutkhd-0709]

Redisにキーが存在しない場合NotFoundでアサートしてますが、RedisはNotFoundではなくRedis.nilを返すようです
(github.com/go-redis/redis/v8 v8.11.5)
該当コード

go_todo_app/store/kvs_test.go

Lines 61 to 63 in 61b3a6b

	if err == nil || !errors.Is(err, ErrNotFound) {
	t.Errorf("want %v, but got %v(value = %d)", ErrNotFound, err, got)
	}

go-redis
https://github.com/redis/go-redis/blob/233f97accde15bcb2e3d86ee744109b4dce34171/string_commands.go#L51

[budougumi0617]

ご指摘ありがとうございます！
該当のテストはgo-redisのテストではなく、 store.KVS 構造体のテストなので store.NotFound が返ってきますね。
redis. ErrNotFound ではなく ErrNotFound になっているので同一pkg内で定義された宣言だとわかります。

go_todo_app/store/kvs.go

Lines 32 to 38 in 61b3a6b

	func (k *KVS) Load(ctx context.Context, key string) (entity.UserID, error) {
	id, err := k.Cli.Get(ctx, key).Int64()
	if err != nil {
	return 0, fmt.Errorf("failed to get by %q: %w", key, ErrNotFound)
	}
	return entity.UserID(id), nil
	}

CIでもテストはpassしているので。もしテストがfailするならば他の原因があると思います！
https://github.com/budougumi0617/go_todo_app/actions/workflows/test.yml

[stutkhd-0709]

あ、なるほど！

return 0, fmt.Errorf("failed to get by %q: %w", key, ErrNotFound) 

こちらのエラーにErrNotFoundが抜けていたためでした！ありがとうございます！

[budougumi0617]

解決してよかったです！また不明点あればお気軽にご記載ください！（確認・返信タイミングはまちまちです 🙇