Skip to content
This repository has been archived by the owner on Jun 26, 2023. It is now read-only.

Feat / home, create campaign end to end test #12

Open
wants to merge 38 commits into
base: master
Choose a base branch
from

Conversation

RCVZ
Copy link
Contributor

@RCVZ RCVZ commented Mar 8, 2021

This PR adds:

  • home page end to end test
  • create campaign page end to end test

@netlify
Copy link

netlify bot commented Mar 8, 2021

Deploy preview for trompa-campaign-manager ready!

Built with commit 360945e

https://deploy-preview-12--trompa-campaign-manager.netlify.app

@RCVZ RCVZ changed the title Feat/e2e home create campaign Feat / home, create campaign end to end test Mar 8, 2021
@@ -38,6 +38,7 @@ exports[`<SelectComposition /> renders and matches snapshot 1`] = `
</p>
<div>
<button
aria-label="select_composition"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We might prefer without underscore? Then it's also readable for screen readers.

const assert = require('assert');
const { ariaLabel } = require('./locators');
const { tolerance } = require('./settings');

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Minor: extra spaces before ariaLabel and tolerance

Copy link
Contributor

@royschut royschut left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Few minors, but mainly: I would use readable text as aria-label, instead of looking at it as an id.

@RCVZ RCVZ requested a review from royschut March 22, 2021 08:10
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants