Skip to content

Introducing TestType filter #3959

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 5 commits into from
Jun 16, 2025
Merged

Conversation

haoranpb
Copy link
Contributor

@haoranpb haoranpb commented Jun 4, 2025

Allowing filter tests by TestType in CI pipeline, see more in Related to microsoft/BCApps#3821

mazhelez
mazhelez previously approved these changes Jun 4, 2025
mazhelez
mazhelez previously approved these changes Jun 6, 2025
aholstrup1
aholstrup1 previously approved these changes Jun 11, 2025
@haoranpb haoranpb dismissed stale reviews from aholstrup1 and mazhelez via 82932ed June 16, 2025 06:45
haoranpb added a commit to microsoft/BCApps that referenced this pull request Jun 16, 2025
#### Summary 

Introducing `TestType` to TestRunner. We can now select which `TestType`
to run from CLI.

Related to microsoft/navcontainerhelper#3959

There will be a follow-up PR to add this capability in both
`ALTestRunner.psm1` and [navcontainerhelper
Get-Tests](https://github.com/microsoft/navcontainerhelper/blob/df270f9e3ed96260221f918f2deb0fd7af941f07/AppHandling/PsTestFunctions.ps1#L328C10-L328C13)

#### Work Item(s) <!-- Add the issue number here after the #. The issue
needs to be open and approved. Submitting PRs with no linked issues or
unapproved issues is highly discouraged. -->
Fixes
[AB#579962](https://dynamicssmb2.visualstudio.com/1fcb79e7-ab07-432a-a3c6-6cf5a88ba4a5/_workitems/edit/579962)
@aholstrup1 aholstrup1 merged commit c955c92 into main Jun 16, 2025
1 check passed
@aholstrup1 aholstrup1 deleted the private/haoransun/include-testtype branch June 16, 2025 08:35
haoranpb added a commit to microsoft/BCApps that referenced this pull request Jun 19, 2025
#### Summary

Successor of:
1. #3821
2. microsoft/navcontainerhelper#3959

Only run `UnitTest` in the build of BCApps for both System Application
and Business Foundation. It will be possible to define `IntegrationTest`
in BCApps with
[AB#581154](https://dynamicssmb2.visualstudio.com/Dynamics%20SMB/_workitems/edit/581154),
but they will be run in NAV. (We only have `UnitTests` for now)

Tried to take a look at disabled tests, they are not `IntegrationTest`,
will not fix them as part of the effort. Re-enabling some easy fixes.

#### Work Item(s) 
Fixes
[AB#579962](https://dynamicssmb2.visualstudio.com/1fcb79e7-ab07-432a-a3c6-6cf5a88ba4a5/_workitems/edit/579962)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants