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

Issue 60740045: test updates

Can't Edit
Can't Publish+Mail
Start Review
Created:
6 years, 6 months ago by lazypower
Modified:
6 years, 6 months ago
Reviewers:
mbruzek, Marco Ceppi, mp+204059
Visibility:
Public.

Description

test updates added relationship tests for :db relationship between mysql. Adds additional pre-condition installation routine for charmtester https://code.launchpad.net/~lazypower/charms/precise/owncloud/tests/+merge/204059 (do not edit description out of merge proposal)

Patch Set 1 #

Patch Set 2 : test updates #

Total comments: 1

Patch Set 3 : test updates #

Unified diffs Side-by-side diffs Delta from patch set Stats (+154 lines, -2 lines) Patch
A [revision details] View 1 2 1 chunk +2 lines, -0 lines 0 comments Download
M hooks/shared-fs-relation-changed View 1 chunk +2 lines, -2 lines 0 comments Download
A tests/00_setup.sh View 1 2 1 chunk +8 lines, -0 lines 0 comments Download
A tests/100_deploy.test View 1 1 chunk +142 lines, -0 lines 0 comments Download

Messages

Total messages: 6
lazypower
Please take a look.
6 years, 6 months ago (2014-02-07 21:06:02 UTC) #1
lazypower
Please take a look.
6 years, 6 months ago (2014-02-10 21:49:54 UTC) #2
mbruzek
Just a nit, that you might want to add so the script can be called ...
6 years, 6 months ago (2014-02-10 22:34:00 UTC) #3
lazypower
Please take a look.
6 years, 6 months ago (2014-02-10 22:35:56 UTC) #4
lazypower
On 2014/02/10 22:34:00, mbruzek wrote: > Just a nit, that you might want to add ...
6 years, 6 months ago (2014-02-10 22:37:02 UTC) #5
Marco Ceppi
6 years, 6 months ago (2014-02-12 15:28:33 UTC) #6
LGTM
Sign in to reply to this message.

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