RFR: 8267095: Miscellaneous cleanups in vm.runtime.defmeth tests [v2]

[Vladimir Ivanov]

On Tue, 25 May 2021 22:06:45 GMT, Vladimir Ivanov <vlivanov at openjdk.org> wrote:

>> Miscellaneous refactorings/cleanups in `vm.runtime.defmeth` tests.
>> 
>> In particular: 
>>   - replace `test/hotspot/jtreg/vmTestbase/vm/runtime/defmeth/scenarios` with enumeration of different test configurations in tests themselves; 
>>   - pass `TestBuilder` as a parameter and initiate test execution from executor; 
>>   - switch max class file version tested from 52 to 61 (`V17`); 
>>   - minor changes (cleanups, new test cases)
>> 
>> Testing:
>> - [x] vmTestbase/vm/runtime/defmeth/
>> - [x] hs-tier5-rt - hs-tier8-rt
>
> Vladimir Ivanov has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains seven additional commits since the last revision:
> 
>  - Merge branch 'master' into 8267095.defmeth
>  - Remove ACC_STRICT-related code
>  - Cleanup
>  - Remove scenarios
>  - FIXME removal
>  - Fixes
>  - TestBuilder as a parameter

Thanks for taking a look, Harold.
 
> JEP 306 (JDK-8175916) proposes removing ACC_STRICT for class file versions >= 61. See also #3891. So perhaps testing of ACC_STRICT should be removed from these tests?

Good point. I went ahead and removed all ACC_STRICT-related code. 

It could be kept for major version <= 60, but I don't see much sense in introducing additional test configuration.  IMO testing `ACC_STRICT` / `ACC_SYNCHRONIZED` method flags has marginal value.

-------------

PR: https://git.openjdk.java.net/jdk/pull/4008