microsoft / AL-Go

The plug-and-play DevOps solution for Business Central app development on GitHub
MIT License
295 stars 125 forks source link

[Question]: Should it not be possible to use a library app for test apps? #1288

Open kennetlindberg opened 3 weeks ago

kennetlindberg commented 3 weeks ago

Question

Hi,

I've created a library app where I have a bunch of common test related functions and that I have a dependency on from my test app. The dependency is downloaded (from another GitHub repo using the appDependencyProbingPaths setting in the .AL-Go/settings.json).

Downloaded dependencies: C:\actions-runner-2\_work\utility-billing\utility-billing\.dependencies\internal-testing-library-main-Apps-1.5.7.0.zip

However, in the Installing Apps part of the Build step I get

Consilia Solutions_Testing Library_1.5.7.0.app (AppId=90493eca-5e01-47d3-b34a-407eae57bab7) is skipped as it is not referenced

which later in the Compiling Test Apps step leads to this (at least I suspect that is the reason)

Processing dependency Consilia Solutions_Testing Library_1.5.0.0 (90493eca-5e01-47d3-b34a-407eae57bab7)
Downloading symbols: Consilia Solutions_Testing Library_1.5.0.0.app
Url : http://172.20.96.236:7049/BC/dev/packages?appId=90493eca-5e01-47d3-b34a-407eae57bab7&versionText=1.5.0.0&tenant=default
ERROR Exception calling "GetResult" with "0" argument(s): "Response status code does not indicate success: 404 (Not Found)."

I can get it to work if I add a dependency to the library app in the main app but that is of course not how we want it.

Am I missing a configuration somewhere or is it simply not meant to work or is it maybe something that could be fixed?

logs_30338196630.zip

freddydk commented 3 weeks ago

Maybe https://aka.ms/algosettings#installOnlyReferencedApps can help you?

freddydk commented 3 weeks ago

but... you state that you have a dependency to that app from your test apps - it should actually download and install it between compiling the apps and compiling the test apps.

Let me have a look at that after Directions,

kennetlindberg commented 3 weeks ago

F.Y.I.: "installOnlyReferencedApps": false worked as a workaround

freddydk commented 3 weeks ago

Thanks - will still investigate, as it should have worked without this.