Passing trim_whitespace=True was my solution to issues like this when I created the patch.
On Wed, May 12, 2021 at 10:14 AM Lijuan Tu <lijuan.tu@intel.com> wrote:
>
> This reverts commit f498f50a62e30f6f8fb9c4e1a759e3d35861b978.
> As it casued some cases failed:
> * flow_classify_softnic
> * test_ipv4_acl_jump
> * test_ipv4_acl_table
> * test_ipv6_hash_jump
> * unit_tests_loopback
> * test_link_mode
> * test_loopback_mode
> * distributor
> * maximum_workers
Any update on having both worlds (UNH lab and those tests) coexist?
I see an exhaustive list of tests that are broken with the new behavior.
Another approach to reverting the whole change is to pass
trim_whitespace= in the affected tests.
Was this idea considered?
--
David Marchand