Skip to content

Added api_version support in get job status method. #405

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

Merged
merged 4 commits into from
Jul 29, 2025

Conversation

sunil-lakshman
Copy link
Contributor

Added api_version support in get job status method.

Copy link

github-actions bot commented Jul 29, 2025

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

Cover ┌─────────────────────────┐ Freq.
   0% │ ░░░░░░░░░░░░░░░░░░░░░░░ │  0.0%
  10% │ ░░░░░░░░░░░░░░░░░░░░░░░ │  0.0%
  20% │ ██░░░░░░░░░░░░░░░░░░░░░ │  2.3%
  30% │ ░░░░░░░░░░░░░░░░░░░░░░░ │  0.0%
  40% │ ░░░░░░░░░░░░░░░░░░░░░░░ │  0.0%
  50% │ ░░░░░░░░░░░░░░░░░░░░░░░ │  0.0%
  60% │ ░░░░░░░░░░░░░░░░░░░░░░░ │  0.0%
  70% │ █████░░░░░░░░░░░░░░░░░░ │  6.8%
  80% │ ███████████████████████ │ 38.6%
  90% │ ███████████████░░░░░░░░ │ 25.0%
 100% │ █████████████████░░░░░░ │ 27.3%
      └─────────────────────────┘
 *Legend:* █ = Current Distribution 
Summary - Lines: 84.81% | Methods: 97.67% | Branches: 69.70%
FilesLinesMethodsBranches
lib
   contentstack.js83.33%100.00%84.62%
   contentstackClient.js80.85%92.86%64.15%
   contentstackCollection.js94.12%100.00%86.67%
   entity.js76.09%100.00%61.33%
lib/core
   Util.js96.36%100.00%83.64%
   concurrency-queue.js86.01%87.18%76.00%
   contentstackError.js100.00%100.00%100.00%
   contentstackHTTPClient.js96.77%100.00%93.65%
   oauthHandler.js88.81%100.00%65.99%
lib/organization
   index.js72.73%100.00%53.09%
lib/organization/teams
   index.js89.47%100.00%65.79%
lib/organization/teams/stackRoleMappings
   index.js78.57%100.00%51.16%
lib/organization/teams/teamUsers
   index.js100.00%100.00%93.75%
lib/query
   index.js73.91%100.00%61.82%
lib/stack/asset/folders
   index.js100.00%100.00%100.00%
lib/stack/asset
   index.js86.25%100.00%74.00%
lib/stack/auditlog
   index.js91.67%100.00%71.88%
lib/stack/branch
   compare.js68.18%100.00%62.50%
   index.js94.87%100.00%77.78%
   mergeQueue.js83.33%100.00%63.16%
lib/stack/branchAlias
   index.js79.31%100.00%60.98%
lib/stack/contentType/entry
   index.js83.04%100.00%59.29%
lib/stack/contentType/entry/variants
   index.js79.31%100.00%62.86%
lib/stack/contentType
   index.js84.62%100.00%70.13%
lib/stack/deliveryToken
   index.js95.24%80.00%89.47%
lib/stack/deliveryToken/previewToken
   index.js21.43%25.00%14.29%
lib/stack/environment
   index.js100.00%100.00%100.00%
lib/stack/extension
   index.js93.88%100.00%85.71%
lib/stack/globalField
   index.js91.67%100.00%82.35%
lib/stack
   index.js80.66%92.86%70.61%
lib/stack/label
   index.js100.00%100.00%100.00%
lib/stack/locale
   index.js100.00%100.00%100.00%
lib/stack/managementToken
   index.js100.00%100.00%94.44%
lib/stack/release
   index.js86.67%100.00%68.09%
lib/stack/roles
   index.js100.00%100.00%100.00%
lib/stack/taxonomy
   index.js84.00%100.00%61.29%
lib/stack/taxonomy/terms
   index.js84.62%100.00%66.67%
lib/stack/variantGroup
   index.js81.58%100.00%60.47%
lib/stack/variantGroup/variants
   index.js77.50%100.00%54.90%
lib/stack/variants
   index.js76.32%100.00%53.19%
lib/stack/webhook
   index.js84.48%100.00%67.80%
lib/stack/workflow
   index.js83.64%100.00%67.69%
lib/stack/workflow/publishRules
   index.js100.00%100.00%100.00%
lib/user
   index.js91.43%100.00%78.57%

🤖 comment via lucassabreu/comment-coverage-clover

Copy link

@Copilot 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 api_version parameter support to the bulk operation job status method to enhance API versioning capabilities.

  • Added api_version parameter to the jobStatus method in the bulk operation module
  • Added comprehensive test coverage for the new api_version functionality with multiple job status scenarios
  • Version bump from 1.23.1 to 1.23.2

Reviewed Changes

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

File Description
lib/stack/bulkOperation/index.js Added api_version parameter support to the jobStatus method with proper header handling
test/sanity-check/api/bulkOperation-test.js Added extensive test coverage for job status functionality with management tokens and API versioning
package.json Version bump to reflect the new feature addition

@sunil-lakshman sunil-lakshman marked this pull request as ready for review July 29, 2025 10:30
@sunil-lakshman sunil-lakshman requested a review from a team as a code owner July 29, 2025 10:30
@sunil-lakshman sunil-lakshman merged commit 72c25a7 into development Jul 29, 2025
13 checks passed
@sunil-lakshman sunil-lakshman deleted the enh/dx-2181 branch July 29, 2025 10:31
@sunil-lakshman sunil-lakshman restored the enh/dx-2181 branch July 29, 2025 17:15
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.

2 participants