The process was quick, even during the holiday period.

After submitting my test, I received a rejection email with the following content:

'Quite good architecture and DI. Good unit tests and UI tests. Good work with offline mode.

What can be better:

1. Use DiffUtil instead of calling notifyDatasetChanged(). Right now, you are reloading the whole list.
2. When I select a currency, it doubles on top. In that case, it sometimes shows non-identical rates for both, which seems strange.
3. UI tests did not pass.
4. No unsubscribe on configuration changes – this can cause a crash.
5. Selected currency is not stored after configuration changes.'

I don't understand how they rejected a senior developer with quite good architecture, DI, and unit tests just because of minor bugs.

I believe the judgment wasn't objective enough.