tye-v2 / tye2

Tye2 is a local orchestrator tool that makes developing and testing microservices and distributed applications easier.
https://www.tye2.org
MIT License
3 stars 2 forks source link

Running Unit Tests #6

Closed razvangoga closed 8 months ago

razvangoga commented 8 months ago
drakon660 commented 8 months ago

This is actually how it looks like in my environment:

image

Did you pushed everything?

razvangoga commented 8 months ago

Yes all is pushed - do you have docker running in the background? Some tests use tye configs that spin up containers.

In any case, you clone locally and use as a control the original tye repo

drakon660 commented 8 months ago

I have docker, we need to add additional info to readme to have docker installed (add volume to appdata) and install azure function tools, dapr cli and dapr init, tonight I will investigate it further, what I have seen so far was containers restarting many times.

drakon660 commented 8 months ago

2 tests are still not working. Downloading solution and installing all missing parts (e.g docker) gives 19 failing e2e test at starting point. Tests definitely doesn't provide fast feedback especially those e2e and are difficult to maintain, there are a lot of places to improve.

drakon660 commented 8 months ago

tests fixed, please check.

razvangoga commented 8 months ago

great job! i've left a couple of comments on the PR

on thing we should add to the readme (and it would be needed also for the CI pipeline is the exact list of prerequisites one would need to successfully run the unit tests locally)

so far we have:

  1. docker
  2. dotnet 6 (tye itself and some tests)
  3. dotnet 7 and 8 (tests)
  4. azure functions v4 (tests)
  5. dapr - ??? -> @drakon660 how do you install the dapr runtime?
  6. others ???
drakon660 commented 8 months ago

I don't see any comments, dapr installed from https://docs.dapr.io/getting-started/install-dapr-cli/ + dapr init.

razvangoga commented 8 months ago

they are inline in the conversation tab of the PR

image
drakon660 commented 8 months ago

You should finish a review

Screenshot 2024-02-19 at 14 45 11
razvangoga commented 8 months ago

try now

drakon660 commented 8 months ago

close the issue?

razvangoga commented 8 months ago

@drakon660 i've merged de PR. thank you for the effort!

I'm working as we speak on #7 to get a working action pipeline that runs the tests.