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

Unknown StepType for input type fields #71

Closed
prashant4be opened this issue May 21, 2024 · 3 comments · Fixed by #72
Closed

Unknown StepType for input type fields #71

prashant4be opened this issue May 21, 2024 · 3 comments · Fixed by #72
Assignees
Labels
bug Something isn't working

Comments

@prashant4be
Copy link

prashant4be commented May 21, 2024

Observe that getting error as unknown StepType for Input fields when I tried to record journey using Recorder,

I tried for standard demo app Browse Order app https://sapui5.hana.ondemand.com/test-resources/sap/m/demokit/orderbrowser/webapp/test/mockServer.html?sap-ui-theme=sap_horizon#

Tried to Search field to input some text but unfortunately not recorded correctly by Recorder, is that version issue?
As I checked couple of videos in that it seems for input enterText("") correctly recorded but for me it didn't worked,
Reference video https://youtu.be/oJCcFkwYyYM?t=675

Is there I missed something? @tabris87

image
@tabris87
Copy link
Collaborator

tabris87 commented Jun 3, 2024

So currently i don't see an issue within your usage.
As the generated code looks not clean as well.

I will check with the provided app.

@tabris87 tabris87 self-assigned this Jun 3, 2024
@tabris87 tabris87 added the bug Something isn't working label Jun 3, 2024
@tabris87 tabris87 linked a pull request Jun 5, 2024 that will close this issue
@tabris87
Copy link
Collaborator

tabris87 commented Jun 5, 2024

@prashant4be: I fixed the intended behaviour of the JourneyRecorder. The Click/Validation Step followed by consecutive KeyPress Steps should be converted into one InputStep.
This also leads to the correct test code

Hopefully this release is published soon by Google

@prashant4be
Copy link
Author

Thank you for the update.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants