Remove default container and version from RestateContainer

[joebowbeer]

As per this issue in the main repo:

testcontainers/testcontainers-node#867

The change will simply make the image name and version mandatory, e.g new PostgreSqlContainer("postgres:X.Y.Z").start().

This should also make it easier to use a custom/enhanced restate image:

sdk-typescript/packages/restate-sdk-testcontainers/src/restate_test_environment.ts

Line 172 in 9f844cb

super(`docker.io/restatedev/restate:${version}`);

[slinkydeveloper]

When you start

sdk-typescript/packages/restate-sdk-testcontainers/src/restate_test_environment.ts

Line 149 in 9f844cb

restateContainerFactory: () => GenericContainer = () =>

you can just provide a generic container. Does that solve it already?

[joebowbeer]

I think the intent of the issue I cited is that neither start() nor RestateContainer() would be supported.

The user would need to pass in an explicit image id.

If that is the case for all other testcontainers, then it might be a good idea to follow suit here.

I created this because of some confusion about how to provide a custom image.

I suggest that start() require an image id -- somehow... If RestateContainer accepted an image id, there would be less need for a factory function.

Maybe the inconsistency is justified, and all that is needed is a doc update.

[slinkydeveloper]

I'll improve the documentation of the start method.

[joebowbeer]

The documentation is sparse:

https://testcontainers.com/modules/restate/

I think a Builder pattern would be a good way to address this in code.

Remove the (telescoping) params from start() and add withMountServicesFn() and withRestateContainerFactory()

https://blogs.oracle.com/javamagazine/post/exploring-joshua-blochs-builder-design-pattern-in-java