Autofixture: अनुरोध: IFixture में क्लोन विधि जोड़ें

को निर्मित 24 सित॰ 2020 · 9टिप्पणियाँ · स्रोत: AutoFixture/AutoFixture

परिचय

मैं चाहता हूं कि IFixture में Clone विधि या स्थिरता की एक प्रति बनाने के समान कुछ हो।

फिलहाल मैं ऐसा करने के लिए एक विस्तार विधि बना सकता हूं:

public static IFixture Clone( this IFixture fixture)
{
   var cloneFixture = new Fixture();

   cloneFixture.Behaviors.Clear();
   foreach ( var behavior in fixture.Behaviors)
   {
      cloneFixture.Behaviors.Add( behavior );
   }

   cloneFixture.Customizations.Clear();
   foreach ( var customization in fixture.Customizations )
   {
      cloneFixture.Customizations.Add( customization );
   }

   cloneFixture.OmitAutoProperties = fixture.OmitAutoProperties;
   cloneFixture.RepeatCount = fixture.RepeatCount;

   cloneFixture.ResidueCollectors.Clear();
   foreach ( var residueCollector in fixture.ResidueCollectors )
   {
      cloneFixture.ResidueCollectors.Add( residueCollector );
   }

   return cloneFixture;
}

लेकिन मुझे लगता है कि कॉपी बनाने की ऐसी क्षमता प्रोजेक्ट में ही होनी चाहिए।

विवरण

मेरा परिदृश्य यह है कि मैं एक परीक्षण में बहुत से मापदंडों के साथ एक वर्ग के कुछ उदाहरणों का निर्माण करता हूं। इसके अलावा मैं चाहता हूं कि एक कंस्ट्रक्टर पैरामीटर एक इंस्टेंस के लिए एक निश्चित मान हो और एक अलग इंस्टेंस के लिए एक अलग कंस्ट्रक्टर वैल्यू हो। मैं इसे कई तरीकों से कर सकता हूं; जिस तरह से मैं इसे कर रहा हूं वह है:

fixture.Customizations.Add( new FilteringSpecimenBuilder(
            new FixedBuilder( mediaId ),
            new ParameterSpecification( 
                typeof( ObjectId ), 
                "mediaId" ) ) );
var mediaVM = fixture.Build<MediaViewModel>()   
                     .With( mvm => mvm.ParentMixerId, mixerId )
                     .With( mvm => mvm.TrackId, trackId )
                     .Create();

_ = fixture.Customizations.Remove( fixture.Customizations.Last() );

//...

अनुकूलन को हटाने का कारण यह है कि मैंने यह सोचे बिना कोशिश की कि शायद जोड़े गए अंतिम अनुकूलन की उच्च प्राथमिकता होगी और इसका उपयोग किया जाएगा; लेकिन ऐसा नहीं था।

अगर मैं इसे किसी प्रकार की विस्तार विधि के साथ सरल बनाने की कोशिश करता हूं जैसे:

public static IFixture AddCustomization<T>( this IFixture fixture, T value, string name )
{
   fixture.Customizations.Add( new FilteringSpecimenBuilder(
         new FixedBuilder( value ),
         new ParameterSpecification(
            typeof( T ),
            name ) ) );
   return fixture;
}

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

मैं अतिरिक्त अनुकूलन को हटाने के लिए एक पंक्ति नहीं रखना पसंद करूंगा। तो अगर इसके बजाय मेरी विस्तार विधि ने मेरी स्थिरता की एक प्रति बनाई है जिसमें मैं प्रतिलिपि में संशोधन जोड़ता हूं तो मेरी रचनाएं अपेक्षित रूप से काम करती हैं और मूल स्थिरता छूटी हुई है।

एक अनुकूलन जोड़ने की क्षमता भी काम करेगी जो ऑब्जेक्ट बनने के बाद स्वचालित रूप से हटा दी जाती है।

आशा है कि यह सब समझ में आता है।

मेरी समस्या पर विचार करने के लिए धन्यवाद: स्माइली:

feature request

स्रोत

ajorians

सबसे उपयोगी टिप्पणी

@ajorians मैं निष्क्रियता के कारण इस मुद्दे को बंद करने वाला था लेकिन मैं उस विशाल निर्माता द्वारा नहीं चल सका।
मुझे पूछना है, क्या आपने अपना डिज़ाइन बदलने पर विचार किया है?

आपके कंस्ट्रक्टर को देखकर मैं आपकी समस्या के कई मुद्दों और संभावित समाधानों को देख सकता हूँ।

सबसे पहले आप नई और इंजेक्शन योग्य संस्थाओं की अवधारणाओं को मिला रहे हैं।
सबसे सरल उपाय यह है कि अपनी इंजेक्शन योग्य निर्भरता (उर्फ सर्विस इंटरफेस) को गुणों में स्थानांतरित करें और संपत्ति इंजेक्शन का उपयोग करके उन्हें इंजेक्ट करें।
यह Fixture प्रतिलिपि बनाने की आपकी तत्काल समस्या का समाधान करेगा और कंस्ट्रक्टर को अधिक प्रबंधनीय बना देगा।

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

चूंकि कक्षा को ViewModel कहा जाता है, मुझे लगता है कि आपके पास एक एमवीवीएम एप्लीकेशन है। यह संभावना है कि आप मैसेजिंग मॉडल पेश करके अपने एप्लिकेशन को और भी अलग कर सकते हैं। शायद एक इवेंट एग्रीगेटर। अधिकांश एमवीवीएम ढांचे ने उन्हें बनाया है ताकि आपको उनसे लाभ उठाने के लिए बहुत सारे सेटअप कार्य करने की आवश्यकता न हो।

मुझे बताएं क्या इससे मदद मिलती है।

aivascu 8 दिस॰ 2020

😄2

सभी 9 टिप्पणियाँ

@ajorians शुभ दिन! मैंने आपके उदाहरण को कैसे समझा, ऐसा लगता है कि हमारे पास पहले से ही वह है जो आपको चाहिए। स्थिरता की Build() विधि वास्तव में आंतरिक ग्राफ़ की एक अपरिवर्तनीय प्रतिलिपि बना रही है, इसलिए आप "वन टाइम" अनुकूलन को बिना बनाए रखे AutoFixture के शीर्ष पर लागू कर सकते हैं।

इस खेल के मैदान के डेमो की समीक्षा करें:

```सी#
पब्लिक क्लास मॉडल
{
सार्वजनिक इंट आईडी {प्राप्त करें; सेट; }
सार्वजनिक स्ट्रिंग नाम {प्राप्त करें; सेट; }
}

[तथ्य]
सार्वजनिक शून्य डेमो ()
{
वर स्थिरता = नई स्थिरता ();
स्थिरता। अनुकूलित करें(सी => सी.विथ (एम => एम.आईडी, 42));

var value1 = fixture.Create<Model>();
Assert.Equal(42, value1.Id);
Assert.NotNull(value1.Name);

var value2 = fixture.Build<Model>()
    .With(m => m.Id, (int genInt) => genInt * 2 /* make it always even */)
    .Without(x => x.Name)
    .Create();
Assert.NotEqual(42, value2.Id);
Assert.Equal(value2.Id % 2, 0);
Assert.Null(value2.Name);

var value3 = fixture.Create<Model>();
Assert.Equal(42, value3.Id);
Assert.NotNull(value3.Name);

}
```

हमें बताएं कि क्या यह आपकी आवश्यकताओं को हल करता है। यदि नहीं, तो कृपया अधिक विशेष रूप से वर्णन करें कि यह आपके लिए काम क्यों नहीं करता है, इसलिए हम मदद करने का प्रयास कर सकते हैं।

धन्यवाद!

zvirja 24 सित॰ 2020

हाय @zvirja ,

शुभ दिवस!

अगर मैं कर सकूं; हमारे Model वर्ग में set होगा और इसे केवल कंस्ट्रक्टर के माध्यम से व्यवस्थित किया जाएगा। तो यदि आप इसे इसके साथ प्रतिस्थापित करना चाहते थे:

public class Model
{
   public Model( int id, string name )
   {
      Id = id;
      Name = name;
   }

   public int Id { get; }
   public string Name { get; }
}

मूल रूप से निर्दिष्ट नहीं करने के लिए खेद है।

इसलिए जैसा कि मैं चीजों को समझता हूं, मुझे कंस्ट्रक्टर मापदंडों को करने के लिए अनुकूलन जोड़ना होगा। और ये अनुकूलन Build() ।

अगर कुछ कारणों में दिलचस्पी है तो हम कंस्ट्रक्टर में बहुत कुछ करते हैं और बिना सेटर्स के इसलिए हम readonly कीवर्ड का उपयोग कर सकते हैं और सुनिश्चित कर सकते हैं कि सदस्य चर सेट हैं और एक विधि के समय तक अनसेट नहीं किया जा सकता है बुलाया।

आशा है कि यह बेहतर समझ में आता है। इसमें मदद करने के लिए धन्यवाद! :स्माइली:

ajorians 24 सित॰ 2020

@ajorians क्या मैं आपकी समस्या को सही ढंग से समझता हूं, कि आपके पास बड़ी मात्रा में इनपुट तर्क वाला मॉडल है, इसलिए आप इसके बारे में स्पष्ट रूप से चिंता किए बिना, इसमें से केवल एक को अनुकूलित करना चाहते हैं। सही?

अन्यथा, आप हमेशा कुछ ऐसा लिख सकते हैं
c# var value2 = fixture.Build<Model>() .FromFactory((int id, string name) => new Model(id, name)) .Create();

दुर्भाग्य से, यह मापदंडों की बढ़ती संख्या के साथ बहुत अच्छा पैमाना नहीं है

zvirja 26 सित॰ 2020

@ajorians क्या मैं आपकी समस्या को सही ढंग से समझता हूं, कि आपके पास बड़ी मात्रा में इनपुट तर्क वाला मॉडल है, इसलिए आप इसके बारे में स्पष्ट रूप से चिंता किए बिना, इसमें से केवल एक को अनुकूलित करना चाहते हैं। सही?

हाँ, यह बिल्कुल सही है।

दुर्भाग्य से, यह मापदंडों की बढ़ती संख्या के साथ बहुत अच्छा पैमाना नहीं है

सौभाग्य से Freeze मापदंडों की बढ़ती संख्या के साथ हमारे कई वर्गों के लिए काम करता है। लेकिन हाँ, जिस वर्ग को मैं ध्यान में रखता था, जहाँ मैं कंस्ट्रक्टर मापदंडों को करने के लिए अनुकूलन का उपयोग कर रहा हूँ, वर्तमान में 30 पैरामीटर हैं:

और संभवत: इस वर्ष के अंत से पहले एक अतिरिक्त 4 या इससे अधिक की वृद्धि होगी।

मुझे लगता है कि यह असामान्य हो सकता है।

लेकिन हां मेरे पास बड़ी मात्रा में बढ़ते इनपुट कंस्ट्रक्टर तर्क हैं और मैं उनमें से बाकी के बारे में सोचे बिना उनमें से सिर्फ एक (या कुछ) को अनुकूलित करना चाहूंगा। पूरे परीक्षण के लिए उस प्रकार को पंजीकृत या फ्रीज किए बिना। और एक संक्षिप्त तरीके से अनुकूलन जोड़ने के रूप में कोड की लगभग 5 पंक्तियाँ हैं।

मैं सौभाग्य से वह सब करने में सक्षम हूँ; लेकिन जिस तरह से मैंने इसे अपने क्लाइंट-साइड कोड में clone एक्सटेंशन विधि बनाने में शामिल किया था।

आशा है कि यह सब समझ में आता है। इसे मेरे साथ देखने के लिए धन्यवाद: स्माइली:

ajorians 28 सित॰ 2020

मुझे बताएं क्या इससे मदद मिलती है।

aivascu 8 दिस॰ 2020

😄2

हाय @aivascu ,

@ajorians मैं निष्क्रियता के कारण इस मुद्दे को बंद करने वाला था

अगर कुछ है तो मैं बस मुझे बता सकता हूं। एक फीचर अनुरोध होने के नाते मैं इसे लागू होने तक देख सकता हूं, इसमें काफी मात्रा में निष्क्रियता हो सकती है।

मुझे पूछना है, क्या आपने अपना डिज़ाइन बदलने पर विचार किया है?

फिलहाल हम प्रिज्म एप्लीकेशन बनने पर विचार कर रहे हैं। इससे आपके द्वारा उल्लिखित कुछ वस्तुओं में मदद मिलेगी। लेकिन दुख की बात है कि मैं हमें कंस्ट्रक्टर में सब कुछ डालने के दर्शन को बदलते हुए नहीं देखता, जैसे कि इसे null एक बार चेक किया जा सके और readonly और हमेशा गैर- null कक्षा के जीवनकाल के माध्यम से। हमें कुछ करने की जरूरत है। लेकिन मुझे नहीं लगता कि अगले एक साल में इसमें सुधार होने वाला है।

मुझे बताएं क्या इससे मदद मिलती है।

यह सब मदद करता है। मैं टीम को बताऊंगा कि दुनिया के लोग भी सोचते हैं कि हमें बदलाव करने पर विचार करने की जरूरत है।

लेकिन अगर मैं पूछ सकता हूं कि IFixture लिए Clone या इसी तरह की विधि होना उचित है? या किसी तरह लोगों को एक अनुकूलन जोड़ने की अनुमति दें और फिर pop अंतिम जोड़ा अनुकूलन बंद करें या किसी अनुकूलन को जोड़ने से पहले किसी तरह राज्य में वापस आएं? मैं सहमत हूं कि इस वांछित व्यवहार को प्रेरित करने वाले परिदृश्य को समझना महत्वपूर्ण है और निश्चित रूप से परिदृश्य में पहली जगह में कई समस्याएं हैं। और मुझे लगता है कि जब तक कोई अच्छा उपयोग मामला न हो, तब तक आप अतिरिक्त जटिलता नहीं जोड़ना चाहते हैं। ठीक है आप चुन सकते हैं कि आगे क्या होता है लेकिन अगर मैं मदद कर सकता हूं तो मुझे बताएं।

पढ़ने और विचार करने के लिए धन्यवाद :स्माइली:

ajorians 8 दिस॰ 2020

@ajorians मेरी राय यह है कि हमें .Clone() विधि को Fixture वर्ग में नहीं जोड़ना चाहिए। जैसा कि मैं देख रहा हूं कि किसी एक परीक्षण के लिए किसी भी समय केवल एक परीक्षण स्थिरता होनी चाहिए।

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

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

public class CountingRelay : ISpecimenBuilder
{
    public CountingRelay(ISpecimenBuilder builder, int maxCount)
    {
        this.MaxCount = maxCount;
        this.Builder = builder;
    }

    public int Count { get; private set; }
    public int MaxCount { get; }
    public ISpecimenBuilder Builder { get; }

    public object Create(object request, ISpecimenContext context)
    {
        if (this.Count == this.MaxCount) return new NoSpecimen();
        var result = this.Builder.Create(request, context);
        if (!(result is NoSpecimen)) this.Count++;
        return result;
    }
}

aivascu 9 दिस॰ 2020

हाय @aivascu ,

@ajorians मेरी राय यह है कि हमें .Clone() विधि को Fixture वर्ग में नहीं जोड़ना चाहिए।

ठीक है। खैर विचार करने के लिए धन्यवाद।

आप क्या कर सकते हैं एक रिले को लागू करना जो सफल प्रस्तावों की एक निश्चित गणना के बाद अनुरोधों को अनदेखा करता है

मैं एक नजर मार लूगां।

एक बार फिर धन्यवाद! :स्माइली:

ajorians 9 दिस॰ 2020

👍1

@ajorians मैंने स्पष्ट और स्वचालित अनुकूलन रीसेट के लिए एक अधिक औपचारिक सुविधा अनुरोध #1214 बनाया है। आप वहां इस सुविधा की प्रगति को ट्रैक कर सकते हैं।

aivascu 19 दिस॰ 2020

क्या यह पृष्ठ उपयोगी था?

0 / 5 - 0 रेटिंग्स

Autofixture: अनुरोध: IFixture में क्लोन विधि जोड़ें

परिचय

विवरण

सबसे उपयोगी टिप्पणी

सभी 9 टिप्पणियाँ

संबंधित मुद्दों