Harmony
  1. Harmony
  2. HARMONY-1670

[classlib][luni] invalid testcase in ClassTest

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Trivial Trivial
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: Classlib
    • Labels:
      None

      Description

      The test_getDeclaredMethodLjava_lang_String$Ljava_lang_Class of modules/luni/src/test/java/org/apache/harmony/luni/tests/java/lang/ClassTest.java
      checks if IllegalAccessException is thrown during invokation of private method of inner class. This is bad due to:
      1) the check is invalid, see HARMONY-1309 for details;
      2) The ClassTest is not appropriate for testing reflection, and this is already covered in j.l.reflect.* tests.

      1. H-1670.patch
        0.9 kB
        Alexey Varlamov

        Activity

        Alexey Varlamov created issue -
        Hide
        Alexey Varlamov added a comment -

        The DRLVM fails this test, accordingly to specifications.

        Show
        Alexey Varlamov added a comment - The DRLVM fails this test, accordingly to specifications.
        Alexey Varlamov made changes -
        Field Original Value New Value
        Attachment H-1670.patch [ 12342173 ]
        Hide
        Alexei Fedotov added a comment -

        [drlvm][unit] Blocks http://wiki.apache.org/harmony/Unit_Tests_Pass_on_DRLVM

        This is a low risk test fix.

        Show
        Alexei Fedotov added a comment - [drlvm] [unit] Blocks http://wiki.apache.org/harmony/Unit_Tests_Pass_on_DRLVM This is a low risk test fix.
        Hide
        Mikhail Loenko added a comment -

        the patch applied in revision 463186
        Alexey, please check that it applied correctly

        Show
        Mikhail Loenko added a comment - the patch applied in revision 463186 Alexey, please check that it applied correctly
        Mikhail Loenko made changes -
        Assignee Mikhail Loenko [ mloenko ]
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Hide
        Alexey Varlamov added a comment -

        Verified.

        Show
        Alexey Varlamov added a comment - Verified.
        Mikhail Loenko made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

          • Assignee:
            Mikhail Loenko
            Reporter:
            Alexey Varlamov
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development