PR546 with Upgrade script

[KevinBKozan]

Description

• Contains contents of #55 Throw Exception when cannot resolve URL attribute #546, just adds an upgrade script for new requirements.

Contribution checklist

• Pull request has a meaningful description of its purpose
• All commits are accompanied by meaningful commit messages
• All new or changed code is covered with unit/verification tests (if applicable)
• All automated tests passed successfully (all builds on Travis CI are green)
• Changes to Framework doesn't have backward incompatible changes for tests or have related Pull Request with fixes to tests

[coveralls]

Coverage increased (+0.0006%) to 50.974% when pulling 1416667 on pr546WithUpgradeScript into f1820ff on develop.

[soumyau]

Implementation looks good. Had a question about why do we need an upgrade script in the first place. Wouldn't tests end up failing anyway without .url. There won't be any instances of such references unless it's a broken build or something, correct?

[KevinBKozan]

@soumya-ashok MFTF actually (erroneously) will replace {{page}} with the url even if you don't manually specify .url}}, so any previous occurences would be getting taken care of by the framework.
As that changes, you then need to change the input to match the expectation

[soumyau]

@soumyau MFTF actually (erroneously) will replace {{page}} with the url even if you don't manually specify .url}}, so any previous occurences would be getting taken care of by the framework.
As that changes, you then need to change the input to match the expectation

@KevinBKozan Below is the current behavior (develop) without .url. MFTF doesn't resolve reference for AdminLoginPage. Test fails due to page not found, doesn't actually go to login page -

Entering Action Group [loginAsAdmin] LoginAsAdmin
I am on page "{{AdminLoginPage}}"
I fill field "#username","admin"
FAIL

[KevinBKozan]

@soumyau I see, I was getting it mixed up, think I understand now.

Goal of this new feature is to fail early and specifically if they forget to specify .url}}. Outcome of tests is the same, so the upgrade script is pointless since presumably those tests would already be failing as well yeah?

In light of that we may be able to just kill this PR.