[TECH] Mise à jour de ember-mocha dans Pix App.

[lisequesnel]

🦄 Problème

Pix avait quelques versions de retard sur ember-mocha, avec des changements importants notamment sur l'utilisation d'une nouvelle API : https://github.com/emberjs/ember-mocha.

🤖 Solution

Cette PR est la troisième et dernière partie pour utiliser la nouvelle API d'ember-mocha. Elle :

• supprime les usages dépréciés
• met à jour ember-mocha et ses dépendances : ember-cli-mirage

Plus techniquement, on avait avant pour les tests d'acceptance :

import { describe, it, beforeEach, afterEach } from 'mocha';
import { expect } from 'chai';
import startApp from 'app/tests/helpers/start-app';
import destroyApp from 'app/tests/helpers/destroy-app';

describe('basic acceptance test', function() {
  let application;

  beforeEach(function() {
    application = startApp();
  });

  afterEach(function() {
    destroyApp(application);
  });

  it('can visit /', function() {
    visit('/');

    return andThen(() => {
      expect(currentURL()).to.equal('/');
    });
  });
});

Maintenant :

import { expect } from 'chai';
import { describe, it } from 'mocha';
import { setupApplicationTest } from 'ember-mocha';
import { visit, currentURL } from '@ember/test-helpers';

describe('basic acceptance test', function() {
  setupApplicationTest();

  it('can visit /', async function() {
    await visit('/');
    expect(currentURL()).to.equal('/');
  });
});

🌈 Remarques

• Avant, ember-auto-import n'existait pas, faker a donc été inclus dans ember-mocha. Avec l'apparition de ember-auto-import, plus de problème, donc faker a été sorti de ember-mocha. Ces deux packages nous sont donc nécessaires maintenant.
• visit() lève une erreur Transition Aborted qui fait échouer les tests : c'est une issue connue visit() throws TransitionAborted error emberjs/ember-test-helpers#332. Pour contourner le problème une méthode visitWithAbortedTransition a été introduite.

[Forster-Groux]

Je ne comprends pas très bien l'écriture de ces stubs. Est-ce que

currentUserStub = {
  user: { organizations: [{ id: 1 }, { id: 2 }] }
};

ne suffit pas ? A quoi sert la notion de Service ?

[lisequesnel]

Il me semble qu'on a besoin de Service pour avoir toutes les méthodes "cachées" comme les listeners d'events, etc

[pix-service]

I'm deploying this PR to these urls:

• App: https://app-pr638.review.pix.fr
• Orga: https://orga-pr638.review.pix.fr
• Certif: https://certif-pr638.review.pix.fr
• Admin: https://admin-pr638.review.pix.fr
• API: https://pix-api-integration-pr638.scalingo.io

Please check it out