I was thinking about the tests. Putting the conditions in the method make it more reusable and easier to test the coditions of showing or not showing the banner. Maybe, to improve the cohesion, we can return a boolean.

What do you think?