Skip to content

Remove skip from studio page bok choy test - #11419

Merged
cahrens merged 1 commit into
masterfrom
christina/enable-test
Feb 3, 2016
Merged

Remove skip from studio page bok choy test#11419
cahrens merged 1 commit into
masterfrom
christina/enable-test

Conversation

@cahrens

@cahrens cahrens commented Feb 3, 2016

Copy link
Copy Markdown

@benpatterson and @jzoldak please review.

While working on a different project, I noticed that this test was skipped. The comment indicated it was because the asset page didn't always load, so given @dianakhuang's recent change, I decided to try removing the skip.

This has passed 41 times. The JIRA ticket referenced doesn't exist, but I did find a couple related to this test where the ticket was closed because nobody had seen the failure in awhile (which perhaps was because it was skipped...).

@cahrens

cahrens commented Feb 3, 2016

Copy link
Copy Markdown
Author

jenkins run bokchoy

@cahrens
cahrens force-pushed the christina/enable-test branch from 9f522b8 to 19162fb Compare February 3, 2016 20:47
@cahrens cahrens changed the title See if this test now passes reliably. Remove skip from studio page bok choy test Feb 3, 2016
@jzoldak

jzoldak commented Feb 3, 2016

Copy link
Copy Markdown
Contributor

👍

cahrens pushed a commit that referenced this pull request Feb 3, 2016
Remove skip from studio page bok choy test
@cahrens
cahrens merged commit 901cafe into master Feb 3, 2016
@cahrens
cahrens deleted the christina/enable-test branch February 3, 2016 21:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants