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

Custom function test cases #1206

Closed
wants to merge 33 commits into from
Closed

Conversation

deepprakash345
Copy link
Contributor

Description

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • I have signed the Adobe Open Source CLA.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes and the overall coverage did not decrease.
  • All unit tests pass on CircleCi.
  • I ran all tests locally and they pass.

@adobe-bot
Copy link

Lighthouse scores (desktop)

Performance Accessibility Best-Practices SEO
Scores 100 96 96 75

@adobe-bot
Copy link

Lighthouse scores (mobile)

Performance Accessibility Best-Practices SEO
Scores 94 96 96 75

@adobe-bot
Copy link

Accessibility Violations Found

Id Impact
label-title-only serious
target-size serious

2 similar comments
@adobe-bot
Copy link

Accessibility Violations Found

Id Impact
label-title-only serious
target-size serious

@adobe-bot
Copy link

Accessibility Violations Found

Id Impact
label-title-only serious
target-size serious

Copy link

codecov bot commented Apr 25, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 80.29%. Comparing base (86fff6f) to head (c28d102).

❗ Current head c28d102 differs from pull request most recent head b1aa879. Consider uploading reports for the commit b1aa879 to get more accurate results

Additional details and impacted files
@@            Coverage Diff            @@
##                eds    #1206   +/-   ##
=========================================
  Coverage     80.29%   80.29%           
  Complexity      777      777           
=========================================
  Files            92       92           
  Lines          2136     2136           
  Branches        289      289           
=========================================
  Hits           1715     1715           
  Misses          263      263           
  Partials        158      158           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

it(" fullName field should update on change in first name and last name ", () => {
if(toggle_array.includes("FT_FORMS-11269") && toggle_array.includes("FT_FORMS-12423")) {
Cypress.config('chromeWebSecurity',false);
console.log("cypress security is "+Cypress.config('chromeWebSecurity'));
Copy link
Collaborator

Choose a reason for hiding this comment

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

Why are we turning off chromeWebSecurity? Ideally, our tests should mimic real-world scenarios. We should be enabling CORS headers in the URL that serves custom functions.

Copy link
Collaborator

@rismehta rismehta left a comment

Choose a reason for hiding this comment

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

check comments

rismehta and others added 18 commits May 7, 2024 03:44
* disable exposing view layer for customization

* adding dam asset for test cases

* disable exposing view layer for customization: fixed test cases
* Invalid event customization

* Adding invalid event

* Updating af-core version
* CQ-4348557 adding another div to stop footer getting resized

* CQ-4348557 removing space

---------

Co-authored-by: Shivam Agarwal <shivama@adobe.com>
#1158)

* FORMS-13810: Add the field to allow custom message for format in date picker

* FORMS-13810: Test case added

* FORMS-13810: Review comments incorporated.
Co-authored-by: dgurjar <dgurjar@adobe.com>
Co-authored-by: Pavitra Khatri <pavitrakhatri@Pavitras-MacBook-Pro.local>
Co-authored-by: Shivam Agarwal <shivama@adobe.com>
)

* datepicker readonly does not work when edit or display format set

* Update datepicker.runtime.spec.js

---------

Co-authored-by: Rishi Mehta <69448117+rismehta@users.noreply.github.com>
…group (#1176)

* Update type property on replacing checkboxgroup to radio group

* FORMS-12667 review comment

---------

Co-authored-by: demaurya <demaurya@adobe.com>
* Move core component to 0.13.0 spec version
- RTC (done)
DOD(yes)

* Move core component to 0.13.0 spec version
    - RTC (done)
    DOD(yes)

* Fixing wrong date picker test case

* Fixing date picker test case
* datepicker timezone issue on view layer fixed

datepicker timezone issue on view layer fixed

datepicker timezone issue on view layer fixed

* datepicker fix test case
* Updating spec version to 0.13.0

* Updating spec version to 0.13.0

* Updating spec version to 0.13.0
* FORMS-12203: Added localization changes and test cases

* FORMS-12203: Updated test case

* FORMS-12203: Altered to contains instead of get

* FORMS-12203: Resolved comments

* FORMS-12203: Updated test cases

* FORMS-12203: Updated delete button

* FORMS-12203: Updated button

* Retry wizard.locale.spec.js tests

* FORMS-12203: Added theme request wait

* Update wizard.locale.spec.js

* Added latestaddon check

* Delete ui.tests/test-module/package-lock.json

* Update commands.js

* Revert "Delete ui.tests/test-module/package-lock.json"

This reverts commit fd97b53.

* Reverted package-lock.json changes

* FORMS-12203: Removed exception inline

* FORMS-12203: Resolved comments

* FORMS-12203: Update index.js

* FORMS-12203: Removed package-lock.json change

* Update index.js

* Update error message in index.js

* Update index.js

* Update wizard.locale.spec.js

* Removed an intercept

* Updated regex patterns

---------

Co-authored-by: Shivangi Mittal <shivangim@Shivangis-MacBook-Pro.local>
rismehta and others added 13 commits May 7, 2024 03:44
* FORMS-13851 Support for language in date picker and number input
@review @vdua
DOD(Yes)

* FORMS-13851 Support for language in date picker and number input
    @review @vdua
    DOD(Yes)

* FORMS-13851 Support for language in date picker and number input
        @review @vdua
        DOD(Yes)

* FORMS-13851 Support for language in date picker and number input
            @review @vdua
            DOD(Yes)

* FORMS-13851 Support for language in date picker and number input
                @review @vdua
                DOD(Yes)

* FORMS-13851 Support for language in date picker and number input
                    @review @vdua
                    DOD(Yes)

* FORMS-13851 Support for language in date picker and number input
                        @review @vdua
                        DOD(Yes)

* FORMS-13851 Support for language in date picker and number input
                            @review @vdua
                            DOD(Yes)

* Update space character for thousands separator in french lang

* FORMS-13851 Support for language in date picker and number input
                                @review @vdua
                                DOD(Yes)

---------

Co-authored-by: Varun Dua <vdua@adobe.com>
#1196)

* FORMS-12822 : format should not be visible when no pattern is selected

* FORMS-12822 : changes var to let as per ES6 in utils.js
…d submit() rule in examples (#1191)

* FORMS-13209: added check to override hardcoded submitSuccess and submitError hadnlers

* FORMS-13209: updated af2 runtime version

* FORMS-13209: added rule editor authoring cypress tests

* FORMS-13209: added submission handler rule tests

* FORMS-13209: replaced submit(custom:submitSuccess, custom:submitError) examples and it rules with submit()

* FORMS-13209: updated af-custom-functions version

* FORMS-13209: updated submission handler test

* FORMS-13209: updated tests

* FORMS-13209: updated tests

* FORMS-13209: updated tests

* FORMS-13209: added rule editor authoring cypress tests

* FORMS-13209: added submission handler rule tests

* FORMS-13209: updated af-custom-functions version

* FORMS-13209: updated tests

* FORMS-13209: updated tests

* FORMS-13209: added rule editor authoring cypress tests

* FORMS-13209: added submission handler rule tests

* FORMS-13209: updated af-custom-functions version

* FORMS-13209: updated tests

* FORMS-13209: updated tests

* FORMS-13209: updated tests

* FORMS-13209: updated tests

* FORMS-13209: added af-custom-functions in FormView and replaced hardcoded submission handlers with defaultSubmit handlers af-custom-functions

* FORMS-13209: minor update

---------

Co-authored-by: Arun Kumar Attri <aattri@adobe.com>
FORMS-13935 update examples for aemcomponents.dev

FORMS-13935 update examples for aemcomponents.dev
…lue without default/placeholder set (#1203)

* Added test case

* Adding actual fix

* Improved test case

* Small fix
* updated code for ui.frontend with test cases

* resolved PR comments

* test case fixed

---------

Co-authored-by: Gulshan Mishra <gulshanm@adobe.com>
Co-authored-by: Rajat Khurana <rajatkhurana@adobe.com>
Co-authored-by: Pavitra Khatri <pavitrakhatri@pavitras-mbp.corp.adobe.com>
Co-authored-by: Pavitra Khatri <pavitrakhatri@Pavitras-MacBook-Pro.local>
…vided (#1210)

* FORMS-14018 - Fix for dropdown not working when enumNames are not provided

* Import fix

* Improved code and test cov

* Revert "Improved code and test cov"

This reverts commit 7a3d69e.

* Refactor for code cov

* Added the removed test case
* Moving to 15860 image

* updating core component version

* Skipping wizard authoring test
* embedding forms-sdk api

* adding hcaptcha and recaptcha models

* :fix hcaptcha model SlingException

* :fix deleting recaptcha model

* FORMS-13811 | Adding rendition for hCaptcha

* FORMS-13811 | Renamed public API

* FORMS-13811 | Embedding hcaptcha js

* FORMS-13811 | Adding unit test for sling model

* FORMS-13811 | Adding cypress test

* FORMS-13811 | Adressing review comments

* Adding e2e test

* FORMS-13811 | fix import statements

* Update pom.xml

* Adding FT for running Tests

* FORMS-13811 | Removing java-8 build for master

* Adding FT in test

* FORMS-13811 | Tests should run only with latest addon

* FORMS-13811 | Tests should run only with latest addon

* increasing coverage

* addressing review comments

---------

Co-authored-by: Navneet Agarwal <navneeta@adobe.com>
@deepprakash345 deepprakash345 deleted the customFunctionTestCases branch May 7, 2024 03:47
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