Kuali Rice Development
  1. Kuali Rice Development
  2. KULRICE-12792

AFT Failure LabsLookupsAft Bookmark has values in input field Nav does not

    Details

    • Type: Bug Fix Bug Fix
    • Status: Closed Closed
    • Priority: Critical Critical
    • Resolution: Fixed
    • Affects Version/s: 2.4.2
    • Fix Version/s: 2.4.2
    • Component/s: Development, Regression
    • Security Level: Public (Public: Anyone can view)
    • Labels:
      None
    • Similar issues:
      KULRICE-12510AFT Failure LabsLookupsAft Lookups, etc. Ajax Field Query Custom Method and Service loading new data over old
      KULRICE-12503AFT Failure CreateNewAgenda Agenda Type input no longer displayed
      KULRICE-13056AFT Failure LabsQuickFinderCallbackAft lots of commas and repeated values in input fields
      KULRICE-11786AFT Failure No values match this search no longer displayed in many tests, DemoLookUpOperatorsAft too, plus commas added to input fields
      KULRICE-12682AFT Failure LabsLookupsAft and DemoClientResponsivenessAjaxFieldQueryAft Ajax Field Query Custom Methods stopped working
      KULRICE-14196AFT Failure: ConfigComponentActionListBkMrkAft testComponentRecallToActionList_WithPendingPersonApprove_Bookmark
      KULRICE-12504AFT Failure ConfigComponent extra methodToCall save input in header
      KULRICE-11538AFT Failure Rice Sampleapp Administration Identity Role Bookmark doesn't render return value for Kim Type Lookup when hideReturnLink=false
      KULRICE-12043Sorting for Light Table with Basic Inputs does not consider updated values of fields
      KULRICE-13735Fill AFT Gap: Kitchen Sink: Input Fields (part 3)
    • Sprint:
      Core 2.5.0-m4 Sprint 1
    • KAI Review Status:
      Not Required
    • KTI Review Status:
      Not Required
    • Code Review Status:
      Not Required

      Description

      Only the bookmark test has values already in the fields.

      AFT Step: Open /kr-krad/uicomponents?viewId=UifCompView&pageId=UifCompView-Page6&lightbox=true#UifCompView-Page6
      Notice first 4 input fields all have values. Navigating to page and 4 first 4 input fields are all empty, behavior should be the same for nav and bookmark.

      LabsLookupsAft.testLookupsBookmark
      org.kuali.rice.krad.labs.kitchensink.LabsLookupsAft.testLookupsBookmark
      http://ci.rice.kuali.org/job/rice-2.4-smoke-test-krad-labs/lastCompletedBuild/testReport/org.kuali.rice.krad.labs.kitchensink/LabsLookupsAft/testLookupsBookmark/

        Issue Links

          Activity

          Hide
          Sona Sona (Inactive) added a comment -

          Erik M has a fix and will be checking it in.

          I am looking at https://jira.kuali.org/browse/KULRICE-12792
          [12:57:49 PM] sona.muthuswamy: and it looks like clearTextByName("field72"); was changed to assertEmptyInputByName
          [12:58:06 PM] sona.muthuswamy: Did the functionality change or was the test wrong?
          [12:59:23 PM] EGHM: Ah, I assumed that the test was wrong, but could be bug. I doubt it was intensional since the bookmark and nav functionality of the page is different
          [1:02:02 PM] sona.muthuswamy: the change was made on 5th June as part of AFT Analysis failures. I am a little confused since the URL for the lookup page and the bookmark URL are slightly different. This is the URL when you go from the kitchen sink

          http://localhost:8080/krad-dev/kr-krad/uicomponents?viewId=UifCompView&formKey=691e067b-ea52-4af1-b9a4-9c2f02905f96&cacheKey=koyn9g17u4vlstgk8x23st76i5&pageId=UifCompView-Page6

          and this is the Bookmark URL

          http://localhost:8080/krad-dev/kr-krad/uicomponents?viewId=UifCompView&formKey=54ee5765-b299-4cdc-92fc-70ef4da067cf&cacheKey=n43hh93ao0nqaa0b0ni12rkny7&pageId=UifCompView-Page6#UifCompView-Page6
          [1:02:19 PM] sona.muthuswamy: Notice there is an extra #UifCompView-Page6 at the end of the bookmark URL
          [1:03:13 PM] EGHM: does the bookmark test pass if the extra bit is removed?
          [1:04:48 PM] sona.muthuswamy: I am not sure where that is coming from. The only difference I see is that the formKey is gone in the latest update.
          [1:08:57 PM] EGHM: okay, looks like the clears were right and the extra bit on the bookmark url needs to come off, I've to it working here, I can commit it
          [1:09:23 PM] sona.muthuswamy: ok. Thanks for taking a look.
          [1:10:54 PM] EGHM: np, thanks for the details

          Show
          Sona Sona (Inactive) added a comment - Erik M has a fix and will be checking it in. I am looking at https://jira.kuali.org/browse/KULRICE-12792 [12:57:49 PM] sona.muthuswamy: and it looks like clearTextByName("field72"); was changed to assertEmptyInputByName [12:58:06 PM] sona.muthuswamy: Did the functionality change or was the test wrong? [12:59:23 PM] EGHM: Ah, I assumed that the test was wrong, but could be bug. I doubt it was intensional since the bookmark and nav functionality of the page is different [1:02:02 PM] sona.muthuswamy: the change was made on 5th June as part of AFT Analysis failures. I am a little confused since the URL for the lookup page and the bookmark URL are slightly different. This is the URL when you go from the kitchen sink http://localhost:8080/krad-dev/kr-krad/uicomponents?viewId=UifCompView&formKey=691e067b-ea52-4af1-b9a4-9c2f02905f96&cacheKey=koyn9g17u4vlstgk8x23st76i5&pageId=UifCompView-Page6 and this is the Bookmark URL http://localhost:8080/krad-dev/kr-krad/uicomponents?viewId=UifCompView&formKey=54ee5765-b299-4cdc-92fc-70ef4da067cf&cacheKey=n43hh93ao0nqaa0b0ni12rkny7&pageId=UifCompView-Page6#UifCompView-Page6 [1:02:19 PM] sona.muthuswamy: Notice there is an extra #UifCompView-Page6 at the end of the bookmark URL [1:03:13 PM] EGHM: does the bookmark test pass if the extra bit is removed? [1:04:48 PM] sona.muthuswamy: I am not sure where that is coming from. The only difference I see is that the formKey is gone in the latest update. [1:08:57 PM] EGHM: okay, looks like the clears were right and the extra bit on the bookmark url needs to come off, I've to it working here, I can commit it [1:09:23 PM] sona.muthuswamy: ok. Thanks for taking a look. [1:10:54 PM] EGHM: np, thanks for the details
          Hide
          Sona Sona (Inactive) added a comment -

          Applied to 2.4

          Show
          Sona Sona (Inactive) added a comment - Applied to 2.4

            People

            • Assignee:
              Sona Sona (Inactive)
              Reporter:
              Erik Meade
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Time Tracking

                Estimated:
                Original Estimate - 4 hours
                4h
                Remaining:
                Time Spent - 1 hour Remaining Estimate - 3 hours
                3h
                Logged:
                Time Spent - 1 hour Remaining Estimate - 3 hours
                1h

                  Agile

                    Structure Helper Panel