Skip to content

Commit cd6680e

Browse files
Address code review feedback: rename test and update comments for clarity
Agent-Logs-Url: https://github.com/webexpress-framework/WebExpress.WebCore/sessions/f69b558f-1a72-4801-809e-0fb3012f0dc9 Co-authored-by: ReneSchwarzer <31061438+ReneSchwarzer@users.noreply.github.com>
1 parent d1f4015 commit cd6680e

1 file changed

Lines changed: 3 additions & 5 deletions

File tree

src/WebExpress.WebCore.Test/Manager/UnitTestIdentityManager.cs

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -258,18 +258,16 @@ public void IIdentityGroupHasIdAndName()
258258
}
259259

260260
/// <summary>
261-
/// Test that CheckAccess via AllGroup provides PublicAccess policy evaluation for any identity.
261+
/// Test that the AllGroup has the expected name and contains the PublicAccess policy.
262262
/// </summary>
263263
[Fact]
264-
public void CheckAccessViaAllGroup()
264+
public void AllGroupHasExpectedProperties()
265265
{
266266
// arrange
267267
var componentHub = UnitTestFixture.CreateAndRegisterComponentHubMock();
268268
var identityManager = componentHub.IdentityManager as IdentityManager;
269-
var applicationContext = componentHub.ApplicationManager.GetApplications(typeof(TestApplicationA)).FirstOrDefault();
270269

271-
// act - the AllGroup has the PublicAccess policy, so checking AllGroup for a permission
272-
// that is linked to PublicAccess should succeed
270+
// act
273271
var allGroup = identityManager.AllGroup;
274272

275273
// assert

0 commit comments

Comments
 (0)