test imports of non-exported things use relative paths

ecraig12345 · ecraig12345 · commit 42052754ed59 · 2020-01-15T01:57:43.000-08:00
diff --git a/packages/react-bindings/test/accesibility/getKeyDownHandlers-test.ts b/packages/react-bindings/test/accesibility/getKeyDownHandlers-test.ts
@@ -1,3 +1,4 @@
+// This is not exported
 import getKeyDownHandlers from '../../src/accessibility/getKeyDownHandlers'
 // @ts-ignore
 import * as keyboardKey from 'keyboard-key'
diff --git a/packages/react-bindings/test/accesibility/shouldHandleOnKeys-test.ts b/packages/react-bindings/test/accesibility/shouldHandleOnKeys-test.ts
@@ -1,3 +1,4 @@
+// This is not exported
 import shouldHandleOnKeys from '../../src/accessibility/shouldHandleOnKeys'
 
 const getEventArg = (
diff --git a/packages/react-bindings/test/styles/resolveStylesAndClasses-test.ts b/packages/react-bindings/test/styles/resolveStylesAndClasses-test.ts
@@ -4,7 +4,8 @@ import {
   emptyTheme,
   ICSSInJSStyle,
 } from '@fluentui/styles'
-import resolveStylesAndClasses from '@fluentui/react-bindings/src/styles/resolveStylesAndClasses'
+// This is not exported
+import resolveStylesAndClasses from '../../src/styles/resolveStylesAndClasses'
 
 const styleParam: ComponentStyleFunctionParam = {
   disableAnimations: false,
diff --git a/packages/react/test/specs/components/Debug/utils-test.ts b/packages/react/test/specs/components/Debug/utils-test.ts
@@ -1,10 +1,11 @@
+// These are not exported
 import {
   find,
   isOverridden,
   filter,
   getValues,
   removeNulls,
-} from '@fluentui/react/src/components/Debug/utils'
+} from '../../../../src/components/Debug/utils'
 
 describe('debugUtils', () => {
   describe('find', () => {
diff --git a/packages/react/test/specs/components/Icon/Icon-test.tsx b/packages/react/test/specs/components/Icon/Icon-test.tsx
@@ -2,7 +2,7 @@ import { ThemeInput } from '@fluentui/styles'
 import * as React from 'react'
 import { isConformant, handlesAccessibility, getRenderedAttribute } from '../../commonTests'
 
-import Icon from '../../../../src/components/Icon/Icon'
+import { Icon } from '@fluentui/react'
 import { mountWithProviderAndGetComponent } from '../../../utils'
 
 describe('Icon', () => {
diff --git a/packages/react/test/specs/components/Portal/Portal-test.tsx b/packages/react/test/specs/components/Portal/Portal-test.tsx
@@ -2,7 +2,8 @@ import * as React from 'react'
 import { domEvent, nextFrame, mountWithProvider } from '../../../utils'
 
 import { Portal } from '@fluentui/react'
-import PortalInner from '@fluentui/react/src/components/Portal/PortalInner'
+// This is not exported
+import PortalInner from '../../../../src/components/Portal/PortalInner'
 
 describe('Portal', () => {
   const testPortalInnerIsOpen = (rootWrapper, visible: boolean) => {
diff --git a/packages/react/test/specs/components/PortalInner/PortalInner-test.tsx b/packages/react/test/specs/components/PortalInner/PortalInner-test.tsx
@@ -1,6 +1,7 @@
 import * as React from 'react'
 
-import PortalInner, { PortalInnerProps } from '@fluentui/react/src/components/Portal/PortalInner'
+// These are not exported
+import PortalInner, { PortalInnerProps } from '../../../../src/components/Portal/PortalInner'
 import { mountWithProvider } from '../../../utils'
 
 const mountPortalInner = (props: PortalInnerProps) =>
diff --git a/packages/react/test/specs/themes/teams/components/Icon/Icon-test.tsx b/packages/react/test/specs/themes/teams/components/Icon/Icon-test.tsx
@@ -2,6 +2,7 @@ import * as _ from 'lodash'
 import * as React from 'react'
 import { shallow } from 'enzyme'
 
+// These are not exported
 import icons, { teamsIconClassNames } from '../../../../../../src/themes/teams/components/Icon/svg'
 import processedIcons from '../../../../../../src/themes/teams/components/Icon/svg/processedIndex'
 import { SvgIconSpecWithStyles } from '../../../../../../src/themes/teams/components/Icon/svg/types'
diff --git a/packages/react/test/specs/utils/accessibility/FocusContainer-test.ts b/packages/react/test/specs/utils/accessibility/FocusContainer-test.ts
@@ -1,4 +1,5 @@
-import { ContainerFocusHandler } from '@fluentui/react/src/utils/accessibility/FocusHandling/FocusContainer'
+// This is not exported
+import { ContainerFocusHandler } from '../../../../src/utils/accessibility/FocusHandling/FocusContainer'
 
 const createFocusContainer = (
   { itemsCount, setFocusAtFn }: { itemsCount: number; setFocusAtFn?: () => void } = {
diff --git a/packages/react/test/specs/utils/felaDisableAnimationsPlugin-test.ts b/packages/react/test/specs/utils/felaDisableAnimationsPlugin-test.ts
@@ -1,4 +1,5 @@
-import felaDisableAnimationsPlugin from '@fluentui/react/src/utils/felaDisableAnimationsPlugin'
+// This is not exported
+import felaDisableAnimationsPlugin from '../../../src/utils/felaDisableAnimationsPlugin'
 
 const disableAnimationsPlugin = felaDisableAnimationsPlugin()
 
diff --git a/packages/react/test/specs/utils/felaExpandCssShorthandsPlugin-test.ts b/packages/react/test/specs/utils/felaExpandCssShorthandsPlugin-test.ts
@@ -1,4 +1,5 @@
-import felaExpandCssShorthandsPlugin from '@fluentui/react/src/utils/felaExpandCssShorthandsPlugin'
+// This is not exported
+import felaExpandCssShorthandsPlugin from '../../../src/utils/felaExpandCssShorthandsPlugin'
 
 const expandCssShorthands = felaExpandCssShorthandsPlugin()
 
diff --git a/packages/react/test/specs/utils/felaFocusVisibleEnhancer-test.ts b/packages/react/test/specs/utils/felaFocusVisibleEnhancer-test.ts
@@ -2,7 +2,8 @@ import { createRenderer } from 'fela'
 import { renderToString } from 'fela-tools'
 import { RULE_TYPE } from 'fela-utils'
 
-import felaFocusVisibleEnhancer from '@fluentui/react/src/utils/felaFocusVisibleEnhancer'
+// This is not exported
+import felaFocusVisibleEnhancer from '../../../src/utils/felaFocusVisibleEnhancer'
 
 describe('felaFocusVisibleEnhancer', () => {
   test('replaces :focus-visible', () => {
diff --git a/packages/react/test/specs/utils/felaInvokeKeyframesPlugin-test.ts b/packages/react/test/specs/utils/felaInvokeKeyframesPlugin-test.ts
@@ -1,4 +1,5 @@
-import felaInvokeKeyframesPlugin from '@fluentui/react/src/utils/felaInvokeKeyframesPlugin'
+// This is not exported
+import felaInvokeKeyframesPlugin from '../../../src/utils/felaInvokeKeyframesPlugin'
 
 const renderInvokeKeyframes = felaInvokeKeyframesPlugin()
 
diff --git a/packages/react/test/specs/utils/felaSanitizeCssPlugin-test.ts b/packages/react/test/specs/utils/felaSanitizeCssPlugin-test.ts
@@ -1,4 +1,5 @@
-import sanitizeCss from '@fluentui/react/src/utils/felaSanitizeCssPlugin'
+// This is not exported
+import sanitizeCss from '../../../src/utils/felaSanitizeCssPlugin'
 
 const assertCssPropertyValue = (value: string, isValid: boolean) => {
   test(`assert that '${value}' is ${isValid ? 'valid' : 'invalid'}`, () => {
diff --git a/packages/react/test/specs/utils/mergeProviderContexts/mergeBooleanValues-test.ts b/packages/react/test/specs/utils/mergeProviderContexts/mergeBooleanValues-test.ts
@@ -1,3 +1,4 @@
+// This is not exported
 import { mergeBooleanValues } from '../../../../src/utils/mergeProviderContexts'
 
 describe('mergeBooleanValues', () => {
diff --git a/packages/react/test/specs/utils/mergeProviderContexts/mergeProviderContexts-test.ts b/packages/react/test/specs/utils/mergeProviderContexts/mergeProviderContexts-test.ts
@@ -1,4 +1,5 @@
-import { mergeRenderers } from '@fluentui/react/src/utils/mergeProviderContexts'
+// This is not exported
+import { mergeRenderers } from '../../../../src/utils/mergeProviderContexts'
 import { felaRenderer, mergeProviderContexts } from '@fluentui/react'
 
 describe('mergeRenderers', () => {
diff --git a/packages/react/test/specs/utils/positioner/getScrollParent-test.ts b/packages/react/test/specs/utils/positioner/getScrollParent-test.ts
@@ -1,6 +1,7 @@
 import * as _ from 'lodash'
 
-import getScrollParent from '@fluentui/react/src/utils/positioner/getScrollParent'
+// This is not exported
+import getScrollParent from '../../../../src/utils/positioner/getScrollParent'
 
 const overflowStyles: Partial<CSSStyleDeclaration>[] = [
   { overflow: 'scroll' },
diff --git a/packages/react/test/specs/utils/positioner/positioningHelper-test.ts b/packages/react/test/specs/utils/positioner/positioningHelper-test.ts
@@ -1,10 +1,8 @@
 import { Placement } from 'popper.js'
 
 import { Alignment, Position } from '@fluentui/react'
-import {
-  getPlacement,
-  applyRtlToOffset,
-} from '@fluentui/react/src/utils/positioner/positioningHelper'
+// These are not exported
+import { getPlacement, applyRtlToOffset } from '../../../../src/utils/positioner/positioningHelper'
 
 type PositionTestInput = {
   align: Alignment
diff --git a/packages/styles/test/mergeThemes/mergeComponentStyles-test.ts b/packages/styles/test/mergeThemes/mergeComponentStyles-test.ts
@@ -1,5 +1,6 @@
 import { ComponentStyleFunctionParam, emptyTheme, withDebugId } from '@fluentui/styles'
 
+// These are not exported
 import * as debugEnabled from '../../src/debugEnabled'
 import { mergeComponentStyles__PROD, mergeComponentStyles__DEV } from '../../src/mergeThemes'
 
diff --git a/packages/styles/test/mergeThemes/mergeComponentVariables-test.ts b/packages/styles/test/mergeThemes/mergeComponentVariables-test.ts
@@ -1,5 +1,6 @@
 import { objectKeyToValues, withDebugId } from '@fluentui/styles'
 
+// These are not exported
 import * as debugEnabled from '../../src/debugEnabled'
 import { mergeComponentVariables__PROD, mergeComponentVariables__DEV } from '../../src/mergeThemes'
 
diff --git a/packages/styles/test/mergeThemes/mergeSiteVariables-test.ts b/packages/styles/test/mergeThemes/mergeSiteVariables-test.ts
@@ -1,5 +1,6 @@
 import { withDebugId } from '@fluentui/styles'
 
+// These are not exported
 import * as debugEnabled from '../../src/debugEnabled'
 import { mergeSiteVariables__PROD, mergeSiteVariables__DEV } from '../../src/mergeThemes'
 
diff --git a/packages/styles/test/mergeThemes/mergeThemeVariables-test.ts b/packages/styles/test/mergeThemes/mergeThemeVariables-test.ts
@@ -1,6 +1,7 @@
 import { withDebugId } from '@fluentui/styles'
 import * as _ from 'lodash'
 
+// These are not exported
 import * as debugEnabled from '../../src/debugEnabled'
 import { mergeThemeVariables__PROD, mergeThemeVariables__DEV } from '../../src/mergeThemes'
 
diff --git a/packages/styles/test/mergeThemes/mergeThemes-test.ts b/packages/styles/test/mergeThemes/mergeThemes-test.ts
@@ -8,6 +8,7 @@ import {
   emptyTheme,
 } from '@fluentui/styles'
 
+// This is not exported
 import * as debugEnabled from '../../src/debugEnabled'
 
 const styleParam: ComponentStyleFunctionParam = {

Original file line number	Diff line number	Diff line change
`@@ -1,3 +1,4 @@`
	`1`	`+// This is not exported`
`1`	`2`	`import getKeyDownHandlers from '../../src/accessibility/getKeyDownHandlers'`
`2`	`3`	`// @ts-ignore`
`3`	`4`	`import * as keyboardKey from 'keyboard-key'`
Original file line number	Diff line number	Diff line change
`@@ -1,3 +1,4 @@`
	`1`	`+// This is not exported`
`1`	`2`	`import shouldHandleOnKeys from '../../src/accessibility/shouldHandleOnKeys'`
`2`	`3`
`3`	`4`	`const getEventArg = (`