Polaris-react: Percy false positives and test flakiness

Created on 13 Aug 2018  路  13Comments  路  Source: Shopify/polaris-react

Experiencing Percy false positives and test flakiness? Post a comment with the test that incorrectly failed.

Guidelines

  1. Read though the existing flaky test comments before posting
  2. Don鈥檛 post the same flaky test failure as someone else, instead upvote 馃憤 the other persons comment
  3. One test per comment with:

    • screenshot of the actual and expected

    • a link to the build

鈿楋笍 Development

Most helpful comment

/drop-zone/7

screen shot 2018-08-13 at 1 58 53 pm
screen shot 2018-08-13 at 1 58 34 pm

https://percy.io/Shopify/polaris-react/builds/930221

Looks like it took a screenshot before the component finished rendering. When visiting /drop-zone/7, the drop zone component flashes "unstyled" for a moment.

All 13 comments

/drop-zone/7

screen shot 2018-08-13 at 1 58 53 pm
screen shot 2018-08-13 at 1 58 34 pm

https://percy.io/Shopify/polaris-react/builds/930221

Looks like it took a screenshot before the component finished rendering. When visiting /drop-zone/7, the drop zone component flashes "unstyled" for a moment.

/drop-zone/6

screen shot 2018-08-15 at 1 52 57 am

PR changes did not relate in any way to the the drop zone component.

/resource-list/3

In a PR for resource list, but I'm not sure why they're doubled up.

screen shot 2018-08-16 at 12 41 55 pm

/resource-list/2

screen shot 2018-08-16 at 12 46 25 pm

/resource-list/1

screen shot 2018-08-16 at 12 47 11 pm

Unchanged Badge stuff (no badge stuff in PR)
screen shot 2018-08-17 at 11 36 36 am
screen shot 2018-08-17 at 11 36 47 am
screen shot 2018-08-17 at 11 37 03 am

From this PR which didn't make any visual changes
/drop-zone/0
screen shot 2018-09-20 at 2 23 07 pm

Drop zone may be giving false positives because of this: https://github.com/Shopify/polaris-react/pull/1434

This one is fun because it will be a different width almost every time 馃槃

/loading

image

https://percy.io/Shopify/polaris-react/builds/1108793

鈽濓笍 I wish we could provide a progress: number.

@amrocha can you make an issue in the public repo and update the bit.ly link in Percy? Removing the link from the Percy title is also an option, whatever you think works best.

Gonna transfer this

@amrocha does https://github.com/Shopify/polaris-react/pull/1290 fix this? I don't have enough info to know for sure.

It kinda does, this was more of an ongoing issue for tracking problems.

I haven't noticed any issues recently though, we might be good to close it!

I'm gonna close and if more issues come up, we can open a separate ticket for them, since this might not be super visible anymore.

Was this page helpful?
0 / 5 - 0 ratings