From 8e24a47b69f940c31ac95451b4584b95de7c8670 Mon Sep 17 00:00:00 2001
From: Tavian Barnes <tavianator@tavianator.com>
Date: Sat, 4 Oct 2014 12:41:07 -0400
Subject: lazy: Test best practices.

---
 .../tavianator/sangria/lazy/LazySingletonTest.java  | 21 ++++++++++++++++-----
 1 file changed, 16 insertions(+), 5 deletions(-)

(limited to 'sangria-lazy/src/test/java/com/tavianator/sangria/lazy/LazySingletonTest.java')

diff --git a/sangria-lazy/src/test/java/com/tavianator/sangria/lazy/LazySingletonTest.java b/sangria-lazy/src/test/java/com/tavianator/sangria/lazy/LazySingletonTest.java
index 69fabce..2895c7d 100644
--- a/sangria-lazy/src/test/java/com/tavianator/sangria/lazy/LazySingletonTest.java
+++ b/sangria-lazy/src/test/java/com/tavianator/sangria/lazy/LazySingletonTest.java
@@ -20,12 +20,14 @@ package com.tavianator.sangria.lazy;
 import javax.inject.Inject;
 import javax.inject.Provider;
 
+import com.google.inject.AbstractModule;
 import com.google.inject.Guice;
 import com.google.inject.Injector;
 import com.google.inject.Module;
 import com.google.inject.Stage;
 import org.junit.Test;
 
+import static com.tavianator.sangria.test.SangriaMatchers.*;
 import static org.hamcrest.Matchers.*;
 import static org.junit.Assert.*;
 
@@ -62,11 +64,6 @@ public class LazySingletonTest {
         test(Stage.PRODUCTION, new SangriaLazyModule());
     }
 
-    @Test
-    public void testDuplicateModule() {
-        test(Stage.PRODUCTION, new SangriaLazyModule(), new SangriaLazyModule());
-    }
-
     private void test(Stage stage, Module... modules) {
         int before = Scoped.INSTANCES.get();
 
@@ -80,4 +77,18 @@ public class LazySingletonTest {
         assertThat(provider.get(), sameInstance(instance));
         assertThat(Scoped.INSTANCES.get(), equalTo(before + 1));
     }
+
+    @Test
+    public void testBestPractices() {
+        Module module = new AbstractModule() {
+            @Override
+            protected void configure() {
+                install(new SangriaLazyModule());
+                bind(Scoped.class);
+            }
+        };
+
+        assertThat(module, is(atomic()));
+        assertThat(module, followsBestPractices());
+    }
 }
-- 
cgit v1.2.3