Add some test for determinePropertyType

ember-cli · May 12, 2021 · c8044d0 · c8044d0
1 parent d8eb285
commit c8044d0
Showing 1 changed file with 28 additions and 0 deletions.
diff --git a/tests/lib/utils/property-order-test.js b/tests/lib/utils/property-order-test.js
@@ -62,6 +62,20 @@ describe('determinePropertyType', () => {
  ).toStrictEqual('service');
  });
 
+ it('should determine controller-type props with full import', () => {
+ const context = new FauxContext(
+ `import Ember from 'ember';
+ export default Controller.extend({
+ application: Ember.inject.controller(),
+ });`
+ );
+ const importEmberName = context.ast.body[0].specifiers[0].local.name;
+ const node = context.ast.body[1].declaration.arguments[0].properties[0];
+ expect(
+ propertyOrder.determinePropertyType(node, 'controller', [], importEmberName)
+ ).toStrictEqual('controller');
+ });
+
  it('should determine controller-type props', () => {
  const context = new FauxContext(
  `export default Controller.extend({
@@ -136,6 +150,20 @@ describe('determinePropertyType', () => {
  expect(propertyOrder.determinePropertyType(node, 'model')).toStrictEqual('relationship');
  });
 
+ it('should determine observer-type props with full import', () => {
+ const context = new FauxContext(
+ `import Ember from 'ember';
+ export default Controller.extend({
+ someObvs: Ember.observer(),
+ });`
+ );
+ const importEmberName = context.ast.body[0].specifiers[0].local.name;
+ const node = context.ast.body[1].declaration.arguments[0].properties[0];
+ expect(
+ propertyOrder.determinePropertyType(node, 'controller', [], importEmberName)
+ ).toStrictEqual('observer');
+ });
+
  it('should determine observer-type props', () => {
  const context = new FauxContext(
  `export default Controller.extend({