right_develop 1.2.0 → 1.2.2

data/Rakefile

@@ -56,6 +56,10 @@ Jeweler::Tasks.new do |gem|
   gem.email = "support@rightscale.com"
   gem.authors = ["Tony Spataro"]
   gem.rubygems_version = "1.3.7"
+  gem.files.exclude ".rspec"
+  gem.files.exclude "Gemfile*"
+  gem.files.exclude "features/**/*"
+  gem.files.exclude "spec/**/*"
 end
 
 # This is a closed-source gem; omit gemcutter tasks so people don't accidentally

data/VERSION

@@ -1 +1 @@
-1.2.0
+1.2.2

data/lib/right_develop/ci/java_cucumber_formatter.rb

@@ -36,12 +36,12 @@ module RightDevelop::CI
     def build_testcase(duration, status, exception = nil, suffix = "")
       @time += duration
       # Use "cucumber" as a pseudo-package, and the feature name as a pseudo-class
-      classname = "cucumber.#{@feature_name}"
+      classname = "cucumber.#{RightDevelop::CI::Util.pseudo_java_class_name(@feature_name)}"
       name = "#{@scenario}#{suffix}"
       pending = [:pending, :undefined].include?(status)
       passed = (status == :passed || (pending && !@options[:strict]))
 
-      @builder.testcase(:classname => classname, :name => name, :time => "%.6f" % duration) do
+      @builder.testcase(:classname => classname.to_sym, :name => name, :time => "%.6f" % duration) do
         unless passed
           @builder.failure(:message => "#{status.to_s} #{name}", :type => status.to_s) do
             @builder.cdata! @output

data/lib/right_develop/ci/java_spec_formatter.rb

@@ -61,6 +61,7 @@ module RightDevelop::CI
 
       def classname_for(example)
         klass = example.example_group.described_class || example.example_group.top_level_description
+        klass = RightDevelop::CI::Util.pseudo_java_class_name(klass.to_s)
         "rspec.#{klass}"
       end
 
@@ -80,7 +81,7 @@ module RightDevelop::CI
               full_description = full_description[classname.length..-1].strip
             end
 
-            builder.testcase(:classname => classname, :name => full_description, :time => time) do
+            builder.testcase(:classname => classname.to_sym, :name => full_description, :time => time) do
               case test.metadata[:execution_result][:status]
               when "failed"
                 builder.failure :message => "failed #{full_description}", :type => "failed" do
@@ -147,7 +148,7 @@ module RightDevelop::CI
               full_description = full_description[classname.length..-1].strip
             end
 
-            builder.testcase(:classname => classname, :name => full_description, :time => @test_times[test]) do
+            builder.testcase(:classname => classname.to_sym, :name => full_description, :time => @test_times[test]) do
               case result
               when "failed"
                 builder.failure :message => "failed #{full_description}", :type => "failed" do
@@ -182,6 +183,7 @@ module RightDevelop::CI
         # and assuming the first word is a class name
         group = @test_groups[example]
         klass = group.description.split(/\s+/).first
+        klass = RightDevelop::CI::Util.pseudo_java_class_name(klass)
         "rspec.#{klass}"
       end
     end

data/lib/right_develop/ci/util.rb

@@ -0,0 +1,30 @@
+module RightDevelop::CI
+  module Util
+    module_function
+
+    # Regular expression used to determine which characters of a string are allowed
+    # in Java class names.
+    JAVA_CLASS_NAME = /[A-Za-z0-9_]/
+
+    # Make a string suitable for parsing by Jenkins JUnit display plugin by escaping any non-valid
+    # Java class name characters as an XML entity. This prevents Jenkins from interpreting "hi1.2"
+    # as a package-and-class name.
+    #
+    # @param [String] name
+    # @return [String] string with all non-alphanumerics replaced with an equivalent XML hex entity
+    def pseudo_java_class_name(name)
+      result = ''
+
+      name.each_char do |chr|
+        if chr =~ JAVA_CLASS_NAME
+          result << chr
+        else
+          chr = chr.unpack('U')[0].to_s(16)
+          result << "&#x#{chr};"
+        end
+      end
+
+      result
+    end
+  end
+end

data/lib/right_develop/ci.rb

@@ -36,5 +36,6 @@ module RightDevelop
 end
 
 # Explicitly require everything else to avoid overreliance on autoload (1-module-deep rule)
+require 'right_develop/ci/util'
 require 'right_develop/ci/java_cucumber_formatter'
 require 'right_develop/ci/java_spec_formatter'

data/right_develop.gemspec

@@ -5,11 +5,11 @@
 
 Gem::Specification.new do |s|
   s.name = %q{right_develop}
-  s.version = "1.2.0"
+  s.version = "1.2.2"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["Tony Spataro"]
-  s.date = %q{2013-04-29}
+  s.date = %q{2013-04-30}
   s.default_executable = %q{right_develop}
   s.description = %q{A toolkit of development tools created by RightScale.}
   s.email = %q{support@rightscale.com}
@@ -19,31 +19,18 @@ Gem::Specification.new do |s|
     "README.rdoc"
   ]
   s.files = [
-    ".rspec",
     "CHANGELOG.rdoc",
-    "Gemfile",
-    "Gemfile.lock",
     "LICENSE",
     "README.rdoc",
     "Rakefile",
     "VERSION",
     "bin/right_develop",
-    "features/cucumber.feature",
-    "features/rake_integration.feature",
-    "features/rspec1.feature",
-    "features/rspec2.feature",
-    "features/step_definitions/http_client_steps.rb",
-    "features/step_definitions/request_balancer_steps.rb",
-    "features/step_definitions/ruby_steps.rb",
-    "features/step_definitions/serialization_steps.rb",
-    "features/step_definitions/server_steps.rb",
-    "features/support/env.rb",
-    "features/support/file_utils_bundler_mixin.rb",
     "lib/right_develop.rb",
     "lib/right_develop/ci.rb",
     "lib/right_develop/ci/java_cucumber_formatter.rb",
     "lib/right_develop/ci/java_spec_formatter.rb",
     "lib/right_develop/ci/rake_task.rb",
+    "lib/right_develop/ci/util.rb",
     "lib/right_develop/commands.rb",
     "lib/right_develop/commands/git.rb",
     "lib/right_develop/git.rb",
@@ -56,9 +43,7 @@ Gem::Specification.new do |s|
     "lib/right_develop/parsers/sax_parser.rb",
     "lib/right_develop/parsers/xml_post_parser.rb",
     "right_develop.gemspec",
-    "right_develop.rconf",
-    "spec/right_develop/parsers/sax_parser_spec.rb",
-    "spec/spec_helper.rb"
+    "right_develop.rconf"
   ]
   s.homepage = %q{https://github.com/rightscale/right_develop}
   s.licenses = ["MIT"]
@@ -137,6 +122,34 @@ Gem::Specification.new do |s|
       s.add_development_dependency(%q<rdoc>, [">= 2.4.2"])
       s.add_development_dependency(%q<syntax>, ["~> 1.0.0"])
       s.add_development_dependency(%q<nokogiri>, ["~> 1.5"])
+      s.add_development_dependency(%q<flexmock>, ["~> 0.8.7"])
+      s.add_development_dependency(%q<activesupport>, [">= 0"])
+      s.add_development_dependency(%q<libxml-ruby>, ["~> 2.4.0"])
+      s.add_development_dependency(%q<jeweler>, ["~> 1.8.3"])
+      s.add_development_dependency(%q<rdoc>, [">= 2.4.2"])
+      s.add_development_dependency(%q<syntax>, ["~> 1.0.0"])
+      s.add_development_dependency(%q<nokogiri>, ["~> 1.5"])
+      s.add_development_dependency(%q<flexmock>, ["~> 0.8.7"])
+      s.add_development_dependency(%q<activesupport>, [">= 0"])
+      s.add_development_dependency(%q<libxml-ruby>, ["~> 2.4.0"])
+      s.add_development_dependency(%q<jeweler>, ["~> 1.8.3"])
+      s.add_development_dependency(%q<rdoc>, [">= 2.4.2"])
+      s.add_development_dependency(%q<syntax>, ["~> 1.0.0"])
+      s.add_development_dependency(%q<nokogiri>, ["~> 1.5"])
+      s.add_development_dependency(%q<flexmock>, ["~> 0.8.7"])
+      s.add_development_dependency(%q<activesupport>, [">= 0"])
+      s.add_development_dependency(%q<libxml-ruby>, ["~> 2.4.0"])
+      s.add_development_dependency(%q<jeweler>, ["~> 1.8.3"])
+      s.add_development_dependency(%q<rdoc>, [">= 2.4.2"])
+      s.add_development_dependency(%q<syntax>, ["~> 1.0.0"])
+      s.add_development_dependency(%q<nokogiri>, ["~> 1.5"])
+      s.add_development_dependency(%q<flexmock>, ["~> 0.8.7"])
+      s.add_development_dependency(%q<activesupport>, [">= 0"])
+      s.add_development_dependency(%q<libxml-ruby>, ["~> 2.4.0"])
+      s.add_development_dependency(%q<jeweler>, ["~> 1.8.3"])
+      s.add_development_dependency(%q<rdoc>, [">= 2.4.2"])
+      s.add_development_dependency(%q<syntax>, ["~> 1.0.0"])
+      s.add_development_dependency(%q<nokogiri>, ["~> 1.5"])
       s.add_development_dependency(%q<jeweler>, ["~> 1.8.3"])
       s.add_development_dependency(%q<rdoc>, [">= 2.4.2"])
       s.add_development_dependency(%q<syntax>, ["~> 1.0.0"])
@@ -213,6 +226,34 @@ Gem::Specification.new do |s|
       s.add_dependency(%q<rdoc>, [">= 2.4.2"])
       s.add_dependency(%q<syntax>, ["~> 1.0.0"])
       s.add_dependency(%q<nokogiri>, ["~> 1.5"])
+      s.add_dependency(%q<flexmock>, ["~> 0.8.7"])
+      s.add_dependency(%q<activesupport>, [">= 0"])
+      s.add_dependency(%q<libxml-ruby>, ["~> 2.4.0"])
+      s.add_dependency(%q<jeweler>, ["~> 1.8.3"])
+      s.add_dependency(%q<rdoc>, [">= 2.4.2"])
+      s.add_dependency(%q<syntax>, ["~> 1.0.0"])
+      s.add_dependency(%q<nokogiri>, ["~> 1.5"])
+      s.add_dependency(%q<flexmock>, ["~> 0.8.7"])
+      s.add_dependency(%q<activesupport>, [">= 0"])
+      s.add_dependency(%q<libxml-ruby>, ["~> 2.4.0"])
+      s.add_dependency(%q<jeweler>, ["~> 1.8.3"])
+      s.add_dependency(%q<rdoc>, [">= 2.4.2"])
+      s.add_dependency(%q<syntax>, ["~> 1.0.0"])
+      s.add_dependency(%q<nokogiri>, ["~> 1.5"])
+      s.add_dependency(%q<flexmock>, ["~> 0.8.7"])
+      s.add_dependency(%q<activesupport>, [">= 0"])
+      s.add_dependency(%q<libxml-ruby>, ["~> 2.4.0"])
+      s.add_dependency(%q<jeweler>, ["~> 1.8.3"])
+      s.add_dependency(%q<rdoc>, [">= 2.4.2"])
+      s.add_dependency(%q<syntax>, ["~> 1.0.0"])
+      s.add_dependency(%q<nokogiri>, ["~> 1.5"])
+      s.add_dependency(%q<flexmock>, ["~> 0.8.7"])
+      s.add_dependency(%q<activesupport>, [">= 0"])
+      s.add_dependency(%q<libxml-ruby>, ["~> 2.4.0"])
+      s.add_dependency(%q<jeweler>, ["~> 1.8.3"])
+      s.add_dependency(%q<rdoc>, [">= 2.4.2"])
+      s.add_dependency(%q<syntax>, ["~> 1.0.0"])
+      s.add_dependency(%q<nokogiri>, ["~> 1.5"])
       s.add_dependency(%q<jeweler>, ["~> 1.8.3"])
       s.add_dependency(%q<rdoc>, [">= 2.4.2"])
       s.add_dependency(%q<syntax>, ["~> 1.0.0"])
@@ -290,6 +331,34 @@ Gem::Specification.new do |s|
     s.add_dependency(%q<rdoc>, [">= 2.4.2"])
     s.add_dependency(%q<syntax>, ["~> 1.0.0"])
     s.add_dependency(%q<nokogiri>, ["~> 1.5"])
+    s.add_dependency(%q<flexmock>, ["~> 0.8.7"])
+    s.add_dependency(%q<activesupport>, [">= 0"])
+    s.add_dependency(%q<libxml-ruby>, ["~> 2.4.0"])
+    s.add_dependency(%q<jeweler>, ["~> 1.8.3"])
+    s.add_dependency(%q<rdoc>, [">= 2.4.2"])
+    s.add_dependency(%q<syntax>, ["~> 1.0.0"])
+    s.add_dependency(%q<nokogiri>, ["~> 1.5"])
+    s.add_dependency(%q<flexmock>, ["~> 0.8.7"])
+    s.add_dependency(%q<activesupport>, [">= 0"])
+    s.add_dependency(%q<libxml-ruby>, ["~> 2.4.0"])
+    s.add_dependency(%q<jeweler>, ["~> 1.8.3"])
+    s.add_dependency(%q<rdoc>, [">= 2.4.2"])
+    s.add_dependency(%q<syntax>, ["~> 1.0.0"])
+    s.add_dependency(%q<nokogiri>, ["~> 1.5"])
+    s.add_dependency(%q<flexmock>, ["~> 0.8.7"])
+    s.add_dependency(%q<activesupport>, [">= 0"])
+    s.add_dependency(%q<libxml-ruby>, ["~> 2.4.0"])
+    s.add_dependency(%q<jeweler>, ["~> 1.8.3"])
+    s.add_dependency(%q<rdoc>, [">= 2.4.2"])
+    s.add_dependency(%q<syntax>, ["~> 1.0.0"])
+    s.add_dependency(%q<nokogiri>, ["~> 1.5"])
+    s.add_dependency(%q<flexmock>, ["~> 0.8.7"])
+    s.add_dependency(%q<activesupport>, [">= 0"])
+    s.add_dependency(%q<libxml-ruby>, ["~> 2.4.0"])
+    s.add_dependency(%q<jeweler>, ["~> 1.8.3"])
+    s.add_dependency(%q<rdoc>, [">= 2.4.2"])
+    s.add_dependency(%q<syntax>, ["~> 1.0.0"])
+    s.add_dependency(%q<nokogiri>, ["~> 1.5"])
     s.add_dependency(%q<jeweler>, ["~> 1.8.3"])
     s.add_dependency(%q<rdoc>, [">= 2.4.2"])
     s.add_dependency(%q<syntax>, ["~> 1.0.0"])