Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(826)

Issue 6636057: state: add password test

Can't Edit
Can't Publish+Mail
Start Review
Created:
13 years ago by rog
Modified:
13 years ago
Reviewers:
mp+128997
Visibility:
Public.

Description

state: add password test Add a test to check that appropriate entities can set the passwords they need to set. Currently there is no restriction on capabilities, but this test will be more relevant when we move to an API-based state. https://code.launchpad.net/~rogpeppe/juju-core/119-add-password-test/+merge/128997 (do not edit description out of merge proposal)

Patch Set 1 #

Patch Set 2 : state: add password test #

Patch Set 3 : state: add password test #

Patch Set 4 : state: add password test #

Unified diffs Side-by-side diffs Delta from patch set Stats (+74 lines, -0 lines) Patch
A [revision details] View 1 2 1 chunk +2 lines, -0 lines 0 comments Download
M state/unit_test.go View 1 2 2 chunks +72 lines, -0 lines 0 comments Download

Messages

Total messages: 4
rog
Please take a look.
13 years ago (2012-10-10 17:01:08 UTC) #1
rog
Please take a look.
13 years ago (2012-10-10 17:35:59 UTC) #2
niemeyer
LGTM
13 years ago (2012-10-10 17:37:39 UTC) #3
rog
13 years ago (2012-10-11 10:35:09 UTC) #4
*** Submitted:

state: add password test

Add a test to check that appropriate entities
can set the passwords they need to set.
Currently there is no restriction on capabilities,
but this test will be more relevant when we
move to an API-based state.

R=niemeyer
CC=
https://codereview.appspot.com/6636057
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b