Skip to content

Introducing TestType to TestRunner #3821

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 4 commits into from
Jun 16, 2025
Merged

Conversation

haoranpb
Copy link
Contributor

@haoranpb haoranpb commented Jun 4, 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

Work Item(s)

Fixes AB#579962

@haoranpb haoranpb requested a review from a team as a code owner June 4, 2025 06:49
@github-actions github-actions bot added this to the Version 27.0 milestone Jun 4, 2025
Copy link
Contributor

@stkillen stkillen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should we also expose the test type in the UI page? E.g. filtering to test type could be useful

stkillen
stkillen previously approved these changes Jun 10, 2025
nikolakukrika
nikolakukrika previously approved these changes Jun 10, 2025
@haoranpb haoranpb dismissed stale reviews from nikolakukrika and stkillen via bf307bf June 16, 2025 06:35
@haoranpb haoranpb enabled auto-merge (squash) June 16, 2025 06:51
@haoranpb haoranpb merged commit 986c344 into main Jun 16, 2025
25 checks passed
@haoranpb haoranpb deleted the private/haoransun/include-testtype branch June 16, 2025 08:30
aholstrup1 pushed a commit to microsoft/navcontainerhelper that referenced this pull request Jun 16, 2025
Allowing filter tests by `TestType` in CI pipeline, see more in Related
to microsoft/BCApps#3821
haoranpb added a commit 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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants