Tidied up readme, fixed a bug in the cache test

dwmkerr · Jun 28, 2014 · 626b6b6 · 626b6b6
1 parent 8296462
commit 626b6b6
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 13 deletions.
diff --git a/README.md b/README.md
@@ -171,4 +171,4 @@ Thanks
 
 Thanks go the the following contributors:
 
-[https://github.com/joshvillbrandt](joshvillbrant) - Adding support for `$templateCache`.
+[joshvillbrandt](https://github.com/joshvillbrandt) - Adding support for `$templateCache`.
diff --git a/test/template.spec.js b/test/template.spec.js
@@ -63,21 +63,21 @@ describe('template', function() {
       controller: "TemplateController",
       templateUrl: "templatetobecached.html"
     }).then(function(modal) {
-      expect(modal).not.toBe(null);
-    });
 
-    //  The template should now be cached...
-    spyOn($templateCache, 'get');
+      //  The template should now be cached...
+      spyOn($templateCache, 'get').and.callThrough();
 
-    ModalService.showModal({
-      controller: "TemplateController",
-      templateUrl: "templatetobecached.html"
-    }).then(function(modal) {
-      expect(modal).not.toBe(null);
-    });
+      ModalService.showModal({
+        controller: "TemplateController",
+        templateUrl: "templatetobecached.html"
+      }).then(function(modal) {
+        expect(modal).not.toBe(null);
+      });
 
-    //  ...so get should have been called.
-    expect($templateCache.get).toHaveBeenCalledWith('templatetobecached.html');
+      //  ...so get should have been called.
+      expect($templateCache.get).toHaveBeenCalledWith('templatetobecached.html');
+
+    });
 
     $httpBackend.flush();