jimmymcp / al-test-runner

VS Code Extension to help run AL tests against Business Central container
https://jpearson.blog/al-test-runner-for-visual-studio-code/
MIT License
11 stars 16 forks source link

Test run doesnt start from workspace (overrrides config.json) #102

Closed VadymPatuk closed 1 year ago

VadymPatuk commented 1 year ago

Hi James,

Has same issue https://github.com/jimmymcp/al-test-runner/issues/98

Figured out that using workspaces with this definition your extensión works well (same if only app.Test openned in VS)

{
  "folders": [
    {
      "path": "app"
    },
    {
      "path": "app.Test"
    },
    {
      "path": ".AL-Go"
    }
  ],
  "settings": {}
}

But if you change with this one extension stop working. After run test its loading indefinitely and never ends.

{
    "folders":  [
                    {
                        "path":  ".AL-Go"
                    },
                    {
                        "path":  "app"
                    },
                    {
                        "path":  "app.Test"
                    }
                ],
    "settings":  {

                 }
}

Did you try format workspace file @alexandr-gladkiy @vengo2021 ?

vengo2021 commented 1 year ago

Hi, managed to make it working by removing .Al-Go from workspace.

{
    "folders":  [
                    {
                        "path":  "app"
                    },
                    {
                        "path":  "app.test"
                    }
                ],
    "settings": {

                }
}

My config looks like:


{
    "containerResultPath":  "C:\\ProgramData\\BcContainerHelper",
    "securePassword":  "01000000d08c9ddf0115d1118c7a00c04fc297eb01000000bef05587031dd94482a03ddfa577a3b50000000002000000000010660000000100002000000063a660bb6b5cbf02b7eddbf26dbf15c3cd28acbf69933656a687362db004af85000000000e8000000002000020000000f22315d5748faedc6987152a43516ab746a7e3c8477a45e5c0e84942e842a071100000002986afc98aced0a05402ba677e7b53ae40000000b344400fe40bdd64211162e89a7b572cb7bca907a297fe32437b147ea2288b8410a6129434a4e3018b75af5edcc75d4595369fc886d2bc5eb9b6a25b90ee982e",
    "userName":  "Admin",
    "userPassword":  "admin",
    "companyName":  "CRONUS UK Ltd.",
    "testSuiteName":  "",
    "vmUserName":  "",
    "vmSecurePassword":  "",
    "remoteContainerName":  "",
    "dockerHost":  "",
    "newPSSessionOptions":  "",
    "testRunnerServiceUrl":  "https://xxx:7048/BC/ODataV4/TestRunner?company=CRONUS UK Ltd.\u0026tenant=default",
    "codeCoveragePath":  ".altestrunner//codecoverage.json",
    "culture":  "en-US",
    "launchConfigName":  "V22R3SSL"
}
VadymPatuk commented 1 year ago

@vengo2021 ok same here, excluding .AL-Go folder from workspace fixes issue.

Worth mention that both app was added with Actions from AL-GO Pte template https://github.com/microsoft/AL-Go-PTE (example https://github.com/VadymPatuk/example-al-test-runner/actions)

Maybe workflow action somehow rearrange data in workspace file and make it incompatible with AL-Test-Runner :confused:

image

jimmymcp commented 1 year ago

Thanks for the repo to reproduce the problem, very helpful. It's a combination of

Fix coming, I'll push into the pre-release first and then into the release later on.