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

[cumulusci.robotframework.Salesforce] Input form data keyword returns a "No form handler" error with new SalesForce API = 59.0 #3657

Closed
frankeetou opened this issue Sep 20, 2023 · 9 comments · Fixed by #3822
Labels

Comments

@frankeetou
Copy link

Describe the bug

We run a WebDriver test for one of our LWC (Account Creation) and it was working and populating the text fields on a prior apiVersion (58), now on newly created scratch orgs based on apiVersion 59, it's returning the following error:

image

The keyword implementation we are using

Fill in all required fields for the Quick Create form
    ${firstName}        Get fake data       first_name
    ${lastName}         Get fake data       last_name
    Input form data
    ...     First Name                  ${firstName}
    ...     Last Name                   ${lastName}
    ...     Contact Type                Alumna/Alumnus.     #This is a dropdown field
    ...     Account Relationship        Head of Household.     #This is a dropdown field
    ...     Legal Credit %              100
    ...     Gift Recognition Credit %   100

That tag: 'lightning-primitive-input-simple' was introduced within the new v 59 API

We also have the following WARN when executing the tests:
image

Reproduction steps

  1. Identify a LWC with text input or checkbox in your org
  2. Create a sample robot test and use the "input form data" keyword from cumulusci library
  3. Execute the test

Your CumulusCI and Python versions

MacOs
CumulusCI version: 3.79.0
Python version: 3.11.3

Windows 11
CumulusCI version: 3.79.0
Python version: 3.10

Operating System

MacOs Ventura 13.0 and Windows 10

Windows environment

PowerShell

CumulusCI installation method

None

Error Gist

No response

Additional information

NOTE: Multipick (dropdown) fields are working fine! with this keyword

@frankeetou frankeetou added the bug label Sep 20, 2023
@RobertoMartinez-districtm

Did anyone find a way to work around that bug? @frankeetou , what are you doing in the meantime?

@frankeetou
Copy link
Author

Not a quick one , but the workaround would be using selenium commands or keywords with different locators (depending on the locators strategy your are using). Unfortunately this is one of the most important UI keywords from CCI Salesforce Library (in my opinion). Still waiiting to see if they are going to tackle it or not

@RobertoMartinez-districtm

I agree, it's definitely not trivial keyword given that pretty much any new records creation test would need to have an Name field populated. Let's hope it gets fixed quickly.

@frankeetou
Copy link
Author

I agree, it's definitely not trivial keyword given that pretty much any new records creation test would need to have an Name field populated. Let's hope it gets fixed quickly.

@RobertoMartinez-districtm just FYI, the workaround we are doing right now is using the "Populate Form" keyword instead.. which is field specific.

@RobertoMartinez-districtm

Thanks! Will try it out

@RobertoMartinez-districtm

Has this been fixed with v3.83.0?

@chanyahn-st
Copy link
Contributor

@jstvz (please let me know if I should defer this question to a different contributor) I am working on this issue right now, and I'm having trouble testing the fix using the project directory I'm currently working on. How do you go about testing the features that are currently being worked on outside of the tests?

@chanyahn-st
Copy link
Contributor

@vsbharath I see you made the most recent commit to the repo, would you mind letting me know about how the PR process? I am ready to open one for this issue, but I couldn't find a way to contact the relevant personnel to open a new feature branch. Any help is appreciated!

@vsbharath
Copy link
Contributor

@chanyahn-st Please fork it and open a PR once your changes are done. We can review it and merge once everything is good.

Note: Due to limited support, PR review and merge might take longer time.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants