From 26e3f40759836391f01da9594ed23e677f5b6963 Mon Sep 17 00:00:00 2001 From: Sebastian Date: Sun, 27 Nov 2016 01:01:32 +0100 Subject: [PATCH] 1) Test coverage fixed 2) Test fixed --- tests/RunCoverage.bat | 1 - .../Squidex.Write.Tests/Apps/AppDomainObjectTests.cs | 12 ++++++++++++ 2 files changed, 12 insertions(+), 1 deletion(-) diff --git a/tests/RunCoverage.bat b/tests/RunCoverage.bat index 49e385228..fa5116fe5 100644 --- a/tests/RunCoverage.bat +++ b/tests/RunCoverage.bat @@ -33,7 +33,6 @@ exit /b %errorlevel% -skipautoprops ^ -output:"%~dp0\GeneratedReports\Core.xml" ^ -oldStyle -exit /b %errorlevel% "%UserProfile%\.nuget\packages\OpenCover\4.6.519\tools\OpenCover.Console.exe" ^ -register:user ^ diff --git a/tests/Squidex.Write.Tests/Apps/AppDomainObjectTests.cs b/tests/Squidex.Write.Tests/Apps/AppDomainObjectTests.cs index ff1b01860..a49ecba30 100644 --- a/tests/Squidex.Write.Tests/Apps/AppDomainObjectTests.cs +++ b/tests/Squidex.Write.Tests/Apps/AppDomainObjectTests.cs @@ -75,6 +75,12 @@ namespace Squidex.Write.Tests.Apps Assert.Throws(() => sut.AssignContributor(new AssignContributor { ContributorId = contributorId })); } + [Fact] + public void AssignContributor_should_throw_if_contributor_id_not_valid() + { + Assert.Throws(() => sut.AssignContributor(new AssignContributor())); + } + [Fact] public void AssignContributor_should_throw_if_single_owner_becomes_non_owner() { @@ -106,6 +112,12 @@ namespace Squidex.Write.Tests.Apps Assert.Throws(() => sut.RemoveContributor(new RemoveContributor { ContributorId = contributorId })); } + [Fact] + public void RemoveContributor_should_throw_if_contributor_id_not_valid() + { + Assert.Throws(() => sut.RemoveContributor(new RemoveContributor())); + } + [Fact] public void RemoveContributor_should_throw_if_all_owners_removed() {