Skip to content

Fix(tests): resolve nullable reference warnings in SqlTests #56

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.

Sign up for GitHub

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

Jump to bottom
Merged
abbasmhd merged 46 commits into from
Mar 1, 2025
Merged

Fix(tests): resolve nullable reference warnings in SqlTests #56

abbasmhd merged 46 commits into from
Mar 1, 2025

Conversation

@abbasmhd
Copy link
Owner

@abbasmhd abbasmhd commented Mar 1, 2025

This PR addresses nullable reference warnings in the SQL test suite by adding proper nullable annotations while maintaining the existing test structure and formatting.

Changes

  • Updated Values_ShouldReturnCorrectResult parameter to object?[]
  • Modified SelectQuoted_ShouldReturnCorrectResult array declaration
  • Added nullable annotations to test data arrays
  • Preserved existing test formatting and alignment

Testing

  • All tests pass
  • No nullable reference warnings
  • Existing functionality remains unchanged

Checklist

  • Code builds without warnings
  • Tests pass
  • Formatting preserved
  • No breaking changes

Related Issues

Closes #55

abbasmhd added 30 commits April 8, 2023 10:21
@abbasmhd
Use FluentMigration to populate the database
e20f47b
@abbasmhd
Update dotnet.yml
0f4e256 
Bump to .Net 7
@abbasmhd
async optimisation
b274984
@abbasmhd
refactoring
b966df4
@abbasmhd
update test file
433c00b
@abbasmhd
merge
95c9572
@abbasmhd
update test
9ead329
@abbasmhd
merge
aa395f6
@abbasmhd
don't run in build
f707ca7
@abbasmhd
Add XML documentations
aded1b9
@abbasmhd
merge
ed7a32c
@abbasmhd
Merge branch 'main' into development
68d386f
@abbasmhd
move test project
8399f87
@abbasmhd
fix reference and bulild
60a8607
@abbasmhd
Merge branch 'main' into development
ff726fb
@abbasmhd
Add more tests for sql extention
4dbe501
@abbasmhd
Add ColumnDefinitionBuilder
d7eb07e
@abbasmhd
merge
2bae954
@abbasmhd
Upgrade to .Net 8.0
b85e676
@abbasmhd
rem file
4108c92
@abbasmhd
format
3f80aba
@abbasmhd
update global json file
82b2849
@abbasmhd
fix build
a9ef8fd
@abbasmhd
build
b472a6a
@abbasmhd
Merge branch 'main' into development
5aa9c96
@abbasmhd
Docs(readme): update readme with comprehensive project documentation
b5ab051
@abbasmhd
Docs(guides): add comprehensive documentation guides
6d64561 
- Security Guide
  - Database security
  - Parameter security
  - Authentication & Authorization
  - HTTPS and TLS
  - API security
  - Monitoring and logging
  - Best practices checklist

- Examples Guide
  - Basic CRUD operations
  - Advanced queries
  - Multi-table operations
  - Integration examples
  - Real-world scenarios
  - Complete code samples

- Troubleshooting Guide
  - Common issues and solutions
  - Debugging tips
  - Performance optimization
  - Security fixes
  - Configuration troubleshooting
  - Best practices
@abbasmhd
Chore(deps): update actions/upload-artifact to v4
71e593c
@abbasmhd
Fix(ci): update GitHub Pages workflow to use official actions
e16a54c
@abbasmhd
Merge branch 'main' into development
f042bfd
@abbasmhd
Fix(ci): simplify GitHub Pages workflow to use pure GitHub Actions
a40bbc7
@abbasmhd
Merge branch 'main' into development
d9b4036
@abbasmhd
Fix(ci): remove no-build flag to resolve NETSDK1085 error
b18f5f1
@abbasmhd
Merge branch 'development' of https://github.com/abbasmhd/ProxyR into…
e7d5403 
… development
@abbasmhd
Fix(ci): remove Jekyll and simplify GitHub Pages deployment
6267660 
- Remove Jekyll dependencies and configuration
- Simplify GitHub Actions workflow
- Use direct static file deployment
- Consolidate to single job workflow

Fixes #43
@abbasmhd
Merge branch 'main' into development
4cf5eed
@abbasmhd
Feat(branding): add project metadata and icon
83cb99b 
- Add SVG icon for project branding
- Update NuGet package metadata with enhanced description
- Add Source Link and symbol package support
- Update documentation site with icon and branding
- Add favicon and header logo
@abbasmhd
Fix(branding): correct SVG icon format
4af5879
@abbasmhd
Fix(branding): correct SVG icon format
90d9f32
@abbasmhd
Style(branding): update icon color to brand blue
3118172
@abbasmhd
Merge branch 'main' into development
873665e
@abbasmhd
Fix(config): remove comments from appsettings.json to resolve linting…
0526f6c 
… errors
@abbasmhd
Docs(extensions): add XML documentation comments to extension classes
1feb099
@abbasmhd
Fix(tests): add nullable annotations while preserving formatting
c9cb262
@abbasmhd
Fix(tests): align test data with expected output in Values_WhenObject…
7e83c99 
… test
@abbasmhd abbasmhd self-assigned this Mar 1, 2025
@abbasmhd abbasmhd merged commit 093ac4b into main Mar 1, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@abbasmhd abbasmhd

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Fix Nullable Reference Issues in SQL Tests

2 participants

@abbasmhd