kuality-coeus 0.0.5 → 0.0.6

data/features/step_definitions/proposal_development/proposal_permissions.rb

@@ -7,7 +7,7 @@ Then /^the (.*) user is listed as an? (.*) in the proposal permissions$/ do |use
 end
 
 When /^I? ?assign the (.*) user as an? (.*) in the proposal permissions$/ do |system_role, role|
-  set(system_role, (make UserObject, role: system_role))
+  make_user role: system_role
   @proposal.permissions.send(snake_case(role+'s')) << get(system_role).user_name
   @proposal.permissions.assign
 end
@@ -93,7 +93,7 @@ end
 
 And /^their proposal permissions allow them to delete the proposal$/ do
   on(Proposal).proposal_actions
-  lambda{@proposal.delete}.should_not raise_error
+  expect{@proposal.delete}.should_not raise_error
 end
 
 Then /^there should be an error message that says not to select other roles alongside aggregator$/ do
@@ -125,3 +125,21 @@ Then /^the (.*) user should not be listed as an? (.*) in the second proposal$/ d
     page.assigned_to_role(role).should_not include "#{user.first_name} #{user.last_name}"
   end
 end
+
+Then /^the user should be able to create a proposal$/ do
+  # Note that since this stepdef doesn't specify WHICH user, it's
+  # assuming that the one to use is the last one that was
+  # created/defined.
+  $users[-1].sign_in
+  expect{create ProposalDevelopmentObject}.not_to raise_error
+end
+
+Then /^I? ?can override the cost sharing amount$/ do
+  @proposal.view 'Proposal Actions'
+  on ProposalActions do |page|
+    page.expand_all
+    expect{page.budget_field.select 'Cost Sharing Amount'}.not_to raise_error
+    page.budget_changed_value.set '100'
+    expect{page.add_budget_change_data}.not_to raise_error
+  end
+end

data/features/step_definitions/proposal_logs/create_proposal_log.rb

@@ -0,0 +1,13 @@
+When(/^I initiate a proposal log document but I miss a required field$/) do
+  # Pick a field at random for the test...
+  @required_field = ['Title', 'Proposal Type', 'Lead Unit'
+          ].sample
+  # Properly set the nil value depending on the field type...
+  @required_field=~/Type/ ? value='select' : value=''
+  # Transform the field name to the appropriate symbol...
+  field =snake_case(@required_field)
+  @proposal_log = create ProposalLogObject, field=>value
+end
+When(/^I create a proposal log document$/) do
+  @proposal_log = create ProposalLogObject
+end

data/features/step_definitions/protocol_development/irb_create_protocol.rb

@@ -0,0 +1,28 @@
+When(/^I initiate an irb protocol$/) do
+  @irb_protocol = create IRBProtocolDevelopmentObject
+end
+When(/^I initiate a proposal with an invalid lead unit code$/) do
+  @irb_protocol = create IRBProtocolDevelopmentObject, :lead_unit=>'000000'
+end
+Then(/^I should see an error that says my lead unit code is invalid$/) do
+  on(ProtocolOverview).errors.should include 'Lead Unit is invalid.'
+end
+
+# Note: I created this step, in addition to create_proposal.rb, Line: 143 because
+# the errors on this page are lame and include the name of the field twice.
+# ex. "Doc Description (Description) is required."
+Then /^an error should appear that says the field is required$/ do
+  text="#{@required_field} (#{@required_field}) is a required field."
+  @required_field=='Description' ? error='Document '+text : error=text
+  on(ProtocolOverview).errors.should include error
+end
+When(/^I? ?initiate an irb protocol but I miss a required field$/) do
+  # Pick a field at random for the test...
+  @required_field = ['Description', 'Title', 'Lead Unit'
+          ].sample
+  # Properly set the nil value depending on the field type...
+  @required_field= value=''
+  # Transform the field name to the appropriate symbol...
+  field = snake_case(@required_field)
+  @irb_protocol = create IRBProtocolDevelopmentObject, field=>value
+end

data/features/step_definitions/roles.rb

@@ -0,0 +1,17 @@
+When /^create a role with permission to create proposals$/ do
+  role = make_role permissions: %w{842}
+  role.create
+end
+
+When /^I? ?add the group to the role$/ do
+  @role.add_assignee type_code: 'Group', member_identifier: @group.id
+end
+
+When /^the '(.*)' role exists$/ do |role|
+  make_role name: role
+  get(role).create unless get(role).exists?
+end
+
+When /^I? ?add the group to the '(.*)' role$/ do |role|
+  get(role).add_assignee type_code: 'Group', member_identifier: @group.id
+end

data/features/step_definitions/users.rb

@@ -1,14 +1,21 @@
+# We need this step def because of the special case of
+# the admin user. Its data object is already created
+# and added to the Users collection at the start of the
+# scripts.
+Given /^I'm( signed)? in as the admin$/ do |x|
+  $users.admin.sign_in
+end
+
 # Note the difference between the following three
 # step definitions...
 
-# This step definition
+# This step definition is a bit dangerous
 # 1) Assumes that the user already exists in the system
 # 2) Assumes the user object does not exist, so it creates it, sticking it into
 #    a class instance variable based on the username
 # 3) Logs that user in (if they're not already)...
 Given /^I'm logged in with (.*)$/ do |username|
-  user = make_user user: username
-  user.sign_in
+  make_user(user: username).sign_in
 end
 
 # Whereas, this step def
@@ -52,12 +59,40 @@ Given /^users exist with the following roles: (.*)$/ do |roles|
 end
 
 Given /^a user exists that can be a PI for Grants.gov proposals$/ do
-  @grants_gov_pi = make_user user: UserObject::USERS.grants_gov_pi
-  @grants_gov_pi.create unless @grants_gov_pi.exists?
+  make_user(user: UserObject::USERS.grants_gov_pi, type: 'Grants.gov PI')
+  $users[-1].create unless $users[-1].exists?
 end
 
 Given /^an AOR user exists$/ do
-  # TODO: Using the quickstart user here is cheating. Fix this.
-  @aor = make_user user: 'quickstart'
+  # TODO: Using the username here is cheating. Fix this.
+  @aor = make_user(user: 'warrens', type: 'AOR')
   @aor.create unless @aor.exists?
+end
+
+When /^I? ?create an? '(.*)' user$/ do |type|
+  $users << create(UserObject, type: type)
+end
+
+Given /^I? ?create a user with an? (.*) role in the (.*) unit$/ do |role, unit|
+  role_num = RoleObject::ROLES[role]
+  $users << create(UserObject, rolez: [{ id: role_num, name: role, qualifiers: [{:unit=>unit}] }] )
+end
+
+Given /^I? ?log in as the user with the (.*) role in (.*)$/ do |role, unit|
+  $users.with_role_in_unit(role, unit).sign_in
+end
+
+When /^I? ?log in with that user$/ do
+  $users[-1].sign_in
+end
+
+And /^I add the (.*) role in the (.*) unit to that user$/ do |role, unit|
+  role_num = RoleObject::ROLES[role]
+  $users[-1].add_role id: role_num, name: role, qualifiers: [{:unit=>unit}], user_name: $users[-1].user_name
+end
+
+# Use this step def when you know the role doesn't take a qualifier
+And /^I add the (.*) role to that user$/ do |role|
+  role_num = RoleObject::ROLES[role]
+  $users[-1].add_role id: role_num, name: role, qualifiers: [], user_name: $users[-1].user_name
 end

data/features/support/env.rb

@@ -26,10 +26,25 @@ World DateFactory
 World Utilities
 
 kuality = Kuality.new @config[:browser]
+$users = Users.instance
 
 Before do
   @browser = kuality.browser
+  $users.clear
+  # Add the admin user to the Users...
+  $users << UserObject.new(@browser)
+end
+
+After do |scenario|
+
+  if scenario.failed?
+    @browser.screenshot.save 'screenshot.png'
+    embed 'screenshot.png', 'image/png'
+  end
+
+  $users.current_user.sign_out unless $users.current_user==nil
+
 end
 
 # Comment out to help with debugging...
-# at_exit { kuality.browser.close }
+at_exit { kuality.browser.close }

data/kuality-coeus.gemspec

@@ -1,6 +1,6 @@
 spec = Gem::Specification.new do |s|
   s.name = 'kuality-coeus'
-  s.version = '0.0.5'
+  s.version = '0.0.6'
   s.summary = %q{rSmart's test framework for BDD testing of Kuali Coeus}
   s.description = %q{This gem is used for creating test scripts for Kuali Coeus.}
   s.files = Dir.glob("**/**/**")
@@ -8,6 +8,6 @@ spec = Gem::Specification.new do |s|
   s.authors = ["Abraham Heward", "Jon Utter"]
   s.email = %w{"aheward@rsmart.com" "jutter@rsmart.com"}
   s.homepage = 'https://github.com/rSmart'
-  s.add_dependency 'test-factory', '>= 0.2.9'
+  s.add_dependency 'test-factory', '>= 0.3.1'
   s.required_ruby_version = '>= 1.9.3'
 end

data/lib/kuality-coeus.rb

@@ -1,3 +1,4 @@
+require 'singleton'
 require 'test-factory'
 require 'date'
 require 'yaml'

data/lib/kuality-coeus/data_objects/award/award.rb

@@ -22,15 +22,15 @@ class AwardObject
       award_title:           random_alphanums,
       activity_type:         '::random::',
       award_type:            '::random::',
-      project_start_date:    right_now, # Note how this is the date hash, for increased flexibility of validations or tests, if necessary
-      project_end_date:      in_a_year, # Same goes for this variable. It's the entire date hash
+      project_start_date:    right_now[:date_w_slashes],
+      project_end_date:      in_a_year[:date_w_slashes],
       sponsor_id:            '::random::',
       lead_unit:             '::random::',
-      obligation_start_date: right_now, # Date hash, again
-      obligation_end_date:   in_a_year, # and again
+      obligation_start_date: right_now[:date_w_slashes],
+      obligation_end_date:   in_a_year[:date_w_slashes],
       anticipated_amount:    '1000000',
       obligated_amount:      '1000000',
-      transactions:       TransactionCollection.new
+      transactions:          collection('Transaction')
     }
 
     set_options(defaults.merge(opts))
@@ -42,11 +42,9 @@ class AwardObject
     on Award do |create|
       create.expand_all
       fill_out create, :description, :transaction_type, :award_status, :award_title,
-               :activity_type, :award_type, :obligated_amount, :anticipated_amount
-      create.project_start_date.fit @project_start_date[:date_w_slashes]
-      create.project_end_date.fit @project_end_date[:date_w_slashes]
-      create.obligation_start_date.fit @obligation_start_date[:date_w_slashes]
-      create.obligation_end_date.fit @obligation_end_date[:date_w_slashes]
+               :activity_type, :award_type, :obligated_amount, :anticipated_amount,
+               :project_start_date, :project_end_date, :obligation_start_date,
+               :obligation_end_date
       set_sponsor_id
       set_lead_unit
       create.save
@@ -66,10 +64,7 @@ class AwardObject
 
   def add_transaction opts={}
     defaults={award_id: @award_id}
-    options = defaults.merge(opts)
-    trans = make AwardTransactionObject, options
-    trans.create
-    @transactions << trans
+    @transactions.add defaults.merge(opts)
   end
 
   # ========
@@ -106,6 +101,10 @@ class AwardObject
     end
   end
 
+  # ==========
+  private
+  # ==========
+
   def navigate
     doc_search unless on_award?
     on(TimeAndMoney).return_to_award if on_tm?

data/lib/kuality-coeus/data_objects/award/award_transaction.rb

@@ -52,6 +52,8 @@ class AwardTransactionObject
 
 end # AwardTransactionObject
 
-class TransactionCollection < Array
+class TransactionCollection < CollectionsFactory
+
+  contains AwardTransactionObject
 
 end # TransactionCollection

data/lib/kuality-coeus/data_objects/budget/budget_periods.rb

@@ -8,18 +8,20 @@ class BudgetPeriodObject
   attr_accessor :number, :start_date, :end_date, :total_sponsor_cost,
                 :direct_cost, :f_and_a_cost, :unrecovered_f_and_a,
                 :cost_sharing, :cost_limit, :direct_cost_limit, :datified,
-                :budget_name
+                :budget_name, :cost_sharing_distribution_list
 
   def initialize(browser, opts={})
     @browser = browser
 
     defaults = {
-      doc_type: 'Budget Document ' # Note: the trailing space is not a typo!
+      doc_type: 'Budget Document ', # Note: the trailing space is not a typo!
+      cost_sharing_distribution_list: collection('CostSharing')
     }
 
     set_options(defaults.merge(opts))
     requires :start_date, :budget_name
     datify
+    add_cost_sharing @cost_sharing
   end
 
   def create
@@ -50,6 +52,7 @@ class BudgetPeriodObject
     end
     set_options(opts)
     datify
+    add_cost_sharing opts[:cost_sharing]
   end
 
   def delete
@@ -93,9 +96,20 @@ class BudgetPeriodObject
     @datified=Date.parse(@start_date[/(?<=\/)\d+$/] + '/' + @start_date[/^\d+\/\d+/])
   end
 
+  def add_cost_sharing(cost_sharing)
+    if !cost_sharing.nil? && cost_sharing.to_f > 0
+      cs = make CostSharingObject, project_period: @number,
+                amount: cost_sharing, source_account: '',
+                index: @cost_sharing_distribution_list.length
+      @cost_sharing_distribution_list << cs
+    end
+  end
+
 end # BudgetPeriodObject
 
-class BudgetPeriodsCollection < Array
+class BudgetPeriodsCollection < CollectionsFactory
+
+  contains BudgetPeriodObject
 
   def period(number)
     self.find { |period| period.number==number }

data/lib/kuality-coeus/data_objects/budget/budget_versions.rb

@@ -21,12 +21,10 @@ class BudgetVersionsObject
     @browser = browser
 
     defaults = {
-      name:              random_alphanums_plus(40),
-      cost_sharing:      '0.00',
-      f_and_a:           '0.00',
-      budget_periods:    BudgetPeriodsCollection.new,
-      subaward_budgets:  SubawardBudgetCollection.new,
-      personnel:         BudgetPersonnelCollection.new
+      name:                           random_alphanums_plus(40),
+      budget_periods:                 collection('BudgetPeriods'),
+      subaward_budgets:               collection('SubawardBudget'),
+      personnel:                      collection('BudgetPersonnel')
     }
 
     set_options(defaults.merge(opts))
@@ -71,11 +69,8 @@ class BudgetVersionsObject
         budget_name: @name,
         doc_type: @doc_header
     }
-    opts.merge!(defaults)
-
-    bp = create BudgetPeriodObject, opts
+    @budget_periods.add defaults.merge(opts)
     return if on(Parameters).errors.size > 0 # No need to continue the method if we have an error
-    @budget_periods << bp
     @budget_periods.number! # This updates the number value of all periods, as necessary
   end
 
@@ -98,7 +93,7 @@ class BudgetVersionsObject
     confirmation
     on Parameters do |edit|
       edit.final.fit opts[:final]
-      edit.budget_status.fit opts[:budget_status]
+      edit.budget_status.fit opts[:status]
       edit.total_direct_cost_limit.fit opts[:total_direct_cost_limit]
       # TODO: More to add here...
       edit.save
@@ -219,7 +214,9 @@ class BudgetVersionsObject
 
 end # BudgetVersionsObject
 
-class BudgetVersionsCollection < Array
+class BudgetVersionsCollection < CollectionsFactory
+
+  contains BudgetVersionsObject
 
   def budget(name)
     self.find { |budget| budget.name==name }

data/lib/kuality-coeus/data_objects/budget/cost_sharing.rb

@@ -0,0 +1,56 @@
+class CostSharingObject
+
+  include Foundry
+  include DataFactory
+  include StringFactory
+  include Navigation
+
+  attr_accessor :project_period, :percentage, :source_account, :amount, :index
+
+  def initialize(browser, opts={})
+    @browser = browser
+
+    defaults = {
+        percentage:     '0.00',
+        source_account: random_alphanums
+    }
+    set_options(defaults.merge(opts))
+  end
+
+  def create
+    view
+    on DistributionAndIncome do |page|
+      page.add_cost_share_period.set @project_period
+      # TODO: Add more here when needed
+      page.add
+    end
+  end
+
+  def view
+    # Note: Currently assumes we're already viewing
+    # the budget document!
+    on(Parameters).distribution__income
+  end
+
+  def edit(opts)
+    view
+    on DistributionAndIncome do |page|
+      page.expand_all
+      page.cost_sharing_project_period(@index).fit opts[:project_period]
+      page.cost_sharing_percentage(@index).fit opts[:percentage]
+      page.cost_sharing_source_account(@index).fit opts[:source_account]
+      page.cost_sharing_amount(@index).fit opts[:amount]
+      page.save
+    end
+    update_options(opts)
+  end
+
+end
+
+class CostSharingCollection < CollectionsFactory
+
+  contains CostSharingObject
+
+  #TODO: Write code that will update indexes when items change their order in the list.
+
+end