Skip to content
This repository was archived by the owner on Oct 21, 2020. It is now read-only.

refactor(unit tests): DRY up the unit tests for invalid community events #197

Merged
merged 1 commit into from
Jan 15, 2019

Conversation

utsab
Copy link

@utsab utsab commented Jan 9, 2019

This is a fix for #188. We're looking for feedback on this refactor to DRY up the unit tests. Please let us know if we are going in the right direction. Thank you.

Copy link
Contributor

@ojongerius ojongerius left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@ojongerius
Copy link
Contributor

This will need f32d0b7 to go green

@ojongerius
Copy link
Contributor

@utsab could you please merge staging or cherry-pick f32d0b7?

@ValeraS
Copy link

ValeraS commented Jan 15, 2019

@ojongerius close/open PR to update config in Travis CI

@ojongerius
Copy link
Contributor

@ValeraS that commit went in via #199.

@ValeraS
Copy link

ValeraS commented Jan 15, 2019

@ojongerius It doesn't matter

@ojongerius
Copy link
Contributor

@ValeraS I'm not sure what you mean, could please elaborate?

@ValeraS
Copy link

ValeraS commented Jan 15, 2019

@ojongerius I see, You do not merge it yet.

@ValeraS
Copy link

ValeraS commented Jan 15, 2019

@ojongerius If close/open PR Travis CI will start a new build and take for this build config from the master branch.

Restart build in Travis CI UI does not update config.

@ValeraS ValeraS closed this Jan 15, 2019
@ValeraS ValeraS reopened this Jan 15, 2019
@ojongerius
Copy link
Contributor

@ValeraS aha! Nice trick 👍

@ojongerius ojongerius closed this Jan 15, 2019
@ojongerius ojongerius reopened this Jan 15, 2019
@ojongerius ojongerius merged commit 68b2ae5 into freeCodeCamp:staging Jan 15, 2019
@ValeraS
Copy link

ValeraS commented Jan 15, 2019

@ojongerius After updating the configuration or if the cofig is up to date, you can simply restart the build in Travis itself, if you need to. No need to close/open PR.

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.

3 participants