refactor: Restrict subclassing of RelicAddress using 'abstract final'

README.md

@@ -37,7 +37,7 @@ void main() async {
 
   var server = await serve(
     handler,
-    RelicHostnameAddress(hostname: 'localhost'),
+    RelicAddress.fromHostname('localhost'),
     8080,
   );
 

example/example.dart

@@ -1,13 +1,12 @@
 import 'package:relic/relic.dart';
-import 'package:relic/src/address/relic_address.dart';
 
 void main() async {
   var handler =
       const Pipeline().addMiddleware(logRequests()).addHandler(_echoRequest);
 
   var server = await serve(
     handler,
-    RelicHostnameAddress(hostname: 'localhost'),
+    RelicAddress.fromHostname('localhost'),
     8080,
   );
 

lib/relic.dart

@@ -28,8 +28,7 @@ export 'src/middleware/middleware.dart' show Middleware, createMiddleware;
 export 'src/middleware/middleware_extensions.dart' show MiddlewareExtensions;
 
 /// Relic server related exports
-export 'src/address/relic_address.dart'
-    show RelicAddress, RelicHostnameAddress, RelicInternetAddress;
+export 'src/address/relic_address.dart' show RelicAddress;
 export 'dart:io' show InternetAddress;
 export 'src/relic_server.dart' show RelicServer;
 export 'src/relic_server_serve.dart' show serve;

lib/src/address/relic_address.dart

@@ -1,29 +1,29 @@
 import 'dart:io';
 
 /// A class that represents an address.
-abstract class RelicAddress<T> {
+abstract final class RelicAddress<T> {
   /// Returns the address as an [Object].
   T get address;
 
   /// Creates a [RelicAddress] from a [String].
   static RelicAddress<String> fromHostname(String hostname) {
-    return RelicHostnameAddress(hostname: hostname);
+    return _RelicHostnameAddress(hostname: hostname);
   }
 
   /// Creates a [RelicAddress] from an [InternetAddress].
   static RelicAddress<InternetAddress> fromInternetAddress(
     InternetAddress address,
   ) {
-    return RelicInternetAddress(internetAddress: address);
+    return _RelicInternetAddress(internetAddress: address);
   }
 }
 
 /// A class that represents a hostname address.
-class RelicHostnameAddress implements RelicAddress<String> {
+final class _RelicHostnameAddress implements RelicAddress<String> {
   /// The hostname of the address.
   final String hostname;
 
-  RelicHostnameAddress({
+  _RelicHostnameAddress({
     required this.hostname,
   });
 
@@ -36,11 +36,11 @@ class RelicHostnameAddress implements RelicAddress<String> {
 }
 
 /// A class that represents an internet address.
-class RelicInternetAddress implements RelicAddress<InternetAddress> {
+final class _RelicInternetAddress implements RelicAddress<InternetAddress> {
   /// The internet address of the address.
   final InternetAddress internetAddress;
 
-  RelicInternetAddress({
+  _RelicInternetAddress({
     required this.internetAddress,
   });
 

test/headers/headers_test_utils.dart

@@ -3,7 +3,6 @@ import 'dart:io';
 
 import 'package:relic/relic.dart';
 import 'package:http/http.dart' as http;
-import 'package:relic/src/address/relic_address.dart';
 
 /// Thrown when the server returns a 400 status code.
 class BadRequestException implements Exception {
@@ -22,13 +21,13 @@ Future<RelicServer> createServer({
 }) async {
   try {
     return RelicServer.createServer(
-      RelicInternetAddress(internetAddress: InternetAddress.loopbackIPv6),
+      RelicAddress.fromInternetAddress(InternetAddress.loopbackIPv6),
       0,
       strictHeaders: strictHeaders,
     );
   } on SocketException catch (_) {
     return RelicServer.createServer(
-      RelicInternetAddress(internetAddress: InternetAddress.loopbackIPv4),
+      RelicAddress.fromInternetAddress(InternetAddress.loopbackIPv4),
       0,
       strictHeaders: strictHeaders,
     );

test/relic_server_serve_test.dart

@@ -10,7 +10,6 @@ import 'package:async/async.dart';
 import 'package:http/http.dart' as http;
 import 'package:http_parser/http_parser.dart' as parser;
 import 'package:relic/relic.dart';
-import 'package:relic/src/address/relic_address.dart';
 import 'package:relic/src/method/request_method.dart';
 import 'package:relic/src/relic_server_serve.dart' as relic_server;
 import 'package:test/test.dart';
@@ -286,7 +285,7 @@ void main() {
           Future(() => throw StateError('oh no'));
           return syncHandler(request);
         },
-        RelicHostnameAddress(hostname: 'localhost'),
+        RelicAddress.fromHostname('localhost'),
         0,
       );
 
@@ -306,7 +305,7 @@ void main() {
           Future(() => throw StateError('oh no'));
           return syncHandler(request);
         },
-        RelicHostnameAddress(hostname: 'localhost'),
+        RelicAddress.fromHostname('localhost'),
         0,
       );
 
@@ -417,7 +416,7 @@ void main() {
     test('can be set at the server level', () async {
       _server = await relic_server.serve(
         syncHandler,
-        RelicHostnameAddress(hostname: 'localhost'),
+        RelicAddress.fromHostname('localhost'),
         0,
         poweredByHeader: 'ourServer',
       );
@@ -438,7 +437,7 @@ void main() {
             ),
           );
         },
-        RelicHostnameAddress(hostname: 'localhost'),
+        RelicAddress.fromHostname('localhost'),
         0,
         poweredByHeader: 'ourServer',
       );
@@ -654,7 +653,7 @@ Future<void> _scheduleServer(
   assert(_server == null);
   _server = await relic_server.serve(
     handler,
-    RelicHostnameAddress(hostname: 'localhost'),
+    RelicAddress.fromHostname('localhost'),
     0,
     securityContext: securityContext,
   );

test/relic_server_test.dart

@@ -17,12 +17,12 @@ void main() {
   setUp(() async {
     try {
       server = await RelicServer.createServer(
-        RelicInternetAddress(internetAddress: InternetAddress.loopbackIPv6),
+        RelicAddress.fromInternetAddress(InternetAddress.loopbackIPv6),
         0,
       );
     } on SocketException catch (_) {
       server = await RelicServer.createServer(
-        RelicInternetAddress(internetAddress: InternetAddress.loopbackIPv4),
+        RelicAddress.fromInternetAddress(InternetAddress.loopbackIPv4),
         0,
       );
     }