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
prefer to use util_spec in Gem::TestCase
#2227
Conversation
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 is awesome!
@@ -866,6 +867,19 @@ def util_spec name, version = 2, deps = nil # :yields: specification | |||
end | |||
end | |||
|
|||
unless files.empty? then |
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.
should this check s.files
instead?
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.
@segiddins s
isn't defined at this point. (it'd be spec
, here.)
@hsbt can you leave a comment here confirming whether or not this unless
statement is correct?
if you confirm that's correct, lgtm.
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.
It is the different condition with spec.files
. I tried to replace spec.files
, after that some tests were failed.
I will go current implementation as first.
@bundlerbot r+ |
📌 Commit a33365b has been approved by |
prefer to use util_spec in `Gem::TestCase` # Description: `Gem::Testcase` has `new_spec` and `util_spec` for building stub specification. I migrated them to util_spec and marked new_spec to deprecated. ______________ # Tasks: - [ ] Describe the problem / feature - [ ] Write tests - [ ] Write code to solve the problem - [ ] Get code review from coworkers / friends I will abide by the [code of conduct](https://github.com/rubygems/rubygems/blob/master/CODE_OF_CONDUCT.md).
💔 Test failed - status-travis |
☀️ Test successful - status-travis |
Description:
Gem::Testcase
hasnew_spec
andutil_spec
for building stub specification. I migrated them to util_spec and marked new_spec to deprecated.Tasks:
I will abide by the code of conduct.