-
-
Notifications
You must be signed in to change notification settings - Fork 41
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
Snapshot service is not initialized #656
Comments
I have some time this weekend to look into it, but can you provide a reproducible example, with the way you've setup your complete config, capabilities and a simple test so I can reproduce it? |
Hi @wswebcreation, Thank you for your help |
Awesome! I'll take a look |
Hello, My example are finally ready. PackageExamplesThe package is for both: ScriptsPackage scripts are available:
Notes
Thank you @wswebcreation and have a nice day, |
I found the issue for the malformed elementId, it was a stupid mistake. I'm now busy with the other issue regarding the undefined error with the custom matcher. This is related to poor multiremote support in the module for the custom marchers. I know how to solve that I started working on this. I'll update you there |
Hi @P-Courteille |
Hi @wswebcreation, For now I will stay in version 9.3.0 to avoid the issue because I really need to move on my test production after a week of Typescript migration and issue investigation. Sometime I've the following message at start-up but i've it when it work or not so I don't think it's the origin of the issue:
Another log when it fail:
|
I'll check this. This log
means that it can't find a local installed version of Tesseract. It will now use the NodeJS dependency. If you have the option to use a native installed version of Tessaract you will see it will be much faster. See also https://webdriver.io/docs/ocr-testing/more-test-optimization I'll try to run my tests again with only the NodeJS vesion |
Thanks, most of the changes have been implemented by me 😅 , but as far as I can see this doesn't impact the OCR service |
Those lines are the reason:
The irony is the change is related to another of my ticket about issue with addCommand in multiremote... You are innocent @wswebcreation, it's not your code ^^ |
Hi @P-Courteille I'll go ahead and close this issue and release a new visual service that fixes the multiremote support |
I've released a new version if you use 6.2.3 most of the issues should be gone |
On my side the previous example loop in 100% with local-runner 9.3.1 and work as expected with local-runner 9.3.0. I still don't understand the behavior but it's like the foreach loop to addCommand to all browser from https://github.com/webdriverio/webdriverio/pull/13904/files is in conflict with the code in class WdioOcrService with the same pupose. |
@wswebcreation I think you should remove your #extendMultiremoteBrowser from class WdioOcrService in service.js With For-each without #extendMultiremoteBrowser -> It work Why ?
So technically, #extendMultiremoteBrowser was a fix for webdriverio/webdriverio#13836 but only in ocr-service scope. When @ccharnkij fixed it in the main project, the conflict started. |
Hi @wswebcreation, I think the update introduce another issue with the instance in case of mismatch: |
Awesome, can you file a new one? |
@wswebcreation, Thank you again for your assistance, |
Thanks, a fix for #679 will be released today |
Environment (please complete the following information):
@wdio/visual-service
version: 6.2.2@wdio/mocha-framework
version: 9.2.8@wdio/local-runner
version: 9.4.1@wdio/globals
version: 9.4.1@wdio/cli
version: 9.4.1Appium
version: 2.11.5tsx
version: 4.19.2Config of WebdriverIO +
@wdio/visual-service
Describe the bug
After my typescript migration, it no longer work.
(The issue is not explicit.)
💡It work as expected with saveScreen and checkScreen
Only snapshot service seems to be in issue.
Expected behavior
Snapshot service is initialized correctly, "toMatchSnapshot" and "toMatchElementSnapshot" work as expected
Additional context
I'm using typescript and my tsconfig contain:
Links:
My discord question about this subject:
Someone with similar issue in discord:
The text was updated successfully, but these errors were encountered: