MicroJIT Regression Test Suite

[harpreetbamrah]

Regression tests for the MicroJIT compiler #9578
See also:
ibmruntimes/openj9-openjdk-jdk8#413
Issue #9582

[llxia]

Copyright error is about .class files and junit.jar. https://openj9-jenkins.osuosl.org/job/PullRequest-CopyrightCheck-OpenJ9/8096/console
We should not deliver .class file and 3rd party jar into the openj9 repo.

[harpreetbamrah]

Copyright error is about .class files and junit.jar. https://openj9-jenkins.osuosl.org/job/PullRequest-CopyrightCheck-OpenJ9/8096/console We should not deliver .class file and 3rd party jar into the openj9 repo.

Hi Lan, thanks for the clarification!
Actually we need JUnit 5 framework for running MicroJIT tests, hence the jar file as the TKG has JUnit 4. What could be a possible wayout for that?

[llxia]

@harpreetbamrah we can add junit5 jar download url in https://github.com/adoptium/TKG/blob/master/scripts/getDependencies.pl. Similar to junit4 and use it in build.xml

openj9/test/functional/Jsr292/build.xml

Lines 43 to 44 in b975e82

	<property name="LIB" value="junit4,asm_all,jcommander,testng"/>
	<import file="${TEST_ROOT}/TKG/scripts/getDependencies.xml"/>

openj9/test/functional/Jsr292/build.xml

Line 52 in b975e82

<target name="compile_generator" depends="init,getDependentLibs" description="Compile SimpleIndyGenerator">

[harpreetbamrah]

@harpreetbamrah we can add junit5 jar download url in https://github.com/adoptium/TKG/blob/master/scripts/getDependencies.pl. Similar to junit4 and use it in build.xml

openj9/test/functional/Jsr292/build.xml

Lines 43 to 44 in b975e82

	<property name="LIB" value="junit4,asm_all,jcommander,testng"/>
	<import file="${TEST_ROOT}/TKG/scripts/getDependencies.xml"/>

openj9/test/functional/Jsr292/build.xml

Line 52 in b975e82

<target name="compile_generator" depends="init,getDependentLibs" description="Compile SimpleIndyGenerator">

Hi Lan, do we need a separate pull request to add JUnit 5 jar to TKG?

[llxia]

Yes, we need separate PR in TKG for downloading junit5 jar. An example is in #16726 (comment)

Also, we have to clean up .class files in the PR.