Add path aliases and test against dist

Author: aryaemami59

.github/workflows/test.yml

@@ -25,9 +25,6 @@ jobs:
       - name: Install dependencies
         run: yarn install
 
-      - name: Run test suite
-        run: yarn test
-
       - name: Collect coverage
         run: yarn coverage
 
@@ -42,6 +39,42 @@ jobs:
           name: package
           path: ./package.tgz
 
+  test-dist:
+    name: Test against dist
+    needs: [build]
+    runs-on: ubuntu-latest
+    strategy:
+      fail-fast: false
+      matrix:
+        node: ['20.x']
+    steps:
+      - name: Checkout repo
+        uses: actions/checkout@v4
+
+      - name: Use node ${{ matrix.node }}
+        uses: actions/setup-node@v4
+        with:
+          node-version: ${{ matrix.node }}
+          cache: 'yarn'
+
+      - name: Install deps
+        run: yarn install
+
+      - uses: actions/download-artifact@v4
+        with:
+          name: package
+          path: .
+
+      - run: ls -lah
+
+      - name: Install build artifact
+        run: yarn add ./package.tgz
+
+      - run: sed -i -e /@remap-prod-remove-line/d ./tsconfig.base.json ./vitest.config.mts
+
+      - name: Run tests, against dist
+        run: yarn test
+
   test-types:
     name: Test Types with TypeScript ${{ matrix.ts }}
 

test/typetests/connect-mapstate-mapdispatch.test-d.tsx

@@ -3,8 +3,8 @@ import type { ActionCreator, Dispatch } from 'redux'
 import type {
   MapDispatchToProps,
   ReactReduxContextValue,
-} from '../../src/index'
-import { connect } from '../../src/index'
+} from 'react-redux'
+import { connect } from 'react-redux'
 
 // Test cases written in a way to isolate types and variables and verify the
 // output of `connect` to make sure the signature is what is expected

test/typetests/connect-options-and-issues.test-d.tsx

@@ -1,15 +1,15 @@
 import PropTypes from 'prop-types'
 import React from 'react'
-import type { ActionCreator, AnyAction, Dispatch, Reducer, Store } from 'redux'
-import { createStore } from 'redux'
 import type {
   Connect,
   ConnectedProps,
   DispatchProp,
   MapStateToProps,
   ReactReduxContextValue,
-} from '../../src/index'
-import { Provider, ReactReduxContext, connect } from '../../src/index'
+} from 'react-redux'
+import { Provider, ReactReduxContext, connect } from 'react-redux'
+import type { ActionCreator, AnyAction, Dispatch, Reducer, Store } from 'redux'
+import { createStore } from 'redux'
 
 // Test cases written in a way to isolate types and variables and verify the
 // output of `connect` to make sure the signature is what is expected

test/typetests/hooks.test-d.tsx

@@ -6,7 +6,7 @@ import type {
   Selector,
   TypedUseSelectorHook,
   UseSelector,
-} from '../../src/index'
+} from 'react-redux'
 import {
   createDispatchHook,
   createSelectorHook,
@@ -15,7 +15,7 @@ import {
   useDispatch,
   useSelector,
   useStore,
-} from '../../src/index'
+} from 'react-redux'
 import type { AppDispatch, RootState } from './counterApp'
 import { incrementAsync } from './counterApp'
 

test/typetests/hooks.withTypes.test-d.tsx

@@ -1,4 +1,4 @@
-import { useDispatch, useSelector, useStore } from '../../src/index'
+import { useDispatch, useSelector, useStore } from 'react-redux'
 import type { AppDispatch, AppStore, RootState } from './counterApp'
 import { incrementAsync } from './counterApp'
 

test/typetests/provider.test-d.tsx

@@ -1,5 +1,5 @@
+import { Provider } from 'react-redux'
 import type { Store } from 'redux'
-import { Provider } from '../../src'
 
 declare const store: Store<{ foo: string }>
 

test/typetests/react-redux-types.test-d.tsx

@@ -3,12 +3,8 @@ import { bindActionCreators } from '@reduxjs/toolkit'
 import type { ReactElement } from 'react'
 import React, { Component } from 'react'
 import { createRoot } from 'react-dom/client'
-import type {
-  ConnectedProps,
-  DispatchProp,
-  MapStateToProps,
-} from '../../src/index'
-import { Provider, connect } from '../../src/index'
+import type { ConnectedProps, DispatchProp, MapStateToProps } from 'react-redux'
+import { Provider, connect } from 'react-redux'
 import type { CounterState } from './counterApp'
 import { increment } from './counterApp'
 

tsconfig.base.json

@@ -15,6 +15,10 @@
     "noFallthroughCasesInSwitch": true,
     "noUnusedLocals": false,
     "noUnusedParameters": false,
+    "paths": {
+      "@internal/*": ["./src/*"],
+      "react-redux": ["./src/index.ts"] // @remap-prod-remove-line
+    },
     "resolveJsonModule": true,
     "skipLibCheck": true,
     "strict": true,

vitest.config.mts

@@ -12,8 +12,9 @@ export default defineConfig({
     setupFiles: ['test/setup.ts'],
     globals: true,
     alias: {
+      'react-redux': new URL('src/index.ts', import.meta.url).pathname, // @remap-prod-remove-line
       // this mapping is disabled as we want `dist` imports in the tests only to be used for "type-only" imports which don't play a role for jest
-      '@internal': path.join(__dirname, 'src'),
+      '@internal': new URL('src', import.meta.url).pathname,
     },
   },
 })