lp:~teknico/juju-gui/run-tests-in-prod-too

Branch merges

Propose for merging

No branches dependent on this one.

Merged into lp:juju-gui/experimental at revision 273

Juju GUI Hackers: Pending requested 2012-12-10

Diff: 102 lines (+25/-13)

3 files modified

Makefile (+18/-8)
test-server.js (+6/-4)
test-server.sh (+1/-1)

api

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related bugs

Bug #1087107: Tests should be runnable on the production build of the code

High

Fix Released

Link a bug report

Related blueprints

275. By Nicola Larosa on 2012-12-10

Apply suggestions from reviewers.

274. By Nicola Larosa on 2012-12-10

Readd the 'make test' entry in the instructions.

273. By Nicola Larosa on 2012-12-10

Autofixed one style error.

272. By Nicola Larosa on 2012-12-10

Merged from trunk, Makefile conflict resolved.

271. By Nicola Larosa on 2012-12-10

Merged from trunk, Makefile conflict resolved.

270. By Nicola Larosa on 2012-12-07

Enable running test in the production environment.

269. By Nicola Larosa on 2012-12-07

Serve runtime environment statically.

Create two new "prod" and "debug" environments, that are served
statically by a HTTP Python server. Keep serving the "devel"
environment dinamically by node.js, it will be made static
later. Make tests run in the "debug" environment. Refactor the
Makefile adding some docs and targets.

R=hazmat, matthew.scott, benji, gary.poster
CC=
https://codereview.appspot.com/6878053

268. By Benji York on 2012-12-05

un-revert r263

267. By Brad Crittenden on 2012-12-05

Tweaks to the charm store picker CSS.

Fixed spacing on text and placement of the activetick.
Also changed expanded widget to drop straight down instead of shifting
to the right.

Changes suggested by Matt C. Self-reviewed.

266. By Brad Crittenden on 2012-12-05

Makefile tweaks

Bug 1078723 addressed problems with changes to third party JS not
forcing a remake of the combined JS files. We really only have one
such file now, the others are handled properly, so this branch adds a
dependency on it, creates a THIRD_PARTY_JS symbol in the Makefile to
make it obvious in the future, and does other clean-up.

I tried removing the special-case for bin/generateTemplates (which
*is* a JavaScript file) by adding a .js extension. That made most of
the make process work but then the linters blew up. I hastily
retreated.

R=benji, frankban
CC=
https://codereview.appspot.com/6873055