test: Investigate removing IsEmptyInterface

Author: eps1lon

__snapshots__/IsEmptyInterface

@@ -1,15 +1,14 @@
-sources/IsEmptyInterface.test.ts(11,27): error TS2554: Expected 1 arguments, but got 0.
 Files:            54
-Lines:         61530
-Nodes:        180467
-Identifiers:   62604
-Symbols:      104007
-Types:         36924
-Memory used: 135129K
-I/O read:      0.02s
+Lines:         61493
+Nodes:        180360
+Identifiers:   62571
+Symbols:      103235
+Types:         36774
+Memory used: 153131K
+I/O read:      0.01s
 I/O write:     0.00s
-Parse time:    0.76s
-Bind time:     0.22s
-Check time:    3.14s
+Parse time:    0.66s
+Bind time:     0.28s
+Check time:    4.42s
 Emit time:     0.00s
-Total time:    4.12s
+Total time:    5.37s

vendor/@material-ui/styles/makeStyles/makeStyles.d.ts

@@ -5,26 +5,9 @@ import {
   Styles,
   WithStylesOptions,
 } from '@material-ui/styles/withStyles';
-import { Omit, IsAny, Or, IsEmptyInterface } from '@material-ui/types';
+import { Omit } from '@material-ui/types';
 import { DefaultTheme } from '../defaultTheme';
 
-/**
- * @internal
- *
- * If a style callback is given with `theme => stylesOfTheme` then typescript
- * infers `Props` to `any`.
- * If a static object is given with { ...members } then typescript infers `Props`
- * to `{}`.
- *
- * So we require no props in `useStyles` if `Props` in `makeStyles(styles)` is
- * inferred to either `any` or `{}`
- */
-export type StylesRequireProps<S> = Or<
-  IsAny<PropsOfStyles<S>>,
-  IsEmptyInterface<PropsOfStyles<S>>
-> extends true
-  ? false
-  : true;
 
 /**
  * @internal
@@ -33,9 +16,7 @@ export type StylesRequireProps<S> = Or<
  * from which the typechecker could infer a type so it falls back to `any`.
  * See the test cases for examples and implications of explicit `any` annotation
  */
-export type StylesHook<S extends Styles<any, any>> = StylesRequireProps<S> extends false
-  ? (props?: any) => ClassNameMap<ClassKeyOfStyles<S>>
-  : (props: PropsOfStyles<S>) => ClassNameMap<ClassKeyOfStyles<S>>;
+export type StylesHook<S extends Styles<any, any>> = (props?: PropsOfStyles<S>) => ClassNameMap<ClassKeyOfStyles<S>>
 
 export default function makeStyles<
   Theme = DefaultTheme,

vendor/@material-ui/styles/withStyles/withStyles.d.ts

@@ -1,5 +1,5 @@
 import * as React from 'react';
-import { PropInjector, CoerceEmptyInterface, IsEmptyInterface } from '@material-ui/types';
+import { PropInjector, CoerceEmptyInterface } from '@material-ui/types';
 import * as CSS from 'csstype';
 import * as JSS from 'jss';
 import { DefaultTheme } from '../defaultTheme';
@@ -43,9 +43,7 @@ export interface CreateCSSProperties<Props extends object = {}>
  */
 export type StyleRules<Props extends object = {}, ClassKey extends string = string> = Record<
   ClassKey,
-  IsEmptyInterface<Props> extends true
-    ? CSSProperties | (() => CSSProperties)
-    : CreateCSSProperties<Props> | ((props: Props) => CreateCSSProperties<Props>)
+  CreateCSSProperties<Props> | ((props: Props) => CreateCSSProperties<Props>)
 >;
 
 /**

vendor/@material-ui/types/index.d.ts

@@ -71,19 +71,3 @@ export type And<A, B, C = true> = A extends true
       : false
     : false
   : false;
-
-/**
- * @internal
- *
- * check if a type is `{}`
- *
- * 1. false if the given type has any members
- * 2. false if the type is `object` which is the only other type with no members
- *  {} is a top type so e.g. `string extends {}` but not `string extends object`
- * 3. false if the given type is `unknown`
- */
-export type IsEmptyInterface<T> = And<
-  keyof T extends never ? true : false,
-  string extends T ? true : false,
-  unknown extends T ? false : true
->;