Skip to content

Commit

Permalink
More stuff came up related to latest kbweb change (#13931)
Browse files Browse the repository at this point in the history
  • Loading branch information
zapu committed Sep 26, 2018
1 parent 57e462f commit 0d62c86
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 9 deletions.
4 changes: 2 additions & 2 deletions go/engine/identify2_test.go
Expand Up @@ -1118,9 +1118,9 @@ func TestResolveAndCheck(t *testing.T) {
// NOTE: Generic proofs are a WIP, this should change to a
// NotFoundError and a success case as the implementation proceeds.
// See CORE-8787
{"foobunny@gubble.social", libkb.AppStatusError{}, false},
{"foobunny@gubble.social", libkb.ResolutionError{}, false},
// TODO set this up for success!
{"t_alice@gubble.social", libkb.AppStatusError{}, false},
{"t_alice@gubble.social", libkb.ResolutionError{}, false},
}
for _, test := range tests {
tc.G.Resolver = goodResolver
Expand Down
8 changes: 1 addition & 7 deletions go/engine/prove_test.go
Expand Up @@ -147,11 +147,5 @@ func _proveGubbleSocial(tc libkb.TestContext, fu *FakeUser, sigVersion libkb.Sig
}
m := libkb.NewMetaContextTODO(g).WithUIs(uis)
err := RunEngine2(m, eng)
// TODO once CORE-8654 is in this will fail and we can condense to a
// single test that runs with promptPosted=true without error.
if promptPosted {
require.Error(tc.T, err)
} else {
require.NoError(tc.T, err)
}
require.NoError(tc.T, err)
}

0 comments on commit 0d62c86

Please sign in to comment.