Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(104)

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: impl/dummy/dummy_test.go

Issue 1525923002: Implement Mail service. (Closed) Base URL: https://github.com/luci/gae.git@filter_user
Patch Set: add tests Created 5 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « impl/dummy/dummy.go ('k') | impl/memory/context.go » ('j') | impl/memory/mail.go » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: impl/dummy/dummy_test.go
diff --git a/impl/dummy/dummy_test.go b/impl/dummy/dummy_test.go
index 40057489571181932731734d0d2a43405470f8be..34ff57986e84ebec819467c9886373d36985e60d 100644
--- a/impl/dummy/dummy_test.go
+++ b/impl/dummy/dummy_test.go
@@ -9,6 +9,7 @@ import (
dsS "github.com/luci/gae/service/datastore"
infoS "github.com/luci/gae/service/info"
+ mailS "github.com/luci/gae/service/mail"
mcS "github.com/luci/gae/service/memcache"
tqS "github.com/luci/gae/service/taskqueue"
userS "github.com/luci/gae/service/user"
@@ -82,5 +83,14 @@ func TestContextAccess(t *testing.T) {
}, ShouldPanicWith, "dummy: method User.IsAdmin is not implemented")
})
+ Convey("Mail", func() {
martiniss 2015/12/16 21:54:15 Bad indentation?
iannucci 2015/12/17 00:01:07 Rietveld bug? Check the new patch, but it looks fi
+ c = mailS.Set(c, Mail())
+ So(mailS.Get(c), ShouldNotBeNil)
+ So(func() {
+ defer p()
+ _ = mailS.Get(c).Send(nil)
+ }, ShouldPanicWith, "dummy: method Mail.Send is not implemented")
+ })
+
})
}
« no previous file with comments | « impl/dummy/dummy.go ('k') | impl/memory/context.go » ('j') | impl/memory/mail.go » ('J')

Powered by Google App Engine
This is Rietveld 408576698