lf-lang / lingo

Package manager and build tool for the Lingua Franca coordination language
BSD 2-Clause "Simplified" License
4 stars 1 forks source link

Testing #38

Closed tanneberger closed 8 months ago

tanneberger commented 10 months ago

Things Done

Future Work

Fixes #42.

tanneberger commented 9 months ago

Regarding Testing different targets, I would say we do this on a per-backend basis so we would have tests for cmake, npm, pnpm and lfc

tanneberger commented 9 months ago

I would add the other backends tests in a seperate pr @lhstrh

tanneberger commented 9 months ago

The CPP Tests are failing becuase the wrong path is in LF_SRC_PKG_PATH:UNINITIALIZED=/home/revol-xut/workspace/lf/lingua-franca/test

It should always point to the src dir

lhstrh commented 8 months ago

What is the status of this PR, @tanneberger?

tanneberger commented 8 months ago

@lhstrh Last Thing I did this weekend was looking into why the typescript CI is still red, and I think its the dockerized tests.

tanneberger commented 8 months ago

Okay, the next problem with the typescript target is that the federated code gen is dropped under fed-gen and not src-gen. Is the fed-gen folder so structurally different from the src-gen folder that its not feasible to remove the fed-gen folder ? @lhstrh

lhstrh commented 8 months ago

Okay, the next problem with the typescript target is that the federated code gen is dropped under fed-gen and not src-gen. Is the fed-gen folder so structurally different from the src-gen folder that its not feasible to remove the fed-gen folder ? @lhstrh

Yes, it is. The fed-gen/src folder actually has generated LF code in it, which is then compiled down to target code in a second step.

tanneberger commented 8 months ago

@lhstrh Would it be possible to rename fed-gen to src-gen

No, there is an src-gen in fed-gen subtree for each federate as well. It is different from the src-gen in the root.