Skip to content

test: Adding API bulk automation to Blazor.Lite#241

Merged
damyanpetev merged 32 commits into
masterfrom
mkirova/bulk-automation
Jul 15, 2026
Merged

test: Adding API bulk automation to Blazor.Lite#241
damyanpetev merged 32 commits into
masterfrom
mkirova/bulk-automation

Conversation

@MayaKirova

@MayaKirova MayaKirova commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

Description

Moving the existing bulk automation (with a few tweaks so they don't use metadata from the model) to public repo.

Motivation / Context

So we reuse what we already have for automation.

Type of Change (check all that apply):

  • Bug fix
  • New functionality
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Refactoring (no functional changes)
  • Documentation
  • Demos
  • CI/CD
  • Tests
  • Changelog

Component(s) / Area(s) Affected:

All component now have logic that checks all props/events/methods/templates for errors during blazor serialization/deserialization.

Checklist:

  • All relevant tags have been applied to this PR
  • This PR includes unit tests covering all the new code
  • This PR includes API docs for newly added methods/properties
  • This PR includes general feature table updates in the root README.MD
  • This PR includes CHANGELOG.MD updates for newly added functionality
  • This PR contains breaking changes
  • This PR includes behavioral changes and the feature specification has been updated with them
  • Accessibility (ARIA, keyboard navigation, focus management) has been verified

Comment thread .github/workflows/ci.yml Outdated
kdinev
kdinev previously requested changes Jul 10, 2026
Comment thread tests/Infragistics.Blazor.Testing/Playwright.Testing/Playwright.Testing.csproj Outdated
@MayaKirova
MayaKirova requested a review from kdinev July 10, 2026 12:48

@damyanpetev damyanpetev left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

At least the project use nullable, which is great, but there seems to be some warnings, that'd be lovely if resolved (should be the standard for all new code).

Overall, looking to clean up any and all "leftover" bits that are either not needed/used/for this source and keep the test reflection as flexible as possible so we can refactor components one by one

We'll still need to adjust some thing as we move them around (such as WCEnumNameAttribute vs the System.Text.Json equivalent we'll likely move to, BaseRendererControl references, etc).

Comment thread .github/workflows/ci.yml Outdated
Comment thread tests/IgniteUI.Blazor.Lite.TestBed/Components/Common/CustomTypes.cs
@MayaKirova
MayaKirova requested a review from damyanpetev July 14, 2026 12:15
Comment thread tests/IgniteUI.Blazor.Lite.TestBed/Components/Common/CustomTypes.cs Outdated
Comment thread tests/Infragistics.Blazor.Testing/Infragistics.Blazor.Testing/.runsettings Outdated
Comment thread tests/IgniteUI.Blazor.Lite.IntegrationTests/README.md Outdated
Comment thread tests/IgniteUI.Blazor.Lite.IntegrationTests/README.md Outdated
Comment thread tests/IgniteUI.Blazor.Lite.IntegrationTests/README.md Outdated
Comment thread tests/IgniteUI.Blazor.Lite.IntegrationTests/ComponentTest.cs
@damyanpetev damyanpetev changed the title Adding bulk automation to Blazor.Lite test: Adding API bulk automation to Blazor.Lite Jul 15, 2026
@damyanpetev damyanpetev added the squash-merge Merge PR with "Squash and Merge" option label Jul 15, 2026
@damyanpetev
damyanpetev merged commit 854e951 into master Jul 15, 2026
9 of 10 checks passed
@damyanpetev
damyanpetev deleted the mkirova/bulk-automation branch July 15, 2026 18:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

squash-merge Merge PR with "Squash and Merge" option 🧪 ci: tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants