Ти трябва да тестваш само метода , а не  цялата програма според мен :)

Според мен трябва да се провери дали обекта е добавен в базата данни, защото това е основната задача на тествания метод. Иначе може да изкараш съобщение, че обектът е добавен, тестът да излизе passed (ако пропуснеш предпоставките за exception-и), a пък обектът реално да не е добавен в базата данни.

При другите решения на изпити, които съм гледал, така беше направено. На подготовката за изпита Наско също така процедираше.

Ами и аз имам такива съмнения, но защо тогава в авторското се тестват върнатите съобщения ?!  Колебая се кое е по - правилно в случая ? :)

И при мене има оценка вече и ей тоя коментар:

Повечето неща са останали с конкретики вместо абстракции, т.е. Няма интерфейси зад тях и се приемат директно като имплементации по методите.StyleCop warning-ите не са оправени. Бъговете не са оправени. Липсва документация. Липсват по-интересни case-ове за някои тестове.Колекциите в базата не са с речници, което е в пъти по-бързо спрямо листовете. Липсва мокинг.Бонус задачата не е изпълнена.

Тоя човек май скоро не е ебал, а?.... Ебахти хейтъра... 

Извинявай, ако е удобно да попитам какви точки са ти дали ....?Аз от коментара разбирам, че си зипнал задачата и си я пуснал :) така, както е скелета.

Оправи ми деня пичаго. И аз честно казано не съм напълно съгласен с рецензията имайки на предвид предни авторски решения, които нали уж трябва да са за пример, а в случая когато правиш нещата по - подобен начин ти се вземат точки и ти разказват играта....

Ако си ползвал dependency injection на други места, не се брои. Единственият начин да покажеш, че знаеш какво е това dependency injection е да го използваш в конкретно поставената задача. 

По принцип е хубаво да имаш на всякъде, където зависиш от "нещо" и това "нещо" може да приема различна конкретика напред във времето. Както често давания пример е с базата. Хубаво е когато очакваш база в някой твой клас, да очакваш база която имплементира интерфейс "IDatabase", например. Така си гарантираш, че дори да ти се подаде друга база данни ако имплементира този интерфейс програмата ти ще знае как да я обработва без да ти  чупи кода и да ти се налага да пипаш по него.

Еми точно това съм направил по принцип. Но в коментара ми пише, че нямам dependency injection, и се зачудих дали не е трябвало навсякъде да се направи.

        public Controller(IUserInterface userInterface, ICommand command)
        {
            this.UserInterface = userInterface;
            this.Command = command;
        }

        public IUserInterface UserInterface { get; private set; }

        public ICommand Command { get; private set; }
 

Здравей,
Аз нямам нито точки, нито рецензия- предполагам, че всичко е въпрос на време! :)

Поздрави!

И аз имам рецензия, без точки. Мисля че в момента се зареждат данните.

При мен са веднага след датата на изпита, аз лично на ооп ги видях от 5тия опит, да не случая да е такъв?