Skip to content

Commit 8f740fe

Browse files
committed
🐛 fix: generate common component interpolated class name
1 parent 789e3b9 commit 8f740fe

File tree

2 files changed

+22
-8
lines changed

2 files changed

+22
-8
lines changed

src/commands/generate/component/common/common.test.ts

Lines changed: 16 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
1-
import { filesystem } from 'gluegun';
1+
import { filesystem, strings } from 'gluegun';
2+
23
import { runNgxdCLI } from '../../../../utils/cli-test-setup';
34

45
describe('Commands: [Generate] => [Component] => [Common]', () => {
@@ -58,7 +59,20 @@ describe('Commands: [Generate] => [Component] => [Common]', () => {
5859
filesystem.remove(`${name}`);
5960
});
6061

61-
test('should not contain ngOnInit on import statment', async () => {
62+
test('should properly interpolate component name on spec file', async () => {
63+
const name = 'sample-style-template-url';
64+
await runNgxdCLI(`g c c ${name}`);
65+
66+
const ts = filesystem.read(`${name}/${name}.component.spec.ts`);
67+
68+
const pascalCaseName = strings.pascalCase(name);
69+
70+
expect(ts).toContain(`describe('${pascalCaseName}Component', () => {`);
71+
72+
filesystem.remove(`${name}`);
73+
});
74+
75+
test('should not contain ngOnInit on import statement', async () => {
6276
const name = 'sample-style-template-url';
6377
await runNgxdCLI(`g c c ${name}`);
6478

src/templates/component.template.spec.ts.ejs

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,19 +1,19 @@
11
import { ComponentFixture, TestBed } from '@angular/core/testing';
2-
import { <%= pascalCase(props.name) %> } from './<%= kebabCase(props.name) %>.component';
2+
import { <%= pascalCase(props.name) %>Component } from './<%= kebabCase(props.name) %>.component';
33

4-
describe('<%= pascalCase(props.name) %>', () => {
5-
let component: <%= pascalCase(props.name) %>;
6-
let fixture: ComponentFixture<<%= pascalCase(props.name) %>>;
4+
describe('<%= pascalCase(props.name) %>Component', () => {
5+
let component: <%= pascalCase(props.name) %>Component;
6+
let fixture: ComponentFixture<<%= pascalCase(props.name) %>Component>;
77

88
beforeEach(async () => {
99
await TestBed.configureTestingModule({
10-
declarations: [ <%= pascalCase(props.name) %> ]
10+
declarations: [ <%= pascalCase(props.name) %>Component ]
1111
})
1212
.compileComponents();
1313
});
1414

1515
beforeEach(() => {
16-
fixture = TestBed.createComponent(<%= pascalCase(props.name) %>);
16+
fixture = TestBed.createComponent(<%= pascalCase(props.name) %>Component);
1717
component = fixture.componentInstance;
1818
fixture.detectChanges();
1919
});

0 commit comments

Comments
 (0)