[PR #631] [MERGED] coverage: add missing App Store metadata relationship endpoints #665

Closed
opened 2026-02-26 22:32:01 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/rudrankriyam/App-Store-Connect-CLI/pull/631
Author: @rudrankriyam
Created: 2/18/2026
Status: Merged
Merged: 2/18/2026
Merged by: @rudrankriyam

Base: mainHead: cursor/issue-622-metadata-relationships


📝 Commits (3)

  • 9428188 fix: isolate git commands from hook env
  • 7b7da66 coverage: add missing App Store metadata relationship endpoints
  • 06ee64b Merge remote-tracking branch 'origin/main' into cursor/issue-622-metadata-relationships

📊 Changes

12 files changed (+1194 additions, -12 deletions)

View changed files

📝 internal/asc/client_app_clips.go (+68 -0)
📝 internal/asc/client_app_info_relationships.go (+35 -0)
📝 internal/asc/client_assets.go (+115 -0)
📝 internal/asc/client_background_assets.go (+70 -0)
internal/asc/client_http_issue_622_metadata_relationships_test.go (+357 -0)
📝 internal/asc/client_product_pages.go (+175 -0)
📝 internal/asc/client_product_pages_experiments.go (+175 -0)
📝 internal/asc/client_version_relationships.go (+89 -0)
📝 internal/asc/client_versions.go (+34 -0)
📝 internal/asc/review_details.go (+35 -0)
📝 internal/cli/cmdtest/release_notes_generate_test.go (+19 -12)
📝 internal/releasenotes/git.go (+22 -0)

📄 Description

Summary

  • Add ASC client methods for 24 missing App Store metadata relationship endpoints (OpenAPI coverage): GET x20, PATCH x4
  • Add request-shaping tests for the newly added endpoints
  • Fix releasenotes/cmdtest git invocation to ignore GIT_DIR/GIT_WORK_TREE set by git hooks/worktrees

Test plan

  • make format
  • make lint
  • ASC_BYPASS_KEYCHAIN=1 make test

Closes #622.


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/rudrankriyam/App-Store-Connect-CLI/pull/631 **Author:** [@rudrankriyam](https://github.com/rudrankriyam) **Created:** 2/18/2026 **Status:** ✅ Merged **Merged:** 2/18/2026 **Merged by:** [@rudrankriyam](https://github.com/rudrankriyam) **Base:** `main` ← **Head:** `cursor/issue-622-metadata-relationships` --- ### 📝 Commits (3) - [`9428188`](https://github.com/rudrankriyam/App-Store-Connect-CLI/commit/94281889af1e60f5c96862444014a69d2e211815) fix: isolate git commands from hook env - [`7b7da66`](https://github.com/rudrankriyam/App-Store-Connect-CLI/commit/7b7da667f905e6debd3d6b40b1aa07e4c765fe2e) coverage: add missing App Store metadata relationship endpoints - [`06ee64b`](https://github.com/rudrankriyam/App-Store-Connect-CLI/commit/06ee64bdda36bc4844f6eea6239b925910cacfcc) Merge remote-tracking branch 'origin/main' into cursor/issue-622-metadata-relationships ### 📊 Changes **12 files changed** (+1194 additions, -12 deletions) <details> <summary>View changed files</summary> 📝 `internal/asc/client_app_clips.go` (+68 -0) 📝 `internal/asc/client_app_info_relationships.go` (+35 -0) 📝 `internal/asc/client_assets.go` (+115 -0) 📝 `internal/asc/client_background_assets.go` (+70 -0) ➕ `internal/asc/client_http_issue_622_metadata_relationships_test.go` (+357 -0) 📝 `internal/asc/client_product_pages.go` (+175 -0) 📝 `internal/asc/client_product_pages_experiments.go` (+175 -0) 📝 `internal/asc/client_version_relationships.go` (+89 -0) 📝 `internal/asc/client_versions.go` (+34 -0) 📝 `internal/asc/review_details.go` (+35 -0) 📝 `internal/cli/cmdtest/release_notes_generate_test.go` (+19 -12) 📝 `internal/releasenotes/git.go` (+22 -0) </details> ### 📄 Description ## Summary - Add ASC client methods for 24 missing App Store metadata relationship endpoints (OpenAPI coverage): GET x20, PATCH x4 - Add request-shaping tests for the newly added endpoints - Fix releasenotes/cmdtest git invocation to ignore GIT_DIR/GIT_WORK_TREE set by git hooks/worktrees ## Test plan - make format - make lint - ASC_BYPASS_KEYCHAIN=1 make test Closes #622. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-02-26 22:32:01 +03:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/App-Store-Connect-CLI#665
No description provided.