[SAT] ignored_fields key for basic read test

[evantahler]

Sometimes sources change data in records for each read, like a last_refresh_time column.

See #21419, #21593 and #21775 for examples

[erohmensing]

Grooming -
This is supported for Test full refresh sync - we think we want it for all tests (not just adding it to basic read test)? Is it only necessary for tests with expected records? It's a bit confusing what the support for ignored_fields in test full refresh sync is for

[alafanechere]

It's a bit confusing what the support for ignored_fields in test full refresh sync is for

You are right, I think we could either merge full_refresh test into basic read (or the opposite). In any case, ignored_fields for full refresh and ignored_fields for basic read should serve the same purpose and can be the same pydantic model field. I think we shall add a bypass_reason to explain why we're ignoring this field.

[erohmensing]

I think we shall add a bypass_reason to explain why we're ignoring this field.

+1