fix confused comment

a-tarasyuk · a-tarasyuk · commit fe2e2c621f1b · 2022-05-13T09:51:59.000+03:00
diff --git a/tests/baselines/reference/jsxExcessPropsAndAssignability.errors.txt b/tests/baselines/reference/jsxExcessPropsAndAssignability.errors.txt
@@ -17,11 +17,11 @@ tests/cases/compiler/jsxExcessPropsAndAssignability.tsx(16,27): error TS2698: Sp
     
         const props: ComposedComponentProps = null as any;
     
-        // Expected no error, got none - good
+        // Expected no error caused by "myProp", got none - good
         <WrapperComponent {...props} myProp={'1000000'} />;
                               ~~~~~
 !!! error TS2698: Spread types may only be created from object types.
-        // Expected error, but got none - bad!
+        // Expected error caused by "myProp", but got none - bad!
         <WrapperComponent {...props} myProp={1000000} />;
          ~~~~~~~~~~~~~~~~
 !!! error TS2322: Type 'ComposedComponentProps & { myProp: number; }' is not assignable to type 'IntrinsicAttributes & IntrinsicClassAttributes<Component<WrapperComponentProps, any, any>> & Readonly<{ children?: ReactNode; }> & Readonly<WrapperComponentProps>'.
diff --git a/tests/baselines/reference/jsxExcessPropsAndAssignability.js b/tests/baselines/reference/jsxExcessPropsAndAssignability.js
@@ -11,9 +11,9 @@ const myHoc = <ComposedComponentProps extends any>(
 
     const props: ComposedComponentProps = null as any;
 
-    // Expected no error, got none - good
+    // Expected no error caused by "myProp", got none - good
     <WrapperComponent {...props} myProp={'1000000'} />;
-    // Expected error, but got none - bad!
+    // Expected error caused by "myProp", but got none - bad!
     <WrapperComponent {...props} myProp={1000000} />;
 };
 
@@ -37,8 +37,8 @@ var React = require("react");
 var myHoc = function (ComposedComponent) {
     var WrapperComponent = null;
     var props = null;
-    // Expected no error, got none - good
+    // Expected no error caused by "myProp", got none - good
     React.createElement(WrapperComponent, __assign({}, props, { myProp: '1000000' }));
-    // Expected error, but got none - bad!
+    // Expected error caused by "myProp", but got none - bad!
     React.createElement(WrapperComponent, __assign({}, props, { myProp: 1000000 }));
 };
diff --git a/tests/baselines/reference/jsxExcessPropsAndAssignability.symbols b/tests/baselines/reference/jsxExcessPropsAndAssignability.symbols
@@ -30,13 +30,13 @@ const myHoc = <ComposedComponentProps extends any>(
 >props : Symbol(props, Decl(jsxExcessPropsAndAssignability.tsx, 10, 9))
 >ComposedComponentProps : Symbol(ComposedComponentProps, Decl(jsxExcessPropsAndAssignability.tsx, 4, 15))
 
-    // Expected no error, got none - good
+    // Expected no error caused by "myProp", got none - good
     <WrapperComponent {...props} myProp={'1000000'} />;
 >WrapperComponent : Symbol(WrapperComponent, Decl(jsxExcessPropsAndAssignability.tsx, 8, 9))
 >props : Symbol(props, Decl(jsxExcessPropsAndAssignability.tsx, 10, 9))
 >myProp : Symbol(myProp, Decl(jsxExcessPropsAndAssignability.tsx, 13, 32))
 
-    // Expected error, but got none - bad!
+    // Expected error caused by "myProp", but got none - bad!
     <WrapperComponent {...props} myProp={1000000} />;
 >WrapperComponent : Symbol(WrapperComponent, Decl(jsxExcessPropsAndAssignability.tsx, 8, 9))
 >props : Symbol(props, Decl(jsxExcessPropsAndAssignability.tsx, 10, 9))
diff --git a/tests/baselines/reference/jsxExcessPropsAndAssignability.types b/tests/baselines/reference/jsxExcessPropsAndAssignability.types
@@ -6,7 +6,7 @@ import * as React from 'react';
 
 const myHoc = <ComposedComponentProps extends any>(
 >myHoc : <ComposedComponentProps extends unknown>(ComposedComponent: React.ComponentClass<ComposedComponentProps, any>) => void
-><ComposedComponentProps extends any>(    ComposedComponent: React.ComponentClass<ComposedComponentProps>,) => {    type WrapperComponentProps = ComposedComponentProps & { myProp: string };    const WrapperComponent: React.ComponentClass<WrapperComponentProps> = null as any;    const props: ComposedComponentProps = null as any;    // Expected no error, got none - good    <WrapperComponent {...props} myProp={'1000000'} />;    // Expected error, but got none - bad!    <WrapperComponent {...props} myProp={1000000} />;} : <ComposedComponentProps extends unknown>(ComposedComponent: React.ComponentClass<ComposedComponentProps, any>) => void
+><ComposedComponentProps extends any>(    ComposedComponent: React.ComponentClass<ComposedComponentProps>,) => {    type WrapperComponentProps = ComposedComponentProps & { myProp: string };    const WrapperComponent: React.ComponentClass<WrapperComponentProps> = null as any;    const props: ComposedComponentProps = null as any;    // Expected no error caused by "myProp", got none - good    <WrapperComponent {...props} myProp={'1000000'} />;    // Expected error caused by "myProp", but got none - bad!    <WrapperComponent {...props} myProp={1000000} />;} : <ComposedComponentProps extends unknown>(ComposedComponent: React.ComponentClass<ComposedComponentProps, any>) => void
 
     ComposedComponent: React.ComponentClass<ComposedComponentProps>,
 >ComposedComponent : React.ComponentClass<ComposedComponentProps, any>
@@ -28,15 +28,15 @@ const myHoc = <ComposedComponentProps extends any>(
 >null as any : any
 >null : null
 
-    // Expected no error, got none - good
+    // Expected no error caused by "myProp", got none - good
     <WrapperComponent {...props} myProp={'1000000'} />;
 ><WrapperComponent {...props} myProp={'1000000'} /> : JSX.Element
 >WrapperComponent : React.ComponentClass<ComposedComponentProps & { myProp: string; }, any>
 >props : ComposedComponentProps
 >myProp : "1000000"
 >'1000000' : "1000000"
 
-    // Expected error, but got none - bad!
+    // Expected error caused by "myProp", but got none - bad!
     <WrapperComponent {...props} myProp={1000000} />;
 ><WrapperComponent {...props} myProp={1000000} /> : JSX.Element
 >WrapperComponent : React.ComponentClass<ComposedComponentProps & { myProp: string; }, any>
diff --git a/tests/cases/compiler/jsxExcessPropsAndAssignability.tsx b/tests/cases/compiler/jsxExcessPropsAndAssignability.tsx
@@ -12,8 +12,8 @@ const myHoc = <ComposedComponentProps extends any>(
 
     const props: ComposedComponentProps = null as any;
 
-    // Expected no error, got none - good
+    // Expected no error caused by "myProp", got none - good
     <WrapperComponent {...props} myProp={'1000000'} />;
-    // Expected error, but got none - bad!
+    // Expected error caused by "myProp", but got none - bad!
     <WrapperComponent {...props} myProp={1000000} />;
 };
