1047330: Fixed b-unit test case failures in CI. - #46
Open
Yokesh-SF4393 wants to merge 23 commits into
Open
Conversation
- Add polished CodeQL workflow (csharp + javascript-typescript + actions) - Add Dependabot configuration for NuGet, npm, and GitHub Actions - Enable weekly scheduled scans and dependency updates
…ntain permissions' Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
…zor-toolkit into ci-confirguration
- Add SLSA build provenance attestation (actions/attest-build-provenance@v2)
for nupkg/*.nupkg and nupkg/*.snupkg, scoped to pushed tag ref.
- Add workflow-level concurrency guard (cancel-in-progress: false) so re-tags
don't race the sign-and-push steps.
- Switch vulnerability scan to exit-code-driven --format json capture, upload
vuln-report artifact for diagnosis on failure.
- Install Node 22 + npm + run gulp themes task on the release runner so the
csproj BeforeBuild target completes on a clean image.
- Pass -p:ContinuousIntegrationBuild=true to restore/build/pack for re-tag
reproducibility.
- Replace hard-coded copyright year with UtcNow-derived property.
- Update .github/{THREAT-MODEL,SECURITY,DEVELOPMENT}.md to reflect new posture.
…ntain permissions' Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
…ntain permissions' Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
…zor-toolkit into ci-confirguration
…at sets the timezone/locale so the runner uses UTC and en_US.UTF-8.
…nto ci-confirguration
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Bug description
Fixed b-unit test case failures in CI for charts
Root cause
Found some test cases are failing due to time delay and text content match.
Solution description
Fixed the test cases by adding the wait for assertion and used contains for text content.
Code Studio usage(Mandatory)
Code Studio used in this PR/MR?
If
Yes: Primary use (choose one)Outcome
If “Cost time” explain in short (1 or 2 lines):
Impact assessment
Reason for not identifying earlier
Provide the reason for not identifying the bug earlier.
Areas tested against this fix
Breaking changes
breaking-issue)If yes, provide breaking commit details link and migration guidance.
Regression testing
Action taken to prevent recurrence
Automation status
Cross-platform verification
Related issues
Is this issue present in EJ2 or other components?
needs-attention-coreteam)Output screenshots
Post the output screenshots if a UI is affected or added due to this bug.
API changes
Performance verification
Reviewer Checklist