remove unnecessary todos
This commit is contained in:
parent
91805bc9ad
commit
a97b95a704
@ -264,8 +264,6 @@ public class CitiesTests : TestBase
|
|||||||
TestContext.Current.CancellationToken));
|
TestContext.Current.CancellationToken));
|
||||||
}
|
}
|
||||||
|
|
||||||
// TODO: Add more tests with user role (copy tests with admin role)
|
|
||||||
|
|
||||||
[Fact]
|
[Fact]
|
||||||
public async Task AddCity_UnAuthnticatedUser_ThrowsForbiddenException()
|
public async Task AddCity_UnAuthnticatedUser_ThrowsForbiddenException()
|
||||||
{
|
{
|
||||||
@ -279,8 +277,6 @@ public class CitiesTests : TestBase
|
|||||||
TestContext.Current.CancellationToken));
|
TestContext.Current.CancellationToken));
|
||||||
}
|
}
|
||||||
|
|
||||||
// TODO: Add more tests with unauthenticated user
|
|
||||||
// (copy tests with admin role)
|
|
||||||
|
|
||||||
[Fact]
|
[Fact]
|
||||||
public async Task UpdateCity_WithAdminRole_CityUpdated()
|
public async Task UpdateCity_WithAdminRole_CityUpdated()
|
||||||
@ -480,8 +476,6 @@ public class CitiesTests : TestBase
|
|||||||
TestContext.Current.CancellationToken));
|
TestContext.Current.CancellationToken));
|
||||||
}
|
}
|
||||||
|
|
||||||
// TODO: Add more tests with user role (copy tests with admin role)
|
|
||||||
|
|
||||||
[Fact]
|
[Fact]
|
||||||
public async Task UpdateCity_UnAuthnticatedUser_ThrowsForbiddenException()
|
public async Task UpdateCity_UnAuthnticatedUser_ThrowsForbiddenException()
|
||||||
{
|
{
|
||||||
@ -495,8 +489,6 @@ public class CitiesTests : TestBase
|
|||||||
TestContext.Current.CancellationToken));
|
TestContext.Current.CancellationToken));
|
||||||
}
|
}
|
||||||
|
|
||||||
// TODO: Add more tests with unauthenticated user
|
|
||||||
// (copy tests with admin role)
|
|
||||||
|
|
||||||
[Fact]
|
[Fact]
|
||||||
public async Task DeleteCity_WithAdminRole_CityDeleted()
|
public async Task DeleteCity_WithAdminRole_CityDeleted()
|
||||||
@ -584,8 +576,6 @@ public class CitiesTests : TestBase
|
|||||||
TestContext.Current.CancellationToken));
|
TestContext.Current.CancellationToken));
|
||||||
}
|
}
|
||||||
|
|
||||||
// TODO: Add more tests with user role (copy tests with admin role)
|
|
||||||
|
|
||||||
[Fact]
|
[Fact]
|
||||||
public async Task DeleteCity_UnAuthnticatedUser_ThrowsForbiddenException()
|
public async Task DeleteCity_UnAuthnticatedUser_ThrowsForbiddenException()
|
||||||
{
|
{
|
||||||
@ -599,8 +589,6 @@ public class CitiesTests : TestBase
|
|||||||
TestContext.Current.CancellationToken));
|
TestContext.Current.CancellationToken));
|
||||||
}
|
}
|
||||||
|
|
||||||
// TODO: Add more tests with unauthenticated user
|
|
||||||
// (copy tests with admin role)
|
|
||||||
|
|
||||||
[Fact]
|
[Fact]
|
||||||
public async Task GetCity_WithAdminRole_CityReturned()
|
public async Task GetCity_WithAdminRole_CityReturned()
|
||||||
@ -703,8 +691,6 @@ public class CitiesTests : TestBase
|
|||||||
TestContext.Current.CancellationToken));
|
TestContext.Current.CancellationToken));
|
||||||
}
|
}
|
||||||
|
|
||||||
// TODO: Add more tests with user role (copy tests with admin role)
|
|
||||||
|
|
||||||
[Fact]
|
[Fact]
|
||||||
public async Task GetCity_UnAuthnticatedUser_ThrowsForbiddenException()
|
public async Task GetCity_UnAuthnticatedUser_ThrowsForbiddenException()
|
||||||
{
|
{
|
||||||
@ -718,8 +704,6 @@ public class CitiesTests : TestBase
|
|||||||
TestContext.Current.CancellationToken));
|
TestContext.Current.CancellationToken));
|
||||||
}
|
}
|
||||||
|
|
||||||
// TODO: Add more tests with unauthenticated user
|
|
||||||
// (copy tests with admin role)
|
|
||||||
|
|
||||||
[Fact]
|
[Fact]
|
||||||
public async Task GetCitiesPage_WithAdminRole_CitiesPageReturned()
|
public async Task GetCitiesPage_WithAdminRole_CitiesPageReturned()
|
||||||
@ -1556,8 +1540,6 @@ public class CitiesTests : TestBase
|
|||||||
TestContext.Current.CancellationToken));
|
TestContext.Current.CancellationToken));
|
||||||
}
|
}
|
||||||
|
|
||||||
// TODO: Add more tests with user role (copy tests with admin role)
|
|
||||||
|
|
||||||
[Fact]
|
[Fact]
|
||||||
public async Task GetCitiesPage_UnAuthnticatedUser_ThrowsForbiddenException()
|
public async Task GetCitiesPage_UnAuthnticatedUser_ThrowsForbiddenException()
|
||||||
{
|
{
|
||||||
@ -1570,7 +1552,4 @@ public class CitiesTests : TestBase
|
|||||||
new GetCityQuery(),
|
new GetCityQuery(),
|
||||||
TestContext.Current.CancellationToken));
|
TestContext.Current.CancellationToken));
|
||||||
}
|
}
|
||||||
|
|
||||||
// TODO: Add more tests with unauthenticated user
|
|
||||||
// (copy tests with admin role)
|
|
||||||
}
|
}
|
||||||
|
@ -248,8 +248,6 @@ public class CountriesTests : TestBase
|
|||||||
}, TestContext.Current.CancellationToken));
|
}, TestContext.Current.CancellationToken));
|
||||||
}
|
}
|
||||||
|
|
||||||
// TODO: Add more tests with user role (copy tests with admin role)
|
|
||||||
|
|
||||||
[Fact]
|
[Fact]
|
||||||
public async Task UpdateCountry_UnAuthnticatedUser_ThrowsForbiddenException()
|
public async Task UpdateCountry_UnAuthnticatedUser_ThrowsForbiddenException()
|
||||||
{
|
{
|
||||||
@ -265,8 +263,6 @@ public class CountriesTests : TestBase
|
|||||||
}, TestContext.Current.CancellationToken));
|
}, TestContext.Current.CancellationToken));
|
||||||
}
|
}
|
||||||
|
|
||||||
// TODO: Add more tests with unauthenticated user
|
|
||||||
// (copy tests with admin role)
|
|
||||||
|
|
||||||
[Fact]
|
[Fact]
|
||||||
public async Task DeleteCountry_WithAdminRole_CountryDeleted()
|
public async Task DeleteCountry_WithAdminRole_CountryDeleted()
|
||||||
@ -341,8 +337,6 @@ public class CountriesTests : TestBase
|
|||||||
}, TestContext.Current.CancellationToken));
|
}, TestContext.Current.CancellationToken));
|
||||||
}
|
}
|
||||||
|
|
||||||
// TODO: Add more tests with user role (copy tests with admin role)
|
|
||||||
|
|
||||||
[Fact]
|
[Fact]
|
||||||
public async Task DeleteCountry_UnAuthnticatedUser_ThrowsForbiddenException()
|
public async Task DeleteCountry_UnAuthnticatedUser_ThrowsForbiddenException()
|
||||||
{
|
{
|
||||||
@ -357,8 +351,6 @@ public class CountriesTests : TestBase
|
|||||||
}, TestContext.Current.CancellationToken));
|
}, TestContext.Current.CancellationToken));
|
||||||
}
|
}
|
||||||
|
|
||||||
// TODO: Add more tests with unauthenticated user
|
|
||||||
// (copy tests with admin role)
|
|
||||||
|
|
||||||
[Fact]
|
[Fact]
|
||||||
public async Task GetCountry_WithAdminRole_CountryReturned()
|
public async Task GetCountry_WithAdminRole_CountryReturned()
|
||||||
@ -434,8 +426,6 @@ public class CountriesTests : TestBase
|
|||||||
}, TestContext.Current.CancellationToken));
|
}, TestContext.Current.CancellationToken));
|
||||||
}
|
}
|
||||||
|
|
||||||
// TODO: Add more tests with user role (copy tests with admin role)
|
|
||||||
|
|
||||||
[Fact]
|
[Fact]
|
||||||
public async Task GetCountry_UnAuthnticatedUser_ThrowsForbiddenException()
|
public async Task GetCountry_UnAuthnticatedUser_ThrowsForbiddenException()
|
||||||
{
|
{
|
||||||
@ -450,8 +440,6 @@ public class CountriesTests : TestBase
|
|||||||
}, TestContext.Current.CancellationToken));
|
}, TestContext.Current.CancellationToken));
|
||||||
}
|
}
|
||||||
|
|
||||||
// TODO: Add more tests with unauthenticated user
|
|
||||||
// (copy tests with admin role)
|
|
||||||
|
|
||||||
[Fact]
|
[Fact]
|
||||||
public async Task GetCountriesPage_WithAdminRole_CountriesPageReturned()
|
public async Task GetCountriesPage_WithAdminRole_CountriesPageReturned()
|
||||||
|
@ -216,8 +216,6 @@ public class RegionsTests : TestBase
|
|||||||
TestContext.Current.CancellationToken));
|
TestContext.Current.CancellationToken));
|
||||||
}
|
}
|
||||||
|
|
||||||
// TODO: Add more tests with user role (copy tests with admin role)
|
|
||||||
|
|
||||||
[Fact]
|
[Fact]
|
||||||
public async Task AddRegion_UnAuthnticatedUser_ThrowsForbiddenException()
|
public async Task AddRegion_UnAuthnticatedUser_ThrowsForbiddenException()
|
||||||
{
|
{
|
||||||
@ -231,8 +229,6 @@ public class RegionsTests : TestBase
|
|||||||
TestContext.Current.CancellationToken));
|
TestContext.Current.CancellationToken));
|
||||||
}
|
}
|
||||||
|
|
||||||
// TODO: Add more tests with unauthenticated user
|
|
||||||
// (copy tests with admin role)
|
|
||||||
|
|
||||||
[Fact]
|
[Fact]
|
||||||
public async Task UpdateRegion_WithAdminRole_RegionUpdated()
|
public async Task UpdateRegion_WithAdminRole_RegionUpdated()
|
||||||
@ -408,8 +404,6 @@ public class RegionsTests : TestBase
|
|||||||
TestContext.Current.CancellationToken));
|
TestContext.Current.CancellationToken));
|
||||||
}
|
}
|
||||||
|
|
||||||
// TODO: Add more tests with user role (copy tests with admin role)
|
|
||||||
|
|
||||||
[Fact]
|
[Fact]
|
||||||
public async Task UpdateRegion_UnAuthnticatedUser_ThrowsForbiddenException()
|
public async Task UpdateRegion_UnAuthnticatedUser_ThrowsForbiddenException()
|
||||||
{
|
{
|
||||||
@ -423,8 +417,6 @@ public class RegionsTests : TestBase
|
|||||||
TestContext.Current.CancellationToken));
|
TestContext.Current.CancellationToken));
|
||||||
}
|
}
|
||||||
|
|
||||||
// TODO: Add more tests with unauthenticated user
|
|
||||||
// (copy tests with admin role)
|
|
||||||
|
|
||||||
[Fact]
|
[Fact]
|
||||||
public async Task DeleteRegion_WithAdminRole_RegionDeleted()
|
public async Task DeleteRegion_WithAdminRole_RegionDeleted()
|
||||||
@ -505,8 +497,6 @@ public class RegionsTests : TestBase
|
|||||||
TestContext.Current.CancellationToken));
|
TestContext.Current.CancellationToken));
|
||||||
}
|
}
|
||||||
|
|
||||||
// TODO: Add more tests with user role (copy tests with admin role)
|
|
||||||
|
|
||||||
[Fact]
|
[Fact]
|
||||||
public async Task DeleteRegion_UnAuthnticatedUser_ThrowsForbiddenException()
|
public async Task DeleteRegion_UnAuthnticatedUser_ThrowsForbiddenException()
|
||||||
{
|
{
|
||||||
@ -520,8 +510,6 @@ public class RegionsTests : TestBase
|
|||||||
TestContext.Current.CancellationToken));
|
TestContext.Current.CancellationToken));
|
||||||
}
|
}
|
||||||
|
|
||||||
// TODO: Add more tests with unauthenticated user
|
|
||||||
// (copy tests with admin role)
|
|
||||||
|
|
||||||
[Fact]
|
[Fact]
|
||||||
public async Task GetRegion_WithAdminRole_RegionReturned()
|
public async Task GetRegion_WithAdminRole_RegionReturned()
|
||||||
@ -610,8 +598,6 @@ public class RegionsTests : TestBase
|
|||||||
TestContext.Current.CancellationToken));
|
TestContext.Current.CancellationToken));
|
||||||
}
|
}
|
||||||
|
|
||||||
// TODO: Add more tests with user role (copy tests with admin role)
|
|
||||||
|
|
||||||
[Fact]
|
[Fact]
|
||||||
public async Task GetRegion_UnAuthnticatedUser_ThrowsForbiddenException()
|
public async Task GetRegion_UnAuthnticatedUser_ThrowsForbiddenException()
|
||||||
{
|
{
|
||||||
@ -625,8 +611,6 @@ public class RegionsTests : TestBase
|
|||||||
TestContext.Current.CancellationToken));
|
TestContext.Current.CancellationToken));
|
||||||
}
|
}
|
||||||
|
|
||||||
// TODO: Add more tests with unauthenticated user
|
|
||||||
// (copy tests with admin role)
|
|
||||||
|
|
||||||
[Fact]
|
[Fact]
|
||||||
public async Task GetRegionsPage_WithAdminRole_RegionsPageReturned()
|
public async Task GetRegionsPage_WithAdminRole_RegionsPageReturned()
|
||||||
@ -1141,8 +1125,6 @@ public class RegionsTests : TestBase
|
|||||||
TestContext.Current.CancellationToken));
|
TestContext.Current.CancellationToken));
|
||||||
}
|
}
|
||||||
|
|
||||||
// TODO: Add more tests with user role (copy tests with admin role)
|
|
||||||
|
|
||||||
[Fact]
|
[Fact]
|
||||||
public async Task GetRegionsPage_UnAuthnticatedUser_ThrowsForbiddenException()
|
public async Task GetRegionsPage_UnAuthnticatedUser_ThrowsForbiddenException()
|
||||||
{
|
{
|
||||||
@ -1155,7 +1137,4 @@ public class RegionsTests : TestBase
|
|||||||
new GetRegionQuery(),
|
new GetRegionQuery(),
|
||||||
TestContext.Current.CancellationToken));
|
TestContext.Current.CancellationToken));
|
||||||
}
|
}
|
||||||
|
|
||||||
// TODO: Add more tests with unauthenticated user
|
|
||||||
// (copy tests with admin role)
|
|
||||||
}
|
}
|
||||||
|
Loading…
Reference in New Issue
Block a user