神楽坂しえる @Clworld@md.ggtea.org

にゃー(つかれた)

さっきまでめっちゃ にゃーにゃー 言ってたのは Pubsubhubbub::DeliveryWorker のテストをしていたんですよ…

@unarist
raise e.class, "～" 回避したのも上げました https://github.com/tootsuite/mastodon/pull/4283

にゃー

@unarist ぐぬぬ😑

@unarist 何がなんでも元のExceptionのclassで投げる方向での実装で投げてみました。Wrapするとrspecでのraise_errorチェックをどうすればいいか分からんという理由による…。

ねこ…。

@unarist 次善策: Mastodon::WorkerError を作ってそれを投げる(perform_asyncで本来は例外返らないし大丈夫とみなす)

@unarist raise e.class, "～" していいものか迷ってる

@unarist 例外時無視してnil返す奴から呼ばれている => 例外変わったから無視されなくなった と UnexpectedResponseError をちゃんと返すテストケース の二つ

@unarist むずい

@unarist されちゃいますかー。

Q: rspecだとperform_asyncだった物が同期実行されます？

やっべこれ面倒くさいやつか

Exceptionチェックしてましたかー

あっCI死んだ

フォローしてる人のサーバが死んでると永久にSubscribeしようとしつづけるんよね…

Sidekiqのエラー見てもどのホストでエラーになったのか全く分からんぞ対応です。 / "Add callback_url/acct information for Sidekiq PuSH workers Exception." / https://github.com/tootsuite/mastodon/pull/4281

にゃー…

にゃー