[PR #1249] [MERGED] add codegen for nodejs native and instrument snapshot tests #3172

New issue

Closed

opened 2026-03-17 00:48:43 +03:00 by kerem · 0 comments

kerem commented

2026-03-17 00:48:43 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/hoppscotch/hoppscotch/pull/1249
Author: @yq314
Created: 10/7/2020
Status: ✅ Merged
Merged: 10/8/2020
Merged by: @liyasthomas

Base: master ← Head: add-nodejs-native-codegen

📝 Commits (3)

30250e1 add codegen for nodejs native and instrument snapshot tests
fe1618d move test data to fixtures
6135bc7 Merge branch 'master' into add-nodejs-native-codegen

📊 Changes

5 files changed (+268 additions, -0 deletions)

View changed files

📝 helpers/codegen/codegen.js (+2 -0)
➕ helpers/codegen/generators/__fixtures__/test-data.js (+16 -0)
➕ helpers/codegen/generators/__tests__/__snapshots__/nodejs-native.spec.js.snap (+98 -0)
➕ helpers/codegen/generators/__tests__/nodejs-native.spec.js (+74 -0)
➕ helpers/codegen/generators/nodejs-native.js (+78 -0)

📄 Description

This PR adds codegen for NodeJs Native and also performs snapshot test on the generated code.

I put the test data in __fixtures__ so it can be reused in tests for other generators.

_{🔄 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/1249 **Author:** [@yq314](https://github.com/yq314) **Created:** 10/7/2020 **Status:** ✅ Merged **Merged:** 10/8/2020 **Merged by:** [@liyasthomas](https://github.com/liyasthomas) **Base:** `master` ← **Head:** `add-nodejs-native-codegen` --- ### 📝 Commits (3) - [`30250e1`](https://github.com/hoppscotch/hoppscotch/commit/30250e1afac86a76811f28ac21c41b09cf73bc81) add codegen for nodejs native and instrument snapshot tests - [`fe1618d`](https://github.com/hoppscotch/hoppscotch/commit/fe1618d1d8822acf20ca919cf815d65eefbf30cb) move test data to __fixtures__ - [`6135bc7`](https://github.com/hoppscotch/hoppscotch/commit/6135bc716ede57f7817580cff35ba78d3b46ca19) Merge branch 'master' into add-nodejs-native-codegen ### 📊 Changes **5 files changed** (+268 additions, -0 deletions) <details> <summary>View changed files</summary> 📝 `helpers/codegen/codegen.js` (+2 -0) ➕ `helpers/codegen/generators/__fixtures__/test-data.js` (+16 -0) ➕ `helpers/codegen/generators/__tests__/__snapshots__/nodejs-native.spec.js.snap` (+98 -0) ➕ `helpers/codegen/generators/__tests__/nodejs-native.spec.js` (+74 -0) ➕ `helpers/codegen/generators/nodejs-native.js` (+78 -0) </details> ### 📄 Description This PR adds codegen for NodeJs Native and also performs snapshot test on the generated code. I put the test data in `__fixtures__` so it can be reused in tests for other generators. <img width="724" alt="Screenshot 2020-10-08 at 12 09 24 AM" src="https://user-images.githubusercontent.com/2336852/95357669-bb2bfb00-08fa-11eb-8b38-8439857ebf3d.png"> --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>