Skip to content

Issue 36459 include postmantest - #36505

Closed
freddyDOTCMS wants to merge 11 commits into
mainfrom
issue-36459-include-postmantest
Closed

Issue 36459 include postmantest#36505
freddyDOTCMS wants to merge 11 commits into
mainfrom
issue-36459-include-postmantest

Conversation

@freddyDOTCMS

@freddyDOTCMS freddyDOTCMS commented Jul 10, 2026

Copy link
Copy Markdown
Member

Summary

Fixes a stray backtick character (`) appended to a closing }); line in the FolderResource Postman collection test script, introduced in #36497. The extra character made the JavaScript test body syntactically invalid, causing the Postman test runner to fail on that request.

What changed

  • FolderResource.postman_collection.json — removed the trailing backtick from the }); line in the folder search test script.

Checklist

  • Tests
  • Translations
  • Security Implications Contemplated (n/a — test-only fix)

Related to #36459

@claude

claude Bot commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

Claude finished @freddyDOTCMS's task in 37s —— View job


PR Review — Issue 36459 include postmantest

  • Gather context (diff, changed file)
  • Analyze the change
  • Post review

New Issues

  • 🔴 Critical: dotcms-postman/src/main/resources/postman/FolderResource.postman_collection.json:155 — A stray backtick (`) was appended to the closing }); of the "Checking that all expected folder attributes are present" test. The line reads "});",\`` instead of "});",. This backtick is **outside** the JSON string literal, so it corrupts the collection file: it is no longer valid JSON and Newman/Postman will fail to parse and run the FolderResource` collection. This looks like an accidental keystroke rather than an intended change.

    Fix: remove the trailing backtick so line 155 is just:

    "});",
    

    Fix this →

Note: aside from this typo, the intended goal (the PR title mentions including a hasChildren Postman test) doesn't appear in this diff — the only change here is the accidental backtick. Please confirm the intended test additions were committed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

AI: Safe To Rollback Area : Backend PR changes Java/Maven backend code

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant