@futdevpro/dynamo-eslint 1.15.25 → 1.15.28
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/.c8rc.json +26 -26
- package/.cursor/rules/__assistant_guide.mdc +30 -30
- package/.cursor/rules/__main.mdc +64 -64
- package/.cursor/rules/_ag_backend-structure.mdc +85 -85
- package/.cursor/rules/_ag_backend.mdc +16 -16
- package/.cursor/rules/_ag_debug.mdc +7 -7
- package/.cursor/rules/_ag_documentation_writing_rules.mdc +372 -372
- package/.cursor/rules/_ag_file-refactoring.mdc +113 -113
- package/.cursor/rules/_ag_fixes_rules.mdc +5 -5
- package/.cursor/rules/_ag_frontend-structure.mdc +86 -86
- package/.cursor/rules/_ag_frontend.mdc +39 -39
- package/.cursor/rules/_ag_import-rules.mdc +44 -44
- package/.cursor/rules/_ag_naming.mdc +115 -115
- package/.cursor/rules/_ag_running_commands.mdc +4 -4
- package/.cursor/rules/_ag_server-controller.mdc +5 -5
- package/.cursor/rules/_ag_should-be.mdc +6 -6
- package/.cursor/rules/_ag_swearing.mdc +47 -47
- package/.cursor/rules/ai_development_guide.md +60 -60
- package/.cursor/rules/ai_directives.md +114 -114
- package/.cursor/rules/cursor-rules.md +160 -160
- package/.cursor/rules/default-command.mdc +464 -464
- package/.cursor/rules/error_code_pattern.md +39 -39
- package/.cursor/rules/saved rule mcp server use.md +15 -15
- package/.dynamo/pipeline.cicd.config.json +184 -184
- package/.dynamo/version-bump.config.json +5 -5
- package/.eslintrc.json +4 -4
- package/.github/workflows/main.yml +436 -436
- package/.husky/pre-commit +2 -2
- package/.vscode/settings.json +11 -11
- package/README.md +319 -319
- package/__documentations/2026-04-18-ccap-review-integration.md +173 -173
- package/__documentations/2026-08-18-bfr-ctm-005-eslint9-validator-report.md +45 -0
- package/build/configs/base.d.ts +9 -9
- package/build/configs/base.js +150 -150
- package/build/configs/fsm.d.ts +1 -1
- package/build/configs/fsm.js +7 -7
- package/build/configs/ngx-package.d.ts +1 -1
- package/build/configs/ngx-package.js +7 -7
- package/build/configs/ngx.d.ts +4 -4
- package/build/configs/ngx.js +91 -91
- package/build/configs/nts-package.d.ts +1 -1
- package/build/configs/nts-package.js +17 -17
- package/build/configs/nts.d.ts +1 -1
- package/build/configs/nts.js +31 -31
- package/build/index.d.ts +1 -1
- package/build/index.js +2 -2
- package/build/plugin/index.d.ts +52 -52
- package/build/plugin/index.js +80 -80
- package/build/plugin/rules/angular/angular-for-track-logic.d.ts +7 -7
- package/build/plugin/rules/angular/angular-for-track-logic.js +81 -81
- package/build/plugin/rules/angular/effect-signal-loop.d.ts +9 -9
- package/build/plugin/rules/angular/effect-signal-loop.js +112 -112
- package/build/plugin/rules/angular/no-async-ngoninit.d.ts +8 -8
- package/build/plugin/rules/angular/no-async-ngoninit.js +42 -42
- package/build/plugin/rules/angular/no-component-direct-api.d.ts +9 -9
- package/build/plugin/rules/angular/no-component-direct-api.js +75 -75
- package/build/plugin/rules/angular/no-method-call-in-template.d.ts +8 -8
- package/build/plugin/rules/angular/no-method-call-in-template.js +79 -79
- package/build/plugin/rules/angular/no-ngfor-directive.d.ts +7 -7
- package/build/plugin/rules/angular/no-ngfor-directive.js +59 -59
- package/build/plugin/rules/angular/no-ngif-directive.d.ts +7 -7
- package/build/plugin/rules/angular/no-ngif-directive.js +59 -59
- package/build/plugin/rules/explicit-types.d.ts +3 -3
- package/build/plugin/rules/explicit-types.js +178 -178
- package/build/plugin/rules/import/import-order.d.ts +3 -3
- package/build/plugin/rules/import/import-order.js +154 -154
- package/build/plugin/rules/import/no-import-type.d.ts +3 -3
- package/build/plugin/rules/import/no-import-type.js +45 -45
- package/build/plugin/rules/import/no-js-import.d.ts +3 -3
- package/build/plugin/rules/import/no-js-import.js +40 -40
- package/build/plugin/rules/naming-patterns.d.ts +7 -7
- package/build/plugin/rules/naming-patterns.js +366 -366
- package/build/plugin/rules/no-getter-logic.d.ts +3 -3
- package/build/plugin/rules/no-getter-logic.js +194 -194
- package/build/plugin/rules/prefer-enum-over-string-union.d.ts +3 -3
- package/build/plugin/rules/prefer-enum-over-string-union.js +289 -289
- package/build/plugin/rules/prefer-interface-over-duplicate-types.d.ts +3 -3
- package/build/plugin/rules/prefer-interface-over-duplicate-types.js +264 -264
- package/build/plugin/rules/require-jsdoc-description.d.ts +3 -3
- package/build/plugin/rules/require-jsdoc-description.js +496 -496
- package/build/plugin/rules/require-test-description-prefix.d.ts +3 -3
- package/build/plugin/rules/require-test-description-prefix.js +135 -135
- package/build/plugin/rules/ts/data-model-extends.d.ts +8 -8
- package/build/plugin/rules/ts/data-model-extends.js +94 -94
- package/build/plugin/rules/ts/no-any-type.d.ts +8 -8
- package/build/plugin/rules/ts/no-any-type.js +36 -36
- package/build/plugin/rules/ts/no-as-cast.d.ts +8 -8
- package/build/plugin/rules/ts/no-as-cast.js +49 -49
- package/build/plugin/rules/ts/no-dynamic-imports.d.ts +8 -8
- package/build/plugin/rules/ts/no-dynamic-imports.js +50 -50
- package/build/plugin/rules/ts/no-native-browser-dialogs.d.ts +8 -8
- package/build/plugin/rules/ts/no-native-browser-dialogs.js +78 -78
- package/build/plugin/rules/ts/no-plain-function-export.d.ts +9 -9
- package/build/plugin/rules/ts/no-plain-function-export.js +69 -69
- package/build/plugin/rules/ts/one-export-per-file.d.ts +8 -8
- package/build/plugin/rules/ts/one-export-per-file.js +73 -73
- package/build/scripts/dynamo-fix.d.ts +26 -26
- package/build/scripts/dynamo-fix.js +146 -146
- package/build/scripts/eslintrc-audit.d.ts +26 -26
- package/build/scripts/eslintrc-audit.js +100 -100
- package/build/scripts/fix-return-types.d.ts +74 -74
- package/build/scripts/fix-return-types.js +226 -226
- package/build/scripts/validate-imports.d.ts +26 -26
- package/build/scripts/validate-imports.js +157 -146
- package/build/scripts/validate-imports.js.map +1 -1
- package/build/scripts/validate-naming.d.ts +26 -26
- package/build/scripts/validate-naming.js +157 -131
- package/build/scripts/validate-naming.js.map +1 -1
- package/eslint.config.js +15 -15
- package/package.json +105 -105
- package/samples/.vscode/settings.json +13 -13
- package/samples/base/.eslintrc.json +6 -6
- package/samples/base/eslint.config.js +3 -3
- package/samples/fsm/.eslintrc.json +4 -4
- package/samples/fsm/eslint.config.js +3 -3
- package/samples/ngx/.eslintrc.json +6 -6
- package/samples/ngx/eslint.config.js +3 -3
- package/samples/ngx-package/.eslintrc.json +4 -4
- package/samples/ngx-package/eslint.config.js +3 -3
- package/samples/nts/.eslintrc.json +6 -6
- package/samples/nts/eslint.config.js +3 -3
- package/samples/nts-package/.eslintrc.json +4 -4
- package/samples/nts-package/eslint.config.js +3 -3
- package/samples/package.json.example +26 -26
- package/samples/poc-sample.ts +38 -38
- package/samples/poc-violations.ts +54 -54
- package/scripts/run-coverage-tests.js +24 -24
- package/spec/support/helpers/ts-node-helper.js +9 -9
- package/spec/support/jasmine.coverage.json +23 -23
- package/spec/support/jasmine.json +24 -24
- package/src/configs/base.ts +157 -157
- package/src/configs/fsm.ts +10 -10
- package/src/configs/ngx-package.ts +10 -10
- package/src/configs/ngx.ts +96 -96
- package/src/configs/nts-package.ts +20 -20
- package/src/configs/nts.ts +34 -34
- package/src/index.ts +4 -4
- package/src/plugin/index.ts +86 -86
- package/src/plugin/rules/angular/angular-for-track-logic.spec.ts +44 -44
- package/src/plugin/rules/angular/angular-for-track-logic.ts +88 -88
- package/src/plugin/rules/angular/effect-signal-loop.spec.ts +89 -89
- package/src/plugin/rules/angular/effect-signal-loop.ts +125 -125
- package/src/plugin/rules/angular/no-async-ngoninit.spec.ts +50 -50
- package/src/plugin/rules/angular/no-async-ngoninit.ts +43 -43
- package/src/plugin/rules/angular/no-component-direct-api.spec.ts +78 -78
- package/src/plugin/rules/angular/no-component-direct-api.ts +82 -82
- package/src/plugin/rules/angular/no-method-call-in-template.spec.ts +63 -63
- package/src/plugin/rules/angular/no-method-call-in-template.ts +89 -89
- package/src/plugin/rules/angular/no-ngfor-directive.spec.ts +44 -44
- package/src/plugin/rules/angular/no-ngfor-directive.ts +62 -62
- package/src/plugin/rules/angular/no-ngif-directive.spec.ts +44 -44
- package/src/plugin/rules/angular/no-ngif-directive.ts +62 -62
- package/src/plugin/rules/explicit-types.spec.ts +190 -190
- package/src/plugin/rules/explicit-types.ts +186 -186
- package/src/plugin/rules/import/import-order.spec.ts +219 -219
- package/src/plugin/rules/import/import-order.ts +197 -197
- package/src/plugin/rules/import/no-import-type.spec.ts +69 -69
- package/src/plugin/rules/import/no-import-type.ts +46 -46
- package/src/plugin/rules/import/no-js-import.spec.ts +82 -82
- package/src/plugin/rules/import/no-js-import.ts +45 -45
- package/src/plugin/rules/naming-patterns.spec.ts +189 -189
- package/src/plugin/rules/naming-patterns.ts +426 -426
- package/src/plugin/rules/no-getter-logic.spec.ts +591 -591
- package/src/plugin/rules/no-getter-logic.ts +220 -220
- package/src/plugin/rules/prefer-enum-over-string-union.spec.ts +583 -583
- package/src/plugin/rules/prefer-enum-over-string-union.ts +333 -333
- package/src/plugin/rules/prefer-interface-over-duplicate-types.spec.ts +404 -404
- package/src/plugin/rules/prefer-interface-over-duplicate-types.ts +314 -314
- package/src/plugin/rules/require-jsdoc-description.spec.ts +779 -779
- package/src/plugin/rules/require-jsdoc-description.ts +572 -572
- package/src/plugin/rules/require-test-description-prefix.spec.ts +459 -459
- package/src/plugin/rules/require-test-description-prefix.ts +154 -154
- package/src/plugin/rules/ts/data-model-extends.spec.ts +84 -84
- package/src/plugin/rules/ts/data-model-extends.ts +100 -100
- package/src/plugin/rules/ts/no-any-type.spec.ts +28 -28
- package/src/plugin/rules/ts/no-any-type.ts +36 -36
- package/src/plugin/rules/ts/no-as-cast.spec.ts +51 -51
- package/src/plugin/rules/ts/no-as-cast.ts +51 -51
- package/src/plugin/rules/ts/no-dynamic-imports.spec.ts +54 -54
- package/src/plugin/rules/ts/no-dynamic-imports.ts +49 -49
- package/src/plugin/rules/ts/no-native-browser-dialogs.spec.ts +54 -54
- package/src/plugin/rules/ts/no-native-browser-dialogs.ts +84 -84
- package/src/plugin/rules/ts/no-plain-function-export.spec.ts +72 -72
- package/src/plugin/rules/ts/no-plain-function-export.ts +73 -73
- package/src/plugin/rules/ts/one-export-per-file.spec.ts +64 -64
- package/src/plugin/rules/ts/one-export-per-file.ts +77 -77
- package/src/scripts/dynamo-fix.ts +172 -172
- package/src/scripts/eslintrc-audit.ts +116 -116
- package/src/scripts/fix-return-types.spec.ts +65 -65
- package/src/scripts/fix-return-types.ts +288 -288
- package/src/scripts/validate-imports.ts +278 -261
- package/src/scripts/validate-naming.ts +204 -187
- package/src/scripts/validator-cli.integration.spec.ts +96 -0
- package/tsconfig.json +31 -31
|
@@ -1,190 +1,190 @@
|
|
|
1
|
-
import explicitTypesRule from './explicit-types';
|
|
2
|
-
|
|
3
|
-
describe('| explicit-types', () => {
|
|
4
|
-
it('| should be a valid ESLint rule', () => {
|
|
5
|
-
expect(explicitTypesRule.meta?.type).toBe('suggestion');
|
|
6
|
-
expect(explicitTypesRule.meta?.docs?.description).toContain('explicit type annotations');
|
|
7
|
-
});
|
|
8
|
-
|
|
9
|
-
it('| should have create function that returns visitor object', () => {
|
|
10
|
-
const mockContext = {
|
|
11
|
-
report: () => {},
|
|
12
|
-
} as any;
|
|
13
|
-
|
|
14
|
-
const result = explicitTypesRule.create(mockContext);
|
|
15
|
-
|
|
16
|
-
expect(typeof result).toBe('object');
|
|
17
|
-
expect(typeof result.FunctionDeclaration).toBe('function');
|
|
18
|
-
expect(typeof result.ArrowFunctionExpression).toBe('function');
|
|
19
|
-
expect(typeof result.VariableDeclarator).toBe('function');
|
|
20
|
-
expect(typeof result.PropertyDefinition).toBe('function');
|
|
21
|
-
});
|
|
22
|
-
|
|
23
|
-
it('| should report missing return types on function declarations', () => {
|
|
24
|
-
const mockContext = {
|
|
25
|
-
report: (options: any) => {
|
|
26
|
-
expect(options.messageId).toBe('missingReturnType');
|
|
27
|
-
expect(options.data.name).toBe('testFunction');
|
|
28
|
-
},
|
|
29
|
-
} as any;
|
|
30
|
-
|
|
31
|
-
const mockNode = {
|
|
32
|
-
type: 'FunctionDeclaration' as const,
|
|
33
|
-
id: { name: 'testFunction' },
|
|
34
|
-
returnType: null,
|
|
35
|
-
params: [],
|
|
36
|
-
} as any;
|
|
37
|
-
|
|
38
|
-
const rule = explicitTypesRule.create(mockContext);
|
|
39
|
-
rule.FunctionDeclaration(mockNode);
|
|
40
|
-
});
|
|
41
|
-
|
|
42
|
-
it('| should not report when function has return type', () => {
|
|
43
|
-
const mockContext = {
|
|
44
|
-
report: (options: any) => {
|
|
45
|
-
fail('Should not report when function has return type');
|
|
46
|
-
},
|
|
47
|
-
} as any;
|
|
48
|
-
|
|
49
|
-
const mockNode = {
|
|
50
|
-
type: 'FunctionDeclaration' as const,
|
|
51
|
-
id: { name: 'testFunction' },
|
|
52
|
-
returnType: { type: 'TSTypeAnnotation' },
|
|
53
|
-
params: [],
|
|
54
|
-
} as any;
|
|
55
|
-
|
|
56
|
-
const rule = explicitTypesRule.create(mockContext);
|
|
57
|
-
rule.FunctionDeclaration(mockNode);
|
|
58
|
-
});
|
|
59
|
-
|
|
60
|
-
it('| should report missing return types on arrow functions', () => {
|
|
61
|
-
const mockContext = {
|
|
62
|
-
report: (options: any) => {
|
|
63
|
-
expect(options.messageId).toBe('missingArrowReturnType');
|
|
64
|
-
},
|
|
65
|
-
} as any;
|
|
66
|
-
|
|
67
|
-
const mockNode = {
|
|
68
|
-
type: 'ArrowFunctionExpression' as const,
|
|
69
|
-
returnType: null,
|
|
70
|
-
params: [],
|
|
71
|
-
} as any;
|
|
72
|
-
|
|
73
|
-
const rule = explicitTypesRule.create(mockContext);
|
|
74
|
-
rule.ArrowFunctionExpression(mockNode);
|
|
75
|
-
});
|
|
76
|
-
|
|
77
|
-
it('| should report missing types on variable declarations', () => {
|
|
78
|
-
const mockContext = {
|
|
79
|
-
report: (options: any) => {
|
|
80
|
-
expect(options.messageId).toBe('missingVariableType');
|
|
81
|
-
expect(options.data.name).toBe('testVar');
|
|
82
|
-
},
|
|
83
|
-
} as any;
|
|
84
|
-
|
|
85
|
-
const mockNode = {
|
|
86
|
-
type: 'VariableDeclarator' as const,
|
|
87
|
-
id: { name: 'testVar', typeAnnotation: null },
|
|
88
|
-
init: null,
|
|
89
|
-
} as any;
|
|
90
|
-
|
|
91
|
-
const rule = explicitTypesRule.create(mockContext);
|
|
92
|
-
rule.VariableDeclarator(mockNode);
|
|
93
|
-
});
|
|
94
|
-
|
|
95
|
-
it('| should not report when variable has type annotation', () => {
|
|
96
|
-
const mockContext = {
|
|
97
|
-
report: (options: any) => {
|
|
98
|
-
fail('Should not report when variable has type annotation');
|
|
99
|
-
},
|
|
100
|
-
} as any;
|
|
101
|
-
|
|
102
|
-
const mockNode = {
|
|
103
|
-
type: 'VariableDeclarator' as const,
|
|
104
|
-
id: { name: 'testVar', typeAnnotation: { type: 'TSTypeAnnotation' } },
|
|
105
|
-
init: null,
|
|
106
|
-
} as any;
|
|
107
|
-
|
|
108
|
-
const rule = explicitTypesRule.create(mockContext);
|
|
109
|
-
rule.VariableDeclarator(mockNode);
|
|
110
|
-
});
|
|
111
|
-
|
|
112
|
-
it('| should not report when variable has typed initializer', () => {
|
|
113
|
-
const mockContext = {
|
|
114
|
-
report: (options: any) => {
|
|
115
|
-
fail('Should not report when variable has typed initializer');
|
|
116
|
-
},
|
|
117
|
-
} as any;
|
|
118
|
-
|
|
119
|
-
const mockNode = {
|
|
120
|
-
type: 'VariableDeclarator' as const,
|
|
121
|
-
id: { name: 'testVar', typeAnnotation: null },
|
|
122
|
-
init: { type: 'CallExpression' },
|
|
123
|
-
} as any;
|
|
124
|
-
|
|
125
|
-
const rule = explicitTypesRule.create(mockContext);
|
|
126
|
-
rule.VariableDeclarator(mockNode);
|
|
127
|
-
});
|
|
128
|
-
|
|
129
|
-
it('| should report missing parameter types', () => {
|
|
130
|
-
const mockContext = {
|
|
131
|
-
report: (options: any) => {
|
|
132
|
-
expect(options.messageId).toBe('missingParameterType');
|
|
133
|
-
expect(options.data.name).toBe('param');
|
|
134
|
-
},
|
|
135
|
-
} as any;
|
|
136
|
-
|
|
137
|
-
const mockNode = {
|
|
138
|
-
type: 'FunctionDeclaration' as const,
|
|
139
|
-
id: { name: 'testFunction' },
|
|
140
|
-
returnType: { type: 'TSTypeAnnotation' },
|
|
141
|
-
params: [
|
|
142
|
-
{ type: 'Identifier', name: 'param', typeAnnotation: null }
|
|
143
|
-
],
|
|
144
|
-
} as any;
|
|
145
|
-
|
|
146
|
-
const rule = explicitTypesRule.create(mockContext);
|
|
147
|
-
rule.FunctionDeclaration(mockNode);
|
|
148
|
-
});
|
|
149
|
-
|
|
150
|
-
it('| should report missing class property types', () => {
|
|
151
|
-
const mockContext = {
|
|
152
|
-
report: (options: any) => {
|
|
153
|
-
expect(options.messageId).toBe('missingPropertyType');
|
|
154
|
-
expect(options.data.name).toBe('property');
|
|
155
|
-
},
|
|
156
|
-
} as any;
|
|
157
|
-
|
|
158
|
-
const mockNode = {
|
|
159
|
-
type: 'PropertyDefinition' as const,
|
|
160
|
-
key: { name: 'property' },
|
|
161
|
-
typeAnnotation: null,
|
|
162
|
-
value: null,
|
|
163
|
-
} as any;
|
|
164
|
-
|
|
165
|
-
const rule = explicitTypesRule.create(mockContext);
|
|
166
|
-
rule.PropertyDefinition(mockNode);
|
|
167
|
-
});
|
|
168
|
-
|
|
169
|
-
it('| should report missing destructuring types', () => {
|
|
170
|
-
const mockContext = {
|
|
171
|
-
report: (options: any) => {
|
|
172
|
-
expect(options.messageId).toBe('missingDestructuringType');
|
|
173
|
-
},
|
|
174
|
-
} as any;
|
|
175
|
-
|
|
176
|
-
const objectPatternNode = {
|
|
177
|
-
type: 'ObjectPattern' as const,
|
|
178
|
-
typeAnnotation: null,
|
|
179
|
-
} as any;
|
|
180
|
-
|
|
181
|
-
const arrayPatternNode = {
|
|
182
|
-
type: 'ArrayPattern' as const,
|
|
183
|
-
typeAnnotation: null,
|
|
184
|
-
} as any;
|
|
185
|
-
|
|
186
|
-
const rule = explicitTypesRule.create(mockContext);
|
|
187
|
-
rule.ObjectPattern(objectPatternNode);
|
|
188
|
-
rule.ArrayPattern(arrayPatternNode);
|
|
189
|
-
});
|
|
190
|
-
});
|
|
1
|
+
import explicitTypesRule from './explicit-types';
|
|
2
|
+
|
|
3
|
+
describe('| explicit-types', () => {
|
|
4
|
+
it('| should be a valid ESLint rule', () => {
|
|
5
|
+
expect(explicitTypesRule.meta?.type).toBe('suggestion');
|
|
6
|
+
expect(explicitTypesRule.meta?.docs?.description).toContain('explicit type annotations');
|
|
7
|
+
});
|
|
8
|
+
|
|
9
|
+
it('| should have create function that returns visitor object', () => {
|
|
10
|
+
const mockContext = {
|
|
11
|
+
report: () => {},
|
|
12
|
+
} as any;
|
|
13
|
+
|
|
14
|
+
const result = explicitTypesRule.create(mockContext);
|
|
15
|
+
|
|
16
|
+
expect(typeof result).toBe('object');
|
|
17
|
+
expect(typeof result.FunctionDeclaration).toBe('function');
|
|
18
|
+
expect(typeof result.ArrowFunctionExpression).toBe('function');
|
|
19
|
+
expect(typeof result.VariableDeclarator).toBe('function');
|
|
20
|
+
expect(typeof result.PropertyDefinition).toBe('function');
|
|
21
|
+
});
|
|
22
|
+
|
|
23
|
+
it('| should report missing return types on function declarations', () => {
|
|
24
|
+
const mockContext = {
|
|
25
|
+
report: (options: any) => {
|
|
26
|
+
expect(options.messageId).toBe('missingReturnType');
|
|
27
|
+
expect(options.data.name).toBe('testFunction');
|
|
28
|
+
},
|
|
29
|
+
} as any;
|
|
30
|
+
|
|
31
|
+
const mockNode = {
|
|
32
|
+
type: 'FunctionDeclaration' as const,
|
|
33
|
+
id: { name: 'testFunction' },
|
|
34
|
+
returnType: null,
|
|
35
|
+
params: [],
|
|
36
|
+
} as any;
|
|
37
|
+
|
|
38
|
+
const rule = explicitTypesRule.create(mockContext);
|
|
39
|
+
rule.FunctionDeclaration(mockNode);
|
|
40
|
+
});
|
|
41
|
+
|
|
42
|
+
it('| should not report when function has return type', () => {
|
|
43
|
+
const mockContext = {
|
|
44
|
+
report: (options: any) => {
|
|
45
|
+
fail('Should not report when function has return type');
|
|
46
|
+
},
|
|
47
|
+
} as any;
|
|
48
|
+
|
|
49
|
+
const mockNode = {
|
|
50
|
+
type: 'FunctionDeclaration' as const,
|
|
51
|
+
id: { name: 'testFunction' },
|
|
52
|
+
returnType: { type: 'TSTypeAnnotation' },
|
|
53
|
+
params: [],
|
|
54
|
+
} as any;
|
|
55
|
+
|
|
56
|
+
const rule = explicitTypesRule.create(mockContext);
|
|
57
|
+
rule.FunctionDeclaration(mockNode);
|
|
58
|
+
});
|
|
59
|
+
|
|
60
|
+
it('| should report missing return types on arrow functions', () => {
|
|
61
|
+
const mockContext = {
|
|
62
|
+
report: (options: any) => {
|
|
63
|
+
expect(options.messageId).toBe('missingArrowReturnType');
|
|
64
|
+
},
|
|
65
|
+
} as any;
|
|
66
|
+
|
|
67
|
+
const mockNode = {
|
|
68
|
+
type: 'ArrowFunctionExpression' as const,
|
|
69
|
+
returnType: null,
|
|
70
|
+
params: [],
|
|
71
|
+
} as any;
|
|
72
|
+
|
|
73
|
+
const rule = explicitTypesRule.create(mockContext);
|
|
74
|
+
rule.ArrowFunctionExpression(mockNode);
|
|
75
|
+
});
|
|
76
|
+
|
|
77
|
+
it('| should report missing types on variable declarations', () => {
|
|
78
|
+
const mockContext = {
|
|
79
|
+
report: (options: any) => {
|
|
80
|
+
expect(options.messageId).toBe('missingVariableType');
|
|
81
|
+
expect(options.data.name).toBe('testVar');
|
|
82
|
+
},
|
|
83
|
+
} as any;
|
|
84
|
+
|
|
85
|
+
const mockNode = {
|
|
86
|
+
type: 'VariableDeclarator' as const,
|
|
87
|
+
id: { name: 'testVar', typeAnnotation: null },
|
|
88
|
+
init: null,
|
|
89
|
+
} as any;
|
|
90
|
+
|
|
91
|
+
const rule = explicitTypesRule.create(mockContext);
|
|
92
|
+
rule.VariableDeclarator(mockNode);
|
|
93
|
+
});
|
|
94
|
+
|
|
95
|
+
it('| should not report when variable has type annotation', () => {
|
|
96
|
+
const mockContext = {
|
|
97
|
+
report: (options: any) => {
|
|
98
|
+
fail('Should not report when variable has type annotation');
|
|
99
|
+
},
|
|
100
|
+
} as any;
|
|
101
|
+
|
|
102
|
+
const mockNode = {
|
|
103
|
+
type: 'VariableDeclarator' as const,
|
|
104
|
+
id: { name: 'testVar', typeAnnotation: { type: 'TSTypeAnnotation' } },
|
|
105
|
+
init: null,
|
|
106
|
+
} as any;
|
|
107
|
+
|
|
108
|
+
const rule = explicitTypesRule.create(mockContext);
|
|
109
|
+
rule.VariableDeclarator(mockNode);
|
|
110
|
+
});
|
|
111
|
+
|
|
112
|
+
it('| should not report when variable has typed initializer', () => {
|
|
113
|
+
const mockContext = {
|
|
114
|
+
report: (options: any) => {
|
|
115
|
+
fail('Should not report when variable has typed initializer');
|
|
116
|
+
},
|
|
117
|
+
} as any;
|
|
118
|
+
|
|
119
|
+
const mockNode = {
|
|
120
|
+
type: 'VariableDeclarator' as const,
|
|
121
|
+
id: { name: 'testVar', typeAnnotation: null },
|
|
122
|
+
init: { type: 'CallExpression' },
|
|
123
|
+
} as any;
|
|
124
|
+
|
|
125
|
+
const rule = explicitTypesRule.create(mockContext);
|
|
126
|
+
rule.VariableDeclarator(mockNode);
|
|
127
|
+
});
|
|
128
|
+
|
|
129
|
+
it('| should report missing parameter types', () => {
|
|
130
|
+
const mockContext = {
|
|
131
|
+
report: (options: any) => {
|
|
132
|
+
expect(options.messageId).toBe('missingParameterType');
|
|
133
|
+
expect(options.data.name).toBe('param');
|
|
134
|
+
},
|
|
135
|
+
} as any;
|
|
136
|
+
|
|
137
|
+
const mockNode = {
|
|
138
|
+
type: 'FunctionDeclaration' as const,
|
|
139
|
+
id: { name: 'testFunction' },
|
|
140
|
+
returnType: { type: 'TSTypeAnnotation' },
|
|
141
|
+
params: [
|
|
142
|
+
{ type: 'Identifier', name: 'param', typeAnnotation: null }
|
|
143
|
+
],
|
|
144
|
+
} as any;
|
|
145
|
+
|
|
146
|
+
const rule = explicitTypesRule.create(mockContext);
|
|
147
|
+
rule.FunctionDeclaration(mockNode);
|
|
148
|
+
});
|
|
149
|
+
|
|
150
|
+
it('| should report missing class property types', () => {
|
|
151
|
+
const mockContext = {
|
|
152
|
+
report: (options: any) => {
|
|
153
|
+
expect(options.messageId).toBe('missingPropertyType');
|
|
154
|
+
expect(options.data.name).toBe('property');
|
|
155
|
+
},
|
|
156
|
+
} as any;
|
|
157
|
+
|
|
158
|
+
const mockNode = {
|
|
159
|
+
type: 'PropertyDefinition' as const,
|
|
160
|
+
key: { name: 'property' },
|
|
161
|
+
typeAnnotation: null,
|
|
162
|
+
value: null,
|
|
163
|
+
} as any;
|
|
164
|
+
|
|
165
|
+
const rule = explicitTypesRule.create(mockContext);
|
|
166
|
+
rule.PropertyDefinition(mockNode);
|
|
167
|
+
});
|
|
168
|
+
|
|
169
|
+
it('| should report missing destructuring types', () => {
|
|
170
|
+
const mockContext = {
|
|
171
|
+
report: (options: any) => {
|
|
172
|
+
expect(options.messageId).toBe('missingDestructuringType');
|
|
173
|
+
},
|
|
174
|
+
} as any;
|
|
175
|
+
|
|
176
|
+
const objectPatternNode = {
|
|
177
|
+
type: 'ObjectPattern' as const,
|
|
178
|
+
typeAnnotation: null,
|
|
179
|
+
} as any;
|
|
180
|
+
|
|
181
|
+
const arrayPatternNode = {
|
|
182
|
+
type: 'ArrayPattern' as const,
|
|
183
|
+
typeAnnotation: null,
|
|
184
|
+
} as any;
|
|
185
|
+
|
|
186
|
+
const rule = explicitTypesRule.create(mockContext);
|
|
187
|
+
rule.ObjectPattern(objectPatternNode);
|
|
188
|
+
rule.ArrayPattern(arrayPatternNode);
|
|
189
|
+
});
|
|
190
|
+
});
|