-
Notifications
You must be signed in to change notification settings - Fork 62
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
Artifacts method on build endpoint #3
Conversation
end | ||
|
||
it 'returns all artifacts for build' do | ||
res.body.should be_an_instance_of(Array) |
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.
This test should be updated to test something meaningful such as making expectations on the artifacts for a build.
@mtchavez All updated; the VCR cassette now returns 2 build artifacts, I've added Yardoc and removed the version bump. |
Yeah this looks good and the test is a little more meaningful, thanks. |
Artifacts method on build endpoint
Added a new method to the build class to display of artifact data.
Included updated docs, specs and VCR cassette.