Skip to content

Remove spurious asserts#147

Merged
cstamas merged 1 commit intomainfrom
fix-sonar-warnings
Jun 3, 2024
Merged

Remove spurious asserts#147
cstamas merged 1 commit intomainfrom
fix-sonar-warnings

Conversation

@cstamas
Copy link
Member

@cstamas cstamas commented Jun 3, 2024

These asserts same thing against same thing. Unsure is this call maybe here to trigger some "lazily initialized stuff", but still the test pass OK without these.

@cstamas cstamas requested review from kwin and mcculls June 3, 2024 16:40
@cstamas cstamas self-assigned this Jun 3, 2024
@cstamas cstamas changed the title Remove spurious asserts. Remove spurious asserts Jun 3, 2024
@sonarqubecloud
Copy link

sonarqubecloud bot commented Jun 3, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

@cstamas cstamas merged commit e7f904d into main Jun 3, 2024
@cstamas cstamas deleted the fix-sonar-warnings branch June 3, 2024 16:51
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.

2 participants