Jest: Não é possível alterar window.location usando Object.defineProperty

Criado em 19 dez. 2017 · 78Comentários · Fonte: facebook/jest

Você quer solicitar um _feature_ ou denunciar um _bug_? Reportar um erro

Qual é o comportamento atual?

Chamando o seguinte de dentro de um conjunto de testes:

Object.defineProperty(location, "hostname", {
  value: "example.com",
  writable: true
});

lança o seguinte erro:

    TypeError: Cannot redefine property: hostname
        at Function.defineProperty (<anonymous>)

Qual é o comportamento esperado?

O código não deve lançar uma exceção e window.location.hostname === "example.com" deve ser avaliado como verdadeiro.

Ao que parece, jsdom agora define window.location como impraticável . A única maneira de alterar os valores dentro de window.location é usar reconfigure , mas (por # 2460) Jest não expõe jsdom para testes para brincar.

Forneça sua configuração exata de Jest e mencione seu Jest, nó,versão yarn / npm e sistema operacional.

Versão Jest : 22.0.1
Versão do nó : 8.6.0
Versão do fio : 1.2.0
SO : macOS High Sierra 10.13.2

Discussion Wontfix

Fonte

simon360

👍101

Comentários muito úteis

Publiquei um novo pacote no npm chamado jest-environment-jsdom-global , que pode ajudar com os problemas que algumas pessoas estão tendo com Object.defineProperty .

simon360 em 19 dez. 2017

👍40 🎉12 ❤7 🚀3 😄2 👀1

Todos 78 comentários

Eu tenho um problema semelhante. Você pode criar seu próprio JSDOMEnvironment e expor o objeto jsdom ao global desta forma.

const JSDOMEnvironment = require('jest-environment-jsdom');

module.exports = class CustomizedJSDomEnvironment extends JSDOMEnvironment {
  constructor(config) {
    super(config);
    this.global.jsdom = this.dom;
  }

  teardown() {
    this.global.jsdom = null;
    return super.teardown();
  }
};

E então você pode chamar jsdom.reconfigure em seu caso de teste como quiser

oliverzy em 19 dez. 2017

👍8

Essa é uma boa solução alternativa, obrigado por compartilhar!

Você deve retornar super.teardown(); , pois é uma promessa, aliás

SimenB em 19 dez. 2017

Perfeito, @oliverzy - vou tentar. Obrigado!

Existe um local apropriado para documentar isso? Parece ser uma pergunta que surge com razoável frequência; esperançosamente, problemas futuros poderiam ser eliminados se isso fosse integrado aos documentos.

simon360 em 19 dez. 2017

Esta solução não funcionou muito bem .

Dentro de nossos arquivos de teste, parece que global está definido como o objeto window do JSDom.

Em outras palavras, dentro de um conjunto de testes, global é o mesmo que window , mas dentro da classe que estende JSDOMEnvironment , global vem do ambiente do Node.

Como resultado, ter este:

describe("test suite", () => {
  it("should not fail", () => {
    global.jsdom.reconfigure({
      url: "https://www.example.com/"
    });
  });
});

falha porque global.jsdom é indefinido.

Eu consegui contornar isso fazendo isso, mas não estou muito preocupado com isso.

const JSDOMEnvironment = require("jest-environment-jsdom");

module.exports = class JSDOMEnvironmentGlobal extends JSDOMEnvironment {
  constructor(config) {
    super(config);

    this.dom.window.jsdom = this.dom;
  }
};

Com este ambiente, global.jsdom dentro de um conjunto de testes é igual a this.dom , e o conjunto de testes acima funciona.

Para mim, parece que definir jsdom como uma propriedade própria window objeto está fadado a se desintegrar eventualmente - existe uma maneira mais limpa de fazer isso?

simon360 em 19 dez. 2017

você precisa escrever jsdom vez de global.jsdom em seus testes.

oliverzy em 19 dez. 2017

👍4

@oliverzy Gosta disso?

describe("test suite", () => {
  it("should not fail", () => {
    jsdom.reconfigure({
      url: "https://www.example.com/"
    });
  });
});

Isso gera jsdom is not defined , mas posso estar interpretando mal.

simon360 em 19 dez. 2017

👍5

@ simon360 Configure testEnvironment com o código de @oliverzy , consulte https://facebook.github.io/jest/docs/en/configuration.html#testenvironment -string

danielbayerlein em 19 dez. 2017

@danielbayerlein minha configuração do Jest tem isto:

"testEnvironment": "@wel-ui/jest-environment-jsdom-global"

onde @wel-ui/jest-environment-jsdom-global é o nome de um pacote em nosso monorepo. O ambiente está sendo usado corretamente, porém, porque a solução que define jsdom em window funciona conforme o esperado.

simon360 em 19 dez. 2017

BTW, alguém sabe por que a solução original não funciona na nova versão?
Este:

Object.defineProperty(location, "hostname", {
  value: "example.com",
  writable: true
});