Test automation discussion

Created a test for checking that the discussion module is properly loaded on an article page
For the findByTestAttribute. The implicit wait only waits until the element is present, not displayed/visible. Right? I added this when I was dealing with JavaScript elements which were hidden/unhidden and wanted to make sure that the element is checked that it is displayed as well as present.

On the viewAllComments. Where is the extra explicit wait? If you are referring to line 20 and 26, both are needed because in one case I am clicking on the commentsViewType element, which should make the viewAllComments element dissapear. And on the other one I am clicking on the viewAllComments which should make itself dissapear.

Am I missing something here?