Skip to content

Add support for the path endpoints#186

Merged
astrinski-smartsheet merged 13 commits into
mainlinefrom
feature/add-path-endpoints
Jun 19, 2026
Merged

Add support for the path endpoints#186
astrinski-smartsheet merged 13 commits into
mainlinefrom
feature/add-path-endpoints

Conversation

@astrinski-smartsheet

Copy link
Copy Markdown
Contributor

Pull Request

Description

Added support and WireMock tests for the following endpoints:

  • /2.0/folders/{folderId}/path
  • /2.0/reports/{reportId}/path
  • /2.0/sheets/{sheetId}/path
  • /2.0/sights/{sightId}/path

Type of Change

  • Bug fix
  • New feature
  • Documentation update
  • Code refactoring
  • Other (please describe):

@coveralls

coveralls commented Jun 11, 2026

Copy link
Copy Markdown

Coverage Report for CI Build 27831171269

Warning

No base build found for commit 377ae81 on mainline.
Coverage changes can't be calculated without a base build.
If a base build is processing, this comment will update automatically when it completes.

Coverage: 95.386%

Details

  • Patch coverage: 6 uncovered changes across 3 files (52 of 58 lines covered, 89.66%).

Uncovered Changes

File Changed Covered %
lib/reports/helpers.ts 12 10 83.33%
lib/sheets/helpers.ts 12 10 83.33%
lib/sights/helpers.ts 12 10 83.33%
Total (9 files) 58 52 89.66%

Coverage Regressions

Requires a base build to compare against. How to fix this →


Coverage Stats

Coverage Status
Relevant Lines: 1271
Covered Lines: 1226
Line Coverage: 96.46%
Relevant Branches: 246
Covered Branches: 221
Branch Coverage: 89.84%
Branches in Coverage %: Yes
Coverage Strength: 1735.06 hits per line

💛 - Coveralls

Comment thread lib/folders/types.ts Outdated
Comment thread lib/folders/types.ts
Comment thread lib/folders/types.ts
@astrinski-smartsheet
astrinski-smartsheet merged commit 83839b5 into mainline Jun 19, 2026
5 checks passed
@astrinski-smartsheet
astrinski-smartsheet deleted the feature/add-path-endpoints branch June 19, 2026 14:23
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.

3 participants