Skip to content

Commit 6ef6c86

Browse files
author
Justin Shih
committed
chore(amplify-util-uibuilder): remove codegen-ui usage
1 parent 8b5d433 commit 6ef6c86

13 files changed

+4850
-127
lines changed

packages/amplify-util-uibuilder/package.json

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -17,8 +17,7 @@
1717
"@aws-amplify/amplify-category-api": "^5.6.2",
1818
"@aws-amplify/amplify-cli-core": "4.2.5",
1919
"@aws-amplify/amplify-prompts": "2.8.2",
20-
"@aws-amplify/codegen-ui": "2.14.2",
21-
"@aws-amplify/codegen-ui-react": "2.14.2",
20+
"@aws-amplify/datastore": "^4.7.5",
2221
"amplify-codegen": "^4.4.0",
2322
"aws-sdk": "^2.1426.0",
2423
"fs-extra": "^8.1.0",
@@ -32,6 +31,7 @@
3231
"@types/jest": "^29.5.1",
3332
"@types/semver": "^7.1.0",
3433
"@types/tiny-async-pool": "^2.0.0",
34+
"semver": "^7.5.4",
3535
"ts-jest": "^29.1.0"
3636
},
3737
"jest": {
Lines changed: 37 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,37 @@
1+
import semver from 'semver';
2+
import { ReactRequiredDependencyProvider } from '../commands/utils/codegen-ui-dependency-provider';
3+
4+
describe('ReactStudioDependencyProvider', () => {
5+
const requiredDependencies = new ReactRequiredDependencyProvider().getRequiredDependencies(false);
6+
const requiredDependenciesWithStorageManager = new ReactRequiredDependencyProvider().getRequiredDependencies(true);
7+
8+
describe('getRequiredDependencies', () => {
9+
it('has required dependencies', () => {
10+
expect(requiredDependencies.length).toBeGreaterThan(0);
11+
});
12+
13+
it('includes ui-react', () => {
14+
expect(requiredDependencies.filter((dep) => dep.dependencyName === '@aws-amplify/ui-react')).toBeTruthy();
15+
});
16+
17+
it('includes all valid semver values', () => {
18+
requiredDependencies.forEach((dep) => {
19+
expect(semver.valid(dep.supportedSemVerPattern)).toBeDefined();
20+
});
21+
});
22+
23+
it('includes reasons on all dependencies', () => {
24+
requiredDependencies.forEach((dep) => {
25+
expect(dep.reason.length).toBeGreaterThan(0);
26+
});
27+
});
28+
29+
it('does not include ui-react-storage if user does not use StorageManager', () => {
30+
expect(requiredDependencies.filter((dep) => dep.dependencyName !== '@aws-amplify/ui-react-storage')).toBeTruthy();
31+
});
32+
33+
it('includes ui-react-storage if user is using StorageManager', () => {
34+
expect(requiredDependenciesWithStorageManager.filter((dep) => dep.dependencyName === '@aws-amplify/ui-react-storage')).toBeTruthy();
35+
});
36+
});
37+
});

0 commit comments

Comments
 (0)