React: パブリックAPIを介してより多くのテストを表現する

週末にこれを見て、短期的に取り組むことができるかどうかを確認することができます。

まことにありがとうございます！ 特定のテストを選択する場合は、他の誰かが同じテストで作業を開始しないように、スレッド内のファイル名をコメントにしてください。

私は間違いなくこれに貢献することに非常に興味があります。 私は今週末を調べて、リファクタリングする機会を見つけます！

私も興味があります👍

最初の投稿でテストのリストを公開しました。 どれを取りたいか教えてください。割り当てます。

ReactDOMInput-testは私には問題ありません:)

@SadPandaBear了解しました！

ReactErrorUtils-testで作業できます。 😄

setInnerHTML-test.jsを見てみましょう

getEventCharCode-test.jsをやります。 😀

getEventKey-test.jsで作業できます。

escapeTextContentForBrowser-test.jsを取ることができます。

ChangeEventPlugin-test.jsを試してみたいです:)

SyntheticEvent-test.jsを取ることができます

EnterLeaveEventPlugin-test.jsに取り組みたい

ReactDOMEventListener-test.jsに取り組みたい

BeforeInputEventPlugin-test.jsを取りたいのですが

SyntheticKeyboardEvent-test.jsを取得したいと思います。 ありがとう👍

inputValueTracking-test.jsを取りましょう

SyntheticWheelEvent-test.jsに取り組みたい

取ることができます： ReactBrowserEventEmitter-test.js ？

私はSelectEventPlugin-test.jsを取っています

ReactDOMComponentTree-test.jsに取り組みたい

ReactTreeTraversal-test.jsで作業したい

やあ！ 👋 ReactCoroutine-test.jsに取り組みたい

SyntheticClipboardEvent-test.jsを取ることができます

validateDOMNesting-test.jsに取り組みたい

私はEventPluginRegistry-test.jsを取ることができます

ReactDOMComponent-test.jsを試してみたい

こんにちは！ quoteAttributeValueForBrowser-test.jsを試してみたいです！ ：筋：

こんにちは！ ReactDOMServerIntegration-test.jsを試してみたいです！

ReactIncrementalPerf-test.jsを取ります

getNodeForCharacterOffset-test.jsを実行します

@burnsbeaver FallbackCompisitionState-testとgetNodeForCharacterOffset-test $のどちらですか？ まず、1人につき1回のテストから始めたいと思います。

getNodeForCharacterをお願いします。 他のコメントを削除しました。混乱してすみません！

FallbackCompositionState-test.jsとReactFiberHostContext-test.jsはまだ無料です！

FallbackCompisitionStateを取得します！

ReactFiberHostContext-test.jsを試すことができます

更新：すべてのテストが現在行われています。 この号を購読してください！ 誰かが仕事を終える時間がない場合、彼らは将来解放されるかもしれません。 いくつかのテストが利用可能になったら、再試行するためにコメントします。

テストを受けたすべての人に—ありがとう！ 繰り返しになりますが、解決策は必ずしも明確ではなく、場合によっては、Reactソース自体に変更を加えることなく動作を合理的にテストすることさえできない場合があります。 行き詰まったと感じた場合は、ここにコメントしてください。計画を立てようとします。

@adsonpleal失望させて申し訳ありません。確認したばかりですが、残念ながら、誤ってReactFiberHostContext-testを修正してしまいました。 最初は気づかなかったのですが、TODOは時代遅れです。 これは私が見ることができる唯一のテストであり、すでに修正されています。

@gaearonうわー、 ReactDOMInput-testのリファクタリングには少し時間がかかりますが、今週末にすべてを終えることを願っています。

私は少し疑問があります：
inputValueTracking setUntrackedValue $からReactTestUtils.Simulate.?のようなものに、そのすべての動作を置き換えても大丈夫ですか？

ちなみに、ここで私のPRをチェックして、 inputValueTrackingを削除するために私が何をしたかを確認できます。 これらのテストのいくつかはinputValueTrackingの非常に特殊なものに依存している可能性があることは知っていますが、代わりにReactTestUtilsを効果的に使用できるかどうか疑問に思っていました。

inputValueTrackingの場合、 @ jquenseは、それを最適に行う方法について最も多くのコンテキストを持っている必要があります。 ただし、機能する限り、アプローチが機能するかどうかを確認して確認できます。

他の人に注意してください。古いバージョンのyarn（0.22）をインストールしていて、ローカルでテストを実行しようとしていたところ、すべてが壊れていました。 ヤーンを最新（1.2.1）にアップグレードすると、問題が修正され、テストは現在正常に実行されています。

@sadpandabearテストファイルに同じモジュールがあるので、すぐに実装を確認します

@gaearon setInnerHTMLテストの場合、テストスイートが意味をなすためにsetInnerHTMLの存在に依存しているように感じます。

既存の関数を置き換えるためにいくつかのテストを書くことができますが、テストスイートは、 setInnerHTMLに固有のものではなく、内部htmlの設定に沿ったものになります。 テストスイートの名前付けに少しこだわっていると思います。

編集：私が間違っていない限り...SVGにはinnerHTMLプロパティがあります。 このファイルのテストの2番目のチャンクが必要かどうかさえわかりません。

@silvestrijonathanこれらのテストが実際に検証しようとしているのは、 dangerouslySetInnerHTMLがReactで期待どおりに機能することだと思います。 したがって、ReactDOMを使用するようにこれらのテストを変更し、同じコンテンツをdangerouslySetInnerHTMLでレンダリングする場合は、それで十分です。 テストの名前をdangerouslySetInnerHTML-testに変更する方が適切だと思われる場合は、問題ありません。

テストファイルで数時間遊んだ後、（私のような）初心者に強い提案があります。クリーンなenv（ファイル）でテストを開始してください。
既存のファイルを編集していて、いくつかの内部関数をモックしたので、いくつかのReactTestUtils関数が壊れていました!! 物事が意図したとおりに機能しなかった理由を理解するのに少し時間がかかりました。

@gaearonイベントのシミュレーションを含め、 ReactTestUtilsはプライベートAPIと見なす必要があることを明確にする必要があると思います。 これは間接的に内部APIに到達するため、それ以外の場合にどのように機能するかを適切にテストすることはできません。 たとえば、これらのフックを公開せずにコンパイル済みパッケージをテストする機能が妨げられ、新しい非合成イベントシステムをテストできなくなります。

@dphurley 、あなたがReactIncrementalPerf-testに取り組んでいることに気づきました。 ReactCoroutineを使用するテストスイートにも取り組んでいたので、このPRについてお知らせします：＃11338。
すでにPRを提出しているかどうかはわかりませんが、提出していない場合は、＃11338（受け入れられている場合）に基づいて作成することをお勧めします。

@gaearonアドバイスありがとうございます。 React-DOMとdangerouslySetInnerHTMLをテストに利用することを考えたので、そのようにします。

はい、その場合はおそらくテストの名前をその名前に変更します。 翌日かそこらで私のPRを探してください。

最初に統合されたPRがあります！

それを見てください： https ：//github.com/facebook/react/pull/11309

イベントに関連する何かに取り組んでいる場合は、 https ：//github.com/facebook/react/pull/11309のディスカッションと次のレビューの両方を読むことをお勧めします。

https://github.com/facebook/react/pull/11316
https://github.com/facebook/react/pull/11317
https://github.com/facebook/react/pull/11327
https://github.com/facebook/react/pull/11332

いくつかの共通のテーマがあり、それらの議論を使用して他のPRを正しい方向に導くことが価値があるかもしれません。

参照するのに最適なコンボ-ありがとう！

仕事は少し忙しくなりましたが、今週末はテストのためにPRを開くことができると思います。

私はまだ私が拾ったテストを調べていません。 他のことに忙しくなりました。 あと数日でPRを上げてみますが、いつか買えますか？

いいですね、確かに！

私は同じ問題を抱えていました

別のPRが統合されました！ https://github.com/facebook/react/pull/11316

もう2つダウン。 https://github.com/facebook/react/pull/11317 https://github.com/facebook/react/pull/11385

私がテストの1つにどのようにアプローチしたかについての小さな分析を書きました。 https://github.com/facebook/react/pull/11385#issuecomment -341934588

私が知る限り、それらはすべて取られています、もしあればここに投稿してください。

特にSyntheticEventを扱う2つのPRをマージしました。 関連するテストを受けた場合は、それらをチェックして、スタイルとアプローチを一致させてみてください： https ://github.com/facebook/react/pull/11365およびhttps://github.com/facebook/react/issues/ 11299

@gaearon EventPluginRegistry-test.jsの場合、パブリックAPIを使用してテストをどのように書き直す必要がありますか？ReactDom.js->ReactDOMClientInjection.jsにデフォルトのプラグインを挿入していることがわかります
{{
SimpleEventPlugin、
EnterLeaveEventPlugin、
ChangeEventPlugin、
SelectEventPlugin、
BeforeInputEventPlugin、
}

では、テストモックプラグインをどの程度正確に挿入してEventPluginRegistry機能をテストできるか
カスタムプラグインを検索し、 react-tap-event-pluginを見つけて、カスタムプラグインがどのように挿入されているかを確認しました。
これがプラグインを挿入する唯一の方法ですか

require('react-dom').__SECRET_INTERNALS_DO_NOT_USE_OR_YOU_WILL_BE_FIRED.EventPluginHub.injection.injectEventPluginsByName({ 'TapEventPlugin': require('./TapEventPlugin.js')(shouldRejectClick) });

ReactTreeTraversal-test.jsで頭がおかしくなるのではないかと心配しています
他の誰かがそれを試してみたい場合は、遠慮なくそうしてください。

@aqumus

EventPluginRegistryは注意が必要です。 最初のステップは、実際にどこで使用されているかを把握することだと思います。 確かに、タップイベントプラグインは、私たちがまだサポートに取り組んでいる唯一のプラグインです。 便利なことに、レポに正しくあります。 それで、それが特に機能するテストを追加できるでしょうか？

RNがEventPluginRegistryを使用している可能性もあります。 RNイベントが機能することを確認するテストはありますか？ わからない。 EventPluginRegistryのさまざまな部分をコメントアウトして、テストが失敗するかどうかを確認する価値があります。 同じコードが、それらのコードパスにストレスを与える他のテストですでにカバーされている場合は、プライベートAPIテストを削除できます。 これは意味がありますか？

@gdevincenziいいですね、割り当てを削除します。

また、今週は不在になりますが、これを続けないでください。 来週、さらにいくつかの提出されたPRがマージされることを望んでいます。

私はReactTreeTraversal-test.jsを取ることができます-それはなじみのないAPIですが、それを学ぶのは良い機会のようです。 アドバイスようこそ@gdevincenzi

かなり複雑なものを完成させてくれた@gordydにおめでとうございます： https：//github.com/facebook/react/pull/11383。

これは、本質的に同じことをテストしているときに、コードが前後でどのように異なるかを示す良い例です。 そして、参照を検索し、gitの履歴を調べ、さまざまな行をコメント化することで、コードが何をしているのかを理解する方法を説明します。

以下のユーザーからのPRはまだ受けていません。

@ email2vimalraj
@andrevargas
@tranotheron
@minerado
@ sw-yx
@ king0120
@ aarboleda1
@danilowoz
@ dms1lva
@morajabi
@dphurley
@reznord
@aqumus

これに取り組み始めたかどうか確認していただけますか？ 待ち行列の中で助けたいと思っている人がたくさんいるので、時間がない場合は、他の人にこれに取り組む機会を与えるのは素晴らしいことです。

@andrevargashttps ://github.com/facebook/react/pull/11331であなたの議論に気づきました。 それらの変更をそこで統一しておくことは理にかなっていると思います。

そのキューに私を追加してください@gaearon

@gaearonはい、その通りです。 確認のために、 @ jeremenichelliは＃11331のquoteAttributeValueForBrowser-test.jsに取り組んでおり、私はこれを認識しています。 彼のユーザー名とPRのリンクでこのスレッドを更新できると思います。

@gaearonのキューにも私を追加してください

@gaearon私はそれに取り組み始めました。 今週中に私の調査結果またはPRを投稿することを計画しています。

@gaearonSyntheticKeyboardEvent.jsの作業を開始しました。 今週末にPRを行う予定です。

@gaearon最初に割り当てられた人がそれらに取り組むのに必要な時間がない場合に備えて、私はテストのいずれかを取ることもできました。

@gaearon長い遅れで申し訳ありませんが、今WIPPRを作成しています。 更新：誰かがプライベートAPIに依存しないために何をすべきかを簡単に説明できますか？ また、 top<EventName>についてもわかりません。 ありがとう。

https://github.com/facebook/react/blob/92b7b172cce9958b846844f0b46fd7bbd8c5140d/packages/react-dom/src/events/__tests__/SelectEventPlugin-test.js> extract関数

@gaearon長い遅延でごめんなさい😅。 私は今日これに取り組んでおり、それについて疑問がある場合はあなたにpingします。

@morajabiこれに取り組むための1つの潜在的な方法

• SelectEventPluginを見て、それがユーザーに提供する機能を理解します
• その機能を利用するコードを書く
• SelectEventPluginの読み込みを無効にして（コメントアウト）、何が壊れているかを確認します（何かが壊れていることを確認してください）
• これで、作成したコードがテストの最初のバージョンになり、テストケースに入れることができます。
• SelectEventPluginの読み込みを再度有効にして、テストに合格することを確認します
• コードを最適化する

@gaearon遅延のお詫び
あなたの提案に基づいて、私はEventPluginRegistry.jsのいくつかの部分にコメントしてみましたが、多くのテストケースが失敗していることがわかりました（ SyntheticWheelEvent 、 ReactDOMComponentTree 、 SyntheticClipboardEvent 、 SyntheticWheelEvent 、 inputValueTracking 、 SimpleEventPlugin 、 ChangeEventPlugin 、 ReactDOMComponentTreeなど）、
ReactBrowserEventEmitter.jsがEventPluginRegistry.jsで定義されているregistrationNameDependenciesを使用していることが主な理由だと思います

コメントアウトするときですが

• 208行目
• 38-41行目

失敗したテストケースはありませんでした（ EventPluginRegistryテストのみが失敗しました）
これに基づいて、 EventPluginRegistryはすでに他のテストスイートでカバーされていると思います。

便利なことに、レポに正しくあります。

これが何を意味するのか分かりませんでした

パブリックAPIを使用してEventPluginRegistryを書き換えるために取り組むことができる他の提案はありますか？

他の人にとっては（これが以前に放送されたかどうかはわかりませんが）、 @kentcdoddsビデオポッドキャスト-BenAlpertによるreactイベントコードのウォークスルーが非常に便利であることがわかりました。

申し訳ありませんが@gaearonやろうとしましたが、今はそれを終える時間がありません
したがって、他の誰かが私の問題を取得したい場合は、お気軽に:)

@danilowoz @gaearonよろしければ、BeforeInputEventPlugin-test.jsを使用できます。

@timjacobi ReactDOMComponentTree.getInstanceFromNodeの代わりは何ですか？

また、どこで無効にすればよいですか？

SelectEventPluginの読み込みを無効にし（コメントアウト）、何が壊れているかを確認します（何かが壊れていることを確認します）

@morajabiあなたがこれに正しいレベルで取り組んでいるかどうかはわかりません。 これまでに持っているものでWIPPRを開いてもらえますか？ このスレッドをあまり汚染したくありません。

@timjacobi何も新しいことをコミットしていないときはできません。 誰かがSelectEventPlugin-testを取ることができれば、私は最近かなり忙しいのでごめんなさい。

他の変更と一緒にカバーしたので、 ReactDOMServerIntegration-test （cc @minerado）はもう必要ないようです。

@aqumus分析に基づいて、このリストからEventPluginRegistry-test.jsを削除します。 これはこれらの特定の内部の単体テストとして保持すると思いますが、実際のロジックが他のテストでカバーされていることを知っておくとよいでしょう。

@ king0120 getEventCharCode-testを@ aarboleda1に再割り当てします。彼は、 https：//github.com/facebook/react/pull/11631で非常に関連性の高いテストに取り組んでいるため、一緒に実行する方が簡単な場合があります。

@morajabi @gaearonSelectEventPlugin -test.jsも見ることができます。

@ email2vimalraj参考までに、 inputValueTracking-testを書き直しました。

現在、これらのテストには所有者がいません。

• [] @accordeiroが撮影したgetNodeForCharacterOffset-test.js （3/5）
• [] @madeinfreeが撮影したReactBrowserEventEmitter-test.js （5/5）
• [] BeforeInputEventPlugin-test.js + FallbackCompositionState-test.js （5/5、組み合わせる必要があります）@GordyDが撮影
• [x] @skiritsisが撮影したSelectEventPlugin-test （2/5）
• [x] @timjacobiが撮影したReactTreeTraversal-test.js （4/5）

私は彼らの難しさの認識によってそれらを分類しました。

@timjacobi @kwnccc @skiritsisのキューがありますので、これらのテストのいずれかを受験したい場合は、3人でお知らせください（どれを指定してください）。 合格することもできます。他の人に試してもらいます。

これらはかなり複雑になっているため、初心者向けではないことに注意してください。 それらは機械的な代替品ではありません。 テストが何を検証しようとしているのかを理解してから、同等のパブリックAPIテストを考え出す必要があります（これは非常に異なって見える場合があります）。

これらを取得するには、 @ GordyDがhttps://github.com/facebook/react/pull/11383で行ったのと同様に、自分で物事を快適に調査できる必要があります。 実際、 @ GordyDがこれらのいずれかを受け取りたい場合は、喜んで彼に1つあげます。

@skiritsis興味があったので、 SelectEventPlugin-testを再割り当てします。

ReactBrowerEventEmitter-test.jsに取り組みたいです💪

@gaearon ：それをありがとう、時間通りに応答しなかったことをお詫びします。 他の仕事で忙しくなりました。 私の理解のためにあなたがどのように書き直したかを調べます。

ReactTreeTraversal-test.internal.jsを見てみましょう

@madeinfree @kwncccはすでにキューに入っているので、最初に返信するために数日与えましょう。 彼が興味を持っていない場合は、後で割り当てます。

@gaearon OK！ 了解しました、ありがとうございます〜

@gaearon他にテイカーがいない場合は、 BeforeInputEventPlugin + FallbackCompositionStateの組み合わせを見ていきます。

@GordyDに感謝します！

ちなみに、このコードが通常どのようになるかを理解するために、変換されたイベントテストがさらにいくつかあります：https： //github.com/facebook/react/pull/11631 https：//github。 com / facebook / react / pull / 11525

@gaearon誰も欲しがらなければ、 ReactBrowserEventEmitter-test.jsも喜んで受け取ります。

@madeinfree

まだReactBrowerEventEmitterに興味がありますか？ もしそうなら、それを取り、私たちを最新の状態に保ってください！

@gaearon OK！ 撮りたいです。

了解しました。

残りのgetNodeForCharacterOffset-testを受け取りたい人はいますか？

私はこれを手伝うことができます（ getNodeForCharacterOffset-test ）:)

@gaearon思ったように、 ReactIncrementalPerf-test.jsに飛び込む時間が見つかりませんでした。 長い間お詫び申し上げますが、誰かがそれを望んでいるなら、それは手に入れることができます！

@accordeiro了解しました！

@gaearon本当に申し訳ありませんが、通知を受け取りませんでした。 まだ対処すべきテストがある場合は、私に知らせてください。 簡単に始められるものが残っているといいのですが。

@kwnccc ReactIncrementalPerf-test.jsを調べたいですか？

ReactIncrementalPerf-test.jsを調べてもいいですか？ または、必要に応じて、 @kwncccがテストを作成するのを手伝うことができます
ReactIncrementalPerf.jsという名前のファイルが表示されないので、 ReactPortal.jsのテストを作成する必要がありますか？

ReactIncrementalPerf-test.jsの問題を少しでも理解している場合（間違っている場合は修正してください）、ReactPortal.createPortal（...）からReactDOMとreact-domのようなパブリックAPIに切り替える必要があることを意味します.createPortal（...）？ @gaearon

@gaearon ReactIncrementalPerf-test.jsを試してみたいです。 @ abiduzz420がチェックアウトを開始したので、必要に応じてチームを組むこともできます。

ええ、確かに@kwnccc私はあなたと一緒に仕事ができてうれしいです。 あなたは私の前に列に並んでいたので、あなたのすべてです！

@kwnccc yarn testで1つのテストエラーが発生し、$＃ yarn flowを実行するとフローエラーが発生します。 コードをフォークされたリポジトリにプッシュします： https ：//github.com/abiduzz420/react

 FAIL  packages\react-reconciler\src\__tests__\ReactIncrementalPerf-test.internal.js (11.191s)
  ● ReactDebugFiberPerf › supports portals

    Invariant Violation: Target container is not a DOM element.

      at invariant (node_modules/fbjs/lib/invariant.js:42:15)
      at Object.createPortal (packages/react-dom/src/client/ReactDOM.js:1112:70)
      at Object.<anonymous> (packages/react-reconciler/src/__tests__/ReactIncrementalPerf-test.internal.js:510:116)

$ yarn flow
yarn run v1.3.2
$ node ./scripts/tasks/flow.js
Error: packages/react-reconciler/src/ReactFiberHostContext.js:87
 87:     const nextContext = getChildHostContext(context, fiber.type, rootInstance);
                                                 ^^^^^^^ NoContextT. This type is incompatible with the expected param type of
                    v---------------------------------------------
 31: export default function<T, P, I, TI, HI, PI, C, CC, CX, PL>(
 32:   config: HostConfig<T, P, I, TI, HI, PI, C, CC, CX, PL>,
 33: ): HostContext<C, CX> {
     --------------------^ some incompatible instantiation of `CX`

Error: packages/react-reconciler/src/ReactFiberHostContext.js:87
 87:     const nextContext = getChildHostContext(context, fiber.type, rootInstance);
                                                                      ^^^^^^^^^^^^ NoContextT. This type is incompatible with the expected param type of
                    v---------------------------------------------
 31: export default function<T, P, I, TI, HI, PI, C, CC, CX, PL>(
 32:   config: HostConfig<T, P, I, TI, HI, PI, C, CC, CX, PL>,
 33: ): HostContext<C, CX> {
     --------------------^ some incompatible instantiation of `C`
Found 2 errors
Flow failed
error Command failed with exit code 2.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

@ abiduzz420フローの問題は、Windowsの既知の問題です。 それは無関係です（しかし、なぜそれが起こるのかを理解する必要があります）。 https://github.com/facebook/react/issues/11703を参照してください。

@kwnccc ReactIncrementalPerf-testに関しては、はい、これにはいくつかの考慮が必要です。 おそらくこの時点で、一時的にReactPortalをsharedに移動するのが最善だと思います。 次に、 ReactNoopに独自のcreatePortal()メソッドを提供させます（ ReactDOMのように）。

もう1つダウン！ https://github.com/facebook/react/issues/11299

@gaearon ReactPortalを共有ディレクトリに移動しましたが、すべてのテストに合格しています。 ReactNoop $に対して自分のメソッドcreatePortal()を実装するには、少し支援が必要です。 createPortal()がReactDOM ReactPortal.createPortal(children,container,null,key)を返します。
私はこれらの行について考えています：

function createPortal( children: ReactNodeList, container: Container, key: ?string = null) {
invariant( 
// TODO: Need to figure out this part of the code
);
return ReactPortal.createPortal(children, container, null, key);
}

私が正しくやっているかどうか教えてください。

@gaearon私のコードはまだ仕掛品です。 しかし、私は自分が正しい方向に進んでいるかどうかを再確認したかったのです。 私のコードをざっと見て、いくつかの疑問を明らかにしていただければ、本当に助かります。

1. 私が取ったアプローチは正しいですか？
2. テストは、既存のタグとシナリオでテストされるすべてのタグとシナリオをカバーする必要があると思います
   テストカバレッジを維持するためのテスト。 これらの特別なフォーマットタグを使用してコンポーネントを動的に生成する方法があるかどうかはわかりません。 これを達成するための効率的な方法について、いくつかの指針を教えてください。
3. メソッドisTagValidInContext（）はテストファイル以外では使用されていないため、パブリックAPIを使用して「anytagnocontext」テストの「nocontext」部分をテストする方法がわかりません。 何か案は？

@ abiduzz420はい、これは正しいですね

@anushreesubramani

私が取ったアプローチは正しいですか？

一般的にはそうですが、そこでの重複は避けたいと思います。 理想的には、一連の

expect(isTagStackValid(['a', 'a'])).toBe(false);

一連の

expectInvalidNestingWarning(true, ['a', 'a']);

ここで、 expectInvalidNestingWarningは、コンポーネントを生成し、それをレンダリングすると警告が生成されることを表明するカスタム関数です。

これらの特別なフォーマットタグを使用してコンポーネントを動的に生成する方法があるかどうかはわかりません。 これを達成するための効率的な方法について、いくつかの指針を教えてください。

はい、私は次のようなことを考えています：

function expectInvalidNestingWarning(shouldWarn, tags) {
  let element = null;
  tags = [...tags];
  while (tags.length) {
    element = React.createElement(tags.pop(), null, element);
  }
  const container = document.createElement('div');
  ReactDOM.render(element, container);
  // assert either a warning or lack of one based on shouldWarn
}

配列を調べて最後の項目を取得し、現在の要素を対応するタグを持つ親にラップします。 このようにして、最終的に対応するツリーになります。 私は何か間違ったことを書いたかもしれませんが、それが一般的な考えです。

あなたはおそらく次のようなものを置きたいと思うでしょう

jest.resetModules();
React = require('react');
ReactDOM = require('react-dom');

レンダリングの直前にその関数で。そうしないと、警告が重複排除され、確実にテストできないためです。

メソッドisTagValidInContext（）はテストファイル以外では使用されていないため、パブリックAPIを使用して「anytagnocontext」テストの「nocontext」部分をテストする方法がわかりません。

責任を見ると、 https：//github.com/facebook/react/commit/76bb96ef21b7c665e1b51b6bb90e64ec40c0f16aに追加されており、その時点でテスト外でも呼び出されていました。 しかし、将来のある時点で、それはもはや使用されませんでした。 テストでは、サーバーのレンダリングについて言及しています。

https://github.com/facebook/react/blob/8cbc16f0faedafe4f7424b286af52dafd7a79587/packages/react-dom/src/__tests__/validateDOMNesting-test.internal.js#L144 -L145

ただし、この関数はサーバーレンダリングでも使用されなくなりました。

このコメントから、祖先がわからないときに警告しないようにすることが目標であることがわかりました。 ただし、 nullをancestorInfoとして$ validateDOMNestingに渡すコードパスが見つかりません。 したがって、おそらく現在のバージョンの祖先を常に知っています。 したがって、このテストを完全に削除しても安全だと思います。

@gaearon ReactNoop.js createPortal()のメソッドを作成しました。その前に、提案したようにReactPortal.jsファイルも共有ディレクトリに移動しました。 私のコードを確認してください。変更や追加が必要な場合は、それを実行します。

PRを送ってもらえますか？ レビューで議論する方が簡単です。

@ abiduzz420あなたは素晴らしい仕事をしたようです、申し訳ありませんが、今日までそれを始める時間がありませんでした！ テストを完了できて本当に良かったです！ 👏👏
@gaearon次は私が素晴らしい可用性を持っていることを確認します

@gaearonクイックアップデート： BeforeInputEventPlugin + FallbackCompositionStateテストのリファクタリング/結合を開始しました。 パブリックAPIを介して実装をテストする方法をよく理解しています。 私は現在、さまざまな実行環境/ブラウザーエンジンに応じてさまざまなコードパスを実行するためのテストケースの作成に取り組んでいます。 今週の終わりに向けて最初のPRが予定されています（12月10日までに更新されました。もう少し時間が必要です。12月12日を見てください）。

いいですね、アップデートしてくれてありがとう！

@gaearonのクイックアップデート： getNodeForCharacterOffset-testのテストの書き直しをすでに開始しており、木曜日までPRをプッシュできるはずです。これでよろしいですか？

いいですね

もう1つダウン！ https://github.com/facebook/react/pull/11742

@reznordまだ連絡がありません。何か始めましたか？ 忙しすぎる場合は、他の人に試してみる機会を与えたほうがいいかもしれません。

Ping @reznord

こんにちは@gaearon 、最初の寄付にReactErrorUtils-test.jsを取ることはできますか？

こんにちは、 @gaearonはReactErrorUtils-test.jsでできることがあります

こんにちは、誰かがReactBrowserEventEmitter-test.jsテストを続行するのを手伝ってくれますか？ 今は仕事で忙しいので、続ける時間がないので、PRはhttps://github.com/facebook/react/pull/11713です、どうもありがとうございました!!

みなさん、ありがとうございました！ まだいくつかの未解決のテストがありますが、私たちはそれを閉じることができると思います。

@gaearon ： ReactErrorUtils-test.js私はそれに取り組みたいと思います。 私がそれを拾うことができるかどうか確認できますか？