permission-node: rename static create method to fromConfig

Signed-off-by: MT Lewis <mtlewis@users.noreply.github.com>

packages/backend/src/index.ts

@@ -64,9 +64,8 @@ function makeCreateEnv(config: Config) {
   const reader = UrlReaders.default({ logger: root, config });
   const discovery = SingleHostDiscovery.fromConfig(config);
   const tokenManager = ServerTokenManager.fromConfig(config, { logger: root });
-  const permissions = ServerPermissionClient.create({
+  const permissions = ServerPermissionClient.fromConfig(config, {
     discovery,
-    config,
     tokenManager,
   });
 

plugins/permission-node/api-report.md

@@ -134,10 +134,12 @@ export class ServerPermissionClient implements PermissionAuthorizer {
     options?: AuthorizeRequestOptions,
   ): Promise<AuthorizeResponse[]>;
   // (undocumented)
-  static create(options: {
-    discovery: PluginEndpointDiscovery;
-    config: Config;
-    tokenManager: TokenManager;
-  }): ServerPermissionClient;
+  static fromConfig(
+    config: Config,
+    options: {
+      discovery: PluginEndpointDiscovery;
+      tokenManager: TokenManager;
+    },
+  ): ServerPermissionClient;
 }
 ```

plugins/permission-node/src/ServerPermissionClient.test.ts

@@ -68,9 +68,8 @@ describe('ServerPermissionClient', () => {
   afterEach(() => server.resetHandlers());
 
   it('should bypass authorization if permissions are disabled', async () => {
-    const client = ServerPermissionClient.create({
+    const client = ServerPermissionClient.fromConfig(new ConfigReader({}), {
       discovery,
-      config: new ConfigReader({}),
       tokenManager: ServerTokenManager.noop(),
     });
 
@@ -81,9 +80,8 @@ describe('ServerPermissionClient', () => {
 
   it('should bypass authorization if permissions are enabled and request has valid server token', async () => {
     const tokenManager = ServerTokenManager.fromConfig(config, { logger });
-    const client = ServerPermissionClient.create({
+    const client = ServerPermissionClient.fromConfig(config, {
       discovery,
-      config,
       tokenManager,
     });
 
@@ -96,9 +94,8 @@ describe('ServerPermissionClient', () => {
 
   it('should authorize normally if permissions are enabled and request does not have valid server token', async () => {
     const tokenManager = ServerTokenManager.fromConfig(config, { logger });
-    const client = ServerPermissionClient.create({
+    const client = ServerPermissionClient.fromConfig(config, {
       discovery,
-      config,
       tokenManager,
     });
 
@@ -111,9 +108,8 @@ describe('ServerPermissionClient', () => {
 
   it('should error if permissions are enabled but a no-op token manager is configured', async () => {
     expect(() =>
-      ServerPermissionClient.create({
+      ServerPermissionClient.fromConfig(config, {
         discovery,
-        config,
         tokenManager: ServerTokenManager.noop(),
       }),
     ).toThrowError(

plugins/permission-node/src/ServerPermissionClient.ts

@@ -39,12 +39,14 @@ export class ServerPermissionClient implements PermissionAuthorizer {
   private readonly tokenManager: TokenManager;
   private readonly permissionEnabled: boolean;
 
-  static create(options: {
-    discovery: PluginEndpointDiscovery;
-    config: Config;
-    tokenManager: TokenManager;
-  }) {
-    const { discovery, config, tokenManager } = options;
+  static fromConfig(
+    config: Config,
+    options: {
+      discovery: PluginEndpointDiscovery;
+      tokenManager: TokenManager;
+    },
+  ) {
+    const { discovery, tokenManager } = options;
     const permissionClient = new PermissionClient({ discovery, config });
     const permissionEnabled =
       config.getOptionalBoolean('permission.enabled') ?? false;