feat: processorの分割

[book000]

発話そのものの可否と、メッセージメタデータを使う処理、VoiceTextパラメータの変更を必要とする機能である Processor を Replacer 同様に分割し、processText関数内でのごちゃごちゃを軽減させます。

テストケース名レビュー: https://docs.google.com/spreadsheets/d/13ofOk0xV5aIzA8RJHYRaUE1Bhf84VuaBigPSb2EXhKI/edit?usp=sharing

進捗

• [x] Processor 群のクラス分割
• [x] schedule 内での Processor 呼び出し
• [x] 各 Processor のテスト作成
• [x] 各 Replacer のテスト作成
• [x] まともにコメントを書く

テストケースの整理

• Processor
  • [x] 返信メッセージ
  • [x] スタンプ
  • [x] ファイル
  • [x] インラインVoiceTextパラメーターのパース
  • [x] ignore (Replacer 前)
  • [x] Replacer
  • [x] ignore (Replacer 後)
  • [x] 強調文字の読み上げスピード変更 (speed / 1.3 または 50 のどちらか大きい方)
  • [ ] Markdown Heading の読み上げ音量変更 (h1: 200%, h2: 175%, h3: 150%) #155
  • [x] Markdown の書式文字削除 (リンク、コード、太字、斜字、下線、取消線、スポイラー)
  • [x] 180文字以上の場合180文字に制限
• Replacer
  • [x] エイリアス
  • [x] 絵文字
  • [x] 正規表現エイリアス
  • [x] チャンネルメンション
  • [x] ユーザメンション
  • [x] ロールメンション
  • [x] URL
  • [x] Discord メッセージ URL
  • [x] Discord チャンネル URL
  • [x] Discord イベント直接 URL
  • [x] Discord イベント招待 URL
  • [x] ツイート URL
  • [x] Discord 招待 URL
  • [x] Steam アイテム URL
  • [x] YouTube 動画 URL
  • [x] YouTube プレイリスト URL
  • [x] Google 検索 URL
  • [x] title タグ
  • [x] 拡張子

[book000]

Replacerのテストもう少し作り込まなきゃダメかな…

[book000]

https://kotest.io/docs/framework/integrations/mocking.html

[book000]

さて、結合テストのバターンを考えないと…。

[book000]

Attachment.downloadがですとできないやつ、HttpClient.getをmockすればいいのか？

[book000]

画像ファイルの文字起こし処理テストに断念。とりあえずメモ message.replyがinlineなことにより、モックできないらしい

    test("download") {
        // 画像ファイル image.png
        mockkStatic("com.kotlindiscord.kord.extensions.utils._AttachmentsKt")
        val attachment = mockk<Attachment>()
        every { attachment.isImage } returns true
        every { attachment.isSpoiler } returns false
        every { attachment.filename } returns "image.png"
        coEvery { attachment.download() } returns byteArrayOf(0x00)

        // VisionApi
        mockkObject(VisionApi)
        every { VisionApi.isGoogleAppCredentialsExist() } returns true
        every { VisionApi.getTextAnnotations(any()) } returns listOf(
            VisionTextAnnotation(
                "test",
                "ja",
                1.0f,
                listOf(VisionVertex(0, 0), VisionVertex(0, 0), VisionVertex(0, 0), VisionVertex(0, 0))
            )
        )
        every { VisionApi.drawTextAnnotations(any()) } returns mockk<ImmutableImage> {
            every { output(any<PngWriter>(), any<File>()) } returns mockk()
        }

        mockkObject(VCSpeaker)
        every { VCSpeaker.storeFolder } returns File("./store-test")

        mockkObject(VisionApiCounterStore)
        every { VisionApiCounterStore.get() } returns mockk {
            every { count } returns 0
        }
        every { VisionApiCounterStore.increment() } just Runs

        mockkStatic("dev.kord.core.behavior.MessageBehaviorKt")
        val message = mockk<Message>()
        coEvery { message.reply(any()) } returns mockk<Message>()
        every { message.attachments } returns setOf(attachment)

        val voice = Voice(speaker = Speaker.Hikari)

        val (processedText, processedVoice) = AttachmentProcessor().process(message, "test", voice)

        processedText shouldBe "test 画像ファイル image.png "
        processedVoice shouldBe voice
    }

[book000]

@yuuahp レビューをお願いします。基本的にはテストの追加ばっかりですが、PRあたりの変更がかなり大きくなってしまいごめんなさい

それはそれとして、assign reviewは相変わらずエラーです https://github.com/jaoafa/VCSpeaker.kt/actions/runs/9632849766/job/26566534103

[yuuahp]

もしかしてコミットされるとその部分のレビューのドラフト消える？

[book000]

うそお！？