> Please review this test enhancement.
> Implemented the padding-tolerant Unified Logging assertions.
> A bunch of  Unified Logging assertions do not account for possible extra 
> padding in the `[<tags>]` section.
> 
> Changed short-tag UL assertions from rigid `shouldContain(...)` to 
> `shouldMatch(...)` with patterns like:
> 
> \[class,load *\] ...
> 
> 
> Covered short categories including `class,load`, `nmt`, `pagesize`, `aot`, 
> `jni`, `cds`, and `trimnative`. I left remaining `shouldContain()` hits alone 
> where the tag set is long enough - `class,resolve`,  `verification`, 
> `gc,heap,coops`, etc.).
> 
> Also updated a couple of existing rigid `shouldMatch()` UL regexes for the 
> same padding issue.
> 
> ---------
> - [x] I confirm that I make this contribution in accordance with the [OpenJDK 
> Interim AI Policy](https://openjdk.org/legal/ai).

Ivan Bereziuk has updated the pull request incrementally with two additional 
commits since the last revision:

 - update copyright year
 - fix more places with tolarance for padding in UL checks

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/31338/files
  - new: https://git.openjdk.org/jdk/pull/31338/files/bd3eca0b..f9c5c607

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=31338&range=02
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=31338&range=01-02

  Stats: 41 lines in 27 files changed: 1 ins; 0 del; 40 mod
  Patch: https://git.openjdk.org/jdk/pull/31338.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/31338/head:pull/31338

PR: https://git.openjdk.org/jdk/pull/31338

Reply via email to