refactor(nextcloud_test): mark test targets and fixture interceptor a…

…s internal Signed-off-by: Nikolas Rimikis <leptopoda@users.noreply.github.com>
nextcloud · Sep 11, 2024 · 115d63a · 115d63a
1 parent 441c9d3
commit 115d63a
Show file tree

Hide file tree

Showing 6 changed files with 11 additions and 2 deletions.
diff --git a/packages/nextcloud/packages/nextcloud_test/lib/nextcloud_test.dart b/packages/nextcloud/packages/nextcloud_test/lib/nextcloud_test.dart
@@ -1,2 +1,2 @@
-export 'src/fixtures.dart' hide appendFixture;
+export 'src/fixtures.dart' hide appendFixture, validateFixture;
 export 'src/presets.dart';
diff --git a/packages/nextcloud/packages/nextcloud_test/lib/src/fixture_interceptor.dart b/packages/nextcloud/packages/nextcloud_test/lib/src/fixture_interceptor.dart
@@ -4,8 +4,10 @@ import 'dart:typed_data';
 
 import 'package:http/http.dart' as http;
 import 'package:interceptor_http_client/interceptor_http_client.dart';
+import 'package:meta/meta.dart';
 
 /// An http interceptor that records every request and adds them to a fixture.
+@internal
 final class FixtureInterceptor implements HttpInterceptor {
   /// Creates a new fixture interceptor.
   const FixtureInterceptor({

diff --git a/packages/nextcloud/packages/nextcloud_test/lib/src/fixtures.dart b/packages/nextcloud/packages/nextcloud_test/lib/src/fixtures.dart
@@ -1,4 +1,5 @@
 import 'package:built_collection/built_collection.dart';
+import 'package:meta/meta.dart';
 import 'package:nextcloud/webdav.dart';
 import 'package:nextcloud_test/src/models/models.dart';
 // ignore: implementation_imports
@@ -9,6 +10,7 @@ var _closed = false;
 final _fixture = <String>[];
 
 /// Appends some [data] to the current fixture.
+@internal
 void appendFixture(String data) {
   if (!_closed) {
     _fixture.add(data);
@@ -30,6 +32,7 @@ void resetFixture() {
 /// Validates that the requests match the stored fixtures.
 ///
 /// If there is no stored fixture a new one is created.
+@internal
 void validateFixture(NextcloudTester tester) {
   if (_fixture.isEmpty) {
     return;

diff --git a/packages/nextcloud/packages/nextcloud_test/lib/src/presets.dart b/packages/nextcloud/packages/nextcloud_test/lib/src/presets.dart
@@ -1,5 +1,5 @@
 import 'package:meta/meta.dart';
-import 'package:nextcloud_test/nextcloud_test.dart';
+import 'package:nextcloud_test/src/fixtures.dart';
 import 'package:nextcloud_test/src/models/models.dart';
 import 'package:nextcloud_test/src/test_target/test_target.dart';
 import 'package:test/test.dart';

diff --git a/packages/nextcloud/packages/nextcloud_test/lib/src/test_target/docker_container.dart b/packages/nextcloud/packages/nextcloud_test/lib/src/test_target/docker_container.dart
@@ -89,6 +89,7 @@ final class DockerContainerFactory extends TestTargetFactory<DockerContainerInst
 }
 
 /// Test target representing a docker container.
+@internal
 final class DockerContainerInstance extends TestTargetInstance {
   /// Creates a new Docker container instance.
   DockerContainerInstance({

diff --git a/packages/nextcloud/packages/nextcloud_test/lib/src/test_target/local.dart b/packages/nextcloud/packages/nextcloud_test/lib/src/test_target/local.dart
@@ -3,11 +3,13 @@ import 'dart:convert';
 import 'dart:io';
 
 import 'package:built_collection/built_collection.dart';
+import 'package:meta/meta.dart';
 import 'package:nextcloud_test/src/models/models.dart';
 import 'package:nextcloud_test/src/test_target/test_target.dart';
 import 'package:version/version.dart';
 
 /// Factory for running tests against a local instance.
+@internal
 final class LocalFactory extends TestTargetFactory<LocalInstance> {
   /// Creates a new test factory for a local server.
   LocalFactory({
@@ -86,6 +88,7 @@ final class LocalFactory extends TestTargetFactory<LocalInstance> {
 }
 
 /// Test target representing a local instance.
+@internal
 final class LocalInstance extends TestTargetInstance {
   /// Creates a new test instance for a local server.
   LocalInstance({