Skip to content

dx | 3283 add searchApps functionality#89

Merged
harshithad0703 merged 3 commits intodevelopmentfrom
fix/dx-3283-search-query
Aug 6, 2025
Merged

dx | 3283 add searchApps functionality#89
harshithad0703 merged 3 commits intodevelopmentfrom
fix/dx-3283-search-query

Conversation

@harshithad0703
Copy link
Contributor

No description provided.

@harshithad0703 harshithad0703 requested a review from a team as a code owner August 5, 2025 12:04
@github-actions
Copy link

github-actions bot commented Aug 5, 2025

Coverage report for commit: ad76949
File: coverage/clover.xml

Cover ┌─────────────────────────┐ Freq.
   0% │ ░░░░░░░░░░░░░░░░░░░░░░░ │  0.0%
  10% │ ░░░░░░░░░░░░░░░░░░░░░░░ │  0.0%
  20% │ ░░░░░░░░░░░░░░░░░░░░░░░ │  0.0%
  30% │ ░░░░░░░░░░░░░░░░░░░░░░░ │  0.0%
  40% │ ░░░░░░░░░░░░░░░░░░░░░░░ │  0.0%
  50% │ ░░░░░░░░░░░░░░░░░░░░░░░ │  0.0%
  60% │ ░░░░░░░░░░░░░░░░░░░░░░░ │  0.0%
  70% │ ░░░░░░░░░░░░░░░░░░░░░░░ │  0.0%
  80% │ ███████████████████████ │ 50.0%
  90% │ ████████░░░░░░░░░░░░░░░ │ 16.7%
 100% │ ████████████████░░░░░░░ │ 33.3%
      └─────────────────────────┘
 *Legend:* █ = Current Distribution 
Summary - Lines: 86.69% | Methods: 97.98% | Branches: 67.51%
FilesLinesMethodsBranches
lib
   contentstack.js100.00%100.00%100.00%
   contentstackClient.js100.00%100.00%85.29%
   contentstackCollection.js86.67%100.00%90.00%
   entity.js75.31%100.00%53.45%
lib/core
   Util.js96.36%100.00%83.64%
   concurrency-queue.js92.40%93.75%84.87%
   contentstackError.js100.00%100.00%100.00%
   contentstackHTTPClient.js100.00%100.00%95.56%
   region.js100.00%100.00%75.00%
lib/marketplace/app/hosting
   deployment.js80.88%100.00%63.16%
   index.js80.00%100.00%58.23%
lib/marketplace/app
   index.js80.46%90.00%53.26%
lib/marketplace/app/oauth
   index.js83.33%100.00%52.63%
lib/marketplace/apprequest
   index.js82.86%100.00%58.54%
lib/marketplace/authorization
   index.js83.33%100.00%65.12%
lib/marketplace
   index.js90.48%100.00%82.76%
lib/marketplace/installation
   index.js81.82%100.00%56.52%
lib/marketplace/installation/webhooks
   index.js81.58%100.00%53.19%

🤖 comment via lucassabreu/comment-coverage-clover

@harshithad0703 harshithad0703 marked this pull request as draft August 5, 2025 12:07
@harshithad0703 harshithad0703 requested a review from Copilot August 5, 2025 12:11
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds search functionality to the Contentstack Marketplace SDK, allowing users to query apps by name and other criteria. The implementation introduces a new searchApps method that enables searching through marketplace applications with configurable query parameters.

  • Adds searchApps method with search term validation and query parameter support
  • Includes comprehensive unit and integration tests for the new functionality
  • Updates package version from 1.2.9 to 1.3.0 to reflect the new feature

Reviewed Changes

Copilot reviewed 7 out of 9 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
lib/marketplace/index.js Implements the core searchApps method with search term validation and API integration
test/unit/marketplace-test.js Adds unit tests for successful search and error handling scenarios
test/sanity-check/api/app-test.js Includes integration tests for search functionality with real API responses
test/unit/mock/objects.js Defines mock data structure for search results testing
package.json Bumps version to 1.3.0 for the new feature release
CHANGELOG.md Documents the new search functionality enhancement
.talismanrc Updates checksum for package-lock.json changes

@harshithad0703 harshithad0703 marked this pull request as ready for review August 6, 2025 07:05
@harshithad0703 harshithad0703 merged commit e9af833 into development Aug 6, 2025
10 checks passed
@harshithad0703 harshithad0703 deleted the fix/dx-3283-search-query branch August 6, 2025 07:18
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.

6 participants