-
Notifications
You must be signed in to change notification settings - Fork 9
check_region of entire element fails in versions apper 3.12.0 #32
Comments
Might this be the reason why I can't use Check Region By Element on native apps? And if so, any idea when it might be resolved? |
Hi @nunessf, actually this functionality wasn't tested yet. Which error did you get? |
It throws this exception: On appium logs:
|
Ok, thanks. I see there many things should be changed to support this feature. Currently, I'm busy with other tasks, could make this fix in a week... |
Alright, thank you! |
Hello, Any news about the problem reported by @nunessf ? |
hi @ludovicmcosta! Have you tried 4.x version of sdk? |
Yes, I have the following: eyes-selenium 4.0.8 Should the problem be fixed in this version? |
Oh, sorry, just checked that this is related to native apps. This behave for native apps isn't supported yet. |
Thanks for the quick answer :) The problem reported by @nunessf was also obtained when we execute the example in https://applitools.com/tutorials/appium-native-python.html#run-your-first-test. When method eyes.open() is executed the error below is displayed (appium logs). Is this issue related to native apps as well? [debug] [W3C (14623234)] Calling AppiumDriver.execute() with args: ["\n var height = undefined;\n var width = undefined;\n if (window.innerHeight) {\n height = window.innerHeight;\n } else if (document.documentElement && document.documentElement.clientHeight) {\n height = document.documentElement.clientHeight;\n } else {\n var b = document.getElementsByTagName('body')[0];\n if (b.clientHeight) {height = b.clientHeight;}\n }\n if (window.innerWidth) {\n width = window.innerWidth;\n } else if (document.documentElement && document.documentElement.clientWidth) {\n width = document.documentElement.clientWidth;\n } else {\n var b = document.getElementsByTagName('body')[0];\n if (b.clientWidth) {\n width = b.clientWidth;}\n }\n return [width, height];",[],"14623234-013b-4d3f-b34b-d05282a0d447"] |
Do you use appium python driver or original selenium for mobile testing? Seems it's selenium. Try to use appium-python-client for mobile testing. Also, good option would be to update library to 4.x version
|
I am using appium driver and sdk version = 4.0.8. I did the example https://applitools.com/tutorials/appium-native-java.html and the same problem was observed. It looks like that Android native apps are not supported by applitools SDK. |
@ludovicmcosta I've copied code from tutroial with small modification for version 4.x and I it run as expected. Code:
Could you provide exactly code that you run? |
Below you can see the executed code, logs captured during the execution and the console output. As you can see the sdk is not able to extract device pixel ratio, returning the error message: Method is not implemented. The results are sent to applitools but several logs are created in appium and it can be a problem if I do parallel test with multiple devices. Code:
Logs:
Console output:
|
Ok I see. The log output should be improved.
The warning messages output that you've see after test isn't relevant for your case. Warning don't stop tests execution. My assumption that you've changed relevant API keys here, because with wrong keys program should fail earlier and explicitly.
Regarding results, please check your eyes dashboard or the link below logs in the 'See details at' section. You'll find results of test execution there.
|
Yes, I can see the results on eyes dashboard. However at each eyes.open() the error "Encountered internal error running command: NotImplementedError: Method is not implemented" is logged in the appium file logs. Since this appium method in not yet implemented it should not be better to use always the default ratio to avoid all these error messages from appium? If multiple devices are used (e.g: 200) this error will be replicated for all devices and the log file will increase. |
@ludovicmcosta I've got your point. It will be added in next release so won't bother you anymore :) |
Thanks :) |
Version 3.12.0 is able to take screenshot of the entire element while higher versions can't.
https://trello.com/c/zBqG4fkj/330-python-check-region
The text was updated successfully, but these errors were encountered: