Details

    • Type: Task Task
    • Status: Closed Closed
    • Priority: Major Major
    • Resolution: Incomplete
    • Affects Version/s: None
    • Fix Version/s: 1.0
    • Component/s: Development
    • Labels:
      None
    • Similar issues:
      KULRICE-2269Write Unit Tests for KIM Services
      KULRICE-1889KIM Test Data
      KULRICE-4926Create Unit Test for org.kuali.rice.kim.bo.group.impl.GroupMemberImpl
      KULRICE-4468Create Unit Test for org.kuali.rice.kim.bo.group.impl.GroupAttributeDataImpl
      KULRICE-5343Create additional unit tests for KIM
      KULRICE-2352Create/expand KIM Test Data
      KULRICE-13862Improve the KIM unit tests
      KULRICE-3688Write the unit tests necessary to test data dictionary doc search
      KULRICE-4922Create Unit Test for org.kuali.rice.kns.web.struts.pojo.ArrayUtils
      KULRICE-4466Create Unit Test for org.kuali.rice.kns.util.PatternedStringBuilder
    • Rice Module:
      KIM
    • Application Requirement:
      Rice

      Description

      KIM needs a good set of test data and unit tests on which to run. There is a start on some test data in scripts/ddl/kim_authz_test_data.sql but it needs to be expanded considerably.

      See sub-tasks for more details.

        Issue Links

          Activity

          Hide
          Bryan Hutchinson added a comment -

          Lin-Long/Chitra,

          Here's your first KIM task. It looks like it could be broken up into multiple sub-tasks, so we probably want to think about how to break it down and how to divide up the work.

          Show
          Bryan Hutchinson added a comment - Lin-Long/Chitra, Here's your first KIM task. It looks like it could be broken up into multiple sub-tasks, so we probably want to think about how to break it down and how to divide up the work.
          Hide
          Bryan Hutchinson added a comment -

          Jonathan,

          I've broken this issue down into several sub-tasks. Most of the descriptions are ok, but we need more detail for what is required in the Identity Service Test and Authentication Service Test (the original description only had a ? for those). Thanks.

          Show
          Bryan Hutchinson added a comment - Jonathan, I've broken this issue down into several sub-tasks. Most of the descriptions are ok, but we need more detail for what is required in the Identity Service Test and Authentication Service Test (the original description only had a ? for those). Thanks.
          Hide
          Jonathan Keller added a comment -

          I would say to hold off on those (Identity and Authentication) for now. Identity is still in flux - Scott's looking into that - and I don't know how to test the authN service. Really authZ (which could be split into Role, Permission, and Responsibility now) is the main place that we need some robust testing.

          Also, I just loaded the most recent KFS data into the KR_KIM role and permission data into KULDEV and KULDBA. I know it's not KC data, but it's the best model of how we are planning to load these tables that I have at present.

          Show
          Jonathan Keller added a comment - I would say to hold off on those (Identity and Authentication) for now. Identity is still in flux - Scott's looking into that - and I don't know how to test the authN service. Really authZ (which could be split into Role, Permission, and Responsibility now) is the main place that we need some robust testing. Also, I just loaded the most recent KFS data into the KR_KIM role and permission data into KULDEV and KULDBA. I know it's not KC data, but it's the best model of how we are planning to load these tables that I have at present.
          Hide
          Bryan Hutchinson added a comment -

          I've closed the Identity and Authentication sub-tasks for now. I've also renamed AuthZ to Permission and added a Responsibility test sub-task (need more info on this one), so now we have Role, Permission and Responsibility test sub-tasks.

          Show
          Bryan Hutchinson added a comment - I've closed the Identity and Authentication sub-tasks for now. I've also renamed AuthZ to Permission and added a Responsibility test sub-task (need more info on this one), so now we have Role, Permission and Responsibility test sub-tasks.
          Hide
          Eric Westfall added a comment -

          I've fixed all of the existing KIM unit tests in Rice so that they pass now. However I suspect that there's still some additional unit tests that could be done here but I don't think we are going to have much time to do that for Rice 1.0. I'm going to mark this issue as resolved and create a placeholder issue for 1.1 in regards to improving the coverage of our KIM tests (i'll link it with this issue).

          Show
          Eric Westfall added a comment - I've fixed all of the existing KIM unit tests in Rice so that they pass now. However I suspect that there's still some additional unit tests that could be done here but I don't think we are going to have much time to do that for Rice 1.0. I'm going to mark this issue as resolved and create a placeholder issue for 1.1 in regards to improving the coverage of our KIM tests (i'll link it with this issue).
          Hide
          Eric Westfall added a comment -

          Bulk change of all Rice 1.0 issues to closed after public release.

          Show
          Eric Westfall added a comment - Bulk change of all Rice 1.0 issues to closed after public release.

            People

            • Assignee:
              Unassigned
              Reporter:
              Jonathan Keller
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Due:
                Created:
                Updated:
                Resolved:

                Time Tracking

                Estimated:
                Original Estimate - 4 days, 6 hours
                4d 6h
                Remaining:
                Time Spent - 1 day, 2 hours Remaining Estimate - 3 days, 4 hours
                3d 4h
                Logged:
                Time Spent - 1 day, 2 hours Remaining Estimate - 3 days, 4 hours
                1d 2h

                  Structure Helper Panel