Skip to content

Fixes to issue #415 and #416#442

Merged
mpostol merged 2 commits intompostol:masterfrom
Tycjan-Fortuna-IT:master
Mar 28, 2025
Merged

Fixes to issue #415 and #416#442
mpostol merged 2 commits intompostol:masterfrom
Tycjan-Fortuna-IT:master

Conversation

@Tycjan-Fortuna-IT
Copy link
Contributor

@Tycjan-Fortuna-IT Tycjan-Fortuna-IT commented Mar 27, 2025


The first commit is related to the issue #416.

I've updated NuGet packages to latest stable versions and unified dependencies across projects. Here is a table of updated packages with the corresponding package name and its old and new version.

Additionally the version of package Microsoft.ApplicationInsights was deprecated ⚠️, which I also did update to the long term support one.

Package Old Version New Version
Microsoft.ApplicationInsights 2.22.0 2.23.0
Microsoft.Testing.Extensions.Telemetry 1.2.1 1.6.3
Microsoft.Testing.Extensions.TxrReport.Abstractions 1.2.1 1.6.3
Microsoft.Testing.Extensions.VSTestBridge 1.2.1 1.6.3
Microsoft.Testing.Platform 1.2.1 1.6.3
Microsoft.Testing.Platform.MSBuild 1.2.1 1.6.3
Microsoft.TestPlatform.ObjectModel 17.10.0 17.13.0
MSTest.TestAdapter 3.4.3 3.8.3
MSTest.TestFramework 3.4.3 3.8.3
NuGet.Frameworks 6.10.1 6.13.2
System.Buffers 4.5.1 4.6.1
System.Collections.Immutable 8.0.0 9.0.3
System.Diagnostics.DiagnosticSource 8.0.1 9.0.3
System.Memory 4.5.5 4.6.2
System.Numerics.Vectors 4.5.0 4.6.1
System.Reflection.Metadata 8.0.0 9.0.3
System.Runtime.CompilerServices.Unsafe 6.0.0 6.1.1
System.Security.Cryptography.Xml 8.0.1 9.0.3
System.Security.Permissions 8.0.0 9.0.3
System.Threading.Tasks.Extensions 4.5.4 4.6.2

I additionally, resolved inconsistencies by unifying package versions across different projects within the ExDataManagement solution.


The second commit is related to the issue #415.

I did add the prefix TP.ExDM to all namespaces inside the ExDataManagement solution.

After making changes, the unit tests were used and the tests passed 🟢.

@Tycjan-Fortuna-IT
Copy link
Contributor Author

Tycjan-Fortuna-IT commented Mar 27, 2025

During the regular rebuilding and running tests to see whether I did not destroy anything I found an additional issue with the testing method LambdaSyntaxTest inside the class TP.ExDM.FunctionalProgramming.AnonymousFunctionsUnitTest.

Basically there is a small percentage that this test fails, it is probably due to the usage of the Random class to obtain random numbers and their usage in this test considering the _tolerance variable used in the boolean expression _count > _length / 2 - _tolerance && _count < _length / 2 + _tolerance.

@mpostol
Copy link
Owner

mpostol commented Mar 28, 2025

@Tycjan-Fortuna-IT looks good. I am accepting the pr. Many thanks for your contribution.

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