eater / beater メモ

bouzuya @bouzuya

eater 。reporter もうちょっといい感じにしたいな……。 github.com/yosuke-furukaw…

Yosuke Furukawa @yosuke_furukawa

@bouzuya 僕もそう思ってるんですよ。。。割と適当なんで、要望あればおっしゃっていただければ対応します！！

bouzuya @bouzuya

@yosuke_furukawa 成功は省略、失敗は詳細で最後に件数・失敗をまとめて表示したいみたいな要求なんですが、標準でほしいかと言われるとそうでもない（シンプルさが eater の良さです) し、 既存の reporter API で自作を検討しています。

bouzuya @bouzuya

reporter の API が十分か？という意味の質問だったかもしれない……。

Yosuke Furukawa @yosuke_furukawa

@bouzuya ふむふむ、なるほど。いずれにせよ reporter 周りはなんかもうちょい綺麗にしたいんですよねー。ご意見ありがとうございます！

bouzuya @bouzuya

eater の d.ts ほしいな。

bouzuya @bouzuya

ああ、eater の reporter 。module.exports = Reporter; なのか……。module.exports.Reporter = Reporter; にならないかな……。

bouzuya @bouzuya

@yosuke_furukawa これ、未使用って理解でいいですか？ twitter.com/bouzuya/status…

bouzuya @bouzuya

あれ、Eater の Reporter 。reportSubTestName これ runner.js から testname message 来ないから使われてないのか。

2016-06-04 16:44:46

Yosuke Furukawa @yosuke_furukawa

@bouzuya 過去の名残ですね。必要だったら追加します。 github.com/yosuke-furukaw…

bouzuya @bouzuya

@yosuke_furukawa ありがとうございます！分かりました！いま必要というわけでもないので、大丈夫です。

bouzuya @bouzuya

お、標準とタイミングを変えるだけで、かなり良くなったぞ。

bouzuya @bouzuya

分かりやすいところから Eater の Reporter 改善案を出してみた。 github.com/yosuke-furukaw…

bouzuya @bouzuya

とりあえず SubTest ? で親子関係が取れないといろいろ面倒くさいので、お願いします、という感じ。

bouzuya @bouzuya

__esModule がないと default から探さないのって普通なのかな。 Eater の module.exports = Reporter; を module.exports.default = Reporter; でもいけるようにしようとしているんだけど……。

bouzuya @bouzuya

Eater Reporter の ES Modules の export default 対応を PR したぞ！！！ github.com/yosuke-furukaw…

bouzuya @bouzuya

eater の runner だけど、new Promise(f) の f じゃなくて Promise.resolve().then(g) の g にすれば毎回 resolve 呼ばなくて済むんじゃないの？

bouzuya @bouzuya

eater のいいところはソースコードを読んでから使いはじめても大丈夫（それくらいのサイズでできている）ってことかな。

Yosuke Furukawa @yosuke_furukawa

@bouzuya ありがとうございます！ちょっとおくれましたが件のPRマージして1.7.0で公開しているので使えるかと思います！

bouzuya @bouzuya

@yosuke_furukawa こちらこそ、ありがとうございます！ついでに質問なのですが→ twitter.com/bouzuya/status… のようになっていないのはなぜでしょう？

Yosuke Furukawa @yosuke_furukawa

@bouzuya reject （明示的な fail）をユーザーに公開したかったんですよね。eater はassertがbuilt-inで無いから・・・という。ただまぁ resolve を呼ばなくても良いんじゃない？っていう発想は確かにあって迷いました。

bouzuya @bouzuya

eater-b-reporter が eater の README に追加されたぞ！！ github.com/yosuke-furukaw…

bouzuya @bouzuya

これにまだ悩んでいる。 twitter.com/yosuke_furukaw…

bouzuya @bouzuya

まず、同期処理でも非同期処理でも callback しないとダメと、同期処理でも非同期処理でも callback なし (代わりに非同期処理は Promise を return する、という選択だと後者の方が良いように思える。

bouzuya @bouzuya

で、回答にあった reject (明示的なfail) をユーザーに公開したい場合は throw new Error() でもいいし、それを wrap した関数を提供してもいいし、Promise.reject(new Error()) を return してもいい、だとダメなのか。

bouzuya @bouzuya

ヘルパ。 (s: string, f: <T>() => T | Promise<T>): void => { test(s, (resolve, reject) => Promise.resolve().then(f).then(resolve, reject)); };