feat(go): generated wiremock tests #9314

musicpulpite · 2025-09-10T15:42:33Z

Description

Linear ticket: FER-6290

Following the basic structure laid out for Java here create a WireTestGenerator that creates basic wire tests using go-wiremock. This feature will be enabled using the customConfig.enableWireTests flag.

TO-DO Items

See pending items in the [SDKs] Go Wire tests Linear project. Namely:

Fix existing dynamic snippets
~~Refactor to follow pattern established for Java wire tests~~
Add request body validation and accurate mock response bodies
Refactor to use homebrewed mock server
Potentially refactor to co-locate tests with packages

Changes Made

Add methods to EndpointSnippetGenerator
Create WireTestGenerator class to effectively wrap endpoint snippet generation
Fix golang AsIs file functionality

Testing

I've generated auth0's go-sdk using FROND and played around with it

musicpulpite · 2025-09-11T16:27:38Z

generators/go-v2/ast/src/ast/core/Writer.ts

            return s;
        }
-        return split[0].replace(INVALID_GO_IDENTIFIER_TOKEN, "");
+        return split.map((part) => part.replace(INVALID_GO_IDENTIFIER_TOKEN, "")).join("");


Can't have go-wiremock alias as go

I guess we do need to be careful that this is not breaking behavior for package consumers.

But I don't think it is since it's just changing the alias we use in generated files, it isn't actual changing package names that the consumer might try to import and use without aliasing. But double check me on that!

Yea as this is a writer change can this impact any other code generation parts?

musicpulpite · 2025-09-11T16:28:08Z

generators/go-v2/base/src/asIs/test/main_test.go_

@@ -0,0 +1,57 @@
+package wiremock


This file is useful to run once for the entire test suite since initializing the wiremock test container does incur overhead. It can then be used in all individual test cases in the package

musicpulpite · 2025-09-11T16:29:31Z

generators/go-v2/dynamic-snippets/src/DynamicSnippetsGenerator.ts

+        options?: Options
    ): Promise<FernIr.dynamic.EndpointSnippetResponse> {
-        return super.generate(request);
+        return super.generate(request, options);


Since options is untyped and optional its the best way for me to pass in new options to the dynamic snippets endpoint generator to create "unit test case" snippets without breaking the abstract interface for all other users of DynamicSnippetsGenerator

musicpulpite · 2025-09-11T16:30:24Z

generators/go-v2/dynamic-snippets/src/context/DynamicSnippetsGeneratorContext.ts

        });
    }
+
+    public static chainMethods(


I could be convinced that a utility like this should be added onto the go.FuncInvocation type itself... open to other ideas here but the usefulness should be clear

generators/go-v2/sdk/src/reference/EndpointSnippetsGenerator.ts

musicpulpite · 2025-09-11T19:50:57Z

generators/go-v2/sdk/src/SdkGeneratorCli.ts


        await context.snippetGenerator.populateSnippetsCache();

+        if (context.config.writeUnitTests) {


Is it valid here to use the base config option writeUnitTest (currently unused) or should I create a new more specific enableWireTests etc flag? Will we ever want this specificity in the future?

The realization is that writeUnitTests is enabled by default across all generators (but check me on that) so in order to make this feature "opt-in" to begin with I will need to create a new customConfig flag for the go generator probably called enableWireTests as well!

musicpulpite · 2025-09-16T18:02:15Z

generators/go-v2/sdk/src/wire-tests/WireTestGenerator.ts

+        return response.snippet;
+    }
+
+    private parseEndpointTestCaseSnippet(fileString: string): [string, Map<string, string>] {


The endpoint snippet (with config.outputWiremockTests) writes a complete test file for the single endpoint test method. I think its more meaningful to keep that behavior and then parse out the function body and imports in the consuming code. We can then collate the test methods by service

musicpulpite · 2025-09-16T18:05:40Z

generators/go/sdk/Dockerfile

 RUN git config --global user.name "fern" && git config --global user.email "[email protected]"

-COPY generators/go-v2/sdk/dist/cli.cjs /dist/cli.cjs
+COPY generators/go-v2/sdk/dist/ /dist/


This is necessary for getting the AsIs file behavior to work for the v2 go generator. The AsIs files get copied into /dist/ along with the javascript bundle during pnpm dist:cli.

musicpulpite · 2025-09-16T18:06:24Z

seed/go-sdk/imdb/with-wiremock-tests/test/imdb_test.go

@@ -0,0 +1,72 @@
+package wiremock


Here's the generated tests... which do pass!

musicpulpite · 2025-09-16T18:57:23Z

I'm still looking into why this new seed test fails and why this new dependency adds a "toolchain" line:

[go-sdk:imdb:with-wiremock-tests]:  Running script CGO_ENABLED=0 go test ./... on imdb:with-wiremock-tests
go: errors parsing go.mod:
/imdb_with-wiremock-tests/generated/go.mod:5: unknown directive: toolchain

musicpulpite temporarily deployed to Fern Dev September 10, 2025 15:42 — with GitHub Actions Inactive

musicpulpite temporarily deployed to Fern Dev September 11, 2025 16:26 — with GitHub Actions Inactive

musicpulpite commented Sep 11, 2025

View reviewed changes

generators/go-v2/sdk/src/reference/EndpointSnippetsGenerator.ts Outdated Show resolved Hide resolved

musicpulpite marked this pull request as ready for review September 11, 2025 16:35

musicpulpite commented Sep 11, 2025

View reviewed changes

musicpulpite temporarily deployed to Fern Dev September 11, 2025 20:41 — with GitHub Actions Inactive

musicpulpite requested a review from tjb9dc September 11, 2025 21:01

musicpulpite added 10 commits September 16, 2025 13:43

initial commit

f18b07d

Use only first endpoint example so as to avoid name and file collisions

50ffa00

add support for as is files

6b8d13a

include all dist outputs in go-v2 generator

ea3a828

mvp fixes

2c57276

do not include error case for now

c0682ee

remove unnecessary safe escape code

c30be70

rename tests to avoid collisions

cdbae09

final fixes for now

bcf12ab

add illustrative wire test seed fixture

4e98e59

musicpulpite force-pushed the FER-6290/go-wire-tests-2 branch from 0e09c33 to 4e98e59 Compare September 16, 2025 17:49

musicpulpite temporarily deployed to Fern Dev September 16, 2025 17:49 — with GitHub Actions Inactive

musicpulpite temporarily deployed to Fern Dev September 16, 2025 18:00 — with GitHub Actions Inactive

musicpulpite commented Sep 16, 2025

View reviewed changes

musicpulpite requested a review from tstanmay13 September 16, 2025 18:06

musicpulpite added 2 commits September 16, 2025 14:09

fix biome

0245f97

fix test snapshots to use new aliasing behavior

6c100b6

musicpulpite temporarily deployed to Fern Dev September 16, 2025 18:18 — with GitHub Actions Inactive

musicpulpite had a problem deploying to Fern Dev September 16, 2025 18:18 — with GitHub Actions Error

fully fix snapshots

fd49ee9

musicpulpite temporarily deployed to Fern Dev September 16, 2025 18:22 — with GitHub Actions Inactive

fix dynamic snippet snapshot tests also

7604ad4

musicpulpite temporarily deployed to Fern Dev September 16, 2025 18:27 — with GitHub Actions Inactive

tstanmay13 approved these changes Sep 16, 2025

View reviewed changes

musicpulpite added 2 commits September 16, 2025 15:30

small tweak to main_test

105ae3b

ignore new fixture for now

248b65e

musicpulpite temporarily deployed to Fern Dev September 16, 2025 19:32 — with GitHub Actions Inactive

musicpulpite merged commit e5a3d82 into main Sep 16, 2025
39 checks passed

musicpulpite deleted the FER-6290/go-wire-tests-2 branch September 16, 2025 19:43

musicpulpite mentioned this pull request Sep 16, 2025

feat(go): setters, support for explicit null #9487

Merged

2 tasks

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

feat(go): generated wiremock tests #9314

feat(go): generated wiremock tests #9314

Uh oh!

musicpulpite commented Sep 10, 2025 •

edited

Loading

Uh oh!

musicpulpite Sep 11, 2025

Uh oh!

musicpulpite Sep 16, 2025

Uh oh!

tstanmay13 Sep 16, 2025

Uh oh!

musicpulpite Sep 11, 2025 •

edited

Loading

Uh oh!

musicpulpite Sep 11, 2025

Uh oh!

musicpulpite Sep 11, 2025

Uh oh!

Uh oh!

musicpulpite Sep 11, 2025

Uh oh!

musicpulpite Sep 12, 2025

Uh oh!

musicpulpite Sep 16, 2025 •

edited

Loading

Uh oh!

musicpulpite Sep 16, 2025

Uh oh!

musicpulpite Sep 16, 2025

Uh oh!

musicpulpite commented Sep 16, 2025

Uh oh!

Uh oh!

Uh oh!


		await context.snippetGenerator.populateSnippetsCache();

		if (context.config.writeUnitTests) {

feat(go): generated wiremock tests #9314

feat(go): generated wiremock tests #9314

Uh oh!

Conversation

musicpulpite commented Sep 10, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Description

TO-DO Items

Changes Made

Testing

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

musicpulpite Sep 11, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

musicpulpite Sep 16, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

musicpulpite commented Sep 16, 2025

Uh oh!

Uh oh!

Uh oh!

musicpulpite commented Sep 10, 2025 •

edited

Loading

musicpulpite Sep 11, 2025 •

edited

Loading

musicpulpite Sep 16, 2025 •

edited

Loading