Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix some of the failing tests in #274 #284

Merged

Conversation

mmuesly
Copy link
Collaborator

@mmuesly mmuesly commented Mar 15, 2021

I could fix 4 of the tests in a reasonable way bringing it down to 11 remaining test cases to complete #274:

    ObjectStreamTest. testSimpleReadbackOk
    RuntimeTest. testAvailableProcessors
    StackWalkerTest. testCallerClass
    DateFormatTest. testConversionCycle
    DateFormatTest. testFormatWithTimeZone
    DateFormatTest. testSetAndGetTimeZone
    SimpleDateFormatTest. testFormatWithTimeZone
    RecursiveClinitTest. testNewInstance
    ProxyTest. testAnnoProxy
    ProxyTest. testBasicProxy
    InterruptTest. testPark
    SAXParserTest. testSimpleParse 

mmuesly added 8 commits March 15, 2021 22:20
Nowadays, Java is aware of three different Class Loaders: System, Platform and Thread.
…ngatures

This ClassLoader is used in the service provider loading and helps to port the DateFormat test cases.
This method is ocassinaly used by Java internals requried to make the DateFormat conversion work.
After resoling an error in loading modules from the classes folder, the Unsafe missed some methods.
This fixes CountDownLatchTest, ExecutorServiceTest, SemaphoreTest
JPF requries nowadays a repeatedInvocation for LoadOnJPFRequired errors.
@cyrille-artho
Copy link
Member

Fantastic, thank you very much!
Travis' CI was broken, so I could not see the test results straight away. I've tried to fix it, let's see if it works next time.

@cyrille-artho cyrille-artho merged commit a992619 into javapathfinder:java-10-gradle Mar 16, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants