Skip to content
This repository has been archived by the owner on Nov 30, 2024. It is now read-only.

Get rid of warnings from rspec-core's library and spec code. #907

Merged
merged 5 commits into from
May 16, 2013

Conversation

phiggins
Copy link
Contributor

This pull request makes rspec-core's own code warning free when running its spec suite. The only remaining warnings are from its dependencies:

pete@balloon:~/projects/rspec-core$ rake spec
/home/pete/.rvm/rubies/ruby-1.9.3-p392/bin/ruby -w -S rspec ./spec/autotest/discover_spec.rb ./spec/autotest/failed_results_re_spec.rb ./spec/autotest/rspec_spec.rb ./spec/command_line/order_spec.rb ./spec/rspec/core/backtrace_cleaner_spec.rb ./spec/rspec/core/command_line_spec.rb ./spec/rspec/core/configuration_options_spec.rb ./spec/rspec/core/configuration_spec.rb ./spec/rspec/core/deprecations_spec.rb ./spec/rspec/core/drb_command_line_spec.rb ./spec/rspec/core/drb_options_spec.rb ./spec/rspec/core/dsl_spec.rb ./spec/rspec/core/example_group_spec.rb ./spec/rspec/core/example_spec.rb ./spec/rspec/core/filter_manager_spec.rb ./spec/rspec/core/formatters/base_formatter_spec.rb ./spec/rspec/core/formatters/base_text_formatter_spec.rb ./spec/rspec/core/formatters/documentation_formatter_spec.rb ./spec/rspec/core/formatters/helpers_spec.rb ./spec/rspec/core/formatters/html_formatter_spec.rb ./spec/rspec/core/formatters/json_formatter_spec.rb ./spec/rspec/core/formatters/progress_formatter_spec.rb ./spec/rspec/core/formatters/snippet_extractor_spec.rb ./spec/rspec/core/formatters/text_mate_formatter_spec.rb ./spec/rspec/core/hooks_filtering_spec.rb ./spec/rspec/core/hooks_spec.rb ./spec/rspec/core/kernel_extensions_spec.rb ./spec/rspec/core/memoized_helpers_spec.rb ./spec/rspec/core/metadata_spec.rb ./spec/rspec/core/option_parser_spec.rb ./spec/rspec/core/pending_example_spec.rb ./spec/rspec/core/project_initializer_spec.rb ./spec/rspec/core/rake_task_spec.rb ./spec/rspec/core/reporter_spec.rb ./spec/rspec/core/resources/a_spec.rb ./spec/rspec/core/rspec_matchers_spec.rb ./spec/rspec/core/ruby_project_spec.rb ./spec/rspec/core/runner_spec.rb ./spec/rspec/core/shared_context_spec.rb ./spec/rspec/core/shared_example_group_spec.rb ./spec/rspec/core/world_spec.rb ./spec/rspec/core_spec.rb
/home/pete/.rvm/gems/ruby-1.9.3-p392@rspec-core/gems/childprocess-0.3.9/lib/childprocess/unix/process.rb:46: warning: assigned but unused variable - pid
/home/pete/.rvm/gems/ruby-1.9.3-p392@rspec-core/gems/syntax-1.0.0/lib/syntax/lang/ruby.rb:55: warning: character class has duplicated range: /\?(\\[^\n\r]|[^\\\n\r\s])/
Run options:
  include {:focus=>true}
  exclude {:ruby=>#<Proc:./spec/spec_helper.rb:118>}

All examples were filtered out; ignoring {:focus=>true}
.........................................................................................................................................................................................................................................................................................................................................................................................................................................................................................../home/pete/.rvm/gems/ruby-1.9.3-p392@rspec-core/gems/syntax-1.0.0/lib/syntax/common.rb:99: warning: character class has duplicated range: /\?(\\[^\n\r]|[^\\\n\r\s])/
/home/pete/.rvm/gems/ruby-1.9.3-p392@rspec-core/gems/syntax-1.0.0/lib/syntax/common.rb:99: warning: character class has duplicated range: /\?(\\[^\n\r]|[^\\\n\r\s])/
./home/pete/.rvm/gems/ruby-1.9.3-p392@rspec-core/gems/syntax-1.0.0/lib/syntax/common.rb:99: warning: character class has duplicated range: /\?(\\[^\n\r]|[^\\\n\r\s])/
/home/pete/.rvm/gems/ruby-1.9.3-p392@rspec-core/gems/syntax-1.0.0/lib/syntax/common.rb:99: warning: character class has duplicated range: /\?(\\[^\n\r]|[^\\\n\r\s])/
./home/pete/.rvm/gems/ruby-1.9.3-p392@rspec-core/gems/syntax-1.0.0/lib/syntax/common.rb:99: warning: character class has duplicated range: /\?(\\[^\n\r]|[^\\\n\r\s])/
/home/pete/.rvm/gems/ruby-1.9.3-p392@rspec-core/gems/syntax-1.0.0/lib/syntax/common.rb:99: warning: character class has duplicated range: /\?(\\[^\n\r]|[^\\\n\r\s])/
./home/pete/.rvm/gems/ruby-1.9.3-p392@rspec-core/gems/syntax-1.0.0/lib/syntax/common.rb:99: warning: character class has duplicated range: /\?(\\[^\n\r]|[^\\\n\r\s])/
/home/pete/.rvm/gems/ruby-1.9.3-p392@rspec-core/gems/syntax-1.0.0/lib/syntax/common.rb:99: warning: character class has duplicated range: /\?(\\[^\n\r]|[^\\\n\r\s])/
.......................................................................................*.....................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................

Pending:
  ::DRbCommandLine with server running outputs colorized text when running with --colour option
    # figure out a way to tell the output to say it's tty
    # ./spec/rspec/core/drb_command_line_spec.rb:95

Finished in 1.91 seconds
1163 examples, 0 failures, 1 pending

Randomized with seed 34559

The most questionable change is the one in spec/rspec/core/example_spec.rb, since I wasn't able to figure out how to make those warnings go away in a cleaner way.

Please let me know what you think, and thanks for making the specs so easy to run and understand!

@JonRowe
Copy link
Member

JonRowe commented May 15, 2013

Thanks for this, I've made some refactorings which make me more comfortable with how warnings are suppressed in a few places, once the Travis build completes I'll merge.

@JonRowe JonRowe merged commit 9ac8222 into rspec:master May 16, 2013
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.

2 participants