JoinColony / colonyDapp

Colony dApp client
49 stars 19 forks source link

E2E Test Creating a New Colony #987

Closed rdig closed 4 years ago

rdig commented 5 years ago

As it says on the tin: write a cypress spec that tests updates made to the User profile metadata.

Testing

It should cover:

Best practices

Please note that, as a best practice, prefer to add a new data-test attribute, to hook onto components, rather then relying on elements/class/ids selectors and filtering from there.

Example:

/*
 * Component (not the actual one, simplified)
 */
const AvatarDropdown = () => (
  <Popover ... >
    <button
      ...
      data-test="avatarDropdown"
    />
  </Popover>
);

/*
 * Test (not the actual one, simplified)
 */
it('Open the Claim Username flow', () => {
  ...
  cy.get('button[data-test="avatarDropdown"]').click();
  ...
});

See this example spec: https://github.com/JoinColony/colonyDapp/blob/master/cypress/integration/claim-username.js

Resources

A couple of resources to help you along you're journey:

rdig commented 5 years ago

Note that this is waiting for #960 to be merged before work being able to be started here

karolinkas commented 5 years ago

@rdig This one can be done now, can do it if you want.

rdig commented 5 years ago

Now that you finished the new Colony Workflow, yes it can be tackled.

If you want, sure, go for it!