quarkiverse / quarkus-bon-jova-rockstar

An implementation of Rockstar as a JVM language
https://codewithrockstar.com/
3 stars 2 forks source link

Use consistent resource dir between rock score and compilation #136

Closed holly-cummins closed 7 months ago

holly-cummins commented 7 months ago

I also added code to tolerate a missing src/main/rockstar directory.

Resolves #134.

I implemented a quick fix where I put a check, but then I realised that the rock score calculator and rock file compiler are using different mechanisms to find the rock files; one the source paths reads from the pom, and the other hardcodes it. It means you can see an endpoint in the dev ui, and then get an error when you click on it.

Reading from the pom is 'correct' but adds an extra step to the demo, to specify the source paths.

I realised I could solve both the inconsistency and the extra step, while still honouring the pom, by using both a default and what was in the pom. That seems friendliest, so I've done that.

I noticed that tests mostly passed even though I had some bad breaks in my implementation, so we're seeing the effect of having so many disabled tests. I think I've manually tested all scenarios.