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

CIF-2039 - Missing root category field in catalog page properties #127

Merged
merged 5 commits into from
Apr 20, 2021

Conversation

LSantha
Copy link
Collaborator

@LSantha LSantha commented Apr 16, 2021

Added UI test for catalog page properties extensions.

Related Issue

CIF-2039

How Has This Been Tested?

UI test.

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.

 * added UI test for catalog page properties extensions
@LSantha LSantha added the bug Something isn't working label Apr 16, 2021
 * added UI test for catalog page properties extensions
@codecov
Copy link

codecov bot commented Apr 16, 2021

Codecov Report

Merging #127 (bddc2b5) into dev (9cf3937) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##                dev     #127   +/-   ##
=========================================
  Coverage     85.71%   85.71%           
  Complexity       18       18           
=========================================
  Files             1        1           
  Lines            28       28           
  Branches          4        4           
=========================================
  Hits             24       24           
  Misses            2        2           
  Partials          2        2           
Flag Coverage Δ Complexity Δ
unittests 85.71% <ø> (ø) 18.00 <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.


Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9cf3937...bddc2b5. Read the comment docs.

@mhaack mhaack merged commit 2c5b07b into dev Apr 20, 2021
@mhaack mhaack deleted the CIF-2039 branch April 20, 2021 15:24
@mhaack mhaack mentioned this pull request Apr 22, 2021
mhaack added a commit that referenced this pull request Apr 22, 2021
* @releng - update CIF Core Components dependency to SNAPSHOT version

* CIF-1718 - Adopt Magento GraphQL schema changes - Category UID

- update graphql dependency to 2.4.2 EE to align with CIF components

* CIF-2015 - Migrate to Azul Zulu JDK (#125)

* @trivial fix ui.frontend readme (#128)

* CIF-2039 - Missing root category field in catalog page properties (#127)

* CIF-2039 - Missing root category field in catalog page properties

 * added UI test for catalog page properties extensions

* CIF-2039 - Missing root category field in catalog page properties

 * added UI test for catalog page properties extensions

* CIF-2039 - Missing root category field in catalog page properties

 * fixed root category field label in UI test

* CIF-1907 - Build navigation component based on WCM Core Component (#126)

* CIF-1907 - Build navigation component based on WCM Core Component

 * implemented Venia theme for navigation component v2

* CIF-1907 - Build navigation component based on WCM Core Component

 * fix test failures

Co-authored-by: Markus Haack <mhaack@adobe.com>

* @trivial - update dependencies (#129)

Co-authored-by: Levente Santha <levente@adobe.com>
Co-authored-by: Christophe Jelger <jelger@adobe.com>
Co-authored-by: Mark J. Becker <herzog31@users.noreply.github.com>
dplaton added a commit that referenced this pull request May 24, 2021
* @releng - update CIF Core Components dependency to SNAPSHOT version

* CIF-1718 - Adopt Magento GraphQL schema changes - Category UID

- update graphql dependency to 2.4.2 EE to align with CIF components

* CIF-2015 - Migrate to Azul Zulu JDK (#125)

* @trivial fix ui.frontend readme (#128)

* CIF-2039 - Missing root category field in catalog page properties (#127)

* CIF-2039 - Missing root category field in catalog page properties

 * added UI test for catalog page properties extensions

* CIF-2039 - Missing root category field in catalog page properties

 * added UI test for catalog page properties extensions

* CIF-2039 - Missing root category field in catalog page properties

 * fixed root category field label in UI test

* CIF-1907 - Build navigation component based on WCM Core Component (#126)

* CIF-1907 - Build navigation component based on WCM Core Component

 * implemented Venia theme for navigation component v2

* CIF-1907 - Build navigation component based on WCM Core Component

 * fix test failures

Co-authored-by: Markus Haack <mhaack@adobe.com>

* @trivial - update dependencies (#129)

* CIF-1746 - Create CIF data types for Content Fragment Models (#102)

* enable content fragment models in Venia configuration

* CIF-2007 - Use latest releases of CIF Add-on for IT and UI tests (#132)

* CIF-2007 - Use latest release of CIF add-on from artifactory

* CIF-2007 - Use latest release of CIF add-on from artifactory

* CIF-2007 - Use latest release of CIF add-on from artifactory

* CIF-2007 - Use latest release of CIF add-on for AEM 6.5 from artifactory

* CIF-2007 - Fix UI tests

* @releng Update dependency versions (#134)

Update `AEM Core CIF Components` to 1.11.0 and `@adobe/aem-core-cif-react-components` to 1.11.0

Co-authored-by: Daniel Platon <platon@adobe.com>

* CIF-2086: Update react-components to be in sync with CIF Components version (#133)

* CIF-2086: Sync reac-components version on release

* CIF-2086: Commit updated npm files

* CIF-2086: Fix npm files commit profile

* CIF-2086: Update cif-react-components dependency

* Remove package-lock.json from .gitignore

Co-authored-by: Daniel Platon <platon@adobe.com>

Co-authored-by: Levente Santha <levente@adobe.com>
Co-authored-by: Christophe Jelger <jelger@adobe.com>
Co-authored-by: Mark J. Becker <herzog31@users.noreply.github.com>
Co-authored-by: Markus Haack <mhaack@adobe.com>
Co-authored-by: Daniel Platon <platon@adobe.com>
Co-authored-by: Laurentiu Magureanu <magurean@adobe.com>
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 this pull request may close these issues.

2 participants