[PR #2145] [MERGED] chore: introduce curl parser tests and minor changes #3641

Closed
opened 2026-03-17 01:14:23 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/hoppscotch/hoppscotch/pull/2145
Author: @kyteinsky
Created: 3/2/2022
Status: Merged
Merged: 3/14/2022
Merged by: @AndrewBastin

Base: mainHead: chore/curl-parser-tests


📝 Commits (10+)

  • bf567dd chore: introduce curl parser tests and minor changes
  • a5a8e66 Merge branch 'main' into chore/curl-parser-tests
  • 13868b3 refactor: introduce samples test scaffolding from proper curl command parser testing
  • 8821905 fix: remove redundant console.log
  • 7009ea0 chore: added curl tests and other fixes
  • 3e11a1b fix: response section overflow
  • 0800d67 feat: sign in with microsoft
  • 823059e feat: arbitrary profile picture
  • e91a44c chore(i18n): updated translations
  • ac806aa chore: added additional failing test cases

📊 Changes

6 files changed (+862 additions, -46 deletions)

View changed files

📝 .github/workflows/tests.yml (+1 -1)
packages/hoppscotch-app/helpers/curl/__tests__/curlparser.spec.js (+636 -0)
packages/hoppscotch-app/helpers/curl/__tests__/detectContentType.spec.js (+160 -0)
📝 packages/hoppscotch-app/helpers/curl/contentParser.ts (+17 -9)
📝 packages/hoppscotch-app/helpers/curl/curlparser.ts (+45 -35)
📝 packages/hoppscotch-app/helpers/curl/index.ts (+3 -1)

📄 Description

Description

Curl parser tests and other related changes.

@AndrewBastin

Checks

  • My pull request adheres to the code style of this project
  • All the tests have passed

fixes #2132, fixes #2155


🔄 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/hoppscotch/hoppscotch/pull/2145 **Author:** [@kyteinsky](https://github.com/kyteinsky) **Created:** 3/2/2022 **Status:** ✅ Merged **Merged:** 3/14/2022 **Merged by:** [@AndrewBastin](https://github.com/AndrewBastin) **Base:** `main` ← **Head:** `chore/curl-parser-tests` --- ### 📝 Commits (10+) - [`bf567dd`](https://github.com/hoppscotch/hoppscotch/commit/bf567dd27dc335798698cb309b62debeaae9953d) chore: introduce curl parser tests and minor changes - [`a5a8e66`](https://github.com/hoppscotch/hoppscotch/commit/a5a8e666417747728620c50242e141d7f9601f07) Merge branch 'main' into chore/curl-parser-tests - [`13868b3`](https://github.com/hoppscotch/hoppscotch/commit/13868b3d8df70d5c766faab76fcb7668e7e5c559) refactor: introduce samples test scaffolding from proper curl command parser testing - [`8821905`](https://github.com/hoppscotch/hoppscotch/commit/8821905ab27dca449102dea2c384bae9c61d8367) fix: remove redundant console.log - [`7009ea0`](https://github.com/hoppscotch/hoppscotch/commit/7009ea0d9c0d168b2e974658bf71aa2c838eff76) chore: added curl tests and other fixes - [`3e11a1b`](https://github.com/hoppscotch/hoppscotch/commit/3e11a1b02e940954a1338af45dded3d2f1b7abb7) fix: response section overflow - [`0800d67`](https://github.com/hoppscotch/hoppscotch/commit/0800d67fa6ec700be481aaf73fc21971ecdb9f93) feat: sign in with microsoft - [`823059e`](https://github.com/hoppscotch/hoppscotch/commit/823059e662d8793b849f711f40160b44077eebfb) feat: arbitrary profile picture - [`e91a44c`](https://github.com/hoppscotch/hoppscotch/commit/e91a44c13eb48da44e36506b11ab6e54fe93de86) chore(i18n): updated translations - [`ac806aa`](https://github.com/hoppscotch/hoppscotch/commit/ac806aacde17d88468271e6f29cd57a506452772) chore: added additional failing test cases ### 📊 Changes **6 files changed** (+862 additions, -46 deletions) <details> <summary>View changed files</summary> 📝 `.github/workflows/tests.yml` (+1 -1) ➕ `packages/hoppscotch-app/helpers/curl/__tests__/curlparser.spec.js` (+636 -0) ➕ `packages/hoppscotch-app/helpers/curl/__tests__/detectContentType.spec.js` (+160 -0) 📝 `packages/hoppscotch-app/helpers/curl/contentParser.ts` (+17 -9) 📝 `packages/hoppscotch-app/helpers/curl/curlparser.ts` (+45 -35) 📝 `packages/hoppscotch-app/helpers/curl/index.ts` (+3 -1) </details> ### 📄 Description ### Description Curl parser tests and other related changes. @AndrewBastin ### Checks <!-- Make sure your pull request passes the CI checks and do check the following fields as needed - --> - [x] My pull request adheres to the code style of this project - [x] All the tests have passed fixes #2132, fixes #2155 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-03-17 01:14:23 +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/hoppscotch#3641
No description provided.