Skip to content

Commit

Permalink
Disable known failure for 3.16.x
Browse files Browse the repository at this point in the history
Test is automatically re-enabled for 3.17.x.  We can re-enable it once
the upstream failure (glimmerjs/glimmer-vm#1010) is resolved and into
ember-source.
  • Loading branch information
hjdivad committed Jan 24, 2020
1 parent 98f41fd commit d4eef8f
Showing 1 changed file with 37 additions and 28 deletions.
Original file line number Diff line number Diff line change
@@ -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';
Expand Down Expand Up @@ -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');
});
});
});
}
});

0 comments on commit d4eef8f

Please sign in to comment.