Skip to content

Commit

Permalink
fix(types): extract the correct props type for the DateConstructor (#…
Browse files Browse the repository at this point in the history
  • Loading branch information
HcySunYang committed Feb 5, 2021
1 parent 288ae0a commit 48f0d29
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 2 deletions.
4 changes: 3 additions & 1 deletion packages/runtime-core/src/componentProps.ts
Original file line number Diff line number Diff line change
Expand Up @@ -97,7 +97,9 @@ type InferPropType<T> = T extends null
? Record<string, any>
: T extends BooleanConstructor | { type: BooleanConstructor }
? boolean
: T extends Prop<infer V, infer D> ? (unknown extends V ? D : V) : T
: T extends DateConstructor | { type: DateConstructor }
? Date
: T extends Prop<infer V, infer D> ? (unknown extends V ? D : V) : T

export type ExtractPropTypes<O> = O extends object
? { [K in RequiredKeys<O>]: InferPropType<O[K]> } &
Expand Down
5 changes: 4 additions & 1 deletion test-dts/defineComponent.test-d.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -34,6 +34,7 @@ describe('with object props', () => {
ggg: 'foo' | 'bar'
ffff: (a: number, b: string) => { a: boolean }
validated?: string
date?: Date
}

type GT = string & { __brand: unknown }
Expand Down Expand Up @@ -103,7 +104,8 @@ describe('with object props', () => {
type: String,
// validator requires explicit annotation
validator: (val: unknown) => val !== ''
}
},
date: Date
},
setup(props) {
// type assertion. See https://github.com/SamVerschueren/tsd
Expand All @@ -125,6 +127,7 @@ describe('with object props', () => {
expectType<ExpectedProps['ggg']>(props.ggg)
expectType<ExpectedProps['ffff']>(props.ffff)
expectType<ExpectedProps['validated']>(props.validated)
expectType<ExpectedProps['date']>(props.date)

// @ts-expect-error props should be readonly
expectError((props.a = 1))
Expand Down

0 comments on commit 48f0d29

Please sign in to comment.