From 8553200661f4b98bac1849f179016225b218b460 Mon Sep 17 00:00:00 2001 From: pangratz Date: Sun, 14 Jun 2015 02:06:19 +0200 Subject: [PATCH] [CLEANUP beta] Fix deprecation warnings - register store as Ember.Service in the tests - remove deprecated Ember.View from tests - use store.query instead of deprecated store.findQuery - use store.findAll(type) instead of deprecated store.find(type) - use Object.keys instead of deprecated Ember.keys in tests/index.html --- .../tests/integration/adapter/json-api-adapter-test.js | 2 +- packages/ember-data/tests/integration/application-test.js | 7 ++----- .../ember-data/tests/integration/debug-adapter-test.js | 2 +- .../ember-data/tests/unit/store/adapter-interop-test.js | 2 +- tests/index.html | 2 +- 5 files changed, 6 insertions(+), 9 deletions(-) diff --git a/packages/ember-data/tests/integration/adapter/json-api-adapter-test.js b/packages/ember-data/tests/integration/adapter/json-api-adapter-test.js index 7eb834467b0..5df9574c75f 100644 --- a/packages/ember-data/tests/integration/adapter/json-api-adapter-test.js +++ b/packages/ember-data/tests/integration/adapter/json-api-adapter-test.js @@ -204,7 +204,7 @@ test('find many records', function() { }]); run(function() { - store.find('post', { filter: { id: 1 } }).then(function(posts) { + store.query('post', { filter: { id: 1 } }).then(function(posts) { equal(passedUrl[0], '/posts'); deepEqual(passedHash[0], { data: { filter: { id: 1 } } }); diff --git a/packages/ember-data/tests/integration/application-test.js b/packages/ember-data/tests/integration/application-test.js index 3e08370ecfc..3c2bfd7f974 100644 --- a/packages/ember-data/tests/integration/application-test.js +++ b/packages/ember-data/tests/integration/application-test.js @@ -1,7 +1,6 @@ var run = Ember.run; var Application = Ember.Application; var Controller = Ember.Controller; -var View = Ember.View; var Store = DS.Store; var Namespace = Ember.Namespace; @@ -24,9 +23,8 @@ module("integration/application - Injecting a Custom Store", { setup: function() { run(function() { app = Application.create({ - ApplicationStore: Store.extend({ isCustom: true }), + StoreService: Store.extend({ isCustom: true }), FooController: Controller.extend(), - ApplicationView: View.extend(), BazController: {}, ApplicationController: Controller.extend(), rootElement: '#qunit-fixture' @@ -83,7 +81,6 @@ module("integration/application - Injecting the Default Store", { run(function() { app = Application.create({ FooController: Controller.extend(), - ApplicationView: View.extend(), BazController: {}, ApplicationController: Controller.extend() }); @@ -120,7 +117,7 @@ if (Ember.inject && Ember.inject.service) { setup: function() { run(function() { app = Application.create({ - DoodleService: Ember.Object.extend({ store: Ember.inject.service() }) + DoodleService: Ember.Service.extend({ store: Ember.inject.service() }) }); }); diff --git a/packages/ember-data/tests/integration/debug-adapter-test.js b/packages/ember-data/tests/integration/debug-adapter-test.js index 17322cc66cf..3d7034c5399 100644 --- a/packages/ember-data/tests/integration/debug-adapter-test.js +++ b/packages/ember-data/tests/integration/debug-adapter-test.js @@ -8,7 +8,7 @@ module("DS.DebugAdapter", { App = Ember.Application.create(); App.toString = function() { return 'App'; }; - App.ApplicationStore = DS.Store.extend({ + App.StoreService = DS.Store.extend({ adapter: DS.Adapter.extend() }); diff --git a/packages/ember-data/tests/unit/store/adapter-interop-test.js b/packages/ember-data/tests/unit/store/adapter-interop-test.js index ec49ecad0ba..4d60cc44ea1 100644 --- a/packages/ember-data/tests/unit/store/adapter-interop-test.js +++ b/packages/ember-data/tests/unit/store/adapter-interop-test.js @@ -1034,7 +1034,7 @@ test("store should not reload record array when shouldReloadAll returns false", }); run(function() { - store.find('person'); + store.findAll('person'); }); }); diff --git a/tests/index.html b/tests/index.html index 14a25a44dd8..848cfed509b 100644 --- a/tests/index.html +++ b/tests/index.html @@ -62,7 +62,7 @@ var skipPackageRegexp = new RegExp('^('+skipPackages.join('|')+')/'); - var keys = Ember.keys(requirejs.entries); + var keys = Object.keys(requirejs.entries); for (var i = 0, length = keys.length; i < length; i++) { var moduleName = keys[i];