Do small improvements related to value-based classes

Author: stIncMale

driver-core/src/main/com/mongodb/internal/operation/retry/AttachmentKeys.java

@@ -36,12 +36,11 @@
  * @see AttachmentKey
  */
 public final class AttachmentKeys {
-    private static final AttachmentKey<Integer> MAX_WIRE_VERSION = new DefaultAttachmentKey<>("maxWireVersion");
-    private static final AttachmentKey<BsonDocument> COMMAND = new DefaultAttachmentKey<>("command");
-    private static final AttachmentKey<Boolean> RETRYABLE_WRITE_COMMAND_FLAG = new DefaultAttachmentKey<>("retryableWriteCommandFlag");
-    private static final AttachmentKey<Supplier<String>> COMMAND_DESCRIPTION_SUPPLIER = new DefaultAttachmentKey<>(
-            "commandDescriptionSupplier");
-    private static final AttachmentKey<BulkWriteTracker> BULK_WRITE_TRACKER = new DefaultAttachmentKey<>("bulkWriteTracker");
+    private static final AttachmentKey<Integer> MAX_WIRE_VERSION = DefaultAttachmentKey.of("maxWireVersion");
+    private static final AttachmentKey<BsonDocument> COMMAND = DefaultAttachmentKey.of("command");
+    private static final AttachmentKey<Boolean> RETRYABLE_WRITE_COMMAND_FLAG = DefaultAttachmentKey.of("retryableWriteCommandFlag");
+    private static final AttachmentKey<Supplier<String>> COMMAND_DESCRIPTION_SUPPLIER = DefaultAttachmentKey.of("commandDescriptionSupplier");
+    private static final AttachmentKey<BulkWriteTracker> BULK_WRITE_TRACKER = DefaultAttachmentKey.of("bulkWriteTracker");
 
     public static AttachmentKey<Integer> maxWireVersion() {
         return MAX_WIRE_VERSION;
@@ -72,6 +71,11 @@ private AttachmentKeys() {
         fail();
     }
 
+    /**
+     * A <a href="https://docs.oracle.com/javase/8/docs/api/java/lang/doc-files/ValueBased.html">value-based</a> class.
+     *
+     * @param <V>
+     */
     @Immutable
     private static final class DefaultAttachmentKey<V> implements AttachmentKey<V> {
         private static final Set<String> AVOID_KEY_DUPLICATION = new HashSet<>();
@@ -83,6 +87,10 @@ private DefaultAttachmentKey(final String key) {
             this.key = key;
         }
 
+        static <V> DefaultAttachmentKey<V> of(final String key) {
+            return new DefaultAttachmentKey<>(key);
+        }
+
         @Override
         public boolean equals(final Object o) {
             if (this == o) {

driver-core/src/main/com/mongodb/internal/time/TimePoint.java

@@ -50,6 +50,7 @@ class TimePoint implements Comparable<TimePoint>, StartTime, Timeout {
     @Nullable
     private final Long nanos;
 
+    @VisibleForTesting(otherwise = PRIVATE)
     TimePoint(@Nullable final Long nanos) {
         this.nanos = nanos;
     }