[INTERNAL] package.json: Update dependency opn to open

The project "opn" is deprecated because it got renamed to "open".
SAP · Mar 28, 2019 · 0609e4f · 0609e4f
1 parent f234613
commit 0609e4f
Show file tree

Hide file tree

Showing 4 changed files with 10 additions and 10 deletions.
diff --git a/lib/cli/commands/serve.js b/lib/cli/commands/serve.js
@@ -88,8 +88,8 @@ serve.handler = function(argv) {
 					}
 					browserUrl += relPath;
 				}
-				const opn = require("opn");
-				opn(browserUrl);
+				const open = require("open");
+				open(browserUrl);
 			}
 
 			return browserUrl;

diff --git a/npm-shrinkwrap.json b/npm-shrinkwrap.json
diff --git a/package.json b/package.json
@@ -104,7 +104,7 @@
 		"@ui5/server": "^1.0.0",
 		"import-local": "^2.0.0",
 		"js-yaml": "^3.13.0",
-		"opn": "^6.0.0",
+		"open": "^6.0.0",
 		"semver": "^6.0.0",
 		"supports-color": "^6.0.0",
 		"treeify": "^1.0.1",

diff --git a/test/lib/cli/commands/serve.js b/test/lib/cli/commands/serve.js
@@ -110,9 +110,9 @@ test.serial("ui5 serve --accept-remote-connections", async (t) => {
 test.serial("ui5 serve --open", async (t) => {
 	normalizerStub.resolves(projectTree);
 	serverStub.resolves({port: 8080});
-	mockRequire("opn", function(openedUrl) {
+	mockRequire("open", function(openedUrl) {
 		t.is(openedUrl, "http://localhost:8080/webapp/index.html", `Opens url: ${openedUrl}`);
-		mockRequire.stop("opn");
+		mockRequire.stop("open");
 	});
 	await serve.handler(
 		Object.assign({}, defaultInitialHandlerArgs, {open: "webapp/index.html"})
@@ -122,9 +122,9 @@ test.serial("ui5 serve --open", async (t) => {
 test.serial("ui5 serve --open (opens default url)", async (t) => {
 	normalizerStub.resolves(projectTree);
 	serverStub.resolves({port: 8080});
-	mockRequire("opn", function(openedUrl) {
+	mockRequire("open", function(openedUrl) {
 		t.is(openedUrl, "http://localhost:8080", `Opens url: ${openedUrl}`);
-		mockRequire.stop("opn");
+		mockRequire.stop("open");
 	});
 	await serve.handler(
 		Object.assign({}, defaultInitialHandlerArgs, {open: true})