Skip to content

Commit

Permalink
browserify botframework-streaming (#2942)
Browse files Browse the repository at this point in the history
* browserify botframework-streaming

Fixes #2311

* fix import in WebSocket.test.js

* Fix dom usage

Co-authored-by: stevengum <[email protected]>
  • Loading branch information
joshgummersall and stevengum authored Oct 23, 2020
1 parent e41531d commit b6a74f5
Show file tree
Hide file tree
Showing 7 changed files with 36 additions and 40 deletions.
5 changes: 3 additions & 2 deletions libraries/botframework-connector/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -40,6 +40,7 @@
"mocha": "^6.2.3",
"nock": "^11.9.1",
"nyc": "^15.1.0",
"rimraf": "^3.0.2",
"should": "^13.2.3",
"shx": "^0.3.2",
"source-map-support": "^0.5.3",
Expand All @@ -49,8 +50,8 @@
},
"scripts": {
"build": "tsc",
"postbuild": "shx cp lib/index.js lib/browser.js | browserify --debug lib/browser.js | exorcist lib/browser.map.js | sponge lib/browser.js",
"clean": "erase /q /s .\\lib",
"postbuild": "shx cp lib/index.js lib/browser.js | browserify -s BFC --debug lib/browser.js | exorcist lib/browser.js.map | sponge lib/browser.js",
"clean": "rimraf q s lib",
"set-version": "npm version --allow-same-version ${Version}",
"test": "tsc && nyc mocha tests/ --timeout 60000 --inspect"
},
Expand Down
8 changes: 4 additions & 4 deletions libraries/botframework-streaming/.gitignore
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
/browser
/lib
/node_modules
es5
lib
node_modules
*.js.map
/coverage/**
coverage/**
17 changes: 12 additions & 5 deletions libraries/botframework-streaming/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@
"url": "https://github.com/microsoft/botbuilder-js.git"
},
"main": "lib/index.js",
"browser": "browser/index-browser.js",
"browser": "lib/index-browser.js",
"typings": "lib/index.d.ts",
"dependencies": {
"@types/ws": "^6.0.3",
Expand All @@ -31,24 +31,31 @@
"@microsoft/api-extractor": "^7.7.12",
"@types/chai": "^4.1.7",
"@types/node": "^10.17.27",
"browserify": "16.5.2",
"chai": "^4.2.0",
"exorcist": "^1.0.1",
"mocha": "^6.2.3",
"npm-run-all": "^4.1.5",
"nyc": "^15.1.0",
"rimraf": "^3.0.2",
"sinon": "^7.5.0",
"sponge": "^0.1.0",
"ts-node": "^4.1.0",
"typescript": "3.5.3"
},
"scripts": {
"build": "tsc -p tsconfig.json && tsc -p tsconfig-browser.json",
"build": "npm-run-all -p build:lib build:es5",
"build:lib": "tsc -p tsconfig.json",
"build:es5": "tsc -p tsconfig-es5.json",
"postbuild": "browserify -s BFSE --debug es5/index-browser.js | exorcist lib/index-browser.js.map | sponge lib/index-browser.js",
"build:rollup": "npm run clean && npm run build && api-extractor run --verbose --local",
"clean": "rmdir /q /s lib browser",
"clean": "rimraf q s lib es5",
"set-version": "npm version --allow-same-version ${Version}",
"test": "npm run build && nyc mocha tests/",
"test:compat": "api-extractor run --verbose"
},
"files": [
"lib",
"src",
"browser"
"src"
]
}
2 changes: 1 addition & 1 deletion libraries/botframework-streaming/tests/WebSocket.test.js
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ const { expect } = require('chai');
const { spy } = require('sinon');

const { RequestHandler, StreamingRequest, WebSocketClient, WebSocketServer } = require('../');
const { BrowserWebSocket } = require('../browser/index-browser');
const { BrowserWebSocket } = require('../lib/index-browser');
const { WebSocketTransport } = require('../lib/webSocket/webSocketTransport');

const { FauxSock } = require('./helpers');
Expand Down
23 changes: 0 additions & 23 deletions libraries/botframework-streaming/tsconfig-browser.json

This file was deleted.

11 changes: 11 additions & 0 deletions libraries/botframework-streaming/tsconfig-es5.json
Original file line number Diff line number Diff line change
@@ -0,0 +1,11 @@
{
"extends": "./tsconfig.json",
"compilerOptions": {
"lib": ["es2015", "dom"],
"target": "es5",
"outDir": "es5"
},
"exclude": [
"**/*node*.ts"
]
}
10 changes: 5 additions & 5 deletions libraries/botframework-streaming/tsconfig.json
Original file line number Diff line number Diff line change
Expand Up @@ -10,10 +10,10 @@
"rootDir": "./src",
"types" : ["node"]
},
"include": [
"src/**/*"
],
"exclude": [
"**/*browser*.ts",
"browser",
"lib",
"node_modules"
"**/*browser*.ts"
]
}
}

0 comments on commit b6a74f5

Please sign in to comment.