lerenn / asyncapi-codegen

An AsyncAPI Golang Code generator that generates all Go code from the broker to the application/user. Just plug your application to your favorite message broker!
Apache License 2.0
78 stars 21 forks source link

Add tests in Dagger for brokers #188

Open lerenn opened 3 months ago

lerenn commented 3 months ago

There is tests for brokers in extensions but there is no execution right now.

This should be added in Dagger.

magraef commented 2 months ago

Why do we run each test file using Dagger in a single container and create a map beforehand with selected tests? Wouldn't it be easier to just run all the tests at once using go test -v ./... to avoid future problems and issues like this?

lerenn commented 2 months ago

I have done that to be able to execute specific tests and not the whole set. I had in mind the fact that I could run the tests on different machines if they were too many of them.

But in the end, that was a silly idea, I prefer the go test -v ./... approach and we can still have another mechanism to execute a specific test.

magraef commented 2 months ago

Maybe we could add a optional parameter to the dagger test function to run a specific test and per default we can use ./...

lerenn commented 2 months ago

Yup, I think that's the best option !