-
Notifications
You must be signed in to change notification settings - Fork 21
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Vertical tabs #64
Vertical tabs #64
Conversation
Changes Unknown when pulling 5764ed6 on quincyle:vertical-tabs into ** on ciena-frost:master**. |
.travis.yml
Outdated
@@ -16,10 +16,6 @@ addons: | |||
- g++-4.8 | |||
firefox: 'latest' | |||
|
|||
cache: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We want to leave this cache in. Please do not remove it.
@@ -51,6 +51,7 @@ describe(test.label, function () { | |||
}) | |||
|
|||
it('Renders', function (done) { | |||
this.timeout(5000) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Instead of setting a timeout, this should be written as a describe
with a beforeEach()
where the setProperties()
and render()
are handled and the expects()
are inside the it()
. If needed you can then put the return wait()
in the beforeEach
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
describe('when something is done', function() {
beforeEach(function () {
this.setProperties({})
this.render()
return wait()
})
it('should have expected outcome', function () {
expect()
})
})
Changes Unknown when pulling cd53042 on quincyle:vertical-tabs into ** on ciena-frost:master**. |
Changes Unknown when pulling c3bfda6 on quincyle:vertical-tabs into ** on ciena-frost:master**. |
This project uses semver, please check the scope of this pr:
CHANGELOG