<div dir="ltr"><p dir="ltr">I noticed this when writing tests for lead photo uploads where I had to create a LeadPhotoPage (to represent a page which has no lead photo)... [1]</p><p>I agree we should only use page exists but use more generic names</p>
<p>e.g. instead of Barack Obama article</p><p>PageWithInfobox</p><p>or PageWithMultipleSections etc..</p><p dir="ltr">[1] <a href="https://gerrit.wikimedia.org/r/105106">https://gerrit.wikimedia.org/r/105106</a></p>
<p dir="ltr">~ Be strong. Not bold. Bold has taken a new semantic meaning and it is time for change.</p>
<div class="gmail_quote">On 14 Jan 2014 13:40, "Juliusz Gonera" <<a href="mailto:jgonera@wikimedia.org" target="_blank">jgonera@wikimedia.org</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">

Hi,<br>
<br>
I was wondering if we need that many PageObjects in mobile browser tests. In my opinion we should only have PageObjects for pages that actually exist, such as:<br>
<br>
* HomePage (or MainPage)<br>
* ArticlePage<br>
* UploadsPage<br>
* etc.<br>
<br>
There are a bunch of PageObjects that don't actually represent pages, but kind of mirror features:<br>
<br>
* CreateArticlePage<br>
* EditPage<br>
* LanguagePage<br>
* NotificationPage<br>
* RandomPage<br>
<br>
I think all the elements from those page objects belong to the ArticlePage. Furthermore, to avoid repeating things that are shared everywhere (e.g. notifications icon), we could have a BasePage from which all the other page objects could inherit. What do you think? I can work on refactoring this if you agree.<br>


<br>
-- <br>
Juliusz<br>
<br>
______________________________<u></u>_________________<br>
QA mailing list<br>
<a href="mailto:QA@lists.wikimedia.org" target="_blank">QA@lists.wikimedia.org</a><br>
<a href="https://lists.wikimedia.org/mailman/listinfo/qa" target="_blank">https://lists.wikimedia.org/<u></u>mailman/listinfo/qa</a><br>
</blockquote></div>
</div>