rally_workspace_utils 0.0.1 → 0.0.2

data/bin/translate_workspace

@@ -38,6 +38,7 @@ end
 args = {:base_url => "https://rally1.rallydev.com/slm"}
 
 debug = false
+audit = false
 logfile = STDOUT
 
 opts.each do |opt, arg|
@@ -45,7 +46,6 @@ opts.each do |opt, arg|
   when "--base_url"
     args[:base_url] = arg
   when "--from_workspace"
-    puts "arg = #{arg}"
     args[:from_workspace] = arg
   when "--to_workspace"
     args[:to_workspace] = arg
@@ -58,6 +58,8 @@ opts.each do |opt, arg|
     RestBuilder::DEBUG = true
   when "--logfile"
     logfile = arg
+  when "--audit"
+    audit = true
   when "--help" 
     usage
   else
@@ -84,7 +86,7 @@ end
 args[:logger] = logger
 
 translate = Translate.new(args)
-puts translate.inspect
 translate.run
+translate.audit if audit
 
 

data/lib/translate/copy/object_copy.rb

@@ -1,5 +1,3 @@
-require 'pp'
-
 class ObjectCopy
   def initialize(object)
     @object = object
@@ -56,7 +54,7 @@ class ObjectCopy
 	attrdef = @object.typedef.custom_dropdown_attributes[attribute]
 	unless attrdef.allowed_values.include? value
 	  values[attribute] = nil 
-	  puts "Deleteing #{attribute} = #{value} from @{object.name} because it no longer exists"
+	  @object.rally_rest.logger.info "Deleteing #{attribute} = #{value} from @{object.name} because it no longer exists"
 	end
       end
     end

data/lib/translate/translate.rb

@@ -3,6 +3,8 @@ require File.dirname(__FILE__) + '/translation_audit'
 
 class Translate
   include TranslationAudit
+  
+  attr_reader :logger
 
   def initialize(args)
     @base_url = args[:base_url]
@@ -29,7 +31,7 @@ class Translate
 	$TRANSLATED = Marshal.load(f)
       end
 
-      if $TRANSLATED[:from] != @from_workspace.oid && $TRANSLATED[:to] != @to_workspace.oid
+      if $TRANSLATED[:from] != @from_workspace.oid || $TRANSLATED[:to] != @to_workspace.oid
 	$TRANSLATED = {}
 	$TRANSLATED[:from] = @from_workspace.oid
 	$TRANSLATED[:to] = @to_workspace.oid

data/lib/translate/translation_audit.rb

@@ -1,3 +1,4 @@
+require 'active_support'
 
 module TranslationAudit
   def audit
@@ -8,29 +9,32 @@ module TranslationAudit
     all_copied_fields_should_match_for :test_case_result, [:test_case]
     compare_each :test_case_result do |old, new|
       if old.test_case.name != new.test_case.name
-	puts "#{old.type} --  #{old.name} is different"
+	self.logger.info diff_string(old, new)
       end
     end      
 
     all_copied_fields_should_match_for :test_case_step, [:test_case]
     compare_each :test_case_step do |old, new|
       if old.test_case.name != new.test_case.name
-	puts "#{old.type} --  #{old.name} is different"
+	self.logger.info diff_string(old, new)
       end
     end      
     all_copied_fields_should_match_for :task, [:work_product]
     compare_each :task do |old, new|
+      next if old.work_product.type != "Story"
       if old.card.work_product.name != new.work_product.name
-	puts "#{old.type} --  #{old.name} is different"
+	self.logger.info diff_string(old, new)
       end
     end
 
     compare_each :defect do |old, new|
       if !old.requirement.nil? && old.requirement.type == "Story"
 	if new.requirement.children
-	  old.requirement.name.should == new.requirement.children.values.first.name
+	  if old.requirement.name != new.requirement.children.values.first.name
+	  end
 	else
-	  old.requirement.name.should == new.requirement.name
+	  if old.requirement.name != new.requirement.name
+	  end
 	end
       end
     end
@@ -38,9 +42,11 @@ module TranslationAudit
     compare_each :test_case do |old, new|
       if !old.work_product.nil? && old.work_product.type == "Story"
 	if new.work_product.children
-	  old.work_product.name.should == new.work_product.children.values.first.name
+	  if old.work_product.name != new.work_product.children.values.first.name
+	  end
 	else
-	  old.work_product.name.should == new.work_product.name
+	  if old.work_product.name != new.work_product.name
+	  end
 	end
 
       end
@@ -51,7 +57,7 @@ module TranslationAudit
     @slm.find_all(type, :workspace => @from_workspace).each do |old_object|
       new_object = @slm.find(type, :workspace => @to_workspace) { equal :object_i_d, $TRANSLATED[old_object.oid] }.first
       if new_object.nil?
-	puts "Cound not find an copy for object #{old_object.type} -- #{old_object.name} -- #{old_object.oid}"
+	self.logger.info "Cound not find a copy for object #{old_object.type} -- #{old_object.name} -- #{old_object.oid}"
 	next
       end
       yield old_object, new_object
@@ -60,17 +66,32 @@ module TranslationAudit
 
   def all_copied_fields_should_match_for(type, additional_exceptions = [])
     compare_each(type) do |old_object, new_object| 
-      excepted_attributes = old_object.excepted_attributes + additional_exceptions
-      old_hash = old_object.elements.reject { |k, v| excepted_attributes.include?(k) }
-      new_hash = {}
-      old_hash.each_key do |k|
-	new_hash[k] = new_object.elements[k]
-      end
-      diff = old_hash.diff(new_hash)
+      diff = diff_objects(old_object, new_object, additional_exceptions)
       if diff.length > 0
-	puts "#{old_hash.inspect} shoud == #{new_hash.inspect}"
-	puts "all_copied_fields_should_match_for #{type} diff = #{diff.inspect}"
+	self.logger.info diff_string(old_object, new_object, additional_exceptions)
       end
     end
   end
+
+  def diff_objects(old_object, new_object, additional_exceptions = [])
+    excepted_attributes = old_object.excepted_attributes + additional_exceptions
+    old_hash = old_object.elements.reject { |k, v| excepted_attributes.include?(k) }
+    new_hash = {}
+    old_hash.each_key do |k|
+      new_hash[k] = new_object.elements[k]
+    end
+
+    old_hash.diff(new_hash)
+  end
+
+  def diff_string(old, new, additional_exceptions = [])
+    diff = diff_objects(old, new, additional_exceptions)
+    diff_string = "#{old.type} with name '#{old.name}' differs\n"
+    diff_string << "\told\t\t\tnew\n"
+    diff.each do |key, valye|
+      diff_string << "\t" << old.elements[key] << "\t\t\t" << new.elements[key] << "\n"
+    end
+    diff_string << "\n"
+    diff_string
+  end
 end

data/lib/translate/version.rb

@@ -2,7 +2,7 @@ module Translate #:nodoc:
   module VERSION #:nodoc:
     MAJOR = 0
     MINOR = 0
-    TINY  = 1
+    TINY  = 2
 
     STRING = [MAJOR, MINOR, TINY].join('.')
   end

metadata

@@ -3,7 +3,7 @@ rubygems_version: 0.9.0
 specification_version: 1
 name: rally_workspace_utils
 version: !ruby/object:Gem::Version 
-  version: 0.0.1
+  version: 0.0.2
 date: 2006-12-19 00:00:00 -07:00
 summary: A utility to translate a UseCase workspace to a UserStory workspace
 require_paths: