diff --git a/packages/-ember-data/tests/integration/relationships/one-to-many-test.js b/packages/-ember-data/tests/integration/relationships/one-to-many-test.js index 254947440af..7e1212e3cc3 100644 --- a/packages/-ember-data/tests/integration/relationships/one-to-many-test.js +++ b/packages/-ember-data/tests/integration/relationships/one-to-many-test.js @@ -1,9 +1,10 @@ import { get } from '@ember/object'; import { run } from '@ember/runloop'; -import { module, test } from 'qunit'; +import { module, test, todo } from 'qunit'; import { resolve } from 'rsvp'; +import { gte } from 'ember-compatibility-helpers'; import { setupTest } from 'ember-qunit'; import Adapter from '@ember-data/adapter'; @@ -1566,39 +1567,47 @@ module('integration/relationships/one_to_many_test - OneToMany relationships', f assert.equal(account.get('user'), null, 'Account does not have the user anymore'); }); - test('createRecord updates inverse record array which has observers', function(assert) { - let store = this.owner.lookup('service:store'); - let adapter = store.adapterFor('application'); - - adapter.findAll = () => { - return { - data: [ - { - id: '2', - type: 'user', - attributes: { - name: 'Stanley', + if (gte('3.16.0') && !gte('3.17.0')) { + // known failure in 3.16.x due to upstream glimmer VM issue + // https://github.com/glimmerjs/glimmer-vm/pull/1010 + todo('createRecord updates inverse record array which has observers', function(assert) { + assert.ok(false, 'known failure in ember-source@^3.16.0'); + }); + } else { + test('createRecord updates inverse record array which has observers', function(assert) { + let store = this.owner.lookup('service:store'); + let adapter = store.adapterFor('application'); + + adapter.findAll = () => { + return { + data: [ + { + id: '2', + type: 'user', + attributes: { + name: 'Stanley', + }, }, - }, - ], + ], + }; }; - }; - return store.findAll('user').then(users => { - assert.equal(users.get('length'), 1, 'Exactly 1 user'); + return store.findAll('user').then(users => { + assert.equal(users.get('length'), 1, 'Exactly 1 user'); - let user = users.get('firstObject'); - assert.equal(user.get('messages.length'), 0, 'Record array is initially empty'); + let user = users.get('firstObject'); + assert.equal(user.get('messages.length'), 0, 'Record array is initially empty'); - // set up an observer - user.addObserver('messages.@each.title', () => {}); - user.get('messages.firstObject'); + // set up an observer + user.addObserver('messages.@each.title', () => {}); + user.get('messages.firstObject'); - let message = store.createRecord('message', { user, title: 'EmberFest was great' }); - assert.equal(user.get('messages.length'), 1, 'The message is added to the record array'); + let message = store.createRecord('message', { user, title: 'EmberFest was great' }); + assert.equal(user.get('messages.length'), 1, 'The message is added to the record array'); - let messageFromArray = user.get('messages.firstObject'); - assert.ok(message === messageFromArray, 'Only one message record instance should be created'); + let messageFromArray = user.get('messages.firstObject'); + assert.ok(message === messageFromArray, 'Only one message record instance should be created'); + }); }); - }); + } });