More specs for base service

src/coffee/services/base.coffee

@@ -26,7 +26,8 @@ class BaseService
   ###
   fetch: ->
     deferred = Q.defer()
-    @_rest @_projectEndpoint, (e, r, b)->
+    @_rest.GET @_projectEndpoint, (e, r, b)->
+      # TODO: wrap / handle responses generally
       if e
         deferred.reject e
       else

src/spec/services/base.spec.coffee

@@ -1,11 +1,12 @@
+Q = require('q')
 BaseService = require('../../lib/services/base')
 
 describe 'BaseService', ->
 
   beforeEach ->
     @restMock =
       config: {}
-      GET: ->
+      GET: (endpoint, callback)->
       POST: ->
       PUT: ->
       DELETE: ->
@@ -20,4 +21,23 @@ describe 'BaseService', ->
     expect(base).toBeDefined()
     expect(base._projectEndpoint).toBe '/'
 
-  xit 'should return promise on fetch', ->
+  it 'should return promise on fetch', ->
+    base = new BaseService @restMock
+    promise = base.fetch()
+    expect(Q.isPromise(promise)).toBe true
+
+  it 'should resolve the promise on fetch', (done)->
+    spyOn(@restMock, 'GET').andCallFake((endpoint, callback)-> callback(null, {statusCode: 200}, '{"foo": "bar"}'))
+    base = new BaseService @restMock
+    base.fetch().then (result)->
+      expect(result).toEqual foo: 'bar'
+      done()
+
+  it 'should reject the promise on fetch', (done)->
+    spyOn(@restMock, 'GET').andCallFake((endpoint, callback)-> callback('foo', null, null))
+    base = new BaseService @restMock
+    base.fetch().then (result)->
+      expect(result).not.toBeDefined()
+    .fail (e)->
+      expect(e).toBe 'foo'
+      done()