Skip to content

Commit

Permalink
fix inter-depedentant package imports in test modules
Browse files Browse the repository at this point in the history
Why?: this way no need to build to run tests while changing and refactoring
  • Loading branch information
p10ns11y committed Dec 5, 2024
1 parent 1e91573 commit cb22e48
Show file tree
Hide file tree
Showing 4 changed files with 9 additions and 7 deletions.
4 changes: 2 additions & 2 deletions coverage/coverage-summary.json
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
{"total": {"lines":{"total":199,"covered":199,"skipped":0,"pct":100},"statements":{"total":199,"covered":199,"skipped":0,"pct":100},"functions":{"total":8,"covered":8,"skipped":0,"pct":100},"branches":{"total":84,"covered":77,"skipped":0,"pct":91.66},"branchesTrue":{"total":0,"covered":0,"skipped":0,"pct":100}}
{"total": {"lines":{"total":199,"covered":199,"skipped":0,"pct":100},"statements":{"total":199,"covered":199,"skipped":0,"pct":100},"functions":{"total":8,"covered":8,"skipped":0,"pct":100},"branches":{"total":86,"covered":79,"skipped":0,"pct":91.86},"branchesTrue":{"total":0,"covered":0,"skipped":0,"pct":100}}
,"/Users/peram/code/adaptate/packages/core/src/index.ts": {"lines":{"total":84,"covered":84,"skipped":0,"pct":100},"functions":{"total":3,"covered":3,"skipped":0,"pct":100},"statements":{"total":84,"covered":84,"skipped":0,"pct":100},"branches":{"total":38,"covered":38,"skipped":0,"pct":100}}
,"/Users/peram/code/adaptate/packages/utils/src/load-yaml.ts": {"lines":{"total":14,"covered":14,"skipped":0,"pct":100},"functions":{"total":1,"covered":1,"skipped":0,"pct":100},"statements":{"total":14,"covered":14,"skipped":0,"pct":100},"branches":{"total":1,"covered":1,"skipped":0,"pct":100}}
,"/Users/peram/code/adaptate/packages/utils/src/openapi.ts": {"lines":{"total":101,"covered":101,"skipped":0,"pct":100},"functions":{"total":4,"covered":4,"skipped":0,"pct":100},"statements":{"total":101,"covered":101,"skipped":0,"pct":100},"branches":{"total":45,"covered":38,"skipped":0,"pct":84.44}}
,"/Users/peram/code/adaptate/packages/utils/src/openapi.ts": {"lines":{"total":101,"covered":101,"skipped":0,"pct":100},"functions":{"total":4,"covered":4,"skipped":0,"pct":100},"statements":{"total":101,"covered":101,"skipped":0,"pct":100},"branches":{"total":47,"covered":40,"skipped":0,"pct":85.1}}
}
3 changes: 3 additions & 0 deletions packages/core/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -35,6 +35,9 @@
"package.json",
"README.md"
],
"imports": {
"#utils/*": "../utils/src/*"
},
"exports": {
".": {
"types": "./src/index.d.ts",
Expand Down
6 changes: 2 additions & 4 deletions packages/core/src/__tests__/index.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -2,13 +2,11 @@ import { describe, it, expect } from 'vitest';

import { z } from 'zod';

// NOTE to Self: In test it better import directly from source
// It is easy to forget building the dependencies
// Introduce special import map `#adaptate/utils/openapi`
import {
getDereferencedOpenAPIDocument,
openAPISchemaToZod,
} from '@adaptate/utils/openapi';
} from '#utils/openapi';

import { transformSchema, applyConditionalRequirements } from '../';

describe('transformSchema', () => {
Expand Down
3 changes: 2 additions & 1 deletion tsconfig.json
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,8 @@
"paths": {
"@adaptate/core": ["./packages/core/src"],
"@adaptate/utils": ["./packages/utils/src"],
"@adaptate/utils/*": ["./packages/utils/src/*"]
"@adaptate/utils/*": ["./packages/utils/src/*"],
"#utils/*": ["./packages/utils/src/*"]
}
},
"include": ["packages", "packages/core/src/**/*.ts", "tests/**/*.ts"],
Expand Down

0 comments on commit cb22e48

Please sign in to comment.