Skip to content

Fix netstandard bugs and tidy up tests#817

Merged
mtmk merged 30 commits into
mainfrom
tidy-up-tests
Apr 10, 2025
Merged

Fix netstandard bugs and tidy up tests#817
mtmk merged 30 commits into
mainfrom
tidy-up-tests

Conversation

@mtmk

@mtmk mtmk commented Apr 9, 2025

Copy link
Copy Markdown
Member

Move NATS server process management to the lightweight class. Created the slow tests project for the last bit of tests heavily depending on the old NatsServer class.

  • bug fix: netstandard ArgumentNullException extension fix
  • bug fix: netstandard JetStream name validation fix

@mtmk mtmk changed the title Tidy up tests Fix netstandard bugs and tidy up tests Apr 10, 2025

@galvesribeiro galvesribeiro left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@mtmk
mtmk merged commit 1bc35ac into main Apr 10, 2025
@mtmk
mtmk deleted the tidy-up-tests branch April 10, 2025 21:43
mtmk added a commit that referenced this pull request Apr 16, 2025
* Stopped testing with net6.0 (#821)
* Update JetStream models (#832)
* Add direct request-reply (#829)
* Socket Connection Factory (#828)
* Fix proxy with TLS scenario with SocketConnectionFactory (#826)
* Make NatsJSConstants public (#825)
* Use Stopwatch in PingCommand for more precise timing (#827)
* Fix netstandard bugs and tidy up tests (#817)
@mtmk mtmk mentioned this pull request Apr 16, 2025
mtmk added a commit that referenced this pull request Apr 16, 2025
* Stopped testing with net6.0 (#821)
* Update JetStream models (#832)
* Add direct request-reply (#829)
* Socket Connection Factory (#828)
* Fix proxy with TLS scenario with SocketConnectionFactory (#826)
* Make NatsJSConstants public (#825)
* Use Stopwatch in PingCommand for more precise timing (#827)
* Fix netstandard bugs and tidy up tests (#817)
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