<div dir="ltr"><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Mar 21, 2014 at 2:56 AM, Željko Filipin <span dir="ltr"><<a href="mailto:zfilipin@wikimedia.org" target="_blank">zfilipin@wikimedia.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class=""><div class="gmail_quote">On Thu, Mar 20, 2014 at 11:54 PM, Chris McMahon <span dir="ltr"><<a href="mailto:cmcmahon@wikimedia.org" target="_blank">cmcmahon@wikimedia.org</a>></span> wrote:<br>


<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"><div>Adhere to the Page Object design pattern:  keep your page element identifiers in the /support/pages/*page.rb files ONLY, and NOT in the /step_definitions/*steps.rb files. <br>



</div><div></div></blockquote></div><br></div>Is there a reason for this e-mail? Did you find a lot of violations of the pattern in the code? Since the e-mail message was sent to both QA and mobile mailing lists, my guess is that you have noticed a problem in MobileFrontend[1] repository. :)</div>


<div class="gmail_extra"><br></div><div class="gmail_extra">I am not asking because I want to blame a person or a team, I think fixing such problems is a great first contribution for a new volunteer, and I am always looking for those.</div>
</div></blockquote><div><br></div><div>Right, exactly.  I spent some time yesterday getting a few more MF tests to pass in Chrome and ran into several places where there are element locators in steps. Some are old, some are new.  You can find them in these files in the MobileFrontend/tests/browser/features/step_definitions directory: </div>
<div><br></div><div>editor_ve_steps.rb</div><div>geonotahack_steps.rb</div><div>language_steps.rb</div><div>mainmenu_steps.rb</div><div>nearby_steps.rb</div><div><br></div><div>I hope to make improvements to some of these, but refactorers are welcome. </div>
<div><br></div><div>-Chris   </div></div></div></div>