Sharing app constants

When the app works but the tests fail, we have a false negative. False negatives could happen, especially when the tests rely on something external (like the E2E tests do). Anyway, we must reduce them as much as possible. False negatives are a smell of bad tests and, sooner or later, is one of the reasons that make you a test-hater.

A lot of things can cause false negatives, one of them is the difference between the constants used by the web app and the ones used by the tests. What are the constants used by the actual test?

File: cypress/integration/examples/signup/signup-4-cypress-testing-library.e2e.spec.js

/// <reference types="Cypress" />

context("Signup flow", () => {
  it("The happy path should work", () => {
    cy.visit("/register");
    const random = Math.floor(Math.random() * 100000);
    cy.findByPlaceholderText("Username").type(`Tester${random}`);
    cy.findByPlaceholderText("Email").type(`user+${random}@realworld.io`);
    cy.findByPlaceholderText("Password").type("mysupersecretpassword");
    cy.get("form")
      .within(() => cy.findByText("Sign up").click())
    cy.findByText("No articles are here... yet.", { timeout: 10000 }).should("be.visible");
  });
});

The route and all the contents

  • "/register"
  • "Username"
  • "Email"
  • "Password"
  • "Sign up"
  • "No articles are here... yet."

could change because of a lot of reasons. If some of them change the app still works but the tests would break. How we can solve the problem? Importing them right from the source code!

We adapted the realworld/frontend/src/components/Register.js file exporting all the strings used internally

export const strings = {
  username: "Username",
  email: "Email",
  password: "Password",
  signUp: "Sign up"
};

and we need to update the test too

+import { strings } from "../../../../realworld/frontend/src/components/Register";

context("Signup flow", () => {
  it("The happy path should work", () => {
    cy.visit("/register");
    const random = Math.floor(Math.random() * 100000);
-   cy.findByPlaceholderText("Username").type(`Tester${random}`);
+   cy.findByPlaceholderText(strings.username).type(`Tester${random}`);
-   cy.findByPlaceholderText("Email").type(`user+${random}@realworld.io`);
+   cy.findByPlaceholderText(strings.email).type(`user+${random}@realworld.io`);
-   cy.findByPlaceholderText("Password").type("mysupersecretpassword");
+   cy.findByPlaceholderText(strings.password).type("mysupersecretpassword");
    cy.get("form")
-     .within(() => cy.findByText("Sign up"))
-      .click();
+     .within(() => cy.findByText(strings.signUp).click())
    cy.findByText("No articles are here... yet.", { timeout: 10000 }).should("be.visible");
  });
});

The updated test looks like this

import { strings } from "../../../../realworld/frontend/src/components/Register";

context("Signup flow", () => {
  it("The happy path should work", () => {
    cy.visit("/register");
    const random = Math.floor(Math.random() * 100000);
    cy.findByPlaceholderText(strings.username).type(`Tester${random}`);
    cy.findByPlaceholderText(strings.email).type(`user+${random}@realworld.io`);
    cy.findByPlaceholderText(strings.password).type("mysupersecretpassword");
    cy.get("form")
      .within(() => cy.findByText(strings.signUp).click())
    cy.findByText("No articles are here... yet.", { timeout: 10000 }).should(
      "be.visible"
    );
  });
});

The same logic could be applied to the /register route and the "No articles are here... yet." string, the final test is the following

File: cypress/integration/examples/signup/signup-5-app-constants.e2e.spec.js

/// <reference types="Cypress" />

import { paths } from "../../../../realworld/frontend/src/components/App";
import { noArticles } from "../../../../realworld/frontend/src/components/ArticleList";
import { strings } from "../../../../realworld/frontend/src/components/Register";

context("Signup flow", () => {
  it("The happy path should work", () => {
    cy.visit(paths.register);
    const random = Math.floor(Math.random() * 100000);
    cy.findByPlaceholderText(strings.username).type(`Tester${random}`);
    cy.findByPlaceholderText(strings.email).type(`user+${random}@realworld.io`);
    cy.findByPlaceholderText(strings.password).type("mysupersecretpassword");
    cy.get("form")
      .within(() => cy.findByText(strings.signUp).click());
    cy.findByText(noArticles, { timeout: 10000 }).should("be.visible");
  });
});

The result is that if you update some of the above-cited constants because of whatever reasons... You do not have to update the test accordingly and they still work as expected, one less thing to care about.

Absolute imports

In order to leverage Webpack's aliases to transform the relative imports into absolute ones

-import { strings } from "../../../../realworld/frontend/src/components/Register";
+import { strings } from "@/components/Register";

you should install and setup the cypress-webpack-preprocessor plugin (do not forget that Cypress is all written in JavaScript and can be tweaked in a lot of different ways).

Author: Stefano Magni

results matching ""

    No results matching ""