You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@harmony.apache.org by "Elena Semukhina (JIRA)" <ji...@apache.org> on 2007/03/29 11:34:25 UTC

[jira] Created: (HARMONY-3521) [drlvm][kernel tests] 44 excluded kernel tests should be enabled

[drlvm][kernel tests] 44 excluded kernel tests should be enabled
----------------------------------------------------------------

                 Key: HARMONY-3521
                 URL: https://issues.apache.org/jira/browse/HARMONY-3521
             Project: Harmony
          Issue Type: Bug
          Components: DRLVM
            Reporter: Elena Semukhina


All  RuntimeAdditionalTest* (total 44 cases) are now excluded with the comment that the tests are unstable and hang often.

They should be revised and enabled.


-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (HARMONY-3521) [drlvm][kernel tests] 44 excluded kernel tests should be enabled

Posted by "Elena Semukhina (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/HARMONY-3521?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Elena Semukhina updated HARMONY-3521:
-------------------------------------

    Attachment: H-3521_runtime_tests_excluded.patch
                H-3521_runtime_tests.patch

I attached the patch for 23 tests which work for me and could be enabled.
I also attached the patch for excluded lists which adds the fixed tests to test runs.

I must note here that I consider this patch as a first step towards enabling the excluded tests.


> [drlvm][kernel tests] 44 excluded kernel tests should be enabled
> ----------------------------------------------------------------
>
>                 Key: HARMONY-3521
>                 URL: https://issues.apache.org/jira/browse/HARMONY-3521
>             Project: Harmony
>          Issue Type: Bug
>          Components: DRLVM
>            Reporter: Elena Semukhina
>         Attachments: H-3521_runtime_tests.patch, H-3521_runtime_tests_excluded.patch
>
>
> All  RuntimeAdditionalTest* (total 44 cases) are now excluded with the comment that the tests are unstable and hang often.
> They should be revised and enabled.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (HARMONY-3521) [drlvm][test][kernel] 44 excluded kernel tests should be enabled

Posted by "Gregory Shimansky (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/HARMONY-3521?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Gregory Shimansky updated HARMONY-3521:
---------------------------------------

    Issue Type: Test  (was: Bug)

> [drlvm][test][kernel] 44 excluded kernel tests should be enabled
> ----------------------------------------------------------------
>
>                 Key: HARMONY-3521
>                 URL: https://issues.apache.org/jira/browse/HARMONY-3521
>             Project: Harmony
>          Issue Type: Test
>          Components: DRLVM
>            Reporter: Elena Semukhina
>         Attachments: H-3521_runtime_tests.patch, H-3521_runtime_tests_excluded.patch
>
>
> All  RuntimeAdditionalTest* (total 44 cases) are now excluded with the comment that the tests are unstable and hang often.
> They should be revised and enabled.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (HARMONY-3521) [drlvm][test][kernel] move RuntimeAdditionalTest to DRLVM regression suite

Posted by "Gregory Shimansky (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HARMONY-3521?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12526446 ] 

Gregory Shimansky commented on HARMONY-3521:
--------------------------------------------

I looked at these tests and it seems to me that they have a very low quality.

In particular they rely on stuff that isn't guaranteed to be present, e.g.

1. There is a hardcoded path C:\Documents and Settings
2. On Windows they require Cygwin
3. On Linux they try to find java executable using "find" command, which may return /usr/lib/java (a directory) instead of a java executable which breaks tests again.
4. They rely on some utilities like tree that aren't present in all distributions.

I don't see a value to enable them and suggest to close this bug as won't fix.

> [drlvm][test][kernel] move RuntimeAdditionalTest to DRLVM regression suite
> --------------------------------------------------------------------------
>
>                 Key: HARMONY-3521
>                 URL: https://issues.apache.org/jira/browse/HARMONY-3521
>             Project: Harmony
>          Issue Type: Test
>          Components: DRLVM
>            Reporter: Elena Semukhina
>         Attachments: H-3521_runtime_tests.patch, H-3521_runtime_tests_excluded.patch
>
>
> All  RuntimeAdditionalTest* (total 44 cases) are now excluded with the comment that the tests are unstable and hang often.
> They should be revised and enabled.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (HARMONY-3521) [drlvm][test][kernel] 44 excluded kernel tests should be enabled

Posted by "Vladimir Beliaev (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HARMONY-3521?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12503078 ] 

Vladimir Beliaev commented on HARMONY-3521:
-------------------------------------------

I believe, including back these 44 (or more) Kernel tests is to be done after these tests refactoring (say, after HARMONY-1834 is applied)... 

There should be someone running this (pretty huge) task based on available time, etc... I mean, meanwhile I'm going to move the updated tests to DRLVM Regression Test Suite (as Alexey proposed), rename this bug (to say "Fixing RuntimeAdditionalTest") & then close this bug as FIXED.

So again, the problem with "excluded tests" will be kept, but this should be a different story...

Thanks
Vladimir Beliaev



> [drlvm][test][kernel] 44 excluded kernel tests should be enabled
> ----------------------------------------------------------------
>
>                 Key: HARMONY-3521
>                 URL: https://issues.apache.org/jira/browse/HARMONY-3521
>             Project: Harmony
>          Issue Type: Test
>          Components: DRLVM
>            Reporter: Elena Semukhina
>         Attachments: H-3521_runtime_tests.patch, H-3521_runtime_tests_excluded.patch
>
>
> All  RuntimeAdditionalTest* (total 44 cases) are now excluded with the comment that the tests are unstable and hang often.
> They should be revised and enabled.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (HARMONY-3521) [drlvm][kernel tests] 44 excluded kernel tests should be enabled

Posted by "Alexey Varlamov (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HARMONY-3521?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12485969 ] 

Alexey Varlamov commented on HARMONY-3521:
------------------------------------------

Elena, what about moving these RuntimeAdditionalTest* to regression suite? Looking at their code, I'd say they are really chasing some subtle issues/regressions in Runtime.Process impl and do nothing more.
IMO they are useless in pre-commit testing but take so much time.

> [drlvm][kernel tests] 44 excluded kernel tests should be enabled
> ----------------------------------------------------------------
>
>                 Key: HARMONY-3521
>                 URL: https://issues.apache.org/jira/browse/HARMONY-3521
>             Project: Harmony
>          Issue Type: Bug
>          Components: DRLVM
>            Reporter: Elena Semukhina
>         Attachments: H-3521_runtime_tests.patch, H-3521_runtime_tests_excluded.patch
>
>
> All  RuntimeAdditionalTest* (total 44 cases) are now excluded with the comment that the tests are unstable and hang often.
> They should be revised and enabled.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (HARMONY-3521) [drlvm][test][kernel] move RuntimeAdditionalTest to DRLVM regression suite

Posted by "Alexei Zakharov (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/HARMONY-3521?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Alexei Zakharov updated HARMONY-3521:
-------------------------------------

    Summary: [drlvm][test][kernel] move RuntimeAdditionalTest to DRLVM regression suite  (was: [drlvm][test][kernel] 44 excluded kernel tests should be enabled)

summary changed according to Vladimir's proposal

> [drlvm][test][kernel] move RuntimeAdditionalTest to DRLVM regression suite
> --------------------------------------------------------------------------
>
>                 Key: HARMONY-3521
>                 URL: https://issues.apache.org/jira/browse/HARMONY-3521
>             Project: Harmony
>          Issue Type: Test
>          Components: DRLVM
>            Reporter: Elena Semukhina
>         Attachments: H-3521_runtime_tests.patch, H-3521_runtime_tests_excluded.patch
>
>
> All  RuntimeAdditionalTest* (total 44 cases) are now excluded with the comment that the tests are unstable and hang often.
> They should be revised and enabled.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Closed: (HARMONY-3521) [drlvm][test][kernel] move RuntimeAdditionalTest to DRLVM regression suite

Posted by "Gregory Shimansky (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/HARMONY-3521?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Gregory Shimansky closed HARMONY-3521.
--------------------------------------

    Resolution: Won't Fix
      Assignee: Gregory Shimansky

Ok it seems Alexey agrees with me. I'm closing this issue for now. Maybe some time later these tests may be revised... not likely though.

> [drlvm][test][kernel] move RuntimeAdditionalTest to DRLVM regression suite
> --------------------------------------------------------------------------
>
>                 Key: HARMONY-3521
>                 URL: https://issues.apache.org/jira/browse/HARMONY-3521
>             Project: Harmony
>          Issue Type: Test
>          Components: DRLVM
>            Reporter: Elena Semukhina
>            Assignee: Gregory Shimansky
>         Attachments: H-3521_runtime_tests.patch, H-3521_runtime_tests_excluded.patch
>
>
> All  RuntimeAdditionalTest* (total 44 cases) are now excluded with the comment that the tests are unstable and hang often.
> They should be revised and enabled.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (HARMONY-3521) [drlvm][test][kernel] 44 excluded kernel tests should be enabled

Posted by "Gregory Shimansky (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/HARMONY-3521?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Gregory Shimansky updated HARMONY-3521:
---------------------------------------

    Summary: [drlvm][test][kernel] 44 excluded kernel tests should be enabled  (was: [drlvm][kernel tests] 44 excluded kernel tests should be enabled)

> [drlvm][test][kernel] 44 excluded kernel tests should be enabled
> ----------------------------------------------------------------
>
>                 Key: HARMONY-3521
>                 URL: https://issues.apache.org/jira/browse/HARMONY-3521
>             Project: Harmony
>          Issue Type: Bug
>          Components: DRLVM
>            Reporter: Elena Semukhina
>         Attachments: H-3521_runtime_tests.patch, H-3521_runtime_tests_excluded.patch
>
>
> All  RuntimeAdditionalTest* (total 44 cases) are now excluded with the comment that the tests are unstable and hang often.
> They should be revised and enabled.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (HARMONY-3521) [drlvm][test][kernel] move RuntimeAdditionalTest to DRLVM regression suite

Posted by "Alexey Varlamov (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HARMONY-3521?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12526681 ] 

Alexey Varlamov commented on HARMONY-3521:
------------------------------------------

+1

> [drlvm][test][kernel] move RuntimeAdditionalTest to DRLVM regression suite
> --------------------------------------------------------------------------
>
>                 Key: HARMONY-3521
>                 URL: https://issues.apache.org/jira/browse/HARMONY-3521
>             Project: Harmony
>          Issue Type: Test
>          Components: DRLVM
>            Reporter: Elena Semukhina
>         Attachments: H-3521_runtime_tests.patch, H-3521_runtime_tests_excluded.patch
>
>
> All  RuntimeAdditionalTest* (total 44 cases) are now excluded with the comment that the tests are unstable and hang often.
> They should be revised and enabled.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.