Да, това е грешка! Първоначално се чудех дали трябва да свържа по някакъв начин потребителя с продажбите, за да може тази стойност автоматично да се изчислява, когато е необходимо. Но в последствие реших, че не искам да си усложнявам живот и съм забравил да го подам като параметър. И понеже го направих като library не съм го тествал. Благодаря за откритието!

Нямам VisualStudio и гледам направо в GitHub но мисля, че проблемът е напълно нормален :)

    return String.Format("Student: {0} {1} has faculty number: {2}", this, FirstName, this.LastName, this.FacultyNumber);

Като първи параметър ( {0} ) подаваш "this" което е инстанция от тип Student. Всеки път когато извикаш обект от тип Student като стрингова операция ще се извика ToString(). Лошото обаче е, че цитираният код е В метода ToString() т.е. стигайки до "this" се вика отново ToString(), който стига до this вика отново себе си и така до безкрай.

Залагам, че си искал да напишеш this.FirstName вместо да ги разделиш със запетая.

Ееее, евала за наблюдателността! Аз писах 3-4 часа и като стигнах до тая грешка и тотално изключих и излязох на въздух..  Доста ми помогна, точно щях да почвам пак да се чудя кво съм оплескал! Благодаря!