Move $app->roles() to AppUsers trait

Improves consistency and overview

src/Cms/App.php

@@ -1331,14 +1331,6 @@ public function response(): Responder
 		return $this->response ??= new Responder();
 	}
 
-	/**
-	 * Returns all user roles
-	 */
-	public function roles(): Roles
-	{
-		return $this->roles ??= Roles::load($this->root('roles'));
-	}
-
 	/**
 	 * Returns a system root
 	 */

src/Cms/AppUsers.php

@@ -67,6 +67,14 @@ public function impersonate(
 		}
 	}
 
+	/**
+	 * Returns all user roles
+	 */
+	public function roles(): Roles
+	{
+		return $this->roles ??= Roles::load($this->root('roles'));
+	}
+
 	/**
 	 * Set the currently active user id
 	 *

tests/Cms/App/AppUsersTest.php

@@ -110,7 +110,34 @@ public function testImpersonateErrorMissingUser()
 		$this->app->impersonate('[email protected]');
 	}
 
-	public function testLoad()
+	public function testRolesSet()
+	{
+		$app = new App([
+			'roles' => [
+				[
+					'name'  => 'editor',
+					'title' => 'Editor'
+				]
+			]
+		]);
+
+		$this->assertCount(2, $app->roles());
+		$this->assertSame('editor', $app->roles()->last()->name());
+	}
+
+	public function testRolesLoad()
+	{
+		$app = new App([
+			'roots' => [
+				'site' => static::FIXTURES
+			]
+		]);
+
+		$this->assertCount(2, $app->roles());
+		$this->assertSame('editor', $app->roles()->last()->name());
+	}
+
+	public function testUsersLoad()
 	{
 		$app = $this->app->clone([
 			'roots' => [
@@ -122,7 +149,7 @@ public function testLoad()
 		$this->assertSame('[email protected]', $app->users()->first()->email());
 	}
 
-	public function testSet()
+	public function testUsersSet()
 	{
 		$app = $this->app->clone([
 			'users' => [