Skip to content

Commit

Permalink
Move interface out of internal
Browse files Browse the repository at this point in the history
  • Loading branch information
johannespfeiffer committed Oct 8, 2024
1 parent 7d2e9fb commit 526f09a
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 2 deletions.
1 change: 0 additions & 1 deletion src/NATS.Client.Core/INatsConnection.cs
Original file line number Diff line number Diff line change
@@ -1,6 +1,5 @@
using System.Diagnostics.CodeAnalysis;
using System.Threading.Channels;
using NATS.Client.Core.Internal;

namespace NATS.Client.Core;

Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
namespace NATS.Client.Core.Internal;
namespace NATS.Client.Core;

public interface ISocketConnection : IAsyncDisposable
{
Expand Down
2 changes: 2 additions & 0 deletions tests/NATS.Client.JetStream.Tests/NatsJsContextFactoryTest.cs
Original file line number Diff line number Diff line change
Expand Up @@ -105,6 +105,8 @@ public class MockConnection : INatsConnection
public INatsSubscriptionManager SubscriptionManager { get; } = new TestSubscriptionManager();

public NatsHeaderParser HeaderParser { get; } = new NatsHeaderParser(Encoding.UTF8);
public Func<(string Host, int Port), ValueTask<(string Host, int Port)>>? OnConnectingAsync { get; set; }

Check warning on line 108 in tests/NATS.Client.JetStream.Tests/NatsJsContextFactoryTest.cs

View workflow job for this annotation

GitHub Actions / check

Elements should be separated by blank line

Check warning on line 108 in tests/NATS.Client.JetStream.Tests/NatsJsContextFactoryTest.cs

View workflow job for this annotation

GitHub Actions / check

Elements should be separated by blank line

Check warning on line 108 in tests/NATS.Client.JetStream.Tests/NatsJsContextFactoryTest.cs

View workflow job for this annotation

GitHub Actions / Windows (main)

Check warning on line 108 in tests/NATS.Client.JetStream.Tests/NatsJsContextFactoryTest.cs

View workflow job for this annotation

GitHub Actions / Windows (main)

Check warning on line 108 in tests/NATS.Client.JetStream.Tests/NatsJsContextFactoryTest.cs

View workflow job for this annotation

GitHub Actions / Windows (latest)

Check warning on line 108 in tests/NATS.Client.JetStream.Tests/NatsJsContextFactoryTest.cs

View workflow job for this annotation

GitHub Actions / Windows (latest)

Check warning on line 108 in tests/NATS.Client.JetStream.Tests/NatsJsContextFactoryTest.cs

View workflow job for this annotation

GitHub Actions / Linux (v2.9)

Check warning on line 108 in tests/NATS.Client.JetStream.Tests/NatsJsContextFactoryTest.cs

View workflow job for this annotation

GitHub Actions / Linux (v2.9)

Check warning on line 108 in tests/NATS.Client.JetStream.Tests/NatsJsContextFactoryTest.cs

View workflow job for this annotation

GitHub Actions / Windows (v2.9)

Check warning on line 108 in tests/NATS.Client.JetStream.Tests/NatsJsContextFactoryTest.cs

View workflow job for this annotation

GitHub Actions / Windows (v2.9)

Check warning on line 108 in tests/NATS.Client.JetStream.Tests/NatsJsContextFactoryTest.cs

View workflow job for this annotation

GitHub Actions / Linux (latest)

Check warning on line 108 in tests/NATS.Client.JetStream.Tests/NatsJsContextFactoryTest.cs

View workflow job for this annotation

GitHub Actions / Linux (latest)

Check warning on line 108 in tests/NATS.Client.JetStream.Tests/NatsJsContextFactoryTest.cs

View workflow job for this annotation

GitHub Actions / Linux (main)

Check warning on line 108 in tests/NATS.Client.JetStream.Tests/NatsJsContextFactoryTest.cs

View workflow job for this annotation

GitHub Actions / Linux (main)

public Func<ISocketConnection, ValueTask<ISocketConnection>>? OnSocketAvailableAsync { get; set; }

Check warning on line 109 in tests/NATS.Client.JetStream.Tests/NatsJsContextFactoryTest.cs

View workflow job for this annotation

GitHub Actions / check

Elements should be separated by blank line

Check warning on line 109 in tests/NATS.Client.JetStream.Tests/NatsJsContextFactoryTest.cs

View workflow job for this annotation

GitHub Actions / check

Elements should be separated by blank line

Check warning on line 109 in tests/NATS.Client.JetStream.Tests/NatsJsContextFactoryTest.cs

View workflow job for this annotation

GitHub Actions / Windows (main)

Check warning on line 109 in tests/NATS.Client.JetStream.Tests/NatsJsContextFactoryTest.cs

View workflow job for this annotation

GitHub Actions / Windows (main)

Check warning on line 109 in tests/NATS.Client.JetStream.Tests/NatsJsContextFactoryTest.cs

View workflow job for this annotation

GitHub Actions / Windows (latest)

Check warning on line 109 in tests/NATS.Client.JetStream.Tests/NatsJsContextFactoryTest.cs

View workflow job for this annotation

GitHub Actions / Windows (latest)

Check warning on line 109 in tests/NATS.Client.JetStream.Tests/NatsJsContextFactoryTest.cs

View workflow job for this annotation

GitHub Actions / Linux (v2.9)

Check warning on line 109 in tests/NATS.Client.JetStream.Tests/NatsJsContextFactoryTest.cs

View workflow job for this annotation

GitHub Actions / Linux (v2.9)

Check warning on line 109 in tests/NATS.Client.JetStream.Tests/NatsJsContextFactoryTest.cs

View workflow job for this annotation

GitHub Actions / Windows (v2.9)

Check warning on line 109 in tests/NATS.Client.JetStream.Tests/NatsJsContextFactoryTest.cs

View workflow job for this annotation

GitHub Actions / Windows (v2.9)

Check warning on line 109 in tests/NATS.Client.JetStream.Tests/NatsJsContextFactoryTest.cs

View workflow job for this annotation

GitHub Actions / Linux (latest)

Check warning on line 109 in tests/NATS.Client.JetStream.Tests/NatsJsContextFactoryTest.cs

View workflow job for this annotation

GitHub Actions / Linux (latest)

Check warning on line 109 in tests/NATS.Client.JetStream.Tests/NatsJsContextFactoryTest.cs

View workflow job for this annotation

GitHub Actions / Linux (main)

Check warning on line 109 in tests/NATS.Client.JetStream.Tests/NatsJsContextFactoryTest.cs

View workflow job for this annotation

GitHub Actions / Linux (main)


public ValueTask<TimeSpan> PingAsync(CancellationToken cancellationToken = default) => throw new NotImplementedException();

Expand Down

0 comments on commit 526f09a

Please sign in to comment.