3 commits

Author	SHA1	Message	Date
Cmcmahon	4961e32b53	QA: add modern assertions to flyout tests ... "Then" steps should always contains RSpec assertions. In the past this meant using some version of the word "should", but the modern convention in RSpec is to use "expect()" as I did here. Change-Id: I1c438a310803e29c0985328a76d2ec08578e0cb0	2014-09-23 14:14:47 -07:00
Cmcmahon	b3df4d0814	QA: Refactoring and making the nojs test dtrt ... * add @custom_browser tag so we don't create an extra browser instance * remove @chrome tag because this only works in Firefox * alphabetize within G/W/T buckets in commons_steps.rb * alphabetize flyout_steps.rb * bring env.rb up to date with the modern situation * make assertions about the URL and heading on the final destination page * update Ruby gems, especially mediawiki-selenium with a @custom_browser improvement ** use modern RSpec assert syntax *** we're going to have to update all the repos for this soon Change-Id: I6cb20126a662bd2a20e2b4bbebe91e8f3b04c7df	2014-09-23 13:35:50 -07:00
jdlrobson	bb6eef198c	QA: Add some basic browser tests ... yippee ki yay! Tests basic functionality 1) presence of Echo button 2) behaviour of clicking Echo button Change-Id: I6334264e1ee73a59da1e7b0c3f18a1d43b565f3a	2014-08-04 16:22:01 -07:00