bug: Add missing import to test templates

[ballcoach12]

What does this PR do?

This PR adds a missing import for "github.com/testcontainers/testcontainers-go" to the examples_test.go.tmpl and module_test.go.tmpl files.

Why is it important?

Without this import, generating a new module will return an error code because the required module is not imported in the test files.

Related issues

• Closes [Can't Create a New Module #2816]

How to test this PR

Generate a new module according to the instructions listed here. Verify that the list of imports contains "github.com/testcontainers/testcontainers-go" in the example test file and module test file.

[netlify]

❌ Deploy Preview for testcontainers-go failed.

Name	Link
🔨 Latest commit	7e241d6
🔍 Latest deploy log	https://app.netlify.com/sites/testcontainers-go/deploys/67111589676f400008379337

[mdelapenya]

Nit: the lint will complain if we do not add these blank lines separating imports

Other than that, LGTM

[mdelapenya]

@ballcoach12 just to verify this is working, can you paste the output of generating a random module from your workspace? 🙏

[ballcoach12]

I'm getting an error back, but the generated code appears to be correct.

Error: >> error synchronizing the dependencies: exit status 2
Usage:
  modulegen new module [flags]

Flags:
  -h, --help           help for module
  -i, --image string   Fully-qualified name of the Docker image to be used by the module
  -n, --name string    Name of the module. Only alphabetical characters are allowed.
  -t, --title string   (Optional) Title of the module name, used to override the name in the case of mixed casing (Mongodb -> MongoDB). Use camel-case when needed. Only alphabetical characters are allowed.

>> error synchronizing the dependencies: exit status 2
exit status 1

[mdelapenya]

So it means there is something else in the formatting 🤔 Could you please take a deep look into the generated files and the templates ?

[ballcoach12]

So just debugging the make lint command run that is called from exec.go, I captured the output and it looks like this:

out: go install github.com/golangci/golangci-lint/cmd/golangci-lint@v1.61.0
golangci-lint run --out-format=colored-line-number --path-prefix=. --verbose -c /.golangci.yml --fix
 err: process_begin: CreateProcess(NULL, dirname g:/BITBUCKET/testcontainers-go-github/testcontainers-go/commons-test.mk, ...) failed.
../../commons-test.mk:1: pipe: No error
level=info msg="golangci-lint has version v1.61.0 built with go1.22.2 from (unknown, modified: ?, mod sum: \"h1:VvbOLaRVWmyxCnUIMTbf1kDsaJbTzH20FAMXTAlQGu8=\") on (unknown)"
Error: can't load config: can't read viper config: open /.golangci.yml: The system cannot find the file specified.
Failed executing command with error: can't load config: can't read viper config: open /.golangci.yml: The system cannot find the file specified.
make: *** [../../commons-test.mk:33: lint] Error 3

So maybe a relative path is incorrect?

[mdelapenya]

I think 69594b8 should be reverted, as it does not belong to this branch, shouldn't it?

[ballcoach12]

Yes. My mistake there. Would it be better to delete this PR while I track down and fix the error that is occurring on the module generation? It feels like this PR is drifting a little.

[mdelapenya]

@ballcoach12 thanks for this contribution, but I think #2831 resolved it in a cleaner manner. In any case, thanks for your time and dedication for improving testcontainers-go! 💪

[ballcoach12]

No worries. Sorry I couldn't be of more help.

[mdelapenya]

@ballcoach12 there are and there will be more issues/bugs to work on, so I encourage you to help the community in making progress with your work. Just let us know if we can be of any help