React: सार्वजनिक एपीआई के माध्यम से अधिक परीक्षण व्यक्त करें

मैं इसे सप्ताहांत में देख सकता हूं और देख सकता हूं कि क्या मैं इसे अल्पावधि में निपट सकता हूं।

बहुत बहुत शुक्रिया! यदि आप कोई विशेष परीक्षण चुनते हैं, तो कृपया थ्रेड में फ़ाइल नाम के साथ टिप्पणी करें ताकि कोई अन्य व्यक्ति उसी परीक्षण पर काम करना शुरू न करे।

मुझे इसमें योगदान देने में निश्चित रूप से बहुत दिलचस्पी है। मैं इस सप्ताह के अंत में देखूंगा और रिफैक्टर करने का अवसर ढूंढूंगा!

मुझे भी दिलचस्पी है

मैंने पहली पोस्ट में परीक्षणों की एक सूची प्रकाशित की। बस मुझे बताएं कि आप किसे लेना चाहते हैं और मैं उन्हें आपको सौंप दूंगा।

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 ? हम प्रति व्यक्ति एक परीक्षण के साथ शुरुआत करना चाहेंगे।

getNodeForCharacter कृपया। मैंने दूसरी टिप्पणी हटा दी, भ्रम के लिए खेद है!

हमारे पास अभी भी FallbackCompositionState-test.js और ReactFiberHostContext-test.js मुफ़्त है!

मैं FallbackCompisitionState लूंगा!

मैं ReactFiberHostContext-test.js को आजमा सकता हूं

अद्यतन: सभी परीक्षण अभी लिए गए हैं। इस मुद्दे की सदस्यता लें! यदि किसी के पास काम खत्म करने का समय नहीं है तो वे भविष्य में मुक्त हो सकते हैं। यदि पुन: प्रयास करने के लिए कुछ परीक्षण उपलब्ध हो जाते हैं तो हम टिप्पणी करेंगे।

परीक्षण करने वाले सभी लोगों को-धन्यवाद! मैं दोहराना चाहता हूं कि समाधान हमेशा स्पष्ट नहीं होता है, और कुछ मामलों में रिएक्ट स्रोत में कुछ बदलाव किए बिना व्यवहार का यथोचित परीक्षण करना भी संभव नहीं हो सकता है। यदि आप कभी भी अटका हुआ महसूस करते हैं, तो यहां टिप्पणी करें, और हम एक योजना का पता लगाने की कोशिश करेंगे।

@adsonpleal आपको निराश करने के लिए क्षमा करें—मैंने अभी-अभी जाँच की है, और दुर्भाग्य से हमने गलती से पहले ही ReactFiberHostContext-test तय कर लिया है। मुझे पहले इसका एहसास नहीं हुआ, लेकिन वह TODO पुराना हो चुका है। यह एकमात्र परीक्षण है जिसे मैं देख सकता हूं जो पहले से तय है।

@gaearon वाह, ReactDOMInput-test को फिर से सक्रिय करने में मुझे कुछ समय लगेगा लेकिन मुझे आशा है कि मैं इस सप्ताह के अंत में सब कुछ खत्म कर दूंगा।

मुझे एक छोटा सा संदेह है:
क्या उस सभी व्यवहार को setUntrackedValue फ़ंक्शन से बदलना ठीक है जो inputValueTracking का उपयोग inputValueTracking ReactTestUtils.Simulate.? किसी चीज़ में करता है?

वैसे आप मेरे पीआर को यहां देख सकते हैं कि मैंने inputValueTracking को हटाने के लिए क्या किया। मुझे पता है कि इनमें से कुछ परीक्षण inputValueTracking से बहुत विशिष्ट सामग्री पर भरोसा कर सकते हैं, लेकिन मैं बस सोच रहा था कि इसके बजाय ReactTestUtils का प्रभावी ढंग से उपयोग किया जा सकता है या नहीं।

inputValueTracking के लिए @jquense के पास इसे सर्वोत्तम तरीके से करने के बारे में सबसे अधिक संदर्भ होना चाहिए। लेकिन जब तक आप इसे काम करते हैं, हम समीक्षा कर सकते हैं और देख सकते हैं कि दृष्टिकोण काम करता है या नहीं।

किसी और के लिए बस एक सिर, मेरे पास यार्न का एक पुराना संस्करण स्थापित था (0.22) और स्थानीय स्तर पर परीक्षण चलाने की कोशिश कर रहा था और सब कुछ टूट रहा था। यार्न को नवीनतम (1.2.1) में अपग्रेड करने से समस्या ठीक हो गई और परीक्षण अब ठीक चल रहे हैं।

@sadpandabear मेरे पास मेरी परीक्षण फ़ाइल में एक ही मॉड्यूल है इसलिए मैं जल्द ही आपके कार्यान्वयन की जांच करने जा रहा हूं

@gaearon setInnerHTML परीक्षणों के लिए, उन्हें ऐसा लगता है कि वे परीक्षण सूट के लिए setInnerHTML के अस्तित्व पर निर्भर हैं।

मैं मौजूदा कार्यों को बदलने के लिए कुछ परीक्षण लिख सकता हूं, लेकिन फिर परीक्षण सूट आंतरिक एचटीएमएल, बनाम setInnerHTML के लिए विशिष्ट कुछ भी सेट करने की तर्ज पर अधिक है। मुझे लगता है कि मैं टेस्ट सूट के नामकरण पर थोड़ा लटका हुआ हूं।

संपादित करें: जब तक मैं गलत नहीं हूँ ... एसवीजी के पास innerHTML संपत्ति है। सुनिश्चित नहीं है कि इस फ़ाइल में परीक्षणों का दूसरा हिस्सा भी आवश्यक है।

@silvetrijonathan मुझे लगता है कि वे परीक्षण वास्तव में सत्यापित करने की कोशिश कर रहे हैं कि dangerouslySetInnerHTML रिएक्ट में अपेक्षित रूप से काम करता है। इसलिए यदि आप ReactDOM का उपयोग करने के लिए उन परीक्षणों को संशोधित करते हैं और उसी सामग्री को dangerouslySetInnerHTML के साथ प्रस्तुत करते हैं, तो यह पर्याप्त होना चाहिए। यदि आपको लगता है कि यह अधिक उपयुक्त होगा तो परीक्षण का नाम बदलकर dangerouslySetInnerHTML-test करना ठीक है।

मेरी परीक्षण फ़ाइल के साथ खेलने के कुछ घंटों के बाद मेरे पास नवागंतुकों (मेरे जैसे) के लिए एक मजबूत सुझाव है: एक स्वच्छ एनवी (फ़ाइल) के साथ अपने परीक्षण शुरू करें।
मैं मौजूदा फ़ाइल को संपादित कर रहा था और इसने कुछ आंतरिक कार्यों का मज़ाक उड़ाया था इसलिए कुछ ReactTestUtils फ़ंक्शन टूट गए थे !! मुझे यह महसूस करने में थोड़ा समय लगा कि चीजें इरादे के मुताबिक काम क्यों नहीं कर रही हैं..

@gaearon मुझे लगता है कि हमें यह स्पष्ट करने की आवश्यकता है कि ReactTestUtils , जिसमें घटनाओं का अनुकरण भी शामिल है, को एक निजी एपीआई माना जाना चाहिए। यह परोक्ष रूप से आंतरिक एपीआई तक पहुंचता है, इसलिए यह हमें ठीक से परीक्षण नहीं करने देता कि यह अन्यथा कैसे काम करेगा। उदाहरण के लिए, यह उन हुकों को उजागर किए बिना संकलित पैकेजों का परीक्षण करने की हमारी क्षमता को बाधित करेगा और यह हमें एक नई गैर-सिंथेटिक घटना प्रणाली का परीक्षण नहीं करने देगा।

@dphurley , मैंने देखा है कि आप ReactIncrementalPerf-test पर काम कर रहे हैं। मैं एक परीक्षण सूट पर भी काम कर रहा था जो ReactCoroutine का उपयोग करता है, इसलिए मैं आपको इस पीआर के बारे में जानकारी देना चाहता हूं: #11338।
मुझे यकीन नहीं है कि आपने पहले ही पीआर जमा कर दिया है, लेकिन अगर आपने नहीं किया है, तो इसे #11338 (यदि इसे स्वीकार किया जाता है) के आधार पर आधारित करना एक अच्छा विचार हो सकता है।

@gaearon आपकी सलाह के लिए धन्यवाद। मैंने परीक्षणों के लिए React-DOM के साथ dangerouslySetInnerHTML का उपयोग करने के बारे में सोचा, इसलिए मैं इसके बारे में इस तरह से जाऊंगा!

और हाँ, मैं शायद उस मामले में परीक्षणों का नाम बदल दूंगा। अगले दिन या तो मेरे पीआर की तलाश करें।

हमें पहले मर्ज किया गया पीआर मिल गया है!

इसे देखें: 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

कुछ सामान्य विषय हैं, और अन्य पीआर को सही दिशा में चलाने के लिए उन चर्चाओं का उपयोग करना मूल्यवान हो सकता है।

संदर्भ के लिए महान काफिला-धन्यवाद!

काम थोड़ा व्यस्त हो गया, लेकिन मुझे लगता है कि मैं इस सप्ताह के अंत में अपने परीक्षणों के लिए पीआर खोलने में सक्षम हो जाऊंगा।

मैंने अभी तक जो परीक्षा ली है उस पर गौर करना बाकी है। दूसरे कामों में व्यस्त हो गए। मैं कुछ और दिनों में पीआर बढ़ाने की कोशिश करूंगा, क्या मैं कुछ समय खरीद सकता हूं?

ठीक लगता है, ज़रूर!

मुझे एक ही समस्या थी

एक और पीआर विलय! https://github.com/facebook/react/pull/11316

दो और नीचे। https://github.com/facebook/react/pull/11317 https://github.com/facebook/react/pull/11385

मैंने एक परीक्षण के लिए कैसे संपर्क किया, इस बारे में एक छोटा सा विश्लेषण लिखा। https://github.com/facebook/react/pull/11385#issuecomment -341934588

जहां तक ​​​​मैं कह सकता हूं कि वे सभी ले लिए गए हैं, कृपया यहां पोस्ट करें यदि कोई मुफ्त है और मैं देखूंगा

मैंने दो और पीआर का विलय किया जो विशेष रूप से SyntheticEvent s से संबंधित हैं। यदि आपने कोई संबंधित परीक्षण किया है, तो कृपया उन्हें देखें और शैली और दृष्टिकोण में मिलान करने का प्रयास करें: https://github.com/facebook/react/pull/11365 और https://github.com/facebook/react/issues/ 11299

@gaearon EventPluginRegistry-test.js के लिए, हमें सार्वजनिक एपीआई का उपयोग करके परीक्षण को फिर से कैसे लिखना चाहिए, मैं देखता हूं कि हम ReactDom.js -> ReactDOMClientInjection.js में डिफ़ॉल्ट प्लगइन्स इंजेक्ट कर रहे हैं।
{
सिंपलइवेंटप्लगिन,
EnterLeaveEventPlugin,
चेंजइवेंट प्लगइन,
इवेंट प्लगइन का चयन करें,
पहले इनपुटइवेंट प्लगइन,
}

तो हम वास्तव में अपने परीक्षण-मजाक प्लगइन को कैसे इंजेक्ट कर सकते हैं और EventPluginRegistry कार्यक्षमता का परीक्षण कर सकते हैं
मैंने किसी भी कस्टम प्लगइन की खोज की और प्रतिक्रिया-टैप-इवेंट-प्लगइन पाया कि यह जांचने के लिए कि वास्तव में कस्टम प्लगइन्स कैसे इंजेक्ट किए जाते हैं।
क्या प्लगइन्स को इंजेक्ट करने का यही एकमात्र तरीका है

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 के विभिन्न हिस्सों पर टिप्पणी करना और यह देखना उचित है कि कोई परीक्षण विफल हो गया है या नहीं। हम इसके निजी एपीआई परीक्षणों को हटा सकते हैं यदि समान कोड पहले से ही अन्य परीक्षणों द्वारा कवर किया गया है जो उन कोड पथों पर जोर देते हैं। इसका कोई मतलब भी है क्या?

@gdevincenzi अच्छा लगता है, मैं असाइनमेंट हटा दूंगा।

इसके अलावा, मैं इस सप्ताह दूर रहने जा रहा हूं, लेकिन कृपया इसे आपको इस पर जारी रखने से न रोकें। उम्मीद है कि अगले सप्ताह कुछ और जमा किए गए पीआर का विलय हो जाएगा।

मैं ReactTreeTraversal-test.js ले सकता हूं - यह एक अपरिचित एपीआई है लेकिन इसे सीखने के लिए एक अच्छा opp लगता है। सलाह स्वागत है @gdevincenzi

एक जटिल को खत्म करने के लिए @gordyd को बधाई: https://github.com/facebook/react/pull/11383।

यह एक अच्छा उदाहरण है कि अनिवार्य रूप से एक ही चीजों का परीक्षण करते समय कोड पहले और बाद में कितना अलग दिख सकता है। और आप कैसे पता लगा सकते हैं कि संदर्भों की खोज करके, गिट इतिहास को देखकर, और विभिन्न पंक्तियों पर टिप्पणी करके कोड को क्या करना चाहिए।

हमें निम्नलिखित उपयोगकर्ताओं से अभी तक कोई पीआर प्राप्त नहीं हुआ है:

@ईमेल2विमलराज
@andrevargas
@tranotheron
@minerado
@sw-yx
@ राजा0120
@aarboleda1
@danilowoz
@dms1lva
@मोरजाबी
@dphurley
@reznord
@aqumus

क्या आप कृपया पुष्टि कर सकते हैं कि क्या आपने इस पर काम शुरू किया है? मेरे पास और भी लोग हैं जो कतार में मदद करना चाहते हैं, इसलिए यदि आपके पास समय नहीं है, तो दूसरों को इस पर काम करने का अवसर देना बहुत अच्छा होगा।

@andrevargas ने आपकी चर्चा को https://github.com/facebook/react/pull/11331 में देखा। मुझे लगता है कि उन परिवर्तनों को वहां एकीकृत रखना समझ में आता है।

कृपया मुझे उस कतार में जोड़ें @gaearon

@gaearon हाँ, तुम सही हो। बस पुष्टि करने के लिए, @jeremenichelli #11331 में quoteAttributeValueForBrowser-test.js पर काम कर रहा है, और मुझे इसकी जानकारी है। मुझे लगता है कि आप इस धागे को उसके उपयोगकर्ता नाम और पीआर के लिंक से अपडेट कर सकते हैं।

कृपया मुझे भी कतार में जोड़ें @gaearon

@gaearon मैंने इस पर काम करना शुरू कर दिया है। इस सप्ताह किसी समय या तो मेरे निष्कर्ष या पीआर पोस्ट करने की योजना बना रहे हैं।

@gaearon मैंने SyntheticKeyboardEvent.js पर काम करना शुरू कर दिया है। इस सप्ताह के अंत में पीआर करने की योजना है।

@gaearon मैं उन मामलों में भी कोई भी परीक्षण कर सकता हूं जिन्हें मूल रूप से असाइन किए गए लोगों के पास काम करने के लिए आवश्यक समय नहीं है।

@gaearon लंबे विलंब के लिए क्षमा करें, मैं अभी WIP PR बना रहा हूं। अपडेट: क्या कोई संक्षेप में बता सकता है कि निजी एपीआई पर भरोसा न करने के लिए मुझे क्या करना चाहिए? साथ ही, मुझे top<EventName> s के बारे में पता नहीं है। धन्यवाद।

https://github.com/facebook/react/blob/92b7b172cce9958b846844f0b46fd7bbd8c5140d/packages/react-dom/src/events/__tests__/SelectEventPlugin-test.js > extract समारोह

@gaearon लंबी देरी के लिए क्षमा करें । मैं आज इस पर काम करूंगा और अगर मुझे इसके बारे में कुछ संदेह है तो मैं आपको पिंग करूंगा।

@morajab इससे निपटने का एक संभावित तरीका है

• SelectEventPlugin को देखें और समझें कि यह उपयोगकर्ता को कौन सी कार्यक्षमता प्रदान करता है
• कोड लिखें जो उस कार्यक्षमता का उपयोग करता है
• SelectEventPlugin लोड करना अक्षम करें (टिप्पणी करें) और देखें कि क्या टूटता है (सुनिश्चित करें कि कुछ टूट जाता है)
• आपके द्वारा लिखा गया कोड अब आपके परीक्षण का पहला संस्करण है ताकि आप इसे एक परीक्षण मामले में रख सकें
• SelectEventPlugin फिर से लोड करना सक्षम करें और सुनिश्चित करें कि परीक्षण पास हो गया है
• अपना कोड अनुकूलित करें

@gaearon देरी के लिए माफ़ी
आपके सुझाव के आधार पर, मैंने EventPluginRegistry.js के कुछ हिस्सों पर टिप्पणी करने की कोशिश की और पाया कि बहुत से परीक्षण मामले विफल हो रहे हैं ( SyntheticWheelEvent , ReactDOMComponentTree , SyntheticClipboardEvent , SyntheticWheelEvent , inputValueTracking , SimpleEventPlugin , ChangeEventPlugin , ReactDOMComponentTree आदि),
मैं मुख्य रूप से अनुमान लगाता हूं क्योंकि ReactBrowserEventEmitter.js registrationNameDependencies $ का उपयोग करता है जिसे EventPluginRegistry.js में परिभाषित किया गया है

हालांकि टिप्पणी करते समय

• लाइन 208
• लाइन 38-41

कोई भी परीक्षण मामला विफल नहीं हुआ (केवल EventPluginRegistry परीक्षण विफल रहा)
इसके आधार पर मुझे लगता है कि EventPluginRegistry पहले से ही अन्य परीक्षण सूट द्वारा कवर किया गया है।

सुविधाजनक रूप से हमारे पास यह रेपो में है।

मुझे समझ में नहीं आया कि आपका इससे क्या मतलब है

क्या कोई अन्य सुझाव है जिस पर मैं सार्वजनिक एपीआई का उपयोग करके EventPluginRegistry को फिर से लिखने के लिए काम कर सकता हूं?

दूसरों के लिए (सुनिश्चित नहीं है कि यह पहले प्रसारित किया गया था) मुझे @kentcdodds वीडियो पॉडकास्ट मिला - बेन अल्परट द्वारा ईवेंट कोड वॉकथ्रू प्रतिक्रिया बहुत उपयोगी है।

क्षमा करें @gaearon मैंने करने की कोशिश की, लेकिन अब मेरे पास इसे खत्म करने का समय नहीं है
तो अगर कोई और मेरे मुद्दों को प्राप्त करना चाहता है, तो बेझिझक :)

@danilowoz @gaearon अगर आपको कोई आपत्ति नहीं है तो मैं पहले इनपुटइवेंटप्लगिन-टेस्ट.जेएस ले सकता हूं।

@timjacobi ReactDOMComponentTree.getInstanceFromNode के लिए प्रतिस्थापन क्या है?

साथ ही, मुझे इसे कहां अक्षम करना चाहिए?

SelectEventPlugin लोड करना अक्षम करें (इस पर टिप्पणी करें) और देखें कि क्या टूटता है (सुनिश्चित करें कि कुछ टूट जाता है)

@ मोराजाबी मुझे यकीन नहीं है कि आप इसे सही स्तर पर निपटा रहे हैं। क्या आपके पास अब तक जो कुछ है, उससे आप WIP PR खोल सकते हैं? मैं इस धागे को ज्यादा प्रदूषित नहीं करना चाहता।

@timjacobi मैं नहीं कर सकता जब मैंने कुछ नया नहीं किया है। मुझे खेद है कि मैं इन दिनों बहुत व्यस्त हूँ अगर कोई SelectEventPlugin-test ले सकता है।

ऐसा लगता है कि अब हमें ReactDOMServerIntegration-test (cc @minerado) की आवश्यकता नहीं है क्योंकि मैंने इसे अन्य परिवर्तनों के साथ कवर किया है।

@aqumus आपके विश्लेषण के आधार पर मैं इस सूची से EventPluginRegistry-test.js हटा दूंगा। मुझे लगता है कि हम इसे उन विशेष आंतरिकों के एक इकाई परीक्षण के रूप में रखेंगे, लेकिन यह जानना अच्छा है कि वास्तविक तर्क अन्य परीक्षणों द्वारा कवर किया गया है।

@ King0120 मैं getEventCharCode-test को @ aarboleda1 पर फिर से असाइन करूंगा क्योंकि वह https://github.com/facebook/react/pull/11631 में एक बहुत ही संबंधित परीक्षण पर काम कर रहा है, और उन्हें एक साथ करना आसान हो सकता है।

@morajabi @gaearon मैं SelectEventPlugin-test.js पर भी एक नज़र डाल सकता हूं।

@ email2vimalraj FYI करें, मैंने inputValueTracking-test फिर से लिखा।

इन परीक्षणों का वर्तमान में कोई स्वामी नहीं है:

• [ ] getNodeForCharacterOffset-test.js (3/5) @accordeiro . द्वारा लिया गया
• [ ] ReactBrowserEventEmitter-test.js (5/5) @madeinfree . द्वारा लिया गया
• [ ] BeforeInputEventPlugin-test.js + FallbackCompositionState-test.js (5/5, संयुक्त होना चाहिए) @GordyD द्वारा लिया गया
• [x] SelectEventPlugin-test (2/5) @skiritsis . द्वारा लिया गया
• [x] ReactTreeTraversal-test.js (4/5) @timjacobi . द्वारा लिया गया

मैंने उनकी कठिनाई के बारे में अपनी धारणा के आधार पर उन्हें छाँटा।

हमारे पास @timjacobi @kwnccc @skiritsis के साथ एक कतार है, तो क्या आप तीनों हमें बता सकते हैं कि क्या आप इनमें से कोई भी परीक्षण करना चाहते हैं (कृपया निर्दिष्ट करें कि कौन सा है)। आप भी पास हो सकते हैं, और हम किसी और को कोशिश करने देंगे।

ध्यान दें कि ये काफी जटिल होते जा रहे हैं, इसलिए ये बिल्कुल शुरुआती के अनुकूल नहीं हैं। वे यांत्रिक प्रतिस्थापन नहीं हैं; आपको यह समझने की आवश्यकता होगी कि परीक्षण क्या सत्यापित करने का प्रयास कर रहा है और फिर एक समान सार्वजनिक एपीआई परीक्षण (जो बहुत अलग दिख सकता है) के साथ आ सकता है।

इन्हें लेने के लिए, आपको अपने आप चीजों पर शोध करने में सहज होना चाहिए, जैसा कि @GordyD ने https://github.com/facebook/react/pull/11383 में किया था। वास्तव में अगर @GordyD इनमें से कोई भी लेना चाहता है तो मुझे उसे देने में खुशी होगी!

@skiritsis चूंकि आप रुचि रखते थे, इसलिए मैं आपको SelectEventPlugin-test फिर से सौंप दूंगा।

मैं ReactBrowerEventEmitter-test.js पर काम करना चाहता हूं

@gaearon : इसके लिए धन्यवाद, समय पर प्रतिक्रिया न देने के लिए खेद है। दूसरे काम में व्यस्त हो गए। मैं देखूंगा कि आपने मेरी समझ के लिए फिर से कैसे लिखा।

मैं ReactTreeTraversal-test.internal.js . पर एक नज़र डाल सकता हूँ

@madeinfree पहले उत्तर देने के लिए @kwnccc को कुछ दिन दें क्योंकि वह पहले से ही कतार में है। अगर उसे कोई दिलचस्पी नहीं है, तो मैं इसे बाद में आपको सौंप दूंगा।

@gaearon ठीक है! समझ गया, धन्यवाद ~

@gaearon मैं BeforeInputEventPlugin + FallbackCompositionState के संयोजन पर एक नज़र डालूंगा यदि कोई अन्य लेने वाला नहीं है।

अच्छा लगता है, धन्यवाद @GordyD!

वैसे, यहां कुछ और ईवेंट परीक्षण हैं जो परिवर्तित हो गए हैं, जिससे आपको यह पता चल सके कि यह कोड आमतौर पर कैसा होता है: https://github.com/facebook/react/pull/11631 https://github. कॉम/फेसबुक/प्रतिक्रिया/पुल/11525

@gaearon मुझे ReactBrowserEventEmitter-test.js लेने में भी खुशी हो रही है अगर कोई और इसे नहीं चाहता है।

@madeinfree

क्या आप अभी भी ReactBrowerEventEmitter में रुचि रखते हैं? यदि हां, तो कृपया इसे लें और हमें अपडेट रखें!

@gaearon ठीक है! मैं इसे लेना चाहूंगा।

आपको यह मिला।

कोई भी शेष 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 (...) से एक सार्वजनिक API जैसे React-dom के साथ React-dom पर स्विच करने की आवश्यकता है। .क्रिएटपोर्टल (...) ? @gaearon

@gaearon मैं वास्तव में ReactIncrementalPerf-test.js आज़माना चाहता हूं। चूँकि @ abiduzz420 ने इसकी जाँच शुरू कर दी है, हम ज़रूरत पड़ने पर टीम भी बना सकते हैं।

हाँ ज़रूर @kwnccc मुझे आपके साथ काम करके खुशी हो रही है। तुम मेरे सामने कतार में थे, तो सब तुम्हारा!

@kwnccc जब मैं yarn flow चलाता हूं तो मुझे yarn test में एक परीक्षण त्रुटि और प्रवाह त्रुटि मिल रही है। मैं अपने फोर्कड रेपो पर कोड को आगे बढ़ाऊंगा: 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 प्रवाह समस्या विंडोज़ पर एक ज्ञात समस्या है। यह असंबंधित है (लेकिन हमें यह पता लगाना होगा कि ऐसा क्यों होता है)। https://github.com/facebook/react/issues/11703 देखें।

@kwnccc ReactIncrementalPerf-test के लिए, हाँ इसके लिए कुछ विचार की आवश्यकता होगी। मुझे लगता है कि शायद इस समय अस्थायी रूप से ReactPortal को shared में स्थानांतरित करना सबसे अच्छा है। फिर ReactNoop अपनी createPortal() विधि प्रदान करें (जैसे ReactDOM करता है)।

एक और नीचे! https://github.com/facebook/react/issues/11299

@gaearon मैंने रिएक्टपोर्टल को साझा निर्देशिका में स्थानांतरित कर दिया है, और सभी परीक्षण पास हो गए हैं। $# 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() परीक्षण फ़ाइल को छोड़कर कहीं और उपयोग नहीं किया जाता है। कोई विचार?

@abiduzz420 हाँ यह सही लगता है

@अनुश्रीसुब्रमणि

क्या मैंने जो तरीका अपनाया है वह सही है?

सामान्य तौर पर, हाँ, लेकिन हम वहाँ दोहराव से बचना चाहेंगे। आदर्श रूप से इसे की एक श्रृंखला से बदलना चाहिए

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() परीक्षण फ़ाइल को छोड़कर कहीं और उपयोग नहीं किया जाता है

दोष को देखते हुए, इसे 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 मैंने createPortal() के लिए ReactNoop.js #$ में विधि बनाई है और इससे पहले मैंने आपके सुझाव के अनुसार ReactPortal.js फ़ाइल को साझा निर्देशिका में स्थानांतरित कर दिया था। कृपया मेरे कोड का अध्ययन करें, और यदि मुझे कोई संशोधन या अतिरिक्त परिवर्धन करने की आवश्यकता है, तो मैं इसे करूंगा।

क्या आप कृपया एक पीआर भेज सकते हैं? समीक्षा में चर्चा करना आसान है।

@ abiduzz420 ऐसा लगता है कि आपने बहुत अच्छा काम किया है, क्षमा करें, लेकिन मेरे पास आज से पहले उस पर काम करना शुरू करने का समय नहीं था! वास्तव में खुशी है कि आप परीक्षण पूरा करने में सक्षम थे! मैं
@gaearon आगे मैं यह सुनिश्चित करूँगा कि मेरे पास एक बड़ी उपलब्धता होगी

@gaearon त्वरित अपडेट: मैंने BeforeInputEventPlugin + FallbackCompositionState परीक्षणों को फिर से तैयार करना/संयोजन करना शुरू कर दिया है। सार्वजनिक एपीआई के माध्यम से कार्यान्वयन का परीक्षण करने के तरीके पर मुझे एक अच्छा संभाल मिला है। अब मैं अलग-अलग निष्पादन वातावरण/ब्राउज़र इंजनों पर निर्भर विभिन्न कोड पथों का प्रयोग करने के लिए परीक्षण मामलों को बनाने के माध्यम से काम कर रहा हूं। मुझे इस सप्ताह के अंत में पहला पीआर होने की उम्मीद है (10 दिसंबर तक - अद्यतन मुझे थोड़ा और समय चाहिए - 12 दिसंबर को देखते हुए)।

बढ़िया, अपडेट के लिए धन्यवाद!

@gaearon मेरे अंत में त्वरित अपडेट: मैंने पहले ही getNodeForCharacterOffset-test के लिए परीक्षणों को फिर से लिखने पर काम करना शुरू कर दिया है और गुरुवार तक पीआर को आगे बढ़ाने में सक्षम होना चाहिए - क्या यह ठीक है?

सुनने में तो अच्छा लगता है

एक और नीचे! https://github.com/facebook/react/pull/11742

@reznord हमने अभी तक आपका कोई जवाब नहीं सुना है—क्या आपने कुछ शुरू किया? अगर आप बहुत व्यस्त हैं तो बेहतर होगा कि आप किसी और को कोशिश करने का मौका दें।

पिंग @reznord

नमस्ते @gaearon , क्या मैं अपने पहले योगदान के लिए ReactErrorUtils-test.js ले सकता हूँ?

नमस्ते, @gaearon कुछ ऐसा है जो मैं ReactErrorUtils-test.js . में कर सकता हूँ

नमस्ते, क्या कोई मुझे ReactBrowserEventEmitter-test.js परीक्षण जारी रखने में मदद कर सकता है? क्योंकि मैं अपने काम में व्यस्त हूँ अब चलते रहने का समय नहीं है, पीआर है https://github.com/facebook/react/pull/11713 , बहुत-बहुत धन्यवाद !!

फिर से सभी को धन्यवाद! अभी भी कुछ उत्कृष्ट परीक्षण हैं लेकिन मुझे लगता है कि हम इसे बंद कर सकते हैं।

@gaearon : ReactErrorUtils-test.js मैं इस पर काम करना चाहूंगा। क्या आप पुष्टि कर सकते हैं कि क्या मैं इसे उठा सकता हूं?