Endnu et test formulerings problem

Du mangler at gøre noget ved SugarContainer for at en test skal give nogen mening (f.eks. via en metode på et objekt, hvor SugarContainer er en egenskab af type int).

Typisk vil målet for din test være denne funktion, hvor du efter kaldet til funktionen undersøger om SugarContainer-egenskaben på objektet har den forventede værdi.

Eksempelvis:


Din testcase kunne da se således ud:


Jeg er klar over, at det sidste scenarium ikke er optimalt kodet. Der findes nok en smartere måde i dit testframework (givetvis noget med Assert.Throws). Jeg anvender normalt MSTest og kender derfor ikke metoden i dit testframework... beklager min uvidenhed.

Det meste af det virker, men jeg kan dog ikke få parameterne til at virke, der fremkommer fejl når disse er implementeret.

Koden ser nu således ud:

        [Test]
        public void SugarCountDownTest()
        {
            var target = new SugarContainer(10);

            target.HasContent();

            Assert.AreEqual(9, target.Level);
        }
       
        //The RefillContainerTest shows if the suger container throws                          an exception when the container is empty!
        [Test]
        public void RefillContainerTest()
        {
            var target = new SugarContainer(10);

            try
            {
                target.HasContent();
                Assert.Fail();
            }
            catch (Exception x)
            {
                Assert.AreEqual("Not enough sugar", x.Message);
            }
        }

Jeg bruger Nunit test 2.6.4.

Og SugarContainer klassen ser således ud:

        public class SugarContainer : Container
    {
        /// <summary>
        /// Initiates container
        /// </summary>
        /// <param name="level">Init level</param>
        public SugarContainer(int level)
        {
            Level = level;
        }

        /// <summary>
        /// Get content from container
        /// </summary>
        public override void GetContent()
        {
            if (HasContent())
            {
                Level -= 1;
            }
            else
            {
                throw new EmptyContainerException("Not enough sugar.");
            }
        }

Og container klassen ser således ud:

public abstract class Container
    {
        // Minimum threshold value indicating when a container is empty
        protected const int MIN_CONTAINER_THRESHOLD = 1;

        // Current container level
        protected int containerLevel;       

        /// <summary>
        /// Container level (get/set)
        /// </summary>
        public int Level
        {
            get
            {
                return containerLevel;
            }
            set
            {
                containerLevel = value;
            }
        }

        /// <summary>
        /// Returns whether the chosen container has content
        /// </summary>
        /// <returns>True if there is content, otherwise false</returns>
        public bool HasContent()
        {
            return Level > MIN_CONTAINER_THRESHOLD;
        }

        /// <summary>
        /// Get content from container (no implementation)
        /// </summary>
        public abstract void GetContent();
    }


Håber du stadig har mod på det :-)

Du ændrer jo ikke værdien af containerens level, så testene skal jo fejle...

Du spørger, i dine tests, blot på, om containeren indeholder noget.

Du bør, i dine tests, nok kalde GetContent i stedet for HasContent.

Desuden ville jeg gøre Level (set) protected i stedet for public, så det kun er klassen eller dens nedarvede klasser, der kan ændret denne egenskab (som jeg også indikerede i mit eksempel). Så har du i det mindste sikret dig, at Level ikke ændrer sig udenfor klassehierarkiet.

P.S. Dine kommentarer virker en smuler overflødige. Kommentarer er fine nok, hvis de tilfører koden nogen værdi ude over det koden tydeligt afspejler... :-)

Du kan i øvrigt gøre din test af exception-tilfældet noget mere overskuelig ved at bruge ExpectedExceptionAttribute.


Bemærk, at jeg har initieret containeren med nul, så kaldet til GetContent bør rejse en kørselsfejl.

Læs mere om brugen af denne atribut på http://www.nunit.org/index.php?p=exception&r=2.4.8

Det kunne jeg ikke lige umiddelbart få til at virke.

Hvad har du gjort?

Læste du min kommentar i #3 (med kald af GetContent i stedet for HasContent)... eller læste du kun den sidste kommentar #4?

Jeg læste dem begge. Jeg tror mit problem er noget med "reference" til de forskellige attributer, metoder og klasser...

Kan du evt. vise den tilrettede kode og evt. forklare, hvad det er der ikke fungerer...?

Snarest muligt :-)

Lukker tråden.

Jamen, velbekomme da!