Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Anvil testcase for activityIndicator test Suite #4806

Merged
merged 8 commits into from Nov 13, 2013

Conversation

shameer49
Copy link
Contributor

Testcases:

TIMOB-6092
TIMOB-7024

activityIndicator.color = 'blue';
valueOf(testRun, activityIndicator.color).shouldBe('blue');
valueOf(testRun, function(){
activityIndicator.hide();
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

activityIndicator should be added to the window before calling hide() or show() on it. It would be a no-op otherwise.

@srahim
Copy link
Contributor

srahim commented Oct 25, 2013

Just one minor comment. Did the second test case work as expected on all platforms ?

@shameer49
Copy link
Contributor Author

We are moved this suite into android specific by @lokeshchdhry's suggestion.But it works fine on iphone os also.

@shameer49
Copy link
Contributor Author

@srahim File changed.Thank you

}).shouldNotThrowException();

finish(testRun);
win.open();
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

discussed with @shameer49 about the change required here.

@srahim
Copy link
Contributor

srahim commented Nov 5, 2013

Alright finally everything looks fine approved

Ready for @lokeshchdhry to do final FR and merge when QE is ready.

@lokeshchdhry
Copy link
Contributor

@shameer49 - Tests "editableFalse" ; "hasTextInIfStatement" ; "textFieldInScrollview" ; "focusAndBlurEvents" are timing out. Check the tests if it times out for you too.

@lokeshchdhry
Copy link
Contributor

@shameer49 - Oh ok , I got confused I ran the code from your last PR in the list but you had code for UI_textfield.

@lokeshchdhry
Copy link
Contributor

Will be re-testing once ticket TIMOB-15700 is fixed. Merging as of now.

lokeshchdhry pushed a commit that referenced this pull request Nov 13, 2013
Anvil testcase for activityIndicator test Suite
@lokeshchdhry lokeshchdhry merged commit 0c05b59 into tidev:master Nov 13, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants