Skip to content

Commit c8044d0

Browse files
committed
Add some test for determinePropertyType
1 parent d8eb285 commit c8044d0

File tree

1 file changed

+28
-0
lines changed

1 file changed

+28
-0
lines changed

tests/lib/utils/property-order-test.js

Lines changed: 28 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -62,6 +62,20 @@ describe('determinePropertyType', () => {
6262
).toStrictEqual('service');
6363
});
6464

65+
it('should determine controller-type props with full import', () => {
66+
const context = new FauxContext(
67+
`import Ember from 'ember';
68+
export default Controller.extend({
69+
application: Ember.inject.controller(),
70+
});`
71+
);
72+
const importEmberName = context.ast.body[0].specifiers[0].local.name;
73+
const node = context.ast.body[1].declaration.arguments[0].properties[0];
74+
expect(
75+
propertyOrder.determinePropertyType(node, 'controller', [], importEmberName)
76+
).toStrictEqual('controller');
77+
});
78+
6579
it('should determine controller-type props', () => {
6680
const context = new FauxContext(
6781
`export default Controller.extend({
@@ -136,6 +150,20 @@ describe('determinePropertyType', () => {
136150
expect(propertyOrder.determinePropertyType(node, 'model')).toStrictEqual('relationship');
137151
});
138152

153+
it('should determine observer-type props with full import', () => {
154+
const context = new FauxContext(
155+
`import Ember from 'ember';
156+
export default Controller.extend({
157+
someObvs: Ember.observer(),
158+
});`
159+
);
160+
const importEmberName = context.ast.body[0].specifiers[0].local.name;
161+
const node = context.ast.body[1].declaration.arguments[0].properties[0];
162+
expect(
163+
propertyOrder.determinePropertyType(node, 'controller', [], importEmberName)
164+
).toStrictEqual('observer');
165+
});
166+
139167
it('should determine observer-type props', () => {
140168
const context = new FauxContext(
141169
`export default Controller.extend({

0 commit comments

Comments
 (0)