Updates to usage of PortAttributes VSC API (#13506)

microsoft · May 16, 2023 · cd19546 · cd19546
1 parent 1e4f3ac
commit cd19546
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 9 deletions.
diff --git a/src/kernels/portAttributeProvider.node.ts b/src/kernels/portAttributeProvider.node.ts
@@ -31,7 +31,7 @@ export class PortAttributesProviders implements PortAttributesProvider, IExtensi
     ): PortAttributes | undefined {
         try {
             if (UsedPorts.has(port)) {
-                return new PortAttributes(port, PortAutoForwardAction.Ignore);
+                return new PortAttributes(PortAutoForwardAction.Ignore);
             }
         } catch (ex) {
             // In case proposed API changes.

diff --git a/src/test/mocks/vsc/extHostedTypes.ts b/src/test/mocks/vsc/extHostedTypes.ts
@@ -2398,23 +2398,16 @@ export namespace vscMockExtHostedTypes {
     }
 
     export class PortAttributes {
-        /**
-         * The port number associated with this this set of attributes.
-         */
-        port: number;
-
         /**
          * The action to be taken when this port is detected for auto forwarding.
          */
         autoForwardAction: PortAutoForwardAction;
-
         /**
          * Creates a new PortAttributes object
          * @param port the port number
          * @param autoForwardAction the action to take when this port is detected
          */
-        constructor(port: number, autoForwardAction: PortAutoForwardAction) {
-            this.port = port;
+        constructor(autoForwardAction: PortAutoForwardAction) {
             this.autoForwardAction = autoForwardAction;
         }
     }