Matt Hess

Results 8 comments of Matt Hess

The `TokenListChecks` class is covered except for a private constructor. Not sure what we want to do about that, so moving it out of this ticket

The 'HAPI Tests (Smart Contract)' failures have nothing to do with the unit test changes in this PR. Merging

We'll want to be careful if we change this. Returning `BAD_ENCODING` is part of the public API, so there are likely some people depending on its current behavior.

@AlfredoG87 @jsync-swirlds my understanding was we're not worried about backward compatibility for now, but please confirm

> > @AlfredoG87 @jsync-swirlds my understanding was we're not worried about backward compatibility for now, but please confirm > > If we are not, then why did we kept some...

> > > > @AlfredoG87 @jsync-swirlds my understanding was we're not worried about backward compatibility for now, but please confirm > > > > > > > > > If...

Significant churn coming from doing this work while #17960 is in flight. Pulling this back until that is merged