-> v0.2.10

* WIP

* programmatic navigation

* get tests working

* wait longer

.eslintrc.json

@@ -1,7 +1,6 @@
 {
     "root": true,
     "rules": {
-        "indent": [ 2, "tab", { "SwitchCase": 1 } ],
         "semi": [ 2, "always" ],
         "space-before-blocks": [ 2, "always" ],
         "no-mixed-spaces-and-tabs": [ 2, "smart-tabs" ],
@@ -33,7 +32,7 @@
         "plugin:import/warnings"
     ],
     "parserOptions": {
-        "ecmaVersion": 6,
+        "ecmaVersion": 8,
         "sourceType": "module"
     }
 }

.travis.yml

@@ -1,10 +1,21 @@
 sudo: false
+
 language: node_js
+
 node_js:
   - "stable"
+
 env:
   global:
     - BUILD_TIMEOUT=10000
+
+addons:
+  apt:
+    packages:
+      - xvfb
+
 install:
+  - export DISPLAY=':99.0'
+  - Xvfb :99 -screen 0 1024x768x24 > /dev/null 2>&1 &
   - npm install
   - (cd test/app && npm install)

CHANGELOG.md

@@ -1,5 +1,26 @@
 # sapper changelog
 
+## 0.2.10
+
+* Handle deep links correctly ([#44](https://github.com/sveltejs/sapper/issues/44))
+
+## 0.2.9
+
+* Don't write files to disk in prod mode
+
+## 0.2.8
+
+* Add `goto` function ([#29](https://github.com/sveltejs/sapper/issues/29))
+* Don't use `/tmp` as destination in Now environments
+
+## 0.2.7
+
+* Fix streaming bug
+
+## 0.2.6
+
+* Render main.js back to templates, to allow relative imports ([#40](https://github.com/sveltejs/sapper/issues/40))
+
 ## 0.2.5
 
 * Fix nested routes on Windows ([#39](https://github.com/sveltejs/sapper/pull/39))

lib/build.js

@@ -1,14 +1,15 @@
 const fs = require('fs');
 const path = require('path');
-const glob = require('glob');
 const mkdirp = require('mkdirp');
+const rimraf = require('rimraf');
 const { client, server } = require('./utils/compilers.js');
 const create_app = require('./utils/create_app.js');
 const generate_asset_cache = require('./utils/generate_asset_cache.js');
 const { dest } = require('./config.js');
 
 module.exports = () => {
-	mkdirp(dest);
+	mkdirp.sync(dest);
+	rimraf.sync(path.join(dest, '**/*'));
 
 	// create main.js and server-routes.js
 	create_app();

lib/config.js

@@ -8,14 +8,14 @@ exports.templates = path.resolve(process.env.SAPPER_TEMPLATES || 'templates');
 
 exports.src = path.resolve(process.env.SAPPER_ROUTES || 'routes');
 
-exports.dest = path.resolve(
-	process.env.NOW ? '/tmp' :
-	process.env.SAPPER_DEST || '.sapper'
-);
+exports.dest = path.resolve(process.env.SAPPER_DEST || '.sapper');
 
 if (exports.dev) {
-	mkdirp(exports.dest);
+	mkdirp.sync(exports.dest);
 	rimraf.sync(path.join(exports.dest, '**/*'));
 }
 
-exports.server_routes = path.resolve(exports.dest, 'server-routes.js');
+exports.entry = {
+	client: path.resolve(exports.templates, '.main.rendered.js'),
+	server: path.resolve(exports.dest, 'server-entry.js')
+};

lib/index.js

@@ -1,9 +1,5 @@
 const fs = require('fs');
 const path = require('path');
-const glob = require('glob');
-const rimraf = require('rimraf');
-const mkdirp = require('mkdirp');
-const webpack = require('webpack');
 const route_manager = require('./route_manager.js');
 const templates = require('./templates.js');
 const create_app = require('./utils/create_app.js');
@@ -11,7 +7,7 @@ const create_watcher = require('./utils/create_watcher.js');
 const compilers = require('./utils/compilers.js');
 const generate_asset_cache = require('./utils/generate_asset_cache.js');
 const escape_html = require('escape-html');
-const { src, dest, dev } = require('./config.js');
+const { dest, dev } = require('./config.js');
 
 function connect_dev() {
 	create_app();
@@ -20,7 +16,7 @@ function connect_dev() {
 
 	let asset_cache;
 
-	return compose_handlers([
+	const middleware = compose_handlers([
 		require('webpack-hot-middleware')(compilers.client, {
 			reload: true,
 			path: '/__webpack_hmr',
@@ -59,6 +55,13 @@ function connect_dev() {
 
 		get_not_found_handler(() => asset_cache)
 	]);
+
+	middleware.close = () => {
+		watcher.close();
+		// TODO shut down chokidar
+	};
+
+	return middleware;
 }
 
 function connect_prod() {
@@ -67,7 +70,7 @@ function connect_prod() {
 		read_json(path.join(dest, 'stats.server.json'))
 	);
 
-	return compose_handlers([
+	const middleware = compose_handlers([
 		set_req_pathname,
 
 		get_asset_handler({
@@ -95,6 +98,12 @@ function connect_prod() {
 
 		get_not_found_handler(() => asset_cache)
 	]);
+
+	// here for API consistency between dev, and prod, but
+	// doesn't actually need to do anything
+	middleware.close = () => {};
+
+	return middleware;
 }
 
 module.exports = dev ? connect_dev : connect_prod;
@@ -139,7 +148,7 @@ function get_route_handler(fn) {
 						// TODO detect other stuff we can preload? images, CSS, fonts?
 						res.set('Link', `<${client.main_file}>;rel="preload";as="script", <${client.routes[route.id]}>;rel="preload";as="script"`);
 
-						let data = { params: req.params, query: req.query };
+						const data = { params: req.params, query: req.query };
 
 						if (mod.preload) {
 							const promise = Promise.resolve(mod.preload(data)).then(preloaded => {
@@ -147,7 +156,7 @@ function get_route_handler(fn) {
 								return mod.render(data);
 							});
 
-							templates.stream(res, 200, {
+							await templates.stream(res, 200, {
 								main: client.main_file,
 								html: promise.then(rendered => rendered.html),
 								head: promise.then(({ head }) => `<noscript id='sapper-head-start'></noscript>${head}<noscript id='sapper-head-end'></noscript>`),
@@ -178,28 +187,30 @@ function get_route_handler(fn) {
 
 			next();
 		} catch(err) {
-			res.status(500).end(templates.render(500, {
+			res.status(500);
+			res.end(templates.render(500, {
 				title: (err && err.name) || 'Internal server error',
 				url,
 				error: escape_html(err && (err.details || err.message || err) || 'Unknown error'),
 				stack: err && err.stack.split('\n').slice(1).join('\n')
 			}));
 		}
-	}
+	};
 }
 
 function get_not_found_handler(fn) {
 	return function handle_not_found(req, res) {
 		const asset_cache = fn();
 
-		res.status(404).end(templates.render(404, {
+		res.status(404);
+		res.end(templates.render(404, {
 			title: 'Not found',
 			status: 404,
 			method: req.method,
 			main: asset_cache.client.main_file,
 			url: req.url
 		}));
-	}
+	};
 }
 
 function compose_handlers(handlers) {
@@ -219,7 +230,7 @@ function compose_handlers(handlers) {
 		}
 
 		go();
-	}
+	};
 }
 
 function read_json(file) {

lib/route_manager.js

@@ -21,7 +21,8 @@ update();
 
 if (dev) {
 	const watcher = chokidar.watch(`${src}/**/*.+(html|js|mjs)`, {
-		ignoreInitial: true
+		ignoreInitial: true,
+		persistent: false
 	});
 
 	watcher.on('add', update);

lib/templates.js

@@ -38,7 +38,7 @@ function create_templates() {
 						const start = template.indexOf('%sapper', i);
 
 						if (start === -1) {
-							res.end(template.slice(start));
+							res.end(template.slice(i));
 							return;
 						}
 
@@ -57,7 +57,7 @@ function create_templates() {
 						i = end + 1;
 					} while (i < template.length);
 				}
-			}
+			};
 		})
 		.sort((a, b) => b.specificity - a.specificity);
 }
@@ -66,7 +66,8 @@ create_templates();
 
 if (dev) {
 	const watcher = chokidar.watch('templates/**.html', {
-		ignoreInitial: true
+		ignoreInitial: true,
+		persistent: false
 	});
 
 	watcher.on('add', create_templates);

lib/utils/create_app.js

@@ -2,10 +2,10 @@ const fs = require('fs');
 const path = require('path');
 const chokidar = require('chokidar');
 const route_manager = require('../route_manager.js');
-const { src, dest, server_routes, dev } = require('../config.js');
+const { src, entry, dev } = require('../config.js');
 
 function posixify(file) {
-	return file.replace(/[\/\\]/g, '/');
+	return file.replace(/[/\\]/g, '/');
 }
 
 function create_app() {
@@ -38,13 +38,11 @@ function create_app() {
 			main += `\n\nimport('${hmr_client}?path=/__webpack_hmr&timeout=20000'); if (module.hot) module.hot.accept();`
 		}
 
-		const file = path.resolve(dest, 'main.js');
-
-		fs.writeFileSync(file, main);
+		fs.writeFileSync(entry.client, main);
 
 		// need to fudge the mtime, because webpack is soft in the head
-		const { atime, mtime } = fs.statSync(file);
-		fs.utimesSync(file, new Date(atime.getTime() - 999999), new Date(mtime.getTime() - 999999));
+		const { atime, mtime } = fs.statSync(entry.client);
+		fs.utimesSync(entry.client, new Date(atime.getTime() - 999999), new Date(mtime.getTime() - 999999));
 	}
 
 	function create_server_routes() {
@@ -59,10 +57,10 @@ function create_app() {
 
 		const exports = `export { ${routes.map(route => route.id)} };`;
 
-		fs.writeFileSync(server_routes, `${imports}\n\n${exports}`);
+		fs.writeFileSync(entry.server, `${imports}\n\n${exports}`);
 
-		const { atime, mtime } = fs.statSync(server_routes);
-		fs.utimesSync(server_routes, new Date(atime.getTime() - 999999), new Date(mtime.getTime() - 999999));
+		const { atime, mtime } = fs.statSync(entry.server);
+		fs.utimesSync(entry.server, new Date(atime.getTime() - 999999), new Date(mtime.getTime() - 999999));
 	}
 
 	create_client_main();
@@ -73,7 +71,8 @@ if (dev) {
 	route_manager.onchange(create_app);
 
 	const watcher = chokidar.watch(`templates/main.js`, {
-		ignoreInitial: true
+		ignoreInitial: true,
+		persistent: false
 	});
 
 	watcher.on('add', create_app);

lib/utils/create_routes.js

@@ -74,4 +74,4 @@ module.exports = function create_matchers(files) {
 		});
 
 	return routes;
-}
+};

lib/utils/create_watcher.js

@@ -38,15 +38,11 @@ module.exports = function create_watcher() {
 		);
 	});
 
-	watcher = {
-		ready: invalidate()
-	};
-
 	function watch_compiler(type) {
 		const compiler = compilers[type];
 
 		compiler.plugin('invalid', filename => {
-			console.log(chalk.red(`${type} bundle invalidated, file changed: ${chalk.bold(filename)}`));
+			console.log(chalk.cyan(`${type} bundle invalidated, file changed: ${chalk.bold(filename)}`));
 			deferreds[type] = deferred();
 			watcher.ready = invalidate();
 		});
@@ -55,7 +51,7 @@ module.exports = function create_watcher() {
 			deferreds[type].reject(err);
 		});
 
-		compiler.watch({}, (err, stats) => {
+		return compiler.watch({}, (err, stats) => {
 			if (stats.hasErrors()) {
 				deferreds[type].reject(stats.toJson().errors[0]);
 			} else {
@@ -64,8 +60,16 @@ module.exports = function create_watcher() {
 		});
 	}
 
-	watch_compiler('client');
-	watch_compiler('server');
+	const watcher = {
+		ready: invalidate(),
+		client: watch_compiler('client'),
+		server: watch_compiler('server'),
+
+		close: () => {
+			watcher.client.close();
+			watcher.server.close();
+		}
+	};
 
 	return watcher;
 };

lib/utils/generate_asset_cache.js

@@ -3,7 +3,7 @@ const path = require('path');
 const glob = require('glob');
 const templates = require('../templates.js');
 const route_manager = require('../route_manager.js');
-const { dest } = require('../config.js');
+const { dest, dev } = require('../config.js');
 
 function ensure_array(thing) {
 	return Array.isArray(thing) ? thing : [thing]; // omg webpack what the HELL are you doing
@@ -17,8 +17,10 @@ module.exports = function generate_asset_cache(clientInfo, serverInfo) {
 	const service_worker = generate_service_worker(chunk_files);
 	const index = generate_index(main_file);
 
-	fs.writeFileSync(path.join(dest, 'service-worker.js'), service_worker);
-	fs.writeFileSync(path.join(dest, 'index.html'), index);
+	if (dev) {
+		fs.writeFileSync(path.join(dest, 'service-worker.js'), service_worker);
+		fs.writeFileSync(path.join(dest, 'index.html'), index);
+	}
 
 	return {
 		client: {
@@ -41,7 +43,7 @@ module.exports = function generate_asset_cache(clientInfo, serverInfo) {
 		},
 
 		server: {
-			entry: path.resolve(dest, 'server', serverInfo.assetsByChunkName.server_routes)
+			entry: path.resolve(dest, 'server', serverInfo.assetsByChunkName.main)
 		}
 	};
 };

mocha.opts

@@ -1,2 +1,3 @@
 --recursive
-test/unit/**/*.js
+test/unit/**/*.js
+test/common/test.js

package.json

@@ -1,6 +1,6 @@
 {
   "name": "sapper",
-  "version": "0.2.5",
+  "version": "0.2.10",
   "description": "Military-grade apps, engineered by Svelte",
   "main": "lib/index.js",
   "bin": {
@@ -21,22 +21,23 @@
     "webpack-hot-middleware": "^2.21.0"
   },
   "devDependencies": {
-    "cypress": "^1.3.0",
+    "css-loader": "^0.28.7",
     "eslint": "^4.13.1",
+    "eslint-plugin-import": "^2.8.0",
+    "express": "^4.16.2",
+    "get-port": "^3.2.0",
     "mocha": "^4.0.1",
+    "nightmare": "^2.10.0",
+    "node-fetch": "^1.7.3",
     "npm-run-all": "^4.1.2",
+    "style-loader": "^0.19.1",
+    "svelte": "^1.49.1",
+    "svelte-loader": "^2.3.2",
     "wait-on": "^2.0.2"
   },
   "scripts": {
     "cy:open": "cypress open",
-    "test": "run-s test:unit test:dev test:prod",
-    "test:unit": "mocha --opts mocha.opts",
-    "test:dev": "run-p --race test:launch:dev cy:run:dev",
-    "test:launch:dev": "node test/launch.js --dev",
-    "cy:run:dev": "wait-on http://localhost:3000 && cypress run -s test/cypress/integration/dev.js",
-    "test:prod": "run-p --race test:launch:prod cy:run:prod",
-    "test:launch:prod": "node test/launch.js --prod",
-    "cy:run:prod": "wait-on http://localhost:3000 && cypress run -s test/cypress/integration/prod.js"
+    "test": "mocha --opts mocha.opts"
   },
   "repository": "https://github.com/sveltejs/sapper",
   "keywords": [

runtime/app.js

@@ -6,12 +6,17 @@ export let component;
 let target;
 let routes;
 
+const history = typeof window !== 'undefined' ? window.history : {
+	pushState: () => {},
+	replaceState: () => {},
+};
+
 const scroll_history = {};
 let uid = 1;
 let cid;
 
 if ('scrollRestoration' in history) {
-	history.scrollRestoration = 'manual'
+	history.scrollRestoration = 'manual';
 }
 
 function select_route(url) {
@@ -39,7 +44,7 @@ function render(Component, data, scroll) {
 		} else {
 			// first load — remove SSR'd <head> contents
 			const start = document.querySelector('#sapper-head-start');
-			let end = document.querySelector('#sapper-head-end');
+			const end = document.querySelector('#sapper-head-end');
 
 			if (start && end) {
 				while (start.nextSibling !== end) detach(start.nextSibling);
@@ -75,8 +80,6 @@ function navigate(url, id) {
 
 			id = cid = ++uid;
 			scroll_history[cid] = { x: 0, y: 0 };
-
-			history.pushState({ id }, '', url.href);
 		}
 
 		selected.route.load().then(mod => {
@@ -123,6 +126,7 @@ function handle_click(event) {
 
 	if (navigate(url, null)) {
 		event.preventDefault();
+		history.pushState({ id: cid }, '', url.href);
 	}
 }
 
@@ -173,10 +177,17 @@ export function init(_target, _routes) {
 		inited = true;
 	}
 
-	const scroll = scroll_history[uid] = scroll_state();
+	setTimeout(() => {
+		const { hash, href } = window.location;
 
-	history.replaceState({ id: uid }, '', window.location.href);
-	navigate(new URL(window.location), uid);
+		const deep_linked = hash && document.querySelector(hash);
+		scroll_history[uid] = deep_linked ?
+			{ x: 0, y: deep_linked.getBoundingClientRect().top } :
+			scroll_state();
+
+		history.replaceState({ id: uid }, '', href);
+		navigate(new URL(window.location), uid);
+	});
 }
 
 function which(event) {
@@ -189,4 +200,12 @@ function scroll_state() {
 		x: window.scrollX,
 		y: window.scrollY
 	};
+}
+
+export function goto(href, opts = {}) {
+	if (navigate(new URL(href, window.location.href))) {
+		if (history) history[opts.replaceState ? 'replaceState' : 'pushState']({ id: cid }, '', href);
+	} else {
+		window.location.href = href;
+	}
 }

test/app/.gitignore

@@ -3,3 +3,4 @@ node_modules
 .sapper
 yarn.lock
 cypress/screenshots
+templates/.*

test/app/routes/about.html

@@ -6,14 +6,21 @@
 	<h1>About this site</h1>
 
 	<p>This is the 'about' page. There's not much here.</p>
+
+	<button on:click='goto("/blog/what-is-sapper")'>What is Sapper?</button>
 </Layout>
 
 <script>
 	import Layout from './_components/Layout.html';
+	import { goto } from '../../../runtime/app.js';
 
 	export default {
 		components: {
 			Layout
+		},
+
+		methods: {
+			goto
 		}
 	};
 </script>

test/app/routes/api/blog/_posts.js

@@ -82,6 +82,30 @@ const posts = [
 		html: `
 			<p>We're so glad you asked! Come on over to the <a href='https://github.com/sveltejs/svelte'>Svelte</a> and <a href='https://github.com/sveltejs/sapper'>Sapper</a> repos, and join us in the <a href='https://gitter.im/sveltejs/svelte'>Gitter chatroom</a>. Everyone is welcome, especially you!</p>
 		`
+	},
+
+	{
+		title: 'A very long post with deep links',
+		slug: 'a-very-long-post',
+		html: `
+			<h2 id='one'>One</h2>
+			<p>I'll have a vodka rocks. (Mom, it's breakfast time.) And a piece of toast. Let me out that Queen. Fried cheese… with club sauce.</p>
+			<p>Her lawyers are claiming the seal is worth $250,000. And that's not even including Buster's Swatch. This was a big get for God. What, so the guy we are meeting with can't even grow his own hair? COME ON! She's always got to wedge herself in the middle of us so that she can control everything. Yeah. Mom's awesome. It's, like, Hey, you want to go down to the whirlpool? Yeah, I don't have a husband. I call it Swing City. The CIA should've just Googled for his hideout, evidently. There are dozens of us! DOZENS! Yeah, like I'm going to take a whiz through this $5,000 suit. COME ON.</p>
+
+			<h2 id='two'>Two</h2>
+			<p>Tobias Fünke costume. Heart attack never stopped old big bear.</p>
+			<p>Nellie is blowing them all AWAY. I will be a bigger and hairier mole than the one on your inner left thigh! I'll sacrifice anything for my children.</p>
+			<p>Up yours, granny! You couldn't handle it! Hey, Dad. Look at you. You're a year older…and a year closer to death. Buster: Oh yeah, I guess that's kind of funny. Bob Loblaw Law Blog. The guy runs a prison, he can have any piece of ass he wants.</p>
+
+			<h2 id='three'>Three</h2>
+			<p>I prematurely shot my wad on what was supposed to be a dry run, so now I'm afraid I have something of a mess on my hands. Dead Dove DO NOT EAT. Never once touched my per diem. I'd go to Craft Service, get some raw veggies, bacon, Cup-A-Soup…baby, I got a stew goin'. You're losing blood, aren't you? Gob: Probably, my socks are wet. Sure, let the little fruit do it. HUZZAH! Although George Michael had only got to second base, he'd gone in head first, like Pete Rose. I will pack your sweet pink mouth with so much ice cream you'll be the envy of every Jerry and Jane on the block!</p>
+			<p>Gosh Mom… after all these years, God's not going to take a call from you. Come on, this is a Bluth family celebration. It's no place for children.</p>
+			<p>And I wouldn't just lie there, if that's what you're thinking. That's not what I WAS thinking. Who? i just dont want him to point out my cracker ass in front of ann. When a man needs to prove to a woman that he's actually… When a man loves a woman… Heyyyyyy Uncle Father Oscar. [Stabbing Gob] White power! Gob: I'm white! Let me take off my assistant's skirt and put on my Barbra-Streisand-in-The-Prince-of-Tides ass-masking therapist pantsuit. In the mid '90s, Tobias formed a folk music band with Lindsay and Maebe which he called Dr. Funke's 100 Percent Natural Good Time Family Band Solution. The group was underwritten by the Natural Food Life Company, a division of Chem-Grow, an Allen Crayne acqusition, which was part of the Squimm Group. Their motto was simple: We keep you alive.</p>
+
+			<h2 id='four'>Four</h2>
+			<p>If you didn't have adult onset diabetes, I wouldn't mind giving you a little sugar. Everybody dance NOW. And the soup of the day is bread. Great, now I'm gonna smell to high heaven like a tuna melt!</p>
+			<p>That's how Tony Wonder lost a nut. She calls it a Mayonegg. Go ahead, touch the Cornballer. There's a new daddy in town. A discipline daddy.</p>
+		`
 	}
 ];
 

test/app/templates/2xx.html

@@ -10,9 +10,9 @@
 	<link rel='icon' type='image/png' href='/favicon.png'>
 
 	<script>
-		if ('serviceWorker' in navigator) {
-			navigator.serviceWorker.register('/service-worker.js');
-		}
+		// if ('serviceWorker' in navigator) {
+		// 	navigator.serviceWorker.register('/service-worker.js');
+		// }
 	</script>
 
 	<!-- Sapper generates a <style> tag containing critical CSS

test/app/templates/main.js

@@ -1,10 +1,6 @@
-import { init } from '__app__';
+import { init } from '../../../runtime/app.js';
 
 // `routes` is an array of route objects injected by Sapper
 init(document.querySelector('#sapper'), __routes__);
 
-// if (__dev__) {
-// 	// Enable hot-module reloading
-// 	import('sapper/webpack/hmr');
-// 	if (module.hot) module.hot.accept();
-// }
+window.READY = true;

test/app/webpack.client.config.js

@@ -1,7 +1,5 @@
 const config = require('../../webpack/config.js');
 const webpack = require('webpack');
-const ExtractTextPlugin = require("extract-text-webpack-plugin");
-const UglifyJSPlugin = require('uglifyjs-webpack-plugin');
 
 module.exports = {
 	entry: config.client.entry(),
@@ -24,27 +22,18 @@ module.exports = {
 					}
 				}
 			},
-			config.dev && {
+			{
 				test: /\.css$/,
 				use: [
 					{ loader: "style-loader" },
 					{ loader: "css-loader" }
 				]
-			},
-			!config.dev && {
-				test: /\.css$/,
-				use: ExtractTextPlugin.extract({
-					fallback: 'style-loader',
-					use: [{ loader: 'css-loader', options: { sourceMap: config.dev } }]
-				})
 			}
 		].filter(Boolean)
 	},
 	plugins: [
 		config.dev && new webpack.HotModuleReplacementPlugin(),
-		!config.dev && new ExtractTextPlugin('main.css'),
-		!config.dev && new webpack.optimize.ModuleConcatenationPlugin(),
-		!config.dev && new UglifyJSPlugin()
+		!config.dev && new webpack.optimize.ModuleConcatenationPlugin()
 	].filter(Boolean),
 	devtool: config.dev ? 'inline-source-map' : false
 };

test/app/webpack.server.config.js

@@ -1,7 +1,4 @@
 const config = require('../../webpack/config.js');
-const webpack = require('webpack');
-const ExtractTextPlugin = require("extract-text-webpack-plugin");
-const UglifyJSPlugin = require('uglifyjs-webpack-plugin');
 
 module.exports = {
 	entry: config.server.entry(),

test/common/test.js

@@ -0,0 +1,244 @@
+const path = require('path');
+const assert = require('assert');
+const express = require('express');
+const serve = require('serve-static');
+const Nightmare = require('nightmare');
+const getPort = require('get-port');
+const fetch = require('node-fetch');
+
+run('production');
+run('development');
+
+function run(env) {
+	describe(`env=${env}`, function () {
+		this.timeout(5000);
+
+		let PORT;
+		let server;
+		let nightmare;
+		let middleware;
+		let capture;
+
+		let base;
+
+		function get(url) {
+			return new Promise(fulfil => {
+				const req = {
+					url,
+					method: 'GET'
+				};
+
+				const result = {
+					headers: {},
+					body: ''
+				};
+
+				const res = {
+					set: (headers, value) => {
+						if (typeof headers === 'string') {
+							return res.set({ [headers]: value });
+						}
+
+						Object.assign(result.headers, headers);
+					},
+
+					status: code => {
+						result.status = code;
+					},
+
+					write: data => {
+						result.body += data;
+					},
+
+					end: data => {
+						result.body += data;
+						fulfil(result);
+					}
+				};
+
+				middleware(req, res, () => {
+					fulfil(result);
+				});
+			});
+		}
+
+		before(async () => {
+			process.chdir(path.resolve(__dirname, '../app'));
+
+			process.env.NODE_ENV = env;
+
+			if (env === 'production') {
+				const cli = path.resolve(__dirname, '../../cli/index.js');
+				await exec(`${cli} build`);
+			}
+
+			const resolved = require.resolve('../..');
+			delete require.cache[resolved];
+			const sapper = require(resolved);
+
+			PORT = await getPort();
+			base = `http://localhost:${PORT}`;
+
+			global.fetch = (url, opts) => {
+				if (url[0] === '/') url = `${base}${url}`;
+				return fetch(url, opts);
+			};
+
+			let captured;
+			capture = async fn => {
+				const result = captured = [];
+				await fn();
+				captured = null;
+				return result;
+			};
+
+			const app = express();
+
+			app.use(serve('assets'));
+
+			app.use((req, res, next) => {
+				if (captured) captured.push(req);
+				next();
+			});
+
+			middleware = sapper();
+			app.use(middleware);
+
+			return new Promise((fulfil, reject) => {
+				server = app.listen(PORT, err => {
+					if (err) reject(err);
+					else fulfil();
+				});
+			});
+		});
+
+		after(() => {
+			server.close();
+			middleware.close();
+
+			// give a chance to clean up
+			return new Promise(fulfil => setTimeout(fulfil, 500));
+		});
+
+		describe('basic functionality', () => {
+			beforeEach(() => {
+				nightmare = new Nightmare();
+
+				nightmare.on('console', (type, ...args) => {
+					console[type](...args);
+				});
+
+				nightmare.on('page', (type, ...args) => {
+					if (type === 'error') {
+						console.error(args[1]);
+					} else {
+						console.warn(type, args);
+					}
+				});
+			});
+
+			afterEach(async () => {
+				await nightmare.end();
+			});
+
+			it('serves /', async () => {
+				const title = await nightmare
+					.goto(base)
+					.evaluate(() => document.querySelector('h1').textContent);
+
+				assert.equal(title, 'Great success!');
+			});
+
+			it('serves static route', async () => {
+				const title = await nightmare
+					.goto(`${base}/about`)
+					.evaluate(() => document.querySelector('h1').textContent);
+
+				assert.equal(title, 'About this site');
+			});
+
+			it('serves dynamic route', async () => {
+				const title = await nightmare
+					.goto(`${base}/blog/what-is-sapper`)
+					.evaluate(() => document.querySelector('h1').textContent);
+
+				assert.equal(title, 'What is Sapper?');
+			});
+
+			it('navigates to a new page without reloading', async () => {
+				await nightmare.goto(base).wait(() => window.READY).wait(100);
+
+				const requests = await capture(async () => {
+					await nightmare.click('a[href="/about"]');
+				});
+
+				assert.equal(
+					await nightmare.path(),
+					'/about'
+				);
+
+				assert.equal(
+					await nightmare.evaluate(() => document.title),
+					'About'
+				);
+
+				assert.deepEqual(requests.map(r => r.url), []);
+			});
+
+			it('navigates programmatically', async () => {
+				await nightmare
+					.goto(`${base}/about`)
+					.wait(() => window.READY)
+					.click('button')
+					.wait(() => window.location.pathname === '/blog/what-is-sapper')
+					.wait(100);
+
+				assert.equal(
+					await nightmare.evaluate(() => document.title),
+					'What is Sapper?'
+				);
+			});
+
+			it('scrolls to active deeplink', async () => {
+				const scrollY = await nightmare
+					.goto(`${base}/blog/a-very-long-post#four`)
+					.wait(() => window.READY)
+					.wait(100)
+					.evaluate(() => window.scrollY);
+
+				assert.ok(scrollY > 0, scrollY);
+			});
+		});
+
+		describe('headers', () => {
+			it('sets Content-Type and Link...preload headers', async () => {
+				const { headers } = await get('/');
+
+				assert.equal(
+					headers['Content-Type'],
+					'text/html'
+				);
+
+				assert.ok(
+					/<\/client\/main.\w+\.js>;rel="preload";as="script", <\/client\/_.\d+.\w+.js>;rel="preload";as="script"/.test(headers['Link']),
+					headers['Link']
+				);
+			});
+		});
+	});
+}
+
+function exec(cmd) {
+	return new Promise((fulfil, reject) => {
+		require('child_process').exec(cmd, (err, stdout, stderr) => {
+			if (err) {
+				process.stdout.write(stdout);
+				process.stderr.write(stderr);
+
+				return reject(err);
+			}
+
+			fulfil();
+		});
+	});
+}

test/cypress/fixtures/example.json

@@ -1,5 +0,0 @@
-{
-  "name": "Using fixtures to represent data",
-  "email": "hello@cypress.io",
-  "body": "Fixtures are a great way to mock data for responses to routes"
-}

test/cypress/integration/dev.js

@@ -1,19 +0,0 @@
-describe('dev mode', () => {
-	beforeEach(() => {
-		cy.visit('/')
-	});
-
-	it('has the correct <h1>', () => {
-		cy.contains('h1', 'Great success!')
-	});
-
-	it('navigates to /about', () => {
-		cy.get('nav a').contains('about').click();
-		cy.url().should('include', '/about');
-	});
-
-	it('navigates to /blog', () => {
-		cy.get('nav a').contains('blog').click();
-		cy.url().should('include', '/blog');
-	});
-});

test/cypress/integration/prod.js

@@ -1,19 +0,0 @@
-describe('prod mode', () => {
-	beforeEach(() => {
-		cy.visit('/')
-	});
-
-	it('has the correct <h1>', () => {
-		cy.contains('h1', 'Great success!')
-	});
-
-	it('navigates to /about', () => {
-		cy.get('nav a').contains('about').click();
-		cy.url().should('include', '/about');
-	});
-
-	it('navigates to /blog', () => {
-		cy.get('nav a').contains('blog').click();
-		cy.url().should('include', '/blog');
-	});
-});

test/cypress/plugins/index.js

@@ -1,17 +0,0 @@
-// ***********************************************************
-// This example plugins/index.js can be used to load plugins
-//
-// You can change the location of this file or turn off loading
-// the plugins file with the 'pluginsFile' configuration option.
-//
-// You can read more here:
-// https://on.cypress.io/plugins-guide
-// ***********************************************************
-
-// This function is called when a project is opened or re-opened (e.g. due to
-// the project's config changing)
-
-module.exports = (on, config) => {
-  // `on` is used to hook into various events Cypress emits
-  // `config` is the resolved Cypress config
-}

test/cypress/support/commands.js

@@ -1,25 +0,0 @@
-// ***********************************************
-// This example commands.js shows you how to
-// create various custom commands and overwrite
-// existing commands.
-//
-// For more comprehensive examples of custom
-// commands please read more here:
-// https://on.cypress.io/custom-commands
-// ***********************************************
-//
-//
-// -- This is a parent command --
-// Cypress.Commands.add("login", (email, password) => { ... })
-//
-//
-// -- This is a child command --
-// Cypress.Commands.add("drag", { prevSubject: 'element'}, (subject, options) => { ... })
-//
-//
-// -- This is a dual command --
-// Cypress.Commands.add("dismiss", { prevSubject: 'optional'}, (subject, options) => { ... })
-//
-//
-// -- This is will overwrite an existing command --
-// Cypress.Commands.overwrite("visit", (originalFn, url, options) => { ... })

test/cypress/support/index.js

@@ -1,20 +0,0 @@
-// ***********************************************************
-// This example support/index.js is processed and
-// loaded automatically before your test files.
-//
-// This is a great place to put global configuration and
-// behavior that modifies Cypress.
-//
-// You can change the location of this file or turn off
-// automatically serving support files with the
-// 'supportFile' configuration option.
-//
-// You can read more here:
-// https://on.cypress.io/configuration
-// ***********************************************************
-
-// Import commands.js using ES2015 syntax:
-import './commands'
-
-// Alternatively you can use CommonJS syntax:
-// require('./commands')

test/launch.js

@@ -1,75 +0,0 @@
-const fs = require('fs');
-const path = require('path');
-const rimraf = require('rimraf');
-const child_process = require('child_process');
-
-// ensure sapper doesn't exist in app/node_modules
-rimraf.sync(
-	path.join(__dirname, 'app/node_modules/sapper')
-);
-
-rimraf.sync(
-	path.join(__dirname, 'app/node_modules/.bin/sapper')
-);
-
-// create symlinks
-fs.symlinkSync(
-	path.join(__dirname, '..'),
-	path.join(__dirname, 'app/node_modules/sapper')
-);
-
-fs.symlinkSync(
-	path.join(__dirname, '../cli/index.js'),
-	path.join(__dirname, 'app/node_modules/.bin/sapper')
-);
-
-const app_dir = path.join(__dirname, 'app');
-
-function start_server() {
-	const server = child_process.spawn(process.execPath, ['server.js'], {
-		cwd: app_dir,
-		env: {
-			NODE_ENV: 'development'
-		},
-		stdio: 'pipe'
-	});
-
-	server.stdout.on('data', (data) => {
-		process.stdout.write(data);
-	});
-
-	server.stderr.on('data', (data) => {
-		process.stderr.write(data);
-	});
-}
-
-function launch() {
-	if (process.argv[2] === '--dev') {
-		start_server();
-	} else {
-		child_process.exec(`npm run build`, {
-			cwd: app_dir
-		}, (err, stdout, stderr) => {
-			if (err) throw err;
-
-			process.stdout.write(stdout);
-			process.stderr.write(stderr);
-			start_server();
-		});
-	}
-}
-
-// this is a terrible hack
-if (process.env.APPVEYOR) {
-	child_process.exec(`npm install`, {
-		cwd: app_dir
-	}, (err, stdout, stderr) => {
-		if (err) throw err;
-
-		process.stdout.write(stdout);
-		process.stderr.write(stderr);
-		launch();
-	});
-} else {
-	launch();
-}

webpack/config.js

@@ -1,5 +1,5 @@
 const path = require('path');
-const { src, dest, dev, server_routes } = require('../lib/config.js');
+const { src, dest, dev, entry } = require('../lib/config.js');
 
 module.exports = {
 	dev,
@@ -7,7 +7,7 @@ module.exports = {
 	client: {
 		entry: () => {
 			return {
-				main: `${dest}/main.js`
+				main: entry.client
 			};
 		},
 
@@ -24,7 +24,7 @@ module.exports = {
 	server: {
 		entry: () => {
 			return {
-				server_routes
+				main: entry.server
 			}
 		},